Collection mit ArrayList

[mix2k1]

Hallo,
ich habe eine ArrayList "meineListe" in die ich Array-Objekte "Zahlenfeld" legen will.

meineListe.add(durchlauf-1, zahlenfeld);

Wie kann ich jetzt wieder auf die Arrays zugreifen?
Mit

meineListe.get(i)

bekomme ich nur die Objektreferenz zurück.

 

[Tobias]

meineListe.get(i)[index];

Meinste das? Wenn nein, formulier deine Frage klarer...

mpG
Tobias

 

[mix2k1]

Ja Danke, das war es eigentlich. Und wie kann ich jetzt auch noch auf die "durchlauf-1" zugreifen?

 

[Tobias]

Wenn i == durchlauf-1 bekommst du das entsprechende Zahlenfeld - genauso wie du es in der ArrayList abgelegt hast. Willst du die durchlauf-1ste Zahl des i. Zahlenfelds abfragen, schreibst du

meineListe.get(i)[durchlauf-1]

Mit ein bißchen mehr Code und ein wenig mehr Info darüber, was um alles in der Welt du da eigentlich machst, muss ich nicht so im Trüben fischen und kann dir effektiver helfen - aber vielleicht reicht das ja schon...

mpG
Tobias

 

[mix2k1]

Also geht darum Lottozahlen zu ziehen und dann die Ziehungsnummer samt Array mit den pro Ziehung gezogenen Zahlen in eine ArrayList oder LinkedList zu packen.

Es soll dann über einen Button die komplette Liste in der Konsole und später in einen Textfeld ausgegeben werden.

1 2 43 23 22 12 13 44
.
.
.

