Filereader funktioniert nicht! Fehlersuche

[Sensewell]

Hallo Leute,


hab mal wieder ein Problem was ich eifnach nicht lösen kann.
Ich habe eine eigene Methode geschrieben:

public static boolean readAlias(String dateiname,String alias)
			throws IOException {
		BufferedReader checkAlias = new BufferedReader(new FileReader(dateiname
				+ ".txt"));
		boolean check = true;
		String checker = null;
		while (checkAlias.readLine() != null) {
			checker = checkAlias.readLine();
			if (checker.equals(alias)) {
				check = false;
				break;
			}

		}
		return check;

	}

Die Methode rufe ich in einer Main auf:

boolean checkAlias = false;
		String alias= null;
		while (!checkAlias){
		alias = IOTools
				.readString("Bitte wählen Sie einen Alias(Dieser ist für den Login erforderlich) : ");
		
		
		//Filereader.readAlias("Konto\\Verwaltung\\Aliasse\\aliasse", alias);
		if(Filereader.readAlias("Konto\\Verwaltung\\Aliasse\\aliasse", alias) == false){
			System.out.println("Der Alias ist bereits vergeben!");
			
		}
		else {
			
			Filewriter.writer("Konto\\Verwaltung\\Aliasse\\aliasse", alias);
			
		}

Zum Hintergrund. Ich schreibe bei einer neuen Kontoerstellung den Alias in das textfile: aliasse.txt . Sollte nun ein neues Konto mit dem gleichen Alias erstellt werden soll dies mit:

System.out.println("Der Alias ist bereits vergeben!");

ausgeben werden.

Nur klappt das leider einfach nicht. und ich bekomme eine NullPointException. könnt ihr mir weiterhelfen?es ging schon so weit dass er den alias bei erstellung in das file geschrieben hat. jedoch bei einer neuen anlegung eines neuen Kontos konnte der Namen wieder gewählt werden.


LG,

Sensewell

 

[Der Müde Joe]

while (checkAlias.readLine() != null) { 

//sollte

while( (checker = buffReader.readLine()) !=  null ) {

//sein

EDIT:
sonst liest du ja 2 Zeilen pro durchgang

 

[Sensewell]

Es funktioniert trotzdem nicht. Immernoch:

Bitte wählen Sie einen Alias(Dieser ist für den Login erforderlich) : test123
Exception in thread "main" java.lang.NullPointerException
at Filereader.readAlias(Filereader.java:30)
at Verwaltung.setData(Verwaltung.java:134)
at Verwaltung.showKonten(Verwaltung.java:96)
at Verwaltung.showInstitut(Verwaltung.java:76)
at Verwaltung.main(Verwaltung.java:243)

 

[Der Müde Joe]

hmm....hast aber Zeile 8 nicht gelöscht...oder?

sonst liest du 2 Zeilen pro durchgang

 

[Sensewell]

An Zeile 8 lag es wohl. nur leider schreibt er mir jetzt beim 2. mal den neuen alias über den alten. das sollte nicht sein. Sondern in die näcste zeile den neuen.

LG,

und danke für deine Hilfe!

 

[Sensewell]

Ich adde mal meinen filewriter hinzu den ich benutze wenn ich ein 2. Alias in die Datei schreibe:

	static void writerNewLine(String dateiname, String text) throws IOException {
		BufferedWriter neuerWriter = new BufferedWriter(new FileWriter(
				dateiname + ".txt"));
		neuerWriter.newLine();
		neuerWriter.write(text);
		neuerWriter.close();

	}

}

wenn ich diesen nun aufrufe und den neuen Alias dazuschreiben will dann überschreibt er mir den alten Alias.bzw geht in die 2. Zeile und schreibt den neuen rein aber der alte ist nicht mehr da.


LG

 

[Der Müde Joe]

mfg

import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;

public class X {

	public static boolean hasAliasInFile(File file, String alias)
			throws IOException {
		BufferedReader reader = new BufferedReader(new FileReader(file));
		String checker = null;
		while ((checker = reader.readLine()) != null) {
			if (checker.equals(alias)) {
				return true;
			}
		}
		return false;
	}

	public static void appendToFile(File file, String text) throws IOException {
		String lineseperator = System.getProperty("line.separator");
		FileWriter fstream = new FileWriter(file, true);
		BufferedWriter out = new BufferedWriter(fstream);
		// newline
		out.write(lineseperator);
		out.write(text);
		out.close();
	}

	public static void main(String[] args) throws IOException {
		File file = new File("test.txt");

		System.out.println(X.hasAliasInFile(file, "test"));
		X.appendToFile(file, "abc");
	}
}

EDIT:
hab noch ein wenig geändert...hasAliasInFile gibt true zutück, wenn es existiert!
EDIT2:
eigentlich sollte man der Reader auch schliessen. also noch reader.close() vor beide return

 

[Sensewell]

also ich habs genauso gemacht wie bei dir: nur er schreibt trotzdem keine neue Zeile: nun mal der gesamte code:

static void writerNewLine(String dateiname, String text) throws IOException {
		String lineseperator = System.getProperty("line.separator"); 
		BufferedWriter neuerWriter = new BufferedWriter(new FileWriter(
				dateiname + ".txt"));
		neuerWriter.write(lineseperator);
	    neuerWriter.write(text);
	    neuerWriter.close();
		
	}

public static boolean readAlias(String dateiname,String alias)
			throws IOException {
		BufferedReader checkAlias = new BufferedReader(new FileReader(dateiname
				+ ".txt"));
		String checker = null;
		 while ((checker = checkAlias.readLine()) != null) {
		    if (checker.equals(alias)) {
		       return true;
		    }
		 }
		 return false;
		} 
	}

ausführender Code:

boolean checkAlias = false;
		String alias= null;
		while (!checkAlias){
		alias = IOTools
				.readString("Bitte wählen Sie einen Alias(Dieser ist für den Login erforderlich) : ");
		
		
		
		if(Filereader.readAlias("Konto\\Verwaltung\\Aliasse\\aliasse", alias) == true){
			System.out.println("Der Alias ist bereits vergeben!");
			checkAlias = false;
		}
		else {
			
			Filewriter.writerNewLine("Konto\\Verwaltung\\Aliasse\\aliasse", alias);
			checkAlias =true;
		}
		}

Datei Inhalt nach 2. Alias setzen wieder gelöscht.....

Ich verzweifle...

 

[Der Müde Joe]

Diese Zeilen sind wichtig:

		FileWriter fstream = new FileWriter(file, true);
		BufferedWriter out = new BufferedWriter(fstream);

so wird ans File appendet und nicht überschrieben

Ein kleines Beispiel:

import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.util.Scanner;

public class X {

	public static boolean hasAliasInFile(File file, String alias)
			throws IOException {
		BufferedReader reader = new BufferedReader(new FileReader(file));
		String checker = null;
		while ((checker = reader.readLine()) != null) {
			if (checker.equals(alias)) {
				reader.close();
				return true;
			}
		}
		reader.close();
		return false;
	}

	public static void appendToFile(File file, String text) throws IOException {
		String lineseperator = System.getProperty("line.separator");
		//WICHTIG!! true = append 
		FileWriter fstream = new FileWriter(file, true);
		BufferedWriter out = new BufferedWriter(fstream);
		// newline
		out.write(lineseperator);
		out.write(text);
		out.close();
	}

	public static void main(String[] args) throws IOException {
		File file = new File("test.txt");
		file.createNewFile();
		
		Scanner scanner = new Scanner(System.in);
		
		while(true){
			System.out.println("Wähle alias: ");
			String alias = scanner.nextLine();
			
			if(X.hasAliasInFile(file, alias)) {
				System.out.println("Sorry. wähle anderen");
				continue;
			} else {
				X.appendToFile(file, alias);
			}
			
		}
	}
}

 

[Sensewell]

Super hat geklappt. vielen Dank