Return Problem

[Kilinat]

Hallo Community

Bin gerade dabei ein Tool zu schreiben, mit welchem man selbst ein bischen lernen kann, in dem man selbst Fragen hinzufügen kann, und über diese Fragen auch wieder abgefragt werden kann.

Ich realisiere das ganze mit einem FileWriter und schreibe eine Struktur in eine Textdatei (Frage"/"Antwort) und lese dann auch so wieder aus. Nur habe ich beim Auslesen ein Problem mit dem return. Ich poste einfach mal die Methode.

public Frage lesen(){
		String splitArray[];
		try {
			BufferedReader reader = new BufferedReader(new FileReader("fragen.txt"));
			String zeile = reader.readLine();
			splitArray = zeile.split("/");
			Frage frage1 = new Frage(splitArray[0], splitArray[1]);
			return frage1;
			
		}catch (IOException ex) { ex.printStackTrace(); }
		
	}

Jetzt sagt mir aber die Methode folgendes: this method must return a result of type Frage

Hoffe ihr könnt mir helfen
MfG Kilinat

 

[Crian]

Deine Funktion gibt nicht in jedem Fall etwas zurück (wenn eine Exception auftritt, nämlich nicht).

Eine mögliche Lösung wäre:

public Frage lesen() {
     Frage frage1 = null;
     String splitArray[];
     try {
         BufferedReader reader = new BufferedReader(new FileReader("fragen.txt"));
         String zeile = reader.readLine();
         splitArray = zeile.split("/");
         frage1 = new Frage(splitArray[0], splitArray[1]);
    }
    catch (IOException ex) {
        ex.printStackTrace();
    }
        
    return frage1;
}

Dann muss die aufrufende Stelle aber damit rechnen, dass ein null-Wert zurückgegeben werden kann. Falls du das nicht möchtest, kannst du frage1 natürlich mit irgendwas anderem initialisieren. Etwa

Frage frage1 = new Frage("Dummyfrage", "Dummyantwort");

. Ob das schöner ist, musst du entscheiden.

Alternativ fängst du den Fehler nicht hier ab, sondern kümmerst dich an der Stelle darum, wo diese Funktion aufgerufen wird. Auch diese Entscheidung liegt bei dir.

Dann sieht der Code hier wesentlich übersichtlicher aus, dafür muss du dich an anderer Stelle darum kümmern (wo dies aber u.U. auch besser möglich ist). Das sind Designentscheidungen, die dir vermutlich niemand abnehmen kann.

public Frage lesen() throws IOException {
    BufferedReader reader = new BufferedReader(new FileReader("fragen.txt"));
    String zeile = reader.readLine();
    String[] splitArray = zeile.split("/");
    Frage frage = new Frage(splitArray[0], splitArray[1]);
    return frage;
}

 

[Haave]

Am Ende der Methode steht kein return. Es könnte ja sein, dass der try-Block nicht zuende ausgeführt wird, dann wird auch nichts zurückgegeben. Versuch mal, frage1 vor dem try-Block zu deklarieren (also

Frage frage1;

) und dann hinter dem catch-Block

return frage1;

zu machen.

 

[Kilinat]

Danke für eure Antworten, haben mir geholfen^^

MfG Kilinat