Integer wird falsch in Datei geschrieben

[12f6we448]

Hi Leute,
ich möchte mir eine Kundenkartei anlegen.

D.h. Name, Adresse und Telefonnummer werden über die Konsole eingegeben (String) und in einer .txt gespeichert.
Die Kundennummer wird automatisch generiert.

Allerdings wird die Kundennummer (Integer) nicht richtig in die Dartei geschrieben. Aber warum?

Eingabe:
testname
01905666666
testadresse 88

In der Datei steht:

testname
01905666666
testadresse 88

Da wo das komische Zeichen steht müsste die Kundennummer (in diesem Fall die 1 stehen).

Hier mein Code:

package paket;

import java.io.*;

class Kunde {

	public static String getName() {
		return name;
	}

	public static void setName(String name) {
		Kunde.name = name;
	}

	public static String getTelefonnummer() {
		return telefonnummer;
	}

	public static void setTelefonnummer(String telefonnummer) {
		Kunde.telefonnummer = telefonnummer;
	}

	public static String getAdresse() {
		return adresse;
	}

	public static void setAdresse(String adresse) {
		Kunde.adresse = adresse;
	}

	public static int getKundennummer() {
		return kundennummer;
	}

	public static void setKundennummer(int kundennummer) {
		Kunde.kundennummer = kundennummer;
	}

	private static String name;
	private static String telefonnummer;
	private static String adresse;
	private static int kundennummer = 1;

	public Kunde(String kname, String ktelefonnummer, String kadresse, int kkundennummer) {
		name = kname;
		telefonnummer = ktelefonnummer;
		adresse = kadresse;
		kundennummer = kkundennummer;
	}
	
	public static int vergebeKundennummer() {
		kundennummer ++;		
		return kundennummer;
	}
}

public class Kundenverwaltung {
	
	public static String stringEingabe() {
		
		String strInput = null;

		BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
		try {
			strInput = br.readLine();
		} catch (IOException e) {
			e.printStackTrace();
		}
		
		return strInput;
		
	}
	
	public static Kunde kundeAnlegen () {
		System.out.print("Name:          ");
		String name = stringEingabe();
		System.out.print("Telefonnummer: ");
		String telefonnummer = stringEingabe();
		System.out.print("Adresse:       ");
		String adresse = stringEingabe();
		int kundennummer = Kunde.vergebeKundennummer();
		
		Kunde neuerKunde = new Kunde(name, adresse, telefonnummer, kundennummer);
		
		return neuerKunde;
	}
	
	public static void schreibeInDatei() throws IOException {
		FileWriter writer;
		File file;
		
		try {
			file = new File("Kundenkartei.txt");
			writer = new FileWriter(file, true);
			writer.write(Kunde.getKundennummer());
			writer.write(System.getProperty("line.separator"));
			writer.write(Kunde.getName());
			writer.write(System.getProperty("line.separator"));
			writer.write(Kunde.getAdresse());
			writer.write(System.getProperty("line.separator"));
			writer.write(Kunde.getTelefonnummer());
			writer.write(System.getProperty("line.separator"));
			writer.write(System.getProperty("line.separator"));
			writer.flush();
			writer.close();
		} catch (IOException e) {
			e.printStackTrace();
		}
		
	}

	public static void main(String[] args) throws IOException {

		while (true) {
			System.out.println("[1] Kunde anlegen und Kundendaten speichern");
			System.out.println("[2] Kundendaten auslesen");
			System.out.println("[3] Kundendaten verändern");
			System.out.println("[4] Kunde löschen");
			System.out.println("[0] Programm beenden");
			System.out.print("Bitte wählen Sie die Operation aus: ");
			
			String strInput = null;
			int operation = 0;

			BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
			try {
				strInput = br.readLine();
			} catch (IOException e) {
				e.printStackTrace();
			}

			operation = Integer.parseInt(strInput);
			
			switch (operation) {
			case 0: System.exit(0);
					break;
			case 1:	kundeAnlegen();
					schreibeInDatei();
					break;
			case 2:	
					break;
			case 3:	
					break;
			case 4: 
					break;
			default:	break;			
			}
		}
	}
}

Ich hoffe ihr könnt mir helfen.
DANKE!

 

[Simon_Flagg]

schau dir mal die doku zu filewriter an, da gibts zwar eine methode write(int), aber die schreibt das zeichen, dessen asci wert du übergibst --> schreibe write(Kunde.getKundenNummer() + ""); --> du wandelst den int in string um --> den er dann ausgibt

lg

 

[nrg]

der unterschied zwischen statisch und nicht-statisch ist dir bewusst? jetzt mal abgesehn von der datei ausgabe, hat ein programm einige andere gravierende konzeptionsfehler

 

[12f6we448]

schau dir mal die doku zu filewriter an, da gibts zwar eine methode write(int), aber die schreibt das zeichen, dessen asci wert du übergibst --> schreibe write(Kunde.getKundenNummer() + ""); --> du wandelst den int in string um --> den er dann ausgibt

lg

