TicTacToe, Problem mit den Checkbox-Aktionen

[0blacko0]

Hallo,

ich will ein TicTacToe-Programm programmieren.

Leider passiert nichts wenn ich die Checkboxen anklicke :/

Hier der Code

import java.awt.BorderLayout;
import java.awt.Button;
import java.awt.Checkbox;
import java.awt.Graphics;
import java.awt.GridLayout;
import java.awt.Panel;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;

import javax.swing.JButton;

public class Panell extends Panel{
	private Panel[] p;
	private Checkbox[] cb;
	Kreis kreis;
	Kreuz kreuz;
	
	public Panell()	{
		this.setLayout(new GridLayout(3,3,10,10));
		kreis=new Kreis();
		kreuz=new Kreuz();
		cb=new Checkbox[9];
		p=new Panel[9];
		CheckboxHandler cbh=new CheckboxHandler();
		for(int i=0; i<9; i++)	{
			p[i]=new Panel();
			p[i].setLayout(new GridLayout(1,1));
			cb[i]=new Checkbox();
			p[i].add(cb[i]);
			this.add(p[i]);	
			cb[i].addItemListener(cbh);
			
		}
		////////////////////////////////////////////////
		
		
	}
	
	private class CheckboxHandler implements ItemListener	{

		public void itemStateChanged(ItemEvent e) {
			int a=0;
			for(int i=0; i<9; i++)	{
				if(cb[i].getState()==true)	{
					
						kreis.methode();
						kreis.repaint();
				
						kreuz.methode();
						kreuz.repaint();
					
				}
			}
			
		}
		
	}
	
	public void paint(Graphics g)	{
		g.fillRect(0,0, super.getWidth(), super.getHeight());

	}
}

mfg

 

[Leroy42]

Was sollen denn deine Methoden

kreis.methode(); und kreuz.methode();

machen?

 

[0blacko0]

Was sollen denn deine Methoden

kreis.methode(); und kreuz.methode();

machen?

ähm nichts^^

die methode methode() ist void und hat keinen parameter.

Sie macht garnichts^^

also ohne der methode würde es auch gehen

 

[Schandro]

Das heißt, du machst NICHTS wenn eine Checkbox angeklickt wird (außer repaint()...)
und wunderst dich dann warum nichts passiert?

-------------------

Außerdem: Warum ist Kreuz bzw. Kreis ne eigene Klasse
UND warum instanzierst du sie jeweils nur einmal, das aber am Anfang? Macht irgendwie keinen Sinn.

Der Code davon wäre hilfreich...

 

[0blacko0]

Das heißt, du machst NICHTS wenn eine Checkbox angeklickt wird (außer repaint()...)

Was soll ich dann schreiben?

 

[Schandro]

Wie gesagt, ich glaube dein Grundprinzip mit einer Klasse Kreuz und Kreis ist falsch.

Versuch doch mal zu beschreiben, was dein Code wann machen soll und wie z.b. die Kreise/Kreuze dargestellt werden sollen auf der GUI.

Solche Überlegungen kommen immer bevor man anfängt ein Programm zu schreiben

 

[Marco13]

Was soll denn diese wunderlich überschriebene paint-Methode da?!