Arraylist wird überschrieben?

[java01]

Hallo Leute,

bin neu in Java und bin bis jetzt ganz gut klar gekommen, aber ich finde das Problem einfach nicht. Ich weiß grundlegend wie ArrayList funktioniert aber ich komme nicht auf meinen Fehler. Das Problem ist, es werden zwar 2 oder mehr Einträge angelegt, aber der Inhalt ist immer die Letzte Eingabe. Wie geht denn sowas? Ich wäre für Hilfe sehr dankbar.

import java.util.ArrayList;
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class Telefonliste {
 
    private ArrayList<TelefonEintrag> liste;
 
    private static BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
    private TelefonEintrag Telefon;
    
    // 2 Methoden um einen Antrag einzulegen
    // Die Methode anlegen wird ueberladen
    public static void main(String[] args) throws IOException {
     new Telefonliste();
    }   
    
    public Telefonliste() throws IOException {
        liste = new ArrayList<TelefonEintrag>();
        menue();
    }

    public void menue() throws IOException{
      	 System.out.println("Willkommen zu Ihrem persönlichen Telefonbuch");
    	 System.out.println("Was möchten sie heute tun?");
    	 System.out.println("");
    	 System.out.println ("Bitte wählen Sie eine Nummer:,\n" 
 				+ "1. Neuen Eintrag anlegen\n"
 				+ "2. Einen Eintrag löschen,\n" 
 				+ "3. Einen Eintrag ändern\n"
 				+ "4. Nach einer Person suchen\n"
 				+ "5. Eine Telefonnummer identifizieren");
    	 int auswahlInt;
    	 do {
             auswahlInt = Telefonliste.getInputInt("\n  Bitte geben Sie eine Nummer ein: ");
             switch (auswahlInt) {
             case 1:
            	 anlegen();
                 break;
             case 2:
                 loeschen();
                 break;
     System.out.println("");
             System.out.println("\n Was möchten sie nun tun?");
         } while (auswahlInt != 4 && auswahlInt != 0);
    	
    	}
    
    public void anlegen () {
    String name1 = "Erster";
    String name2 = "Zweiter";
    String telefonnummer1 = "11111111";
    String telefonnummer2 = "22132123";
    TelefonEintrag eintrag1 = new TelefonEintrag(name1, telefonnummer1);
    TelefonEintrag eintrag2 = new TelefonEintrag(name2, telefonnummer2);
    liste.add(eintrag1);
    liste.add(eintrag2);
     
    }
 public void loeschen (){
    	for (TelefonEintrag inhalt : liste) {
    		System.out.println(inhalt.Output());
    }
    }

}

Viele Grüße,

Michaela

 

[0din]

ich hab das ganze ma fix zusammen geschustert, so das ichs laufn lassn kann...
bei mir zickt da nix rum... du hast nur n ziemliches chaos bei deinem menü...

zum einen sin die beiden prints nich zu erreichen, dir fehlen n paar klammern zum schließen... so kleinigkeiten...

die flickschusterei; ne endlosschleife die anlegen aufruft un alle einträge printet... da siehste das es alle einträge sin... un etwas hübsch gemacht... ^^

import java.util.ArrayList;
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class Telefonliste
{

	private ArrayList<TelefonEintrag>	liste;

	private static BufferedReader		reader	= new BufferedReader(new InputStreamReader(System.in));
	private TelefonEintrag				Telefon;

	// 2 Methoden um einen Antrag einzulegen
	// Die Methode anlegen wird ueberladen
	public static void main(String[] args)
	{
		try
		{
			new Telefonliste();
		}
		catch (IOException e)
		{
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	}

	public Telefonliste() throws IOException
	{
		liste = new ArrayList<TelefonEintrag>();
		menue();
	}

	public void menue() throws IOException{
         System.out.println("Willkommen zu Ihrem persönlichen Telefonbuch");
         System.out.println("Was möchten sie heute tun?");
         System.out.println("");
         System.out.println ("Bitte wählen Sie eine Nummer:,\n" 
                + "1. Neuen Eintrag anlegen\n"
                + "2. Einen Eintrag löschen,\n" 
                + "3. Einen Eintrag ändern\n"
                + "4. Nach einer Person suchen\n"
                + "5. Eine Telefonnummer identifizieren");
         int auswahlInt;
         do {
             		auswahlInt = Telefonliste.getInputInt("\n  Bitte geben Sie eine Nummer ein: ");
             		switch (auswahlInt) 
             		{
             		case 1:
             			anlegen();
             			break;
             		case 2:
             			loeschen();
             			break;
                 	
             		} 
         	}while (auswahlInt != 4 && auswahlInt != 0);
        
        }

	private static int getInputInt(String string)
	{
		// TODO Auto-generated method stub
		return 1;
	}

	public void anlegen()
	{
		String name1 = "Erster";
		String name2 = "Zweiter";
		String telefonnummer1 = "11111111";
		String telefonnummer2 = "22132123";
		TelefonEintrag eintrag1 = new TelefonEintrag(name1, telefonnummer1);
		TelefonEintrag eintrag2 = new TelefonEintrag(name2, telefonnummer2);
		liste.add(eintrag1);
		liste.add(eintrag2);
		
		for(TelefonEintrag inahlt : liste)
		{
			System.out.println(inahlt);
		}

	}

	public void loeschen()
	{
		for (TelefonEintrag inhalt : liste)
		{
			System.out.println(inhalt.Output());
		}
	}

}

PS:
ich hab der TelefonEintrag ne toString verpasst mit der ich name / nummer ausgebn kann

 

[java01]

Hi Odin,

danke für deine schnelle Hilfe. Ich wundere mich bloß weil bei meiner Ausgabe kommt immer nur das gleich raus, ich poste mal noch den anderen java code zusammen mit dem neuen Code. Wenn ich im Menü 2 eingebe kommt zweimal der Eintrag 2 raus. Woran liegt das nur?

import java.util.ArrayList;
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
 
public class Telefonliste
{
 
    private ArrayList<TelefonEintrag>   liste;
 
    private static BufferedReader       reader  = new BufferedReader(new InputStreamReader(System.in));
    private TelefonEintrag              Telefon;
 
    // 2 Methoden um einen Antrag einzulegen
    // Die Methode anlegen wird ueberladen
    public static void main(String[] args)
    {
        try
        {
            new Telefonliste();
        }
        catch (IOException e)
        {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
    }
 
    public Telefonliste() throws IOException
    {
        liste = new ArrayList<TelefonEintrag>();
        menue();
    }
 
    public void menue() throws IOException{
         System.out.println("Willkommen zu Ihrem persönlichen Telefonbuch");
         System.out.println("Was möchten sie heute tun?");
         System.out.println("");
         System.out.println ("Bitte wählen Sie eine Nummer:,\n" 
                + "1. Neuen Eintrag anlegen\n"
                + "2. Einen Eintrag löschen,\n" 
                + "3. Einen Eintrag ändern\n"
                + "4. Nach einer Person suchen\n"
                + "5. Eine Telefonnummer identifizieren");
         int auswahlInt;
         do {
                    auswahlInt = Telefonliste.getInputInt("\n  Bitte geben Sie eine Nummer ein: ");
                    switch (auswahlInt) 
                    {
                    case 1:
                        anlegen();
                        break;
                    case 2:
                        loeschen();
                        break;
                    
                    } 
            }while (auswahlInt != 4 && auswahlInt != 0);
        
        }
 
    private static int getInputInt(String text)
    {
        // TODO Auto-generated method stub
        while (true) {
            System.out.print(text);
            try {
                return Integer.parseInt(reader.readLine());
            } catch (Exception e) {
                System.err.println("Fehler bei der Verarbeitung: "
                        + e.getLocalizedMessage());
            }
        }
    }
 
    public void anlegen()
    {
        String name1 = "Erster";
        String name2 = "Zweiter";
        String telefonnummer1 = "11111111";
        String telefonnummer2 = "22132123";
        TelefonEintrag eintrag1 = new TelefonEintrag(name1, telefonnummer1);
        TelefonEintrag eintrag2 = new TelefonEintrag(name2, telefonnummer2);
        liste.add(eintrag1);
        liste.add(eintrag2);
        
        for(TelefonEintrag inhalt : liste)
        {
            System.out.println(inhalt);
        }
 
    }
 
    public void loeschen()
    {
        for (TelefonEintrag inhalt : liste)
        {
            System.out.println(inhalt.Output());
        }
    }
 
}

import java.util.ArrayList;
import java.util.List;
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;



public class TelefonEintrag {

	//Attribute
	
	private static String name;
	private static String telefonnummer;
	private static BufferedReader eingabe = new BufferedReader(new InputStreamReader(System.in));
	//private ArrayList<TelefonEintrag> eintrag;
	
	//Konstruktor
	public TelefonEintrag (String name, String telefonnummer){
		this.name = name;
		this.telefonnummer = telefonnummer;
		//eintrag = new ArrayList<TelefonEintrag>();
	}
	
	//Methoden
	
	public static String nameEingeben (){
		while (true) {
            try {
                return name = eingabe.readLine();
            } catch (Exception e) {
                System.err.println("Fehler bei der Verarbeitung: "
                        + e.getLocalizedMessage());
            }
        }
	}
	
	public static String telefonnummerEingeben (){
		while (true) {
            try {
                return telefonnummer = eingabe.readLine();
            } catch (Exception e) {
                System.err.println("Fehler bei der Verarbeitung: "
                        + e.getLocalizedMessage());
            }
        }
	}
	public static String Output(){
        return "Name: " + name + " Tel: " + telefonnummer;
    }
	
	public static String getName(){
        return name;
    }

	public static String getNummer(){
        return telefonnummer;
    }
}

Viele Grüße,

Michaela

 

[Final_Striker]

Mach das static in den Attributen von TelefonEintrag weg. ;-)

 

[java01]

Ja das ist es. Weiß garnicht mehr warum ich das da reingemacht habe.

Vielen Dank euch beiden,

bis später

Michaela

 

[0din]

dann weiß ich nu au warum meins so anstandslos getan hat ^^ ich setz solche sachn nur bei singleton oder vllt no daos static...

der zweite code is au n ganzes stück schöner zu lesen :toll:

im übrigen,
static nimmste eig sehr selten, grade am anfang... also die methoden etc. brauchn au net unbedingt static zu sein.
das is häufig n fehler den man am anfang macht weil der compiler meckert das man aus der static main methode nich auf die anderen nich static sachn zugreifn kann...