fehlende return anweisung?

[Br4ve]

ICh habe mich mal wieder an einer weiteren Ausnahme versucht, das funktioniert auch so weit ganz gut doch nun gibt mir meine Methode aus es würde eine return anweisung fehlen!?

// gibt das Dokument beim Index zurück
    public Object getElement(int index) {
        try {
            if (index < 0 || index >= getSize()) {
                throw new VarArrayIllegalIndexException(
                "Fehler ungültiges Argument");
            } else {
                return elems[index];
            }
        } catch (VarArrayIllegalIndexException e) {
            System.out.println(e.getMessage());
        }
        
        
    }

 

[eRaaaa]

Du musst in jedme Falle ein Object zurückgeben. Was ist jetzt aber, wenn index z.B. kleiner 0 ist? dann wirfst du eine Exception, fängst sie aber direkt(komisch? )... dort gibst du dann allerdings nichts zurück ?!
Daher wird gemeckert !

 

[Painii]

Wenn du die Exception selbst wirfst macht es ÜBERHAUPT keinen Sinn sie in der gleichen Methode zu fangen wie sie geworfen wird.

Das try/catch sollte an denjenigen fallen der die Methode aufruft.

 

[Ark]

Hm, ich glaube, das mit den Exceptions hast du noch nicht ganz richtig verstanden: Entweder eine Methode wirft in einer ganz bestimmten Situation eine ganz bestimmte Exception, oder sie fängt Ausnahmen, die durch aufgerufene Methoden etc. entstehen, ab. In deinem Fall fängst du jedoch eine Exception ab, die du sogar erst selbst erzeugt hast. Das ist widersinnig. Entferne das try-catch-Konstrukt (also entferne alle catch-Blöcke komplett und umklammere den eigentlichen Code nicht mehr mit try) und lass stattdessen die Methode die Exception werfen.

Wozu ist VarArrayIllegalIndexException überhaupt gut? Und was macht dieses getSize()? Ich denke, das ist alles komplett überflüssig und deine Methode bräuchte nicht länger zu sein als eine Zeile Code.

Ark

 

[Br4ve]

Also zum return erstmal wenn ich in dem Array n negativen index eingebe, kann doch garkein Objekt zurückgegeben werden...
getSize() gibt die Größe des Arrays zurück.
Ich soll da eine ungeprüfte Ausnahme implementieren. Der Aufruf der Methode findet dann ja direkt in der Mainmethode statt, macht es denn sinn die da zu fangen?

 

[eRaaaa]

Also zum return erstmal wenn ich in dem Array n negativen index eingebe, kann doch garkein Objekt zurückgegeben werden...

Deshalb sollst du ja auch wahrscheinlich eine Exception werfen:

    public Object getElement(int index) {
	if (index < 0 || index >= getSize()) {
	    throw new VarArrayIllegalIndexException(
		    "Fehler ungültiges Argument");
	} else {
	    return elems[index];
	}
    }

 

[Br4ve]

Deshalb sollst du ja auch wahrscheinlich eine Exception werfen

Ok wo fange ich das denn ansonsten? Wenn ich den rest weglasse bekomme ich ja den Fehler "Syntax error, insert "Finally" to complete BlockStatements"
Meine main methode dazu:

 public static void main(String[] args) {
        VarArray neuesarray = new VarArray(1);
        Dokument eins = new Dokument("hallo", "dok1", AUDIO, true);
        Dokument zwei = new Dokument("nochmal", "dok2", PDF, false);
        neuesarray.addElement(eins);
        neuesarray.addElement(zwei);
        System.out.println("Hier sollte jetzt eine 2 stehen: " 
            + neuesarray.getSize());
        neuesarray.deleteElement(1);
        System.out.println("Dokument zwei geloescht, zeige Dokument eins an: ");
        System.out.println(neuesarray.getElement(0));
        
        VarArray arraytest = new VarArray(-3);
        neuesarray.getElement(3);
        neuesarray.getElement(2);
        neuesarray.getElement(1);
    }

   
}

 

