Exception abfangen

[babuschka]

Hallo, liebes Forum,

ich beschäftige mich gerade erstmals mit dem Thema "Exceptions" und habe dazu mal folgende Klasse bzw. Methode versucht:

//Ausgegeben werden soll der größte Eintrag einer als Parameter übergebenen ArrayList.
import java.util.ArrayList;

public class Maximum{

Integer maximum(ArrayList<Integer> a){
Integer max=a.get(0);

try{
  
for(int i=1; i<a.size(); i++){
      
              if( a.get(i)>max ){
                           max=a.get(i);
               }
     }
  
    
}

catch(IndexOutOfBoundsException e){
  System.out.println("Leeres Array übergeben. Maximum kann nicht bestimmt werden.");
}

return max;
}

public static void main(String[] args){
	Maximum testclass=new Maximum();
	ArrayList<Integer> b = new ArrayList<Integer>();
	
	System.out.println(testclass.maximum(b));
}

}

Ich habe im Rumpf der main-Methode absichtlich keine Elemente in die ArrayList b eingefügt, weil ich ja diesen Fall abfangen möchte; es funktioniert jedoch nicht.

Sieht jemand, wieso?

LG

 

[HimBromBeere]

Naja, wenn deine Schleife alle Elemente durchlaufen soll, deine Liste aber keine Elemente hat, kann auch keine Exception geworfen werden... die "kristischen" Stellen werden also eh nie ausgeführt...

 

[babuschka]

Es wird aber eine IndexOutOfBoundsException angezeigt.

 

[truesoul]

Ja pack mal dein :
Integer max=a.get(0);

in den try/catch Block. Dann bekommste die gewünschte Ausgabe ("Leeres Array übergeben. Maximum kann nicht bestimmt werden.").

 

[babuschka]

Nein, dann kommt beim Kompilieren die Fehlermeldung:

cannot find symbol
symbol: variable max

 

[Michael...]

Nein, dann kommt beim Kompilieren die Fehlermeldung:

cannot find symbol
symbol: variable max

Doch Du musst es halt anders programmieren ;-)
Die Exception tritt ausserhalb des des try catch Blocks auf (sollte auch in der Fehlermeldung stehen)

Eine Möglichkeit:

