Problem bei TicTacToe

[kulturfenster]

Hallo allerseits,
Ich bin dabei ein TicTacToe zu programmieren, dies allerdings ohne 2DArray.
Das Grundgerüst sieht folgendermassen aus: Der Player macht einen Zug, dieser wird in der PlayerKlasse überprüft und dann an die Move-Klasse weitergegeben, welche alle Moves speichert. Die PlayerKlasse überprüft lediglich, ob der Zug schon mal getägigt wurde. alles unklar soweit??

Ich vermute, dass ich ein (mir leider unbekanntes) Konstruktur - Problem habe.

hier ein paar CodeSchnipsel:

public class TicTacToe {

	private static final int maxMoves = 9;
	
	public static void main(String[] args)
	{
		Scanner in = new Scanner(System.in);
		Random generator = new Random();
		Player p = new Player();
		Computer c = new Computer();
		Moves m = new Moves();
		
		int moves = 1; // counter
		boolean victory = false;
		
		// 		
		while( victory || moves <= maxMoves)
		{
			boolean validhumanMove = false;
			
			
			while(!validhumanMove)
			{
				int humanMove = in.nextInt();
				if(p.isValidMove(humanMove))
				{
					m.addMove(humanMove);	// add humanMove to moves
					validhumanMove = true;
					System.out.println(p.isValidMove(humanMove));
				}
					
			}
			
			moves++;

public class Player {

	/*
	 * testet, ob der menschliche Zug schon getätigt wurde.
	 */
	public boolean isValidMove(int move)
	{
		Moves m = new Moves();
		for(int e : m.getMoveList())
			if(e == move) return false;
		return true;
	}
}

import java.util.ArrayList;

public class Moves {

	public ArrayList<Integer> moves;
	
	public Moves()
	{
		moves = new ArrayList<Integer>();
	}
	
	public void addMove(int move)
	{
		moves.add(move);
	}
	
	public ArrayList<Integer> getMoveList()
	{
		return moves;
	}
}

Wäre super, wenn jemand Zeit hätte, sich da kurz durchzuarbeiten. Ich bin sicher, es liegt nur ein kleines Problem vor. Vielen Dank für alle Tipps!

 

[The_S]

und dein Problem war gleich?

 

[kulturfenster]

oh, hehe, stimmt:
die isValidMove - Methode in der Playerklasse gibt stets "true" zurück.

 

[The_S]

Dann lass dir mal den Inhalt von der moveList ausgeben und schau, was da drin steht. Vermutlich speicherst du die moves falsch ab.

 

[Roar]

wozu denn

Moves m = new Moves();
m.getMoveList()
was soll denn dort großartig in der move list drinstehen :roll:

 

[The_S]

wozu denn

Moves m = new Moves();
m.getMoveList()
was soll denn dort großartig in der move list drinstehen :roll:

lol, stimmt ... ich sollte dazu übergehen, die Beiträge wieder vollständig zu lesen, anstelle sie nur zu überfliegen

 

[kulturfenster]

der Inhalt der MoveList stimmt.

for(int i = 0; i < m.getMoveList().size(); i++)
			{
				System.out.println("Move: " + m.getMoveList().get(i));
			}

dies lieferte die korrekten Angaben.

 

[Roar]

vielleicht in deiner main methode aber nich in deiner isValidMove method

 

[kulturfenster]

nun gelange ich an meine Wissensgrenze...

public boolean isValidMove(int move)
	{
		Moves m = new Moves();
		for(int e : m.getMoveList())
			if(e == move) return false;
		return true;
	}

Ist es falsch, jedes mal ein Objekt zu erzeugen (Moves m = new Moves() ) ? Wie kann ich das umgehen? Ich mein, anders kann ich auf die MoveList ja nicht zugreifen. [/code]

 

[Leroy42]

Wenn du das schon in getrennten Klassen implementierst,
mußt du deine Liste eben übergeben.

   public boolean isValidMove(int move, List<Integer> moves) { 
      for(int e : moves) 
         if (e == move)
             return false; 
      return true; 
   }

...

if (p.isValidMove(humanMove, m.getMoveList()))

 

[kulturfenster]

super, danke für die Antwort!

Ich dachte, das sei Sinn und Zweck von OOP, dass man möglichst viele kleine Klassen macht?

 

[Leroy42]

Ein Glas Sekt am Morgen soll ja gesund für den Kreislauf sein.
Ein Kanister Sekt wohl eher weniger.

==> Man kann's auch übertreiben.