[eRaaaa]

Ok wo fange ich das denn ansonsten?

Da VarArrayIllegalIndexException eine RuntimeException ist, musst du diese nicht fangen, kannst es aber wenn du willst. Wie? Ganz normal wie bei allen anderen Exceptions auch! (hier macht`s denk ich nicht viel Sinn und verfehlt denke ich auch die Aufgabenstellung (meine Meinung))

 

[Br4ve]

Stimmt ich brauch die garnich fangen..:lol:
Aber wie krieg ich denn das mit dem Fehler in den Griff?

 

[Landei]

Stimmt ich brauch die garnich fangen..:lol:
Aber wie krieg ich denn das mit dem Fehler in den Griff?

Du wirfst nur eine Exception, wenn du ein Problem nicht lokal lösen kannst, aber die Vermutung hast, eine höhere Ebene könnte etwas sinnvolles tun. Also kümmert sich der Aufrufer der Methode, oder der Aufrufer des Aufrufers der Methode, oder... darum. Wie bei der Bundeswehr: Melden macht frei!

 

[eRaaaa]

Stimmt ich brauch die garnich fangen..:lol:
Aber wie krieg ich denn das mit dem Fehler in den Griff?

Welchen denn?

VarArray arraytest = new VarArray(-3);

???:L Was soll das ?

 

[Br4ve]

Das ist nur zum Prüfen ob meine andere Exception funktioniert...
Ich versteh leider nich so ganz, was ich da jez genau ändern muss ???:L
Also die main methode unten soll jez

neuesarray.getElement(3);
        neuesarray.getElement(2);

bei diesen beiden eingaben die Fehlermeldung "....ungültiges Argument" ausgeben, da neuesarray ja nur eine größe von 2 hat. Bei getElement(1); jedoch dürfte dann ja die Fehlermeldung nicht kommen...

 

[eRaaaa]

bei diesen beiden eingaben die Fehlermeldung "....ungültiges Argument" ausgeben

Tut es das denn nicht? 🙂 Es sollte doch jetzt eine Exception geworfen werden! (wenn VarArrayIllegalIndexException gemäß implementiert wurde)

Hast du den Code mal laufen lassen?

 

[Br4ve]

Naja der Code so geht ja net

  public Object getElement(int index) {
        try {
            if (index < 0 || index >= getSize()) {
                throw new VarArrayIllegalIndexException(
                "Fehler ungültiges Argument");
            } else {
                return elems[index];
            }
        } 
       
        
    }

Weil mir hier bei der 2t letzten Klammer die Fehlermeldung angezeigt wird "Syntax error, insert "Finally" to complete BlockStatements" aber finally ist doch optional oder nicht?

 

[eRaaaa]

Na das try hat da ja auch nichts zu suchen 😀
Vergleiche mit dem Code von

Heute, 20:44

 

[Br4ve]

Oh man stimmt 😳
Ich bin auch langsam echt durch den Wind...nach paar std Java...
Also das funktioniert und bricht dann komplett ab, was bei einer RuntimeException ja auch so sein soll oder?
Einziges wehwehchen jez ist das wenn ich neuesarray.getElement(1) inner main aufrufe dann passiert genau das gleiche dabei ist doch neuesarray 2 felder groß(0 und 1)
Edit: Ach schon gut ich bin inzwischen wohl sogar zu doof um meine eigenen Code zu lesen, inner main Methode wird ja ein Objekt wieder gelöscht, daher hat das Array die größe 1 also nur index 0 is belegt...
Ich nerv euch dann mal nicht weiter, sollte ne pause machen 😛
Trotzdem vielen Dank für die tolle Hilfe!

 

[eRaaaa]

Wir kennen ja deine Klasse VarArray nicht, aber so wie es aussieht: Du addest 2 mal Elemente, und löschst dann eins,demzufolge ist nach Adam Ries 2-1 = 1 Element noch im Array vorhanden, oder nicht?

 

[Br4ve]

siehe edit ^^
danke