eigene Klasse die JButton extended - als Liste!

[shaax]

Hallo,

Folgendes:

Ich habe eine Klasse erstellt,

public class SitzNode extends JButton {

	public SitzNode next;
        int sitznummer;

	
	public SitzNode(int s, SitzNode n) {
		sitznummer = s;
		next = n;
	}

}

Ich brauche 150 davon, und wollte diese mit einer einfach verketten Liste ansteuern.
Was anderes (Feld oder, kA) steht eigentlich nicht zur debatte (denn sonst muss einiges abändern) wird sich aber, wenn es nicht anders läuft, wohl nicht vermeiden lassen..

Das Problem ist nun:
wenn ich mit folgender for schleife durch die Buttons gehe:

    	SitzNode[] tmp = new SitzNode[150];
    	int counter = 0;
    	
		for(SitzNode sitz = jButton1; sitz!=null; sitz = sitz.next){
			if(sitz.getBackground() == Color.ORANGE){
				sitz.belegt = true;
				tmp[counter]=sitz;
				counter++;
			}
		}

(jButton1 ist der erste der 150 Buttons - die sind fix, haben den namen jButton1 - jButton150)

Wenn ich nun beim debuggen, die Details von jButton1 anzeigen lasse,
hat er bei "next = null"
obwohl ich die Next referenz beim erzeugen des Buttons übergeben habe
(referenz auf jButton2)

Die Frage ist nun:
warum? -.-

So, ich hoffe ich habe nicht zu unverständlich geschrieben
und ich hoffe es kann jemand helfen.

Wenn noch Fragen da sind,
bitte stellen!

 

[Marco13]

Vielleicht weil "next" public ist, und es damit jeder irgendwo auf null setzen kann, ohne dass man es merkt?

150 Buttons in einer selbst zusammengepfuschten LinkedList speichern ist obertrash, aber ... "das steht ja nicht zur Debatte"

Mal im Ernst: Eine Aussage wie
"Ich schreibe
int a = 5;
aber an einer anderen Stelle im Code hat a nicht den Wert 5"
wirkt irgendwie.... ... ... :?

 

[shaax]

Ja, ich weiss dass es wirklich müll is ...

Aber ich hab ne Klasse bekommen und 'musste' mit der weiterarbeiten,
dann hab ich so lang dran rumgemacht,
dass ich einfach nich von neuen anfangen wollte.. -.-

"Ich schreibe
int a = 5;
aber an einer anderen Stelle im Code hat a nicht den Wert 5"

Aber so isses! ^^
Und zwar ohne dass irgendwas mit dem (in diesem Fall) a passiert.
Deswegen ergibt des ja keinen Sinn...



Ich dachte vllt, dass es da nen bestimmten Grund gibt,
so in der Art:

man kann aus "JButtons keine Liste machen" ... oder so.


Trotzdem danke!

 

[Janus]

wenn du dir sicher bist, dass du beim erzeugen des buttons einen weiteren button als parameter übergeben hast, aber das feld next ist zu einem späteren zeitpunkt null, dann kann das nur zwei ursachen haben:

1) du hast null übergeben
2) jemand hat - dank public field - das ding irgendwann auf null gesetzt.

2) lässt sich einfach lösen: feld private machen und getter anbieten.

1) passiert schnell, wenn man java referenzen falsch versteht:

SitzNode a = null;
SitzNode b = new SitzNode( 1, a );
a = new SitzNode( 0, null );

b.next bleibt in diesem fall null.

 

[Janus]

als "quick fix" für das problem in 2 bietet sich übrigens etwas in der art an:

public class SitzNode extends JButton { 

   private SitzNode next; 
        int sitznummer; 

   public SitzNode(int s) { 
      sitznummer = s; 
      next = null; // leaf
   } 
    
   public SitzNode(int s, SitzNode n) { 
      sitznummer = s; 
      next = n;
      if( next == null )
         throw new NullPointerException( "Child is null" );
   } 

   public SitzNode getNext()
   {
       return next;
   }

}

 

[Janus]

achso, nochwas

bei verketteten listen bietet sich aus gründen des einfacheren handlings eher folgender aufbau an

class Node
{
   Node next;
   
   public Node( Node parent )
   {
       if( parent != null )
          parent.next = this;
   }
}

 

[shaax]

Oh, vielen Dank für die Antwort Janus!

Hätte gar nicht mehr mit weiteren Antworten gerechnet,

habe es nun aber doch mittels eines Feldes realisiert
und da gibts, wie zu erwarten, keinerlei Probleme ; )

Vielen Dank aber für die Mühen!