Wie kann ich denn den int-Wert in eine Datei schreiben?

der unterschied zwischen statisch und nicht-statisch ist dir bewusst? jetzt mal abgesehn von der datei ausgabe, hat ein programm einige andere gravierende konzeptionsfehler

lg

ich denke mal du willst auf die getter und setter methoden anspielen, oder?
static methoden werden beim aufrufen der klasse geladen
lasse ich das static weg, meckert der compiler.
"Cannot make a static reference to the non-static method getAdresse() from the type Kunde"

 

[Final_Striker]

Lege mal einfach in deiner main-Methode zwei unterschiedliche Kundenobjekte an und gib ihre Eigenschaften aus. Dann wirst du sehen wo dein Fehler liegt.

 

[Simon_Flagg]

int wert schreiben hab ich dir gerade aufgeschrieben write (Kunde.getKundenNummer() + "");

und ich glaube du hast das mit static noch nicht ganz verstanden, hat sich wie eine auswendig gelernte erklärung angeschaut...

wenn eine methode static ist, dann gibt es sie pro klasse nur 1x, die gehört zur klasse, du kannst mit Klassenname.methodenname() auf sie zugreifen

ist die methode jedoch nicht static, dann gehört sie zum objekt, jedes objekt hat eine, auf die kannst du mit objektname.methodenname() zugreifen, dieses objekt musst du aber vorher mit new erzeugen:
Klassenname objektname = new Klassenname();

dann kannst du die methode aufrufen.

deine kundenkartei ist imho falsch konzeptioniert, als sie nur EINEN kunden haben kann. --> du musst alle methode etc. auf non-static setzen (static weglassen) und pro kunde ein objekt erzeugen

außerdem würde ich den methoden eingabe und infileschreiben ein objekt mitgeben, in das /von dem sie ihre daten schreiben/bekommen

lg

 

[12f6we448]

OK, also nochmal von vorne.
So sieht es momentan aus.

package paket;

import java.io.*;

class Kunde2 {
	
	public int getKundennummer() {
		return kundennummer;
	}

	public void setKundennummer(int kundennummer) {
		this.kundennummer = kundennummer;
	}

	public String getName() {
		return name;
	}

	public void setName(String name) {
		this.name = name;
	}

	public String getTelefonnummer() {
		return telefonnummer;
	}

	public void setTelefonnummer(String telefonnummer) {
		this.telefonnummer = telefonnummer;
	}

	public String getAdresse() {
		return adresse;
	}

	public void setAdresse(String adresse) {
		this.adresse = adresse;
	}

	private int kundennummer = 1;
	private String name, telefonnummer, adresse;
	
	public Kunde2 (int kKundennummer, String kName, String kTelefonnummer, String kAdresse) {
		kundennummer = kKundennummer;
		name = kName;
		telefonnummer = kTelefonnummer;
		adresse = kAdresse;
	}
	
	public String stringEingabe() {
				
		String strInput = null;
		BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
		
		try {
			strInput = br.readLine();
		} catch (IOException e) {
			e.printStackTrace();
		}
		
		return strInput;
		
	}
	
	public int vergebeKundennummer() {
		kundennummer ++;		
		return kundennummer;
	}
	
}

public class Kundenverwaltung2 {

	public static void main(String[] args) throws IOException {
		
		System.out.print("Name:         ");
		System.out.print("Adresse:      ");
		System.out.print("Telfonnummer: ");
		
	}
}

Wie kann ich nun in der main-Methode ein neues Objekt (Kunde2) erzeugen, dessen Atribute per Tastatureingabe in der Konsole eingelesen werden sollen?

Mit

Kunde2 neuerKunde = new Kunde2 (Kunde2.vergebeKundennummer(), Kunde2.stringEingabe(), Kunde2.stringEingabe(), Kunde2.stringEingabe());

wäre es ja möglich, aber dann müssten alle Methoden und Variablen ja wieder als static initialisiert werden.

Wie sieht die Alternative aus?

 

[Simon_Flagg]

du könntest einen konstruktor ohne parameter machen, in denen diese methoden aufgerufen werden

lg

 

[jonius]

Wenn es um das Speichern von Wertepaaren wie in deinem Fall geht, bietet sich auch die Klasse Properties an:
Referenz: java.util.Properties
Sie speichert sehr einfach Wertepaare. Allerdings als Strings. Da müsstest du auch Integer und Strings ineinander umwandeln:
String zu int: Integer.parseInt(String xyz)
int zu String: Integer.toString(int xyz)

 

[Final_Striker]

Wie sieht die Alternative aus?

Die Methode stringEingabe() hat nichts in der Klasse Kunde zu suchen.

 

[jonius]

Das ganze könnte so aussehen:

package paket;
 
import java.io.*;
 
class Kunde2 {
    
    public int getKundennummer() {
        return kundennummer;
    }
 
    public void setKundennummer(int kundennummer) {
        this.kundennummer = kundennummer;
    }
 
    public String getName() {
        return name;
    }
 
    public void setName(String name) {
        this.name = name;
    }
 