Hier sind die Methoden für hinzufügen und auslesen der Liste:

	public void Add2List(int[]a){
	    int [] zahlenfeld = a;
	meineListe.add(durchlauf-1, zahlenfeld);

public void GetList(){
	Iterator <int[]> it = meineListe.iterator();
	for(int i=0;i<meineListe.size();i++){
	    System.out.print("Ziehung " + meineListe.get(i)[durchlauf-1]);
	    it.next();
	    //System.out.print("Ziehung " + (i+1) + ":");
                    for(int j=0;j<7;){
	    System.out.print("  " + meineListe.get(i)[j]);
	    j++;
                    }
	System.out.println();
                }

 

[Tobias]

Ah, ok.

1. Ist es Absicht, das deine Konsolenausgabe sieben Zahlen ausgibt (wenn man die führende 1 als Nummer der Ziehung interpretiert)? Ich denke hier an 6 aus 49 plus Superzahl, die Frage ist also ernstgemeint.

2. durchlauf ist eine Instanzvariable? Oder bekommst du schon Compilerfehler, weil durchlauf nicht deklariert wurde?

3. Was nützt dir ein Iterator, wenn du in dann doch nicht benutzt?

4. Die Nummer deiner Ziehung wird NICHT in der ArrayList gespeichert. Stell dir die ArrayList als eine Reihe von Kisten vor, in die du deine Werte ablegst. Als Konvention hast du und der "Lagerarbeiter", der dir die Kisteninhalte auf Anforderung holt, vereinbart, das mit der Zählung der Kisten immer von links begonnen wird und neue Werte immer in die erste nie belegte Kiste oder in die Kiste, die du ansagst, gelegt wird. Wenn du also drei Zahlenfelder in die (bislang leeren) Kisten legst und dementsprechend drei mal sagst "Lege diesen Wert in die nächste freie Kiste!", kannst du dir sicher sein, das das erste Feld in die erste Kiste von links kommt, das zweite in die zweite Kiste usw.

Moral der Geschicht': Deine auslesen-Methode sollte so aussehen:

public void GetList(){
    // Sehr gut, der richtige Gebrauch von Iteratoren ist wichtig für Java!
    Iterator <int[]> it = meineListe.iterator();
    // Wenn wir einen Iterator nutzen, soll auch dieser entscheiden 
    // dürfen, wann er keine Werte mehr liefert. Die Variable i 
    // brauchen wir trotzdem.
    for(int i=0; it.hasNext(); i++){
        // Kisten und Konventionen und so ...
        System.out.print(i + ". Ziehung: ");
        // Iterator arbeiten lassen
        int[] zahlenfeld = it.next();
        
        // zahlenfeld.length ist flexibler und leichter verständlich
        // Das j++ in den Schleifenkopf und nicht in den Rumpf 
        // zu schreiben ebenso!
        for(int j=0; j<zahlenfeld.length; j++){
            // Wenn ich das Zahlenfeld schon habe, sollte ich es
            // auch verwenden.
            System.out.print("  " + zahlenfeld[j]);
        }
        System.out.println();
    } // Fehlende Klammer ergänzt

}

mpG
Tobias

P.S.: Wo ich schon von Konventionen und so erzähle: Methodennamen schreibt man klein und in CamelCase. Methodennamen, die mit get beginnen, werden per Konvention benutzt, um Referenzen auf Instanzvariablen zurück zu liefern, eine get-Methode mit void-Rückgabetyp ist also ein klarer Verstoß gegen diese Konvention. Da du darüberhinaus eine Ausgabe auf die Kommandozeile machst, solltest du überlegen, ob du die Methode nicht lieber printList() oder ähnlich nennst.

 

[mix2k1]

Danke, so funktionierts.

1. Ja, das ist mein Absicht.

2. Es ist eine Istanzvariable, die pro Methodenaufruf "NeueZiehung()" 1 hochzählt. Einen Compilefehler hab ich njet.

3. Ich wußte nicht genau wo ich Ihn wie einsetzte.

4. Ich sehe grad, das Durchlauf auch garnicht gespeichert werden soll. Wäre ja dann eh doppelt, weil der Index der Liste ja quasi der Durchlauf ist


Aber irgendwie gibt er mir jede Liste doppelt aus. Aber bekomm ich noch raus.

Eine Frage hab ich noch:
Wie kann ich Objektnamen zusammensetzen in einer FOR-Schleife?


So hatte ich gedacht

for(int i=1; i<=4;i++){
JPanel [myPanel[i]] = new JPanel();
}

sollte dieses hier ersetzen:

JPanel myPanel1 = new JPanel();	
			JPanel myPanel2 = new JPanel();	
			JPanel myPanel3 = new JPanel();	
			JPanel myPanel4 = new JPanel();

 

[Tobias]

Sowas wie "variable Variablennamen" gibt es nicht in Java. Du kannst ein Array benutzen (wenn du im Vorraus weißt, wieviele Variablen du brauchst) oder eine Collection wie die von dir ja schon eingesetzte ArrayList.

mpG
Tobias

 

[mix2k1]

Habs jetzt mit nem Array:

			JPanel [] myPanels = new JPanel[4];
			{
			for(int i=0;i<myPanels.length;i++){
				myPanels[i] = new JPanel();}

 

[Tobias]

Sieht gut aus. Wenn alles läuft, Thread abhaken nicht vergessen.

mpG
Tobias

 

[mix2k1]

Ich wollte jetzt noch das ganze in einem Textfeld ausgeben lassen.

		JTextField meinText = new JTextField(30);
		JScrollPane scrollfeld = new JScrollPane(meinText);

Mit

meinText.setText();

ich nur Strings schreiben.

Wie bekomme ich nun meine Collection in das Textfeld
Und wie mache ich einen Zeilenumbruch?

 

[Tobias]

Textfelder können nur einzeiligen Text ausgeben, für mehrzeiligen mußt du TextArea verwenden.

Ein Zeilenumbruch bekommst du, indem du die Steuerzeichensequenz \n für newline verwendest.

Am einfachsten kannst du ein int[] in einen String umwandeln, in dem du die Methode toString() der Utility-Klasse Arrays verwendest. Etwa so:

import java.util.Arrays;

class Foo {

    private int[] meinArray = new int[6] { 1, 2, 3, 4, 5, 6};

    public void ausgeben {
        String meinString = Arrays.toString(meinArray);
        // weiteres hier ...
    }

}

Oder du schreibst dir eine eigene Methode zum Umwandeln, die den Array iteriert und die einzelnen Elemente einem String hinzufügst. Hier solltest du aber mal nach StringBuilder bzw StringBuffer suchen, damit geht das deutlich performanter (auch wenn das für dein Mini-Problem keine Rolle spielt).

mpG
Tobias

 

[mix2k1]

Bei mir steht immer nur die letzte Zeile im dem Ding.

		JTextArea meineTextJTextArea = new JTextArea( 2, 10);
		JScrollPane scroll = new JScrollPane(meineTextJTextArea);

		public void ShowList(){
			//this.setSize(400, 400);
			Iterator <int[]> it = meineListe.iterator();
			for(int i=0;it.hasNext();i++){
				System.out.print((i+1) + ". Ziehung: "); 
		       
		        int[] zahlenfeld = it.next(); 
					for(int j=0; j<zahlenfeld.length; j++){ 
						System.out.print("  " + zahlenfeld[j]); 
						 String meinString = Arrays.toString(zahlenfeld);
						 meineTextJTextArea.setText((i+1) + ". Ziehung: " + meinString + "\n");
					} 
		        System.out.println(); 
		    }
		}

 

[Tobias]

Weil du mit setText() jedesmal den Text im TextArea überschreibst. Erst einen vollständigen String bauen und dann ganz am Ende setText() ausführen.

mpG
Tobias

Edit: Oder benutz append().

 

[mix2k1]

Wo ist denn ganz am Ende? So?

		public void ShowList(){
			
			//this.setSize(400, 400);
			Iterator <int[]> it = meineListe.iterator();
			for(int i=0;it.hasNext();i++){
				System.out.print((i+1) + ". Ziehung: "); 
		        // Iterator arbeiten lassen 
		        int[] zahlenfeld = it.next(); 	// Ein Array aus dem Object it.next() erzeugen
					for(int j=0; j<zahlenfeld.length; j++){ 
						System.out.print("  " + zahlenfeld[j]); 
						test = test + (i+1) + ". Ziehung: " + Arrays.toString(zahlenfeld) + "\n";
									
					}
		        System.out.println(); 
		    }
			meineTextJTextArea.setText(test);
		}

 

[Tobias]

Ah, da war ich voreilig. Neben der setText()-Problematik uberschreibst du nämlich auch deine String-Referenz bei jedem Schleifendurchlauf:

public void ShowList(){
         //this.setSize(400, 400);
         Iterator <int[]> it = meineListe.iterator();
         for(int i=0;it.hasNext();i++){
              int[] zahlenfeld = it.next();
              String meinString = ""; // Objektreferenz hier deklarieren, da sie innerhalb 
                                                // der Schleife immer wieder überschrieben würde.
               for(int j=0; j<zahlenfeld.length; j++){
                  // String zusammenbauen
                  meinString = meinString + (i+1) + ". Ziehung: " + Arrays.toString(zahlenfeld) + "\n";                 
               }
               // Jetzt alles in die Textarea kippen
               meineTextJTextArea.setText(meinString);
               // System.out.println()? Du gibst doch nichts mehr auf der Konsole aus...
          }
}

mpG
Tobias

P.S.: Dein Kommentar zu it.next() ist übrigens falsch. Es handelt sich dabei nicht um ein Objekt, sondern um einen Methodenaufruf, der ein Array zurückgibt. Das aber nur am Rande, denn im großen und ganzen machst du das ziemlich gut.

 

[mix2k1]

Ok klappt so, aber meine Ausgabearea kann ich nicht in der Größe anpassen.
Wenn ich

 JTextArea meineTextJTextArea = new JTextArea(10,1);

eingebe, kommt verschiebt sich mein ganzes Fenster trotz Layout.
Oder wie stelle ich die Größe ein? Mit setSize gings gar nix.

 

[Tobias]

Das hängt vom verwendeten LayoutManager ab. Du kannst dem LayoutManager vorschlagen, wie groß die Komponente dargestellt werden soll - dafür gibt es setPreferredSize(). setSize() wird vom LayoutManager benutzt, um der Komponente die letztendliche, tatsächliche Größe mitzuteilen - deine Änderungen hier werden also sofort vom LayoutManager überschrieben.

Bessere Hilfe bekommst du, wenn du Code postest (und vielleicht einen neuen Thread im AWT und Swing-Board eröffnest, weil das hier arg offtopic wird).

mpG
Tobias