Problem mit Listen

[Der Programmierer]

Hi leutz,

ich hab in meinem Programm ein paar Arrays auf Listen umgesattelt, weil es einfach praktischer ist vor allem wegen der Löschmethoden etc.
Allerdings gibt es dabei noch den ein oder anderen Fehler, der nicht gefunden werden will:

			for(int i = 0; i<anzahl_klassen; i++)
			{
				klassen.name = lnr.readLine();
				liste.add(klassen);
				short_information = short_information + klassen.name + "\n";

			}

das funktioniert wunderbar. In dem String short_information ist genau das drin was ich erwartet hab. Nämlich unterschiedliche Klassennamen:

			for(int i = 0; i<anzahl_klassen; i++)
			{
				klassen = liste.get(i);
				buff.write(klassen.name);
				buff.newLine();	
			}
	
			buff.close();

Hier jedoch gibt das Programm in meinem Textdokument nur das erste Element aus. Zwar genau in der selben Anzahl wie er soll. Also ich hab in meinem Fall 8 Klassen und dann gibt er statt den 8 untersch. Klassennamen 8 mal den selben aus -.-

Das versteh ich nicht ganz ^^

Schonmal danke für eure Hilfe
Der Programmierer

 

[SnooP]

Der Code-Ausschnitt ist zu klein, um da was genaues sagen zu können... liefert denn auch ein system.out 8 mal die gleiche klasse? was sagt ein System.out der liste? vielleicht steht ja doch 8 mal die gleiche klasse in deiner liste drin?

übrigens iteriert sich's bei Collections per Iterator und ab Java5 schöner so:

for (Klasse klassen: liste) {
   buff.write(klassen.name);
   //...
}

nennt sich foreach-Schleife und nutzt den Iterator (über iterator()-Methode einer beliebigen Collection).

 

[Der Programmierer]

Hier ist nochn bisschen code. Aber sagt nichts über meinen Codestil ich hab mich nur mal ausgetobt!^^

public class pr extends JFrame implements ActionListener
{



	public JTextArea kurzinfo;
	public JButton   aktualis;
	public int anzahl_klassen;
	public JMenuBar  menueleiste;
	public String short_information = "Sie haben ";
	LinkedList<klasse> liste = new LinkedList<klasse>();
	public klasse klassen = new klasse();
	public String Sanzahl_klassen =null;

	public void write_klasse()
	{
		BufferedWriter buff=null;
		FileWriter fw=null;
		int aktuelle_anzahl = anzahl_klassen + 1;
		String Saktuelle_anzahl = String.valueOf(aktuelle_anzahl);


		 
		try
		{
			fw = new FileWriter("klassen.txt");
			buff = new BufferedWriter(fw);

			buff.write(Saktuelle_anzahl);
			buff.newLine();

			for(int i = 0; i<anzahl_klassen; i++)
			{
				klassen = liste.get(i);
				buff.write(klassen.name);
				buff.newLine();	
			}
	
			buff.close();



		}

		catch(Exception e)
		{
			System.out.println("TJA");
		}
	}

	public static void main(String[] args)
	{

		pr fenster = new pr("Schülerverwaltungssystem"); //Fenster einrichten
		fenster.pack();
		fenster.setSize(640,480);


		password test=new password(fenster);  //Passwortdialog aufrufen

		test.setLocation(150,150);
		test.setVisible(true);


		fenster.setResizable(false);
		fenster.setVisible(true);
		



		
	}




	public void actionPerformed(ActionEvent e)
	{	
		String label;
		label = e.getActionCommand();
		if(label.equals("Klasse hinzufügen"))
		{
			klasse_hinzufuegen neue_klasse;
			neue_klasse = new klasse_hinzufuegen(null, "Klasse hinzufügen");
			anzahl_klassen = anzahl_klassen;
			neue_klasse.setLocation(150,150);
			neue_klasse.setVisible(true);
			
			

			write_klasse();


			
		}

		if(label.equals("Windows"))
		{
		       	try
			 {
         		UIManager.setLookAndFeel("com.sun.java.swing.plaf.windows.WindowsLookAndFeel");

        		SwingUtilities.updateComponentTreeUI(this);
			}

			catch(Exception ef)
			{
						System.out.println("Tja");
			}
	        }

	}



           



		
	

	


	pr(String titel)
	{
		super(titel);
		setLayout(new BorderLayout());


		try
		{
         		UIManager.setLookAndFeel("com.sun.java.swing.plaf.windows.WindowsLookAndFeel");
        		SwingUtilities.updateComponentTreeUI(this);
		}

		catch(Exception ef)
		{
			System.out.println("Tja");
		}
	
		try
		{
			LineNumberReader lnr = new LineNumberReader(new FileReader("klassen.txt"));	
			Sanzahl_klassen = lnr.readLine(); 

			anzahl_klassen = Integer.parseInt(Sanzahl_klassen);
			short_information = short_information + Sanzahl_klassen;
			short_information = short_information + " Klassen: \n";

			


			


			for(int i = 0; i<anzahl_klassen; i++)
			{
				klassen.name = lnr.readLine();
				liste.add(klassen);
				short_information = short_information + klassen.name + "\n";

			} 

			LineNumberReader lnrI = new LineNumberReader(new FileReader("klassenstufe.txt"));
			
		
		}
		
		catch (Exception e)
		{
			e.printStackTrace();
		}

 

[Gast]

es ist alles ziemlich schwer zulesen, da du deine klassen mit kleinen Buchstaben beginnst

aber davon mal angesehen, du fügst einfach immer wieder das selbe objekt in die Liste ein

klassen.name = lnr.readLine();
            liste.add(klassen);

du hast die ganze zeit nur ein klassen objekt wovon du mehrmals den namen änderst

 

[Der Programmierer]

Ufff bin ich blöd.
Hast natürlich recht, ich schlauer habe das selbe element immer wieder in die liste getan ich blöder.

Trotzdem vielen dank für die schnelle Hilfe!