Integer max = null;
try {
    max = a.get(0);
    ...

Wobei ich in dem Fall eher die size() der List prüfen und dementsprechend eine eigene Exception werfen würde, anstatt eine IOoBException zu provozieren und deren Ausgabe zu überlagern.

 

[pro2]

Du darfst natürlich keine Variable im try/catch Block anlegen. Oben

Integer max;

und dann im Block

max = ...

.

 

[HimBromBeere]

Du darfst natürlich keine Variable im try/catch Block anlegen

... denn dann ist sie ja nur INNERHALB dieses Blockes verwendbar.

 

[Swoop]

Du darfst natürlich keine Variable im try/catch Block anlegen. Oben

Integer max;

und dann im Block

max = ...

.

Anstatt

Integer

könnte man auch

int

verwenden

 

[babuschka]

Jetzt bin ich verwirrt...

Ich habe die Variable jetzt überhalb des try Blocks deklariert, okay und in dem try Block initialisiert.

Aber wieso tritt der Fehler außrhalb davon auf?



Irgendwie verstehe ich es noch überhaupt nicht.

 

[babuschka]

Okay, mit dem Hinweis, daß ich außerhalb deklarieren und innerhalb des try-Blocks initialisieren muss, klappt es jetzt.


Der Grund ist also, daß die Variable nur innerhalb des try-Blocks gelten würde. Was ist falsch daran, ich benötige sie doch auch nur dort?


Und noch eine andere Frage. Der IndexOutOfBoundsException tritt bei einem leeren Array dort auf, wo ich

max=a.get(0)

habe? Weil es so ein erstes Element nicht gibt?

 

[HimBromBeere]

Wenn du nichts in deine Liste reintust, kannst du auch kein erstes Element bekommen...

 

[babuschka]

Okay und die andere Frage: Wieso muss außerhalb des try-Blocks deklariert werden?

 

[HimBromBeere]

MUSS nicht, aber wenn du sie an anderer Stelle (z.B. im catch-Block) brauchst, ist sie dort natürlich nicht bekannt...

 

[babuschka]

Deswegen frage ich ja, weil bei mir klappt es nur, wenn ich die Variable VOR der Methode selbst deklariere.

Und das verstehe ich nicht.

Erstens müsste das doch auch gehen, wenn ich sie in der Methode (noch vor dem try-Block deklariere) und zweitens sogar, wenn ich sie im try-Block deklariere und initialisiere, weil ich sie doch nur dort brauche.

 

[HimBromBeere]

Schreib mal, was du jetzt eigtl. hast... ich verlier´ langsam den Überblick

 

[babuschka]

Dies ist die Version, die bei mir klappt:

//Ausgegeben werden soll der größte Eintrag eines als Parameter übergebenen Arrays.
import java.util.ArrayList;

public class Maximum{
	Integer max;	
Integer maximum(ArrayList<Integer> a){
			

try{
	
	max=a.get(0);	  
for(int i=1; i<a.size(); i++){
      
              if( a.get(i)>max ){
                           max=a.get(i);
               }
     }
  
    
}

catch(IndexOutOfBoundsException e){
	
	System.out.println("Leeres Array: Maximum kann nicht bestimmt werden.");
}

return max;
}

public static void main(String[] args){
	Maximum testclass=new Maximum();
	ArrayList<Integer> b = new ArrayList<Integer>();
	
	System.out.println(testclass.maximum(b));
}

}

 

[HimBromBeere]

Du willst doch auf dein max auch beim return zugreifen, d.h. du könntest es bestenfalls noch lokal in der Methode deklarieren, anstelle von klassenweit.

 

[babuschka]

Eben, aber wenn ich die Deklaration gleich als erstes in den Methodenrumpf schreibe, kommt die Meldung:

might not have been initialized


Wenn ich dann aber das Initialisieren auch wieder dazu schreibe... kommt wieder die gewünschte Meldung mit der Exception nicht...

 

[HimBromBeere]

class Maximum{
	Integer maximum(ArrayList<Integer> a){
	    Integer max = null;    
	    try{
		    max=a.get(0);     
		    for(int i=1; i<a.size(); i++){
		    	if( a.get(i)>max ){max=a.get(i);}
		    }
	    } catch(IndexOutOfBoundsException e){
	    	System.out.println("Leeres Array: Maximum kann nicht bestimmt werden.");
	    }
	    return max;
	}
}

Also bei mir läuft´s... hab aber auch noch ´ne Klammer hinzugefügt.

EDIT: Ich erinnere dich an dieser Stelle nochmal an einen vorigen Beitrag:

Wobei ich in dem Fall eher die size() der List prüfen und dementsprechend eine eigene Exception werfen würde, anstatt eine IOoBException zu provozieren und deren Ausgabe zu überlagern.

 

[babuschka]

Okay, dann deklariert man also zwei Mal?

Edit: Ich meine initialisieren

Erst mit null und dann mit a.get(0)? ???:L

 

[HimBromBeere]

Initialisieren macht man nur einmal... darum heißt es auch INITialisieren. Das, was du mit

max = a.get(0)

machst, ist eine stinknormale Zuweisung (= null ist auch nichts anderes, da du´s aber das erste mal machst, spricht man von der erstmaligen Zuweisung - der Initialisierung).

EDIT: Du kannst aber auch einfach deine Variable trotzdem im try-Block deklarieren und nach dem Abarbeiten der Schleife return max verwenden. Unter den cath-Block schreibst du dann return null.

 

[babuschka]

Okay, ich merke schon.

Es wird zeit, dass ich irgendwie die Prüfung schaffe, dann zwei bis 5 Kreuze in den Kalender mache und mich dann demütig vom Javabereich entferne... :autsch:

 

[xehpuk]

O je, es kompiliert, aber bitte nicht nachmachen! :shock:

int max = maximum(new ArrayList<Integer>()); // schöne NullPointerException

IndexOutOfBoundsException

ist ein Laufzeitfehler. Nicht abfangen!
Mit

int

, nicht mit

Integer

rechnen!

Dass der Compilerfehler angezeigt wurde, hatte auch einen guten semantischen Grund. Wenn man den Fehler abfängt, kann man nichts Sinnvolles zurückgeben.

Variablen sind nur in dem Block sichtbar, in dem sie deklariert wurden.

void a() {
    {
        int a = 0;
    }
    a = 0; // Whoops!
}

 

[babuschka]

Ich kann Deine Einwände leider nicht nachvollziehen.

Ich würde es gerne, verstehe aber nicht, was Du meinst.


Wieso NullPointerException?

Wieso nicht abfangen?...

 

[xehpuk]

Wieso NullPointerException?

Probier es aus:

import java.util.ArrayList;

public class MaxInt {
	public static void main(final String[] args) {
		final int max = maximum(new ArrayList<Integer>());
	}

	static Integer maximum(final ArrayList<Integer> a) {
		Integer max = null;
		try {
			max = a.get(0);
			for (int i = 1; i < a.size(); i++) {
				if (a.get(i) > max) {
					max = a.get(i);
				}
			}
		} catch (final IndexOutOfBoundsException e) {
			e.printStackTrace();
		}
		return max;
	}
}

Oder kürzer:

public static void main(final String[] args) {
	Integer i = null;
	int j = i;
}

Java versucht, aus der

null

den primitiven Wert zu ziehen, was natürlich nicht funktioniert.

Wieso nicht abfangen?...

Gegenfrage: Wieso abfangen? Ist

null

ein guter Rückgabewert?
Laufzeitfehler sind unerwartete Fehler. Sie sollten nur auftreten, wenn der Programmierer rumgemurkst hat.
Unchecked Exceptions — The Controversy (The Java™ Tutorials > Essential Classes > Exceptions)

Eine denkbare Lösung:

static int maximum(final ArrayList<Integer> a) {
	if (a.isEmpty()) {
		throw new IllegalArgumentException("Liste darf nicht leer sein!");
	}
	int max = a.get(0);
	for (int i = 1; i < a.size(); i++) {
		if (a.get(i) > max) {
			max = a.get(i);
		}
	}
	return max;
}

 

[babuschka]

Das geht dann wohl über meine Fähigkeiten hinaus.

Ich sehe nicht, wo ich da was falsch gemacht habe.

Bei mir klappt sowohl der Fall, daß die ArrayList leer ist, als auch der Fall, daß Elemente enthalten sind.

Im ersten Fall wird die Warnung ausgegeben, im zweiten Fall das größte Element.


:bahnhof:

 

[babuschka]

Hallo, ich wollte mal fragen, ob man das Ganze nicht auch so machen kann:

import java.util.ArrayList;

class Maximum{

Integer maximum(ArrayList<Integer> a){
Integer max;
max=a.get(0);
if(a.size()==0){
   max=null;
   return max;
}

for(int i=1; i<a.size();i++)
{
    if(a.get(i)>max){
         max=a.get(i);
  }

return max;

}

Erfüllt das nicht auch die Aufgabenstellung?

LG

 

[babuschka]

Ich habe noch ein paar Verbesserungen vorgenommen.

Könnt Ihr mir bitte sagen, ob folgender Code okay ist bzw. ob er eine Alternative zu dem obigen Krams mit catch und try ist?

Ich habe ein paar Beispiele ausprobiert (leere Liste und verschiedene gefüllte Listen) und bei mir funktioniert es; dennoch interessiert mich Eure Meinung.

import java.util.ArrayList;

class Maximum{

   Integer maximum( ArrayList<Integer> a ) {
      Integer max;

      if( a.isEmpty() ) {
          max = null;
          return max;

      }

   max = a.get(0);
     for( int i = 1; i < a.size(); i++ ) {
           if( a.get(i) > max ) {
                 max = a.get(i);
      }

   return max;

   }

public static void main( String[] args) {
      ArrayList<Integer> testArrayList = new ArrayList<Integer>();
      Maximum testClass = new Maximum();
      testArrayList.add(1);
      testArrayList.add(200);
      testArrayList.add(5);
      testArrayList.add(50);

     System.out.println( testClass.maximum( testArrayList ) );
  }
}

Bei diesem Beispiel wird bei mir auf der Konsole 200 ausgegeben.
Wenn ich in die Liste

testArrayList

keine Elemente einfüge, wird

null

ausgegeben.

 

[babuschka]

Keiner Lust?

 

[HimBromBeere]

Du solltest nur wahrscheinlich statt Integer mal int verwenden... ansonsten fällt mir nichts ein (abgesehen von der unerhebl,ichen Kleinigkeit, dass wohl niemand eine Klasse Maximum nennt und dann auch noch nie einzige Funktion).
Achja, eine Sache noch: wenn man schon das Interface Iterable irgendwann implementiert (z.B. mittels einer Collection wie ArrayList), dann durchlaufe die auch mal mit dem gelieferten Iterator... dafür is er da

 

[babuschka]

Achja, eine Sache noch: wenn man schon das Interface Iterable irgendwann implementiert (z.B. mittels einer Collection wie ArrayList), dann durchlaufe die auch mal mit dem gelieferten Iterator... dafür is er da

Was meinst Du damit?

Welcher gelieferte Iterator?

 

[HimBromBeere]

den hier: List (Java Platform SE 6)
Alle Klassen, die

Collection

implementieren (wozu irgendwie auch deine

ArrayList

gehört), besitzen die Methode

iterator()

, mit welchem du sukzessive jedes Element durchlaufen kannst.
In deinem Fall ist es eher irrelevant, ob du mittels for-Schleife oder Iterator durch die Elemente gehst, aber so machem Benutzer der ersten Variante soll in ähnlichen Anwendungsfällen bereits eine sog

CurrentModificationException

um die Ohren geflogen sein, deren Ursachenfindung immer ein bisschen hakelig ist...

 

[EnHancEd[]]

Oder als Beispiel:

ArrayList<String> list= new ArrayList<String>();

list.add("So ");
list.add("geht ");
list.add("das");


for(String k: list){
System.out.print(k);
}

greetz