JDialog Frage

[Sanix]

Ich erstelle gerade ein 4 Gewinnt, das man übers Netz spielen kann.
Nun habe ich folgendes Problem:
Man kann im Menu auf Neues Netzwerkspiel gehen, dann wird ein JDialog geöffnet. Dort kann man IP Adresse angeben und den gewünschten Port wählen. Danach auf ok klicken und die Verbindung wird hergestellt.
Die Verbindung wird jedoch in der Klasse FourWins gebraucht, welche das Spielfeld erstellt, Methoden für die Züge hat.
Wie kriege ich die Verbindung am Besten in diese Klasse rein? Derzeit habe ich es mit einer statische Variable gemacht, weiss jedoch nicht ob das gut ist oder obs auch anders geht.
Wenn man beim JDialog auf ok klickt, fängt ja der Actionlistener die Aktionen ab. Wie kann ich am besten auf die Textfelder zugreifen, wenn ich gerade im ActionLIstener bin? Muss ich alle an den Listener übergeben oder gibts hier eine schönere Variante?
Ich will eigentlich nicht die ganze Zeit Objekte hin und herschieben, damit sie Schlussendlich am richtigen Ort bekannt sind.


Derzeit siehts so aus:
Neues Menu wird erstellt in dem MainWindow. Dort kann man den Unterpunkt Netzwerkspiel anwählen. Jetzt kommt der erste Listener. Der ruft dann den JDialog auf. Dort kann man eingaben machen. Diese müssen dann abgefragt werden, mit denne eine Verbindungsthread erstellt werden, welcher dann aber in der FourWins Klasse bekannt sein müsste, welche auf dem Frame erstellt wird.

Ist wahrscheinlich bisschen wirr geschrieben aber ich weiss nicht, wie ichs anders erklären soll. Ich kann sonst auch den Code posten.

 

[bygones]

die Textfelder müssen Instanzvariablen des Dialogs sein, dann kannst du in der action Methode darauf zugreifen !

Intuitiv kräultst mir die Verbindung in die FourWins Klasse zu legen. Ohne Code bzw. genaues Design ist es natürlich schwer viel dazu zusagen, aber eben intuitiv würde ich eine Controller Klasse haben, die den Ablauf regelt. D.h. der Dialog wird geöffnet, die Infos ausgelesen, die Klasse übergibt dann die Infos an die FourWins Klasse.

 

[Sanix]

So hier mal den Code. Wie würde das mit dem Controller dann aussehen? Also der erstellt einfach alle Klassen, die Actionlistener melden sich zuerst bei ihm, dann führt er weitere Aktionen aus?
Wie geht das mit den Istanzvariablen?

FourWins.java

public class FourWins 
{
	private JButton[] jbFields;
	private int yFields;
	private int xFields;
	private static MyConnection mc;
	
	/**
	 * 
	 * @param xFields amount of fields on horizontal axis
	 * @param yFields amount of fields on vertical axis
	 * @param jp the buttons are placed on the given jpannel
	 */
	public FourWins(int _yFields, int _xFields, JPanel jp)
	{
		//Need vars for later use
		yFields = _yFields;
		xFields = _xFields;
		//Create buttons, set new actioncommand, add listener, add to the jpanel
		jbFields = new JButton[xFields * yFields];
		PlayFieldListener mblFields = new PlayFieldListener(this);
		for(int i = 0; i < xFields * yFields; i++)
		{
			jbFields[i] = new JButton("");
			jbFields[i].setActionCommand(new Integer(i).toString());
			jbFields[i].addActionListener(mblFields);
			jp.add(jbFields[i]);
		}
	}
	/**
	 * 
	 * @param fieldIndex field identifier
	 * @param strId player id
	 * @return true, if there are 4 in a row
	 * @throws IndexOutOfBoundsException if the field doens't exist
	 */
	public boolean takeField(int fieldIndex, String strId) throws IndexOutOfBoundsException
	{
		//Its only possible to set a coin over an other
		while((int)((fieldIndex + xFields) / yFields) <= yFields 
				&& jbFields[fieldIndex + xFields].getText().equals(""))
		{
			fieldIndex = fieldIndex + xFields;
		}
		
		//Check if field is not already used, if not, set player identifier
		if(jbFields[fieldIndex].getText().equals(""))
		{
			jbFields[fieldIndex].setText(strId);
		}
		
		//Check if there are 4 in a row
		int fieldIARow = 0;
		//Horizontal
		for(int i = fieldIndex - 3; i <= fieldIndex + 3; i++)
		{
			if(i >= 0 && i < yFields * xFields && 
					(int) (fieldIndex / xFields) == (int) (i / xFields))	//Filter impossible indexes
			{
				if(jbFields[i].getText().equals(strId))
				{
					fieldIARow++;
				}
				else
				{
					fieldIARow = 0;
				}
				if(fieldIARow == 4){return true;}
			}
		}
		//Vertical
		fieldIARow = 0; //Reset field
		for(int i = fieldIndex - 3*xFields; i <= fieldIndex + 3*xFields; i+=xFields)
		{
			if(i >= 0 && i < yFields * xFields)	//Filter impossible indexes
			{
				if(jbFields[i].getText().equals(strId))
				{
					fieldIARow++;
				}
				else
				{
					fieldIARow = 0;
				}
				if(fieldIARow == 4){return true;}
			}
		}
		//Diagonal down left to top right
		fieldIARow = 0; //Reset field
		for(int i = fieldIndex - (fieldIndex % xFields) * (xFields+1); 
			i <= fieldIndex + ((xFields-1) - (fieldIndex % xFields)) * (xFields+1); 
			i+=xFields+1)
		{
			if(i >= 0 && i < yFields * xFields)	//Filter impossible indexes
			{
				if(jbFields[i].getText().equals(strId))
				{
					fieldIARow++;
				}
				else
				{
					fieldIARow = 0;
				}
				if(fieldIARow == 4){return true;}
			}
		}
		//Diagonal down right to top left
		fieldIARow = 0; //Reset field
		for(int i = fieldIndex - ((xFields-1) - fieldIndex % xFields) *(xFields - 1);
		i <= fieldIndex + (fieldIndex % xFields) * (xFields - 1);
		i+=xFields-1)
		{
			if(i >= 0 && i < yFields * xFields)	//Filter impossible indexes
			{
				if(jbFields[i].getText().equals(strId))
				{
					fieldIARow++;
				}
				else
				{
					fieldIARow = 0;
				}
				if(fieldIARow == 4){return true;}
			}
		}		
		
		//Nobody won
		return false;
	}
	public static void setConnection(MyConnection _mc)
	{
		
	}
}

