problematische While-Schlaufe

[kulturfenster]

Guten Tag,

Ich bin dabei ein "graphik-loses" TicTacToe zu schreiben, hab aber Probleme beim Computerzug.

Hier erst mal die, meines erachtens, problematische Stelle des Codes:

boolean correct = false;
		Random generator = new Random();
		
		// get a valid comp move
		while(correct != true)
		{
			int compMove = 1 + generator.nextInt(9); // generates a number between 1-9
			
			// testet, ob compMove in der humanMoveList vorkommt.
			for(int i=0; i < humanMoveList.size(); i++)
			{
								
				if(compMove != humanMoveList.get(i))
				{
					// testet, ob compMove in der compMoveList vorkommt.
					for(int j =0; j < compMoveList.size(); j++)
					{
						if(compMove != compMoveList.get(j))
							correct = true;
					}
				}
				
				
			}
	
			compMoveList.add(compMove); // add comp move to compMoveList

Ich habe 2 Listen erstellt, eine für die Züge des Benutzers (humanMoveList) und eine für die Züge des Computers (compMoveList). Bei jedem ComputerZug soll dann getestet werden, ob der entsprechende Zug schon passiert ist.
Leider will das ganze nicht funktionieren. Sieht jemand einen Fehler? Vielen Dank schon im Voraus

 

[AlArenal]

Warum keine zweidimensionales Feld, dass das Spielfeld dsymbolisoert und dessen Felder die Werte 0 (leer), 1 (Spieler), 2 (Computer) haben kann?

Fände ich jetzt irgendwie naheliegender und einfacher abzuprüfen.

 

[SlaterB]

bereits beim ersten Vergleich mit der humanMoveList gehts du in die compMoveListe
und dort reicht dir auch schon ein Vergleich um sofort auf correkt = true zu setzen,

so nicht,
erst GANZE humanMoveList durchlausen,
dann GANZE compMoveList,

mache es so:

correkt = true; // Anfangswert true, nicht false, natürlich erst innerhalb der Schleife auf true setzen
teste humanList
wenn immer noch correkt {
teste compList
}

 

[kulturfenster]

super, vielen Dank für die beiden Tipps! Falls es klappt, kann ich mich ja dann immer noch dem 2D-Array zuwenden...

 

[kulturfenster]

ein kleines Problem hab ich noch:

correkt = true; // Anfangswert true, nicht false, natürlich erst innerhalb der Schleife auf true setzen

okok, ich werd mir nachher gleich noch n Boolean-Tutorial anschauen.

nur kurz: wieso Anfangswert true und dann in der Schlaufe nochmals auf true setzen?? Was brauch ich dann für eine while - Bedingung?

 

[SlaterB]

damit meinte ich, dass du es insgesamt andersrum machen sollst:
Anfangswert true und wenn gefunden, dann auf false setzen
(sowie weitere Bearbeitung nicht mehr nötig)

andersrum gehts nicht, wenn ein Zug nicht übereinstimmt kannst du nicht correct = true setzen,
weil du nicht weißt, ob der Zug vielleicht noch später in der Liste enthalten ist

 

[kulturfenster]

hmm, da sieht doch alles so hübsch und klar aus, und trotzdem "wählt" der Comp immer wieder Felder, die ich schon besetzt habe.
Sieht jemand den Fehler? Hab schon diverse Tester eingefügt - ohne Erfolg.

boolean correct = true;
		Random generator = new Random();
		
		// get a valid comp move
		while(correct != false)
		{
			int compMove = 1 + generator.nextInt(9); // generates a number between 1-9
			
			
			for(int e : humanMoveList)
			{
				if(compMove != e)
				{
					correct = false;
					compMoveList.add(compMove);
				}
			}

			if (correct == true)
			{
				for(int e : compMoveList)
				{
					if(compMove != e)
					{
						correct = false;
						compMoveList.add(compMove);
					}
				}
	
			}
	
		}
	
		return compMoveList.get(compMoveList.size()-1); // return last compMove

 

[Saxony]

Um mal kurz die Fachtermini zu korrigieren:

Mittlerweile gibt es neben der schon berühmten if-Schleife auch noch eine while-Schlaufe.

Ich will ja jetzt nicht nerven, aber man sollte die Neueinsteiger schon mal drauf hinweisen.

Nichts ist peinlicher als wenn ich auf Arbeit mit anderen Programmieren rede und diese trotz 20 jähriger Programmiertätigkeit immer noch if-Schleifen schreiben.

Also:

if-Anweisung (eigentlich Bedingte Anweisung)
und
while-Schleife (eigentlich Kopfgeprüfte Schleife)

Sorry aber das musste raus.

bye Saxony

 

[SlaterB]

@kulturfenster

du machst immer noch das selbe wie vorher, höchstens noch verquirlter

         for(int e : humanMoveList) 
         { 
            if(compMove != e) 
            { 
               correct = false; 
               compMoveList.add(compMove); 
            } 
         }

was soll das?
denn doch erst mal logisch nach was der Code tun soll?

Annahme:
Liste enthält 2,3,4

neues Element ist 4

so, dann kommt der erste Vergleich,
2 != 4

was passiert dann? eigentlich nix,
noch weiß man nicht viel,
wenn das erste Element in der Liste 4 wäre, dann könnte man sich nun sicher sein, dass das Element nicht korrekt ist,

aber bei != ist doch nicht correct = false?
correct ist doch immer noch möglich!

selbst wenn du correct = false feststellen würdest,
wieso fügst du dann den FALSCHEN move in die Liste ein??

alles total verdreht...

 

[AlArenal]

Das kommt davon, wenn man nicht auf mich hört

 

[Leroy42]

Das kommt davon, wenn man nicht auf mich hört

LOL

Stimmt! Da fehlen Saxony und SlaterB wohl noch ein paar Erfahrungswerte.

 

[SlaterB]

vom hören lernt man nix