Label wird ungewollt mehrmals erstellt

[bristtote]

Hallo liebe Leute,

folgendes Problem:

Ich habe ein Array von Choices. Diese werden unter bestimmten Bedingungen gelöscht und wieder erstellt. Hier der Code:

 if   (cerg[index]!=null) panel[index].remove(cerg[index]); 
     	if    (c1[index] !=null) panel[index].remove(c1[index]);
      	if    (c2[index] !=null) panel[index].remove(c2[index]);      	
        if (cerg[index] !=null) panel[index].remove(g_z);
        if (cerg[index] != null ) panel[index].remove(plus);

Das gleiche soll auch für die beiden Labels "g_z" und "plus" geschehen. Die Labels sieht Ihr oben ebefalls im Code. Bei den Choices läuft alles einwandfrei, denn sobald die Bedingung erfüllt ist, werden sie gelöscht und weiter unten dann wieder erstellt. Hier der Code:

    c1[index]  = new Choice();
       c2[index]  = new Choice();
       cerg[index]= new Choice();
       g_z        = new Label("=");
       plus       = new Label("+");
        
       c1[index].add("R 0");
       c1[index].add("R 1");
       c1[index].add("R 2");
       
       c2[index].add("R 0");
       c2[index].add("R 1");
       c2[index].add("R 2");
       
       cerg[index].add("R 1");
       cerg[index].add("R 2");       
       
       panel[index].add(cerg[index]);
       panel[index].add(g_z);	 
       panel[index].add(c1[index]);
       panel[index].add(plus);
       panel[index].add(c2[index]);

Das eigentliches Problem sind die Labels. Wähle ich einen Item eines Choices, wird auch der Label erstellt bzw. gelöscht(so soll es auch sein). Das gleiche gilt auch für alle anderen Choices. Habe ich aber schon einen Item des z.B. ersten Choices und des z.B. zweiten Choices gewählt und möchte nun einen anderen Item des 1. Choices wählen , so entstehen auf einmal neue Labels!!! Bei den Choices funktioniert, wie gesagt alles einwandfrei. Das Ganze ist etwas kompliziert und nicht leicht zu schildern, deshalb bitte ich Euch nicht gleich wegzugehen und mir zu schreiben, wenn Ihr es nicht ganz versteht. Denn ich komme echt nicht weiter.
Hier noch ein Paar Bilder:

So sollte das Applet aussehen:

Aber, so sieht es aus:

Ich würde Euch für jede Hilfe danken.

 

[bristtote]

Wollt Ihr mich verarschen oder so???!?!?!?!?!?!

Kann doch nicht sein, dass in diesem Forum keiner eine Antwort hat!!!!!!!

 

[norman]

dein code ist imho ziemlich schwierig nachzuvollziehen. evtl hilft es schon, das ganze erstmal etwas besser zu strukturieren.
zB prüfst du im ersten codesample 3 (!) mal, ob cerg[index] != null ist. das kannst du locker zusammenfassen und das lesen&verstehen wird erleichtert..

 

[bygones]

Wollt Ihr mich verarschen oder so???!?!?!?!?!?!

Kann doch nicht sein, dass in diesem Forum keiner eine Antwort hat!!!!!!!

gehts noch ?

du erwartest hilfe und dann kommt ne Beleidigung...

na dann - Viel Spaß noch

 

[bristtote]

@Norman

Vielen Dank erst mal Norman.


@deathbyaclown

Wenn Du zu meinem Problem nichts sagen kannst, dann brauchst Du auch nicht hier rein zu schreiben.

 

[bygones]

@deathbyaclown
Wenn Du zu meinem Problem nichts sagen kannst, dann brauchst Du auch nicht hier rein zu schreiben.

ansichtssache - wenn man das Board angeht wehrt es sich ;-)

bygones

welchen LayoutManager verwendest du ?! starte den debugger und überprüfe wo wie welche methode aufgerufen wird die dir neue Labels hinzufügt bzw. wie verlaufen die Actions der choice ?!

 

[Jockel]

@dbac:
Deine Gutmütigkeit ist beneidenswert.

 

[thE_29]

Jo, ich hät den Thread schon geschlossen :bae:


Und was helfen uns deine if statements und dann die addings?

Wie steht das in was für einen Zusammenhang??

Könntest ja mal einen Codeauschnitt posten der zusammenhängend ist und/oder was du aufrufst, wenn dieser Fehler auftritt

 

[bristtote]

@deathbyaclown
Wenn Du zu meinem Problem nichts sagen kannst, dann brauchst Du auch nicht hier rein zu schreiben.

ansichtssache - wenn man das Board angeht wehrt es sich ;-)

bygones

welchen LayoutManager verwendest du ?! starte den debugger und überprüfe wo wie welche methode aufgerufen wird die dir neue Labels hinzufügt bzw. wie verlaufen die Actions der choice ?!

Tut mir sehr Leid wegen vorhin, danke dass Du mir trotzdem hilfst.

Und der Rest (thE_29, Jockel) kann echt seine Meinung für sich behalten, den Thread habe ich nicht geöffnet, damit Ihr entscheidet, ob es wieder geschlossen werden kann.


Also jetzt zu meinem Problem, hier der Code:

public class itemalsfunk extends Applet implements ActionListener {
	
	
	Panel Hauptpanel= new Panel();
	Color bg_co     = new Color(158,158,158);	
	Button b_b      = new Button("Los");
	Panel panel    [] = new Panel[12];
	String Items   [] = {"NOP", "ADD", "LOAD"};	
	Choice choices [] = new Choice [11];
	Choice c1      [] = new Choice [11];
	Choice c2      [] = new Choice [11];
	Choice cerg    [] = new Choice [11];
	Label labels   [] = new Label[11];
	Label g_z       = new Label("=");
	Label plus      = new Label("+");    
	
	
public void init () {
	Hauptpanel.setLayout(new GridLayout(0,1));
	
	setBackground(bg_co);
    add(Hauptpanel);




for (int i=0; i<panel.length; i++){
	panel[i]=new Panel();
	Hauptpanel.add(panel[i]);
 
	}

//panel[0].setBackground(Color.BLUE);
String s="0";
    
for (int i=0; i<labels.length; i++) {	
	labels[i] =new Label(s);
	int s1=Integer.parseInt(s);
	s1++;
	s=Integer.toString(s1);	
	panel[i].add(labels[i]);
	}
		
for ( int i = 0; i < choices.length; i++ ){
	choices[i] = new Choice();	
	panel[i].add(choices[i]);
	}



for (int i=0; i<choices.length; i++) {	
	choices[i].addItem(Items [0]);
	choices[i].addItem(Items [1]);	
	choices[i].addItem(Items [2]);
	}



panel[11].add(b_b);
b_b.addActionListener(this);  

for ( int i=0; i<choices.length; i++) {

  final int index = i;

  choices[i].addItemListener(new ItemListener(){
    public void itemStateChanged(ItemEvent e) {
      aendere( index, e.getItem() );
      
    }
  });
}
	} 			
		
public void aendere( int index, Object item ){
     
      if(item.equals("ADD")) {  	      
      
        if   (cerg[index]!=null) panel[index].remove(cerg[index]); 
     	if    (c1[index] !=null) panel[index].remove(c1[index]);
      	if    (c2[index] !=null) panel[index].remove(c2[index]);      	
        if (cerg[index] !=null) panel[index].remove(g_z);
        if (cerg[index] != null ) panel[index].remove(plus);        
        
       c1[index]  = new Choice();
       c2[index]  = new Choice();
       cerg[index]= new Choice();
       g_z        = new Label("=");
       plus       = new Label("+");
        
       c1[index].add("R 0");
       c1[index].add("R 1");
       c1[index].add("R 2");

       
       c2[index].add("R 0");
       c2[index].add("R 1");

       
       cerg[index].add("R 1");
       cerg[index].add("R 2");
.
      
       panel[index].add(cerg[index]);
       panel[index].add(g_z);	 
       panel[index].add(c1[index]);
       panel[index].add(plus);
       panel[index].add(c2[index]);
       
  
      }
      
    .............................

 

[bristtote]

@deathbyaclown
Wenn Du zu meinem Problem nichts sagen kannst, dann brauchst Du auch nicht hier rein zu schreiben.

ansichtssache - wenn man das Board angeht wehrt es sich ;-)

bygones

welchen LayoutManager verwendest du ?! starte den debugger und überprüfe wo wie welche methode aufgerufen wird die dir neue Labels hinzufügt bzw. wie verlaufen die Actions der choice ?!

Hallo nochmal,

ich wollte eben den Debugger starten und zwar benutze ich Jcreator. Aber irgendwie ist die Taste "Debugger" inaktiv. Ich weiss nicht woran es liegen kann.


danke nochmal

 

[SebiB90]

@deathbyaclown
Wenn Du zu meinem Problem nichts sagen kannst, dann brauchst Du auch nicht hier rein zu schreiben.

ansichtssache - wenn man das Board angeht wehrt es sich ;-)

bygones

welchen LayoutManager verwendest du ?! starte den debugger und überprüfe wo wie welche methode aufgerufen wird die dir neue Labels hinzufügt bzw. wie verlaufen die Actions der choice ?!

Tut mir sehr Leid wegen vorhin, danke dass Du mir trotzdem hilfst.

Und der Rest (thE_29, Jockel) kann echt seine Meinung für sich behalten, den Thread habe ich nicht geöffnet, damit Ihr entscheidet, ob es wieder geschlossen werden kann.

und wir machen uns nicht die mühe dein thread durchzulesen um dann angemotzt zu werden, also wenne hilfe willst mach es das nächste mal bischen netter. ich hättte genauso gehandelt wie thE_29 gesagt hat, thread einfach schließen bei einer solchen unhöflichkeit.

 

[bygones]

ok genug darüber diskutiert -> back top topic. (ansonsten immer aufpassen, man weiß nie wer Mod ist und dann doch schließt )

eine Vermutung: bei Itemlistener werden bei änderungen stets 2 Events gefeuert, einmal weil ein element selektiert wurde, einmal weil eins deselektiert wurde.... daran könnte es liegen, was anderes seh ich jetzt so nicht.

Du kannst über ItemEvent abfragen ob SELECTED wurde oder nicht, und nur dann würde ich änderungen machen... probiers aus.

 

[bristtote]

Ok, deathbyaclown.

Ich probiere es mal, danke

 

[bygones]

wie gesagt - ich würde das lassen und zum Thema sonst schließen wir hier echt.

generell - du kannst nicht erwarten, dass hier sofort und überhaupt eine Antwort kommt. Wir hocken uns hier hin lesen die Threads durch und wenn uns was einfällt wird das auch geschrieben. Selbstverständlich ist es das Recht des Fragenden nachzuhacken, ob einer eine Lösung weiß, aber eine Reaktion ala

Wollt Ihr mich verarschen oder so???!?!?!?!?!?!

ist unpassend und aggressiv PUNKT, ich war auch kurz davor zu schließen...

ok - schluss damit, wenn weiter diskutiert wird schließ ich (wenn mir stevg nicht dann schon vorher mein Status genommen hat *g)

EDIT: hehe ich liebe dieses alle schreiben auf einmal wieder mal für schreib sperre wenn schon einer schreibt ^^

 

[TheJavaKid]

setz einfach das layout auf null und damit hat sichs...

 

[The_S]

Ich hab mir den Thread zwar nicht durchgelesen (da pfuschen einfach scho zuviele mit und er ist mir zu lang ), aber davon

setz einfach das layout auf null und damit hat sichs...

würde ich auf jedenfall abraten. Ein null-Layout kann unangenehme Überraschungen nach sich ziehen wenn es auf anderen Systemen eingesetzt wird, als auf dem, auf dem es erstellt wurde.

 

[bristtote]

ok genug darüber diskutiert -> back top topic. (ansonsten immer aufpassen, man weiß nie wer Mod ist und dann doch schließt )

eine Vermutung: bei Itemlistener werden bei änderungen stets 2 Events gefeuert, einmal weil ein element selektiert wurde, einmal weil eins deselektiert wurde.... daran könnte es liegen, was anderes seh ich jetzt so nicht.

Du kannst über ItemEvent abfragen ob SELECTED wurde oder nicht, und nur dann würde ich änderungen machen... probiers aus.

Ich habe es mit Selected ausprobiert, die Items sind schon gewählt, also ist die Bedingung auch erfüllt. Denn mit den Choices selbst klappt es ja auch, es sind halt nur die Lables. Ich weiss echt nicht mehr, was ich ausprobieren soll!

Ich meine, wenn die folgende Bedingung erfüllt ist, dann muss es ja mit den Choices und den Labels klappen:

 if (cerg[index]!=null) {
        	 panel[index].remove(cerg[index]);
        	 panel[index].remove(g_z); 
        	 panel[index].remove(plus);        	
        	}

Es klappt halt aber nur für die Choices!!

setz einfach das layout auf null und damit hat sichs...

Wie meintest Du es denn eigentlich mit Layout auf null setzen?
Ich habe es nicht ganz verstanden?


danke Euch allen