Programm wird immer langsamer

[Guest]

ICh hab ein Programm auf JFrame basis geschrieben und einige Buttonsmit ActionListener und Labels geadded

je öfters ich auf die Buttons klicke, desto langsamer wird mein Programm.

was is der Grund dafür?

hier ein kleiner Auszug:

class menue{

  public void hauptmenue()
  {
    H_Frage1="\nSpieler " + mSpielera + " ist an der Reihe.";
    l1.setText(H_Frage1);
    l2.setText(H_Frage2);
    b1.setText("Reich betrachten");
    b1.addActionListener(al1);
    b2.setText("Gebaeude (aus)bauen");
    b2.addActionListener(al2);
    b3.setText(" ---- ");
    b4.setText(" ---- ");
    b5.setText(" ---- ");
    b6.setText("Runde beenden");
    b6.addActionListener(al6);
    if (Kaserne[mSpieler]==1){l3.setText(H_Frage3);
    						  b3.setText("Kaserne");
    						  b3.addActionListener(al3);}
    if (Alchemist[mSpieler]==1){l4.setText(H_Frage4);
    							b4.setText("Alchemist");
    							b4.addActionListener(al4);}
    if (K.Soldaten[mSpieler]>0){l5.setText(H_Frage5);
    							b5.setText("Angriff");
    							b5.addActionListener(al5);}
    l6.setText(H_Frage6);
    
  }

  public void betrachten()
  {	  buttonreset();
  	  labelreset();
  	  uebersicht();
	  l1.setText(AusgabeUebersichtA);
      if (K.Soldaten[mSpieler]>0)
      {
        l2.setText(AusgabeUebersichtB);
      }
      else
      {
        l3.setText(AusgabeUebersichtC);
      }
      buttonreset();
  	b02.setText("Zurück ins Hauptmenue");
    b02.addActionListener(al02bet);
  }



  ActionListener al1 = new ActionListener() {
      public void actionPerformed( ActionEvent e ) {
          betrachten();
      }
    };
  ActionListener al2 = new ActionListener() {
      public void actionPerformed( ActionEvent e ) {
      	gebaudebau();
      }
    };


    ActionListener al02bet = new ActionListener() {
      public void actionPerformed( ActionEvent e ) {
      	buttonreset();
      	labelreset();
      	hauptmenue();
      }
    };

}

 

[Ariol]

Zeile 45 solltest du unbedingt umschreiben.

Du addest da jedesmal einen Actionlistener.

 

[SlaterB]

wenn du
b02.addActionListener(al02bet);
in einer häufig ausgeführten Operation drinstehen hast, dann siehst das ja zumindest Unheil versprechend aus,
vor allem auch in Hinblick auf die noch unbekannten Operationen
buttonreset();
labelreset();


möglicherweise erzeugst du ständig neue Buttons, löschst vielleicht gar die alten nicht,
fügst zumindest ständig neue ActionListener hinzu,

das muss alles nicht unbedingt bös enden,
aber wenn du sowas unnötigerweise machst, dann sind gravierende Fehler dabei auch denkbar


edit: Doppelpost?
http://www.java-forum.org/de/viewtopic.php?t=63303

 

[Guest]

Wie sollte ich Zeile 45 denn umschreiben?


Das soll ja alles ne Art Menü sein und wenn ich einen unterpunkt auswähle muss ich ja immerwieder die Möglichkeit haben zu meinem übergeordneten Menü zurückzukehren.

Dafür ist der Button b02 da


Die Veriablen sind alle unten in der Klasse deklariert, werden somit nur einmal erzeugt ....

was genau kann ich denn wie verändern damit es flüssiger wird, denn die Geschwindigkeit nimmt mit der Anzahl der klicke auf die Buttons exponentiell ab .....

was am Schluss ziemlich bös mies endet ... wie ihr euch vorstellen könnt

 

[SlaterB]

erstmal musst du entscheiden, welchen Sinn
b02.addActionListener(al02bet);
überhaupt hat,
hat b02 im Laufe der Zeit unterschiedliche Funktionen so dass der ActionListener wechselt?
oder wird b02 neu erstellt, so dass der ActionListener neu geadded werden muss?
werden die alten ActionListener entfernt bevor du den neuen einfügst?

 

[Guest]

b02 is der Button von dem ich aus den untermenüs immer in das Hauptmenü zurückkehren kann

Dabei muss ich aus jedem Untermenü andere Variablen beim zurückkehren übergeben.

ich brauch also verschiedene ActionListener für diesen einen Button.
In jedem untermenü bekommen die Buttons auch andere Funktionen und somit andere ActionListener
diese müssen im Hauptmenü natürlich zurückgesetzt werden und ich füge die "hauptmenü"-ActionListener wieder hinzu

ich hatte eben die Idee, die hinzugefügten ActionListener von Zeit zu Zeit zu entfernen

wäre ja dringen nötig

würde das mein Programm auch beschleunigen?

 

[SlaterB]

> ich hatte eben die Idee, die hinzugefügten ActionListener von Zeit zu Zeit zu entfernen
> wäre ja dringen nötig

machst du das denn nun bisher schon oder noch nicht?

> würde das mein Programm auch beschleunigen?

da niemand weiß woran es liegt, verbietet sich diese Frage,
aber schwer hoffen kann man natürlich

-----

eine Alternative wäre übrigens, die GUI mal GUI sein zu lassen
und das ganze irgendwo anders intern klären

der ActionListener ruft nur
fuehreAktionXYAus() auf
udn dort drin steht

if (aktueller Programmstatus == Untermenue a) {
// tue irgendwas
} else {
// tue was anderes
}

 

[Guest]

Es ha mein Programm beschleunigt, das Problem war wohl, das der selbe ActionLListener immer wieder geadded wurde und so der Befehl x^5 mal immer ausgeführt wurde ... das dauert natürlich sehr lange nach ein paar mal klicken.

Aber danke für eure schnelle Hilfe ^^

*aufatme^^*