Lesen aus .txt und vergleichen

[EinfachSö]

Hallo,

Ich habe ein kleines Funktionalitätsproblem.

Ich habe eine .txt-Datei, in der zeilenweise "true" oder "false" steht.
Nun möchte ich in einer Methode diese Zeilen auslesen. Wenn in Zeile x "true" steht soll xy gemacht werden. Wenn dort "false" steht etwas anderes.
Hier meine Herangehensweise:

for(int i = 0; i < number; ++i) //Zeilen bis hin zu der gewünschten werden ignoriert.
	br.readLine();
	String lineIWant = br.readLine(); //Ausgabe der Zeile, die gewünscht ist!
	br.close();
			
	// In .txt-Datei steht ob Zahl =prim oder nicht. Falls ja, dann gib true zurück. Andernfalls false!!!
	lineIWant = lineIWant.replace("\n", ""); //Befürchtung: Zeilenumbruch ist das Problem!
	setReturn(lineIWant);

Die Methode "setReturn" überprüft die Zeile nun auf den Inhalt.

public void setReturn(String line)
	{
		if (line == "true")
		{
			returnvar = true;
		}
		else
		{
			returnvar = false;
		}
	}

Mein Bedenken ist, dass ich so auch den Zeilenumbruch mitlese. Ich habe die.trim()-Methode probiert oder wie oben die replace()-Methode.
Auch if (line == "true\n") funktioniert nicht wie gewünscht.
Es wird immer "false" returned.

Aufbau der Text-Datei (Beispiel):

false
true
true
false
true
false
true
false
false
false
true
false
true
false
false
false
true
false
true

Wie kann ich das Problem lösen. Oder gibt es einfachere Wege dies zu lösen?
Ich habe der Übersicht halber den restlichen Code weggelassen. Ich weiß, dass das Problem bei der Überprüfung per if-Schleife liegt.

Got it!

public void setReturn(String line)
{
	Boolean boolean1 = Boolean.valueOf(line);
	if (boolean1)
	{
		returnvar = true;
	}
	else
	{
		returnvar = false;
	}
}

[Zunft]

Originally Posted by EinfachSö

public void setReturn(String line)
	{
		if (line == "true")
		{
			returnvar = true;
		}
		else
		{
			returnvar = false;
		}
	}

Für die Zukunft (Die Lösung hast du ja schon):

Strings werden in Java nicht mit '==' verglichen. Dann nutzt du entweder:

if(s.equals("false"))

oder um die Gefahr von eingelesenen zusätzlichen Leerzeichen, Zeilenumbrüchen, etc. zu umgehen:

if(s.contains("false"))

Originally Posted by EinfachSö

Ich weiß, dass das Problem bei der Überprüfung per if-Schleife liegt.

Dafür müsstest du mir jetzt eigentlich all dein e*Gold als Strafe geben.

[snow]

	Boolean boolean1 = Boolean.valueOf(line);
	if (boolean1)
	{
		returnvar = true;
	}
	else
	{
		returnvar = false;
	}

Warum nicht

returnvar = Boolean.valueOf(line);

?

[EinfachSö]

Danke beiden!
@Zunft Hätte mir früher auffallen müssen.

Originally Posted by snow

Warum nicht

returnvar = Boolean.valueOf(line);

?

Logisch.
Wald vor lauter Bäumen...



Ganz nebenbei noch eine Frage.
Ich habe die Deklaration der Variablen in eine extra Methode packen müssen, weil es nach der for-Schleife nicht funktioniert hat.
Die for-Schleife und der kleine Rest dahinter befindet sich ein einem try-Block. Kann ich den Schritt mit einer seperaten Methode umgehen? Also, dass ich "returnvar" direkt im try-Block deklariere?

Ganzer Code:

 Spoiler

import java.io.BufferedReader;
import java.io.FileInputStream;
import java.io.IOException;
import java.io.InputStreamReader;

public class CheckNumIfPrimeBoolean
{
	public boolean returnvar;
	public boolean isPrimeNumber(int number, String data)
    {
		while (true) /*Zeilennummern in .txt sind von 1 bis 500000.
		number kann >500k sein. Also 500k abziehen, bis number <=500k.
		(.txt-Dateien sind aufgesplittet. Alle 500k neue Datei.*/
		{
			if (number>500000)
			{
				number -= 500000;
			}
			else
			{
				break;
			}
		}
		
		//Zeile auslesen!!!
		try
		{
			FileInputStream fs= new FileInputStream("textdatas/" + data + ".txt"); //Datei auswählen, die gelesen werden soll.
			BufferedReader br = new BufferedReader(new InputStreamReader(fs)); //Lesen starten???
			for(int i = 1; i < number; ++i) //Zeilen bis hin zu der gewünschten werden ignoriert.
				br.readLine();
			String lineIWant = br.readLine(); //Ausgabe der Zeile, die gewünscht ist!
			br.close();
			
			/* In .txt-Datei steht ob Zahl =prim oder nicht. Falls ja, dann gib true zurück. Andernfalls false!!!*/
			setReturn(lineIWant);
		}
		catch (IOException e)
		{
			System.err.println("IOEXception" + e);
		}
		
		return returnvar;
    }
	
	public void setReturn(String line)
	{
		returnvar = Boolean.valueOf(line);
	}
}

Diese Fälle wurden in meinem Tutorial nie erwähnt.

Originally Posted by Zunft

Dafür müsstest du mir jetzt eigentlich all dein e*Gold als Strafe geben.

nicht nachgedacht...

[Zunft]

Du kannst dir die ganze returnvar-Variable sparen:

public boolean setReturn(String line)
	{
		return Boolean.valueOf(line);
	}

bzw

setReturn(lineIWant);

ersetzen mit:

return setReturn(lineIWant);

Diese Fälle wurden in meinem Tutorial nie erwähnt.

[EinfachSö]

Originally Posted by Zunft

Du kannst dir die ganze returnvar-Variable sparen:

public boolean setReturn(String line)
	{
		return Boolean.valueOf(line);
	}

bzw

setReturn(lineIWant);

ersetzen mit:

return setReturn(lineIWant);

Das Problem ist aber, wenn Ich:

return setReturn(lineIWant);

benutze, dann entweder im try-Block oder dahinter.
Meinen ganzen Code habe ich im vorherigen Post ja gespoilert.
Wenn ich es im try-Block nutze, dann bekommt meine Class "CheckNumIfPrimeBoolean" einen Error. ("This method must return a result of type boolean").
Packe Ich deinen return-Befehl hinter den try-Block, so hat meine Class zwar einen return aber "lineIWant cant be resolved to a variable". lineIWant ist ja lokal im try-Block deklariert.

[Zunft]

In dem Fall würde ich es genauso wie du lösen. Die boolesche Variable global bzw. vor dem betreffenden Block deklarieren.