    public String getTelefonnummer() {
        return telefonnummer;
    }
 
    public void setTelefonnummer(String telefonnummer) {
        this.telefonnummer = telefonnummer;
    }
 
    public String getAdresse() {
        return adresse;
    }
 
    public void setAdresse(String adresse) {
        this.adresse = adresse;
    }
    // diese statische Variable ist unabhängig von den einzelnen Kundenobjekten. Sie existiert also genau einmal.
    private static int kundenzaehler=1; 

    // diese nichtstatische Variable gehört jeweils zu einem Objekt Kunde2
    private int kundennummer;
    private String name, telefonnummer, adresse;

// das hier ist schlecht    
//    public Kunde2 () {
//        kundennummer = Kunde2.vergebeKundennummer;
//        name = Kundenverwaltung2.stringEingabe("Name");
//        telefonnummer = Kundenverwaltung2.stringEingabe("Telefonnummer");
//        adresse = Kundenverwaltung2.stringEingabe("Adresse");
//    }
    public void gibEigenschaftenAus() {
        System.out.print("Name:         "+getName());
        System.out.print("Adresse:      "+getAdresse());
        System.out.print("Telfonnummer: "+getTelefonnummer()); 
        System.out.print("Kundennummer: "+getKundennummer());
    }

    public void schreibeInDatei() throws IOException {
        FileWriter writer;
        File file;
        
        try {
            file = new File("Kundenkartei.txt");
            writer = new FileWriter(file, true);
            writer.write(Integer.toString(Kunde.getKundennummer()));
            writer.write(System.getProperty("line.separator"));
            writer.write(Kunde.getName());
            writer.write(System.getProperty("line.separator"));
            writer.write(Kunde.getAdresse());
            writer.write(System.getProperty("line.separator"));
            writer.write(Kunde.getTelefonnummer());
            writer.write(System.getProperty("line.separator"));
            writer.write(System.getProperty("line.separator"));
            writer.flush();
            writer.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
        
    }

    // Die aktuelle Kundennummer wird zurückgegeben und dann wird der Zaehler eins hoch gesetzt
    public static int vergebeKundennummer() {       
        return kundenzaehler++;
    }
    
}
 
public class Kundenverwaltung2 {
 
    public static void main(String[] args) throws IOException {
        Kunde2 kunde=erstelleKunde();
        kunde.gibEigenschaftenAus();
    }
    public static Kunde2 erstelleKunde() {
        Kunde2 kunde=new Kunde2();
        kunde.setKundennummer(Kunde2.vergebeKundennummer);
        kunde.setName(Kundenverwaltung2.stringEingabe("Name"));
        kunde.setTelefonnummer(Kundenverwaltung2.stringEingabe("Telefonnummer"));
        kunde.setAdresse(Kundenverwaltung2.stringEingabe("Adresse"));
        return kunde;
    }
    public static String stringEingabe(String ausgabe) {
        System.out.print(ausgabe+": ");       
        String strInput = null;
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        
        try {
            strInput = br.readLine();
        } catch (IOException e) {
            e.printStackTrace();
        }
        
        return strInput;
        
    }
}

Ich habe das im Forum geschrieben und bin mir nicht sicher, ob Fehler drin sind. Aber ich hoffe, die Verwendung von statisch und nicht-statisch, sowie des Konstruktors wird klar.
In deinem Beispielen wurde entweder eine statische Kundennummer verwendet, was zur Folge hätte, dass alle immer die gleiche Nummer haben, oder du hast versucht, eine Kundennummer zu vergeben, die abhängig war von sich selbst.

 

[Simon_Flagg]

ich würde kundenummervergabe static machen --> hast du scheinbar vergessen, weil du über Kunde darauf zugreifst

lg

 

[Final_Striker]

    public Kunde2 () {
        kundennummer = Kunde2.vergebeKundennummer;
        name = Kundenverwaltung2.stringEingabe("Name");
        telefonnummer = Kundenverwaltung2.stringEingabe("Telefonnummer");
        adresse = Kundenverwaltung2.stringEingabe("Adresse");
    }

Ein Kunde weiß nicht von einer Kundenverwaltung. Aus diesem Grund ist sowas ein sehr, sehr, sehr schlechter Programmierstil.

Die Eingabe gehört in die Kundenverwaltung. Dort wird aus den Eingaben des User eine neues Kundenobjekt erstellt.

 

[jonius]

Ich habe den Code etwas verbessert. Natürlich könnte man auch einen Konstruktor verwenden, der die wichtigsten Informationen erwartet, um sicher zu gehen, dass diese auch wirklich initialisiert werden. Aber es ging ja auch nur um das Grundsätzlich mit dem Statischen und nicht-Statischen. Außerdem könnte man das Kundenzählen der Kundenverwaltung überlassen. Am Besten hätte man ein Objekt Kundenverwaltung, dass die Kunden und die Kundenanzahl verwaltet und Nummer vergibt...

An 12f6we448: Ich hoffe, dein Problem ist damit gelöst.