Window.java

public class Window extends JFrame
{
	private static final long serialVersionUID = 1L;
	
	public static void main(String[] args) 
	{
		new Window();
	}
	public Window()
	{
		setSize(400, 400);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setTitle("Four wins");
		
		//Add menu
		JMenuBar jmbMain = new JMenuBar();
		JMenu jmGame = new JMenu("Game");
		jmbMain.add(jmGame);
		JMenuItem jmiNewGame = new JMenuItem("New game...");
		jmiNewGame.addActionListener(new MyMenuListener());
		jmGame.add(jmiNewGame);
		setJMenuBar(jmbMain);
		
		//Start new game
		//newGame(7, 8);
		
		//Make visible
		setVisible(true);
	}
	/**
	 * 
	 * @param yFields amount of fields on vertical axis
	 * @param xFields amount of fields on horizontal axis
	 */
	public void newGame(int yFields, int xFields)
	{
		JPanel jpField = new JPanel(new GridLayout(yFields, xFields));
		new FourWins(yFields, xFields, jpField);
		add(jpField);
	}
}

MyMenuListener

public class MyMenuListener implements ActionListener
{

	public void actionPerformed(ActionEvent e) 
	{
		if(e.getActionCommand().equals("New game..."))
		{
			new JFConnect();
		}
	}
}

JDialog zum Verbinden

public class JFConnect extends JDialog
{
	private static final long serialVersionUID = -3240453236508016763L;

	/**
	 * Creating the connection window with
	 * Server / Client function
	 * IP / Port field
	 *
	 */
	public JFConnect()
	{	
		super();
		//Window settings
		setModal(true);
		setTitle("JFConnect to...");
		setSize(360, 120);
		
		//JPanel
		JPanel jpMain = new JPanel(new GridBagLayout()); add(jpMain);
		GridBagConstraints gbc = new GridBagConstraints();
		gbc.fill = GridBagConstraints.HORIZONTAL;
		
		//Actionlistener
		ACConnect acc = new ACConnect(this);
		
		//Fields
		JTextField jtfIP = new JTextField("IP");
		gbc.gridy = 0;
		gbc.gridx = 0;
		jpMain.add(jtfIP, gbc);
		JTextField jtfPort = new JTextField("Port"); jpMain.add(jtfPort);
		gbc.gridy = 0;
		gbc.gridx = 1;
		jpMain.add(jtfPort, gbc);
		JButton jbSClient = new JButton("JFConnect as client"); jpMain.add(jbSClient);
		gbc.gridy = 1;
		gbc.gridx = 0;
		jpMain.add(jbSClient, gbc);
		JButton jbSServer = new JButton("Start server...");
		jbSServer.addActionListener(acc);
		gbc.gridy = 1;
		gbc.gridx = 1;
		jpMain.add(jbSServer, gbc);
		
		//Make visible
		pack();
		setVisible(true);
	}
}

ACConnect.java - ActionListener für den JDialog

public class ACConnect implements ActionListener 
{
	public JDialog jd;
	
	public ACConnect(JDialog _jd)
	{
		jd = _jd;
	}
	public void actionPerformed(ActionEvent e) 
	{
		if(e.getActionCommand().toString().equals("Start server..."))
		{
			 //JTextField jtf = (JTextField) jd.getContentPane().getComponent();	//Get port textfield
			 //FourWins.setConnection(new MyConnection(new Integer(jtf.getText()).intValue()));
			 //jd.dispose();
			 //System.out.println("done");
		}
	}
}

Natürlich ist das ganze noch unfertig, da ich erst das mit der Verbindung schön "geregelt" haben wollte.