Java Swing - repaint() funktioniert nicht

[icarus2]

Hi
Ich möchte gerne für meine Maturaarbeit das N-Damenproblem mit einen grafischen Oberfläche in Swing lösen.

Dazu habe ich die Oberfläche jetzt einmal so in etwa gemacht. Das Problem ist jedoch, dass in der Klasse Schachbrett die repaint()-Methode nicht wirklich funktioniert, wie sie sollte.
Problem: Das alte Koordinatengitter bleibt bestehen, wenn man die verschiedenen JRadioButtons anklickt oder die 2 JButtons vergrössern und verkleinern. Das Schachbrett wird erst richtig gezeichnet, wenn ich die Grösse des gesamten Fensters ändere.

Könnte mir da jemand helfen. Ich habe in den weiten des Internes nichts gefunden, das mir geholfen hätte. Ich habe schon einmal etwas in der Art gemacht und dort hat es so geklappt.

Die Klassen sollte befinden sich im Anhang. NDamenProblem enthält die main-Methode. In der Klasse Schachbrett wird alles gezeichnet und dort befindet sich die repaint()-Methode. Ansonsten habe ich alles kommentiert.


Ich wäre euch wirklich sehr dankbar, wenn ihr mir einen Tipp geben könntet (z.B. eine Alternative für repaint() oder so etwas in der Art).

PS: Damit mich niemand falsch versteht... ich erwarte natürlich nicht, dass ihr mir jetzt den ganzen Code umschreibt. Ich hab ihn nur angehängt, damit ihr das Programm selber laufen lassen könnt, falls ich mein Problem nicht gut genug beschrieben habe ;-)

 

[Verjigorm]

ersetze deine paint()-Methode
durch paintComponent()!:

paint() überschreibt man in einem JPanel NICHT!

PS: Da sind generell einige Dinge SEHR grausam

 @Override
	protected void paintComponent(Graphics g) 
  {
		super.paintComponent(g);
		   switch (n) {
		      case 4:
		      Schachbrett4Felder(g);
		      break;
		      case 5:
		      Schachbrett5Felder(g);
		      break;
		      case 6:
		      Schachbrett6Felder(g);
		      break;
		      case 7:
		      Schachbrett7Felder(g);
		      break;
		      case 8:
		      Schachbrett8Felder(g);
		      break;
		      case 9:
		      Schachbrett9Felder(g);
		      break;
		      case 10:
		      Schachbrett10Felder(g);
		      break;
		      case 11:
		      Schachbrett11Felder(g);
		      break;
		      case 12:
		      Schachbrett12Felder(g);
		      break;
		    }
	}

 

[icarus2]

Ersma Danke für die Hilfe.

Was ist denn genau so grausam?

Ist halt mein erstes Projekt... ich programmiere erst seit seeeehr kuzer Zeit.

 

[Verjigorm]

Aktuelles Beispiel:
überleg dir mal was du in den Methoden SchachbrettXFelder(g) passiert und ob du wirklich 10 verschiedene Funktionen brauchst, ober ab es eine Funktion tut mit einem weiteren Übergabeparamter
Dann würde auch die ganze Switch-Anweisung wegfallen.

Genauso deine actionPerformed im RadioButtonlistener
Schleife mit 2-3 Zeilen ersetzt dein ganzes if/else-konstrukt

Zudem einige weitere Kleinigkeiten

 

[icarus2]

Aso, dann isses das, was ich dachte ^^

Da muss ich dir recht geben. Das ist wirklich sehr hässlich gelöst und wird auch noch abgeändert. Ich wollte nur mal den groben Aufbau machen und sehen obs einigermassen geht.

Danke für die Tipps...

kann geschlossen werden : )