Hilfe bei BUG finden.

[japa]

Ich will mir ein Poker-Programmm schreiben, damit ich Java besser beherrsche. Aber ich scheitere schon beim PAARE-ERKENNEN.

	/**
	 * Ace Pik als 1, Ace Kreuz als 2 ... Fünf Herz als 20...  
	 * @param useCards - eine Liste von Karten werden importiert
	 * @return - wenn es ein Paar gefunden wurde, wird true zurückgegeben sonst false
	 */
	public boolean isPair(int[] useCards){
		for (int karte1 = 0; karte1 < (useCards.length - 1) ; karte1++){
			for (int karte2 = 1; karte2 < useCards.length ; karte2++){
				if((useCards[karte1]-1)/4 == (useCards[karte2]-1)/4){
					return true;
				}
			}
		}
		return false;
	}

Ich benutze (int)((intWert - 1)/4) um Ace, Sechs König usw... zuermitteln.
Zuerst wähle ich Karte1 mit Index 0, Karte2 mit Index 1.
Die Methode vergleich aufsteigend Karte1 mit Karte2.

Ich sehe hier leider keinen Logik-Fehler

Als Test habe ich diese QC benutzt

	int[] paar = {1,344,292};
	if(chance.isPair(paar)){
		System.out.println("paar");
	} else {	
		System.out.println("Kein paar");
	}

Es sollte "Kein paar" erscheinen, jedoch kam "paar raus".

Erkennt jemand hier einen Fehler?

 

[Tomate_Salat]

weil 1 != 344 != 292 ist?!

Edit

for (int karte2 = 1; karte2 < useCards.length ; karte2++){

karte2 sollte hier auch bei 0 anfangen nur sollteste du danach prüfen ob karte2==karte1 ist ;-)

Außerdem glaube ich, wäre es mit mit einer HashMap o.ä. sicher einfacher:

int[] cards	= {1,3,3,4};
		
		boolean pair= false;
		boolean tripple = false;		
		
		HashMap<Integer, Integer> hand	= new HashMap<Integer, Integer>();
		
		for(int i=0; i < cards.length; i++)
		{
			if( hand.containsValue( cards[i] ) )
			{
				if(pair)
					tripple = true;
				else
					pair = true;
			}
			else
			{
				hand.put(hand.size(), cards[i]);
			}
				
		}
		
		System.out.println("Paar: " + pair);
		System.out.println("3-er Paar: " + tripple);

 

[japa]

wenn Karte2 bei 0 anfängt, dann würde ich die selbe Karte am Anfang vergleichen, dann kommt nur true raus.

Karte2 = 1 // ich wähle die zweite Karte mit dem Index 1.

 

[Tomate_Salat]

nein, du machst einen denkfehler: wenn die schleife weider geht, ist karte1 bei 1 und karte2 auch also vergleichst du trotzdem die gleichen, du musst überprüfen ob karte2 gleich karte1 ist und in dem fall den schleifengang überspringen, oder du machst es so wie ich oben.

Edit ah hab mir grad deinen code genauer angesehen, vergiss was ich über den Denkfehler gesagt habe, das sollte eigentl. so funktionieren (mit dem aufsteigend vergleichen)

 

[japa]

Ich kenne hashMap noch nicht, und es sieht für mich viel komplizierter aus.

Ich mache mal weiter und finde vielleicht später den Fehler, danke trotzdem

 

[Tomate_Salat]

ich würde dir empfehlen, zu versuchen meinen code zu verstehen, gerne beantworte ich dir auch fragen zu unklarem

 

[eRaaaa]

Ich kenne hashMap noch nicht, und es sieht für mich viel komplizierter aus.

naja, unkomplizierter sieht es mit einem HashSet vllt. aus

    public boolean isPair(int[] useCards) {
	Set<Integer> cards =new HashSet<Integer>();
	for (int integer : useCards) {
	    cards.add(integer);
	}
	return cards.size() != useCards.length;
    }

(mit Integer wärs sogar nur ein Einzeiler )

Allerdings solltest du dir vllt überlegen anstelle irgendwelche Integer-Werte als Karten zu missbrauchen, eine Klasse Cards, Hand und ähnliche zu schreiben oder so?! (nur ein Tipp)(es kann ja auch mehr als ein Paar geben und so weiter...)

 

[Thief]

Schau dir ruhig mal die HashMap an. Nur weil du sie noch nicht kennst heisst es nicht dass es nicht vielleicht deutlich leichter mit ihr wäre. Generell gilt sich so schnell wie möglich Wissen anzueignen.

Zu deinem Problem.... Um Fehler zu finden lohnt es sich meist mit System.out.println() aktuelle Werte auszugeben.
Dann kannst du auch leichter nachvollziehen (besonders in Schleifen) was du so programmiert hast und eventuelle
Denkfehler leicht finden.

 

[Tomate_Salat]

Zu deinem Problem.... Um Fehler zu finden lohnt es sich meist mit System.out.println() aktuelle Werte auszugeben.
Dann kannst du auch leichter nachvollziehen (besonders in Schleifen) was du so programmiert hast und eventuelle
Denkfehler leicht finden.

oder den Debugger nutzen :rtfm:, der ist vllt bie so kleinen Anwendungen leicht übertrieben, aber der Umgang sollte gelernt sein, bei größeren Projekten ist es manchmal verdammt knifflig Bugs zu finden. Da hilft oft nur noch der Debugger.

 

[japa]

	public boolean isPair(int[] useCards){
		for (int karte1 = 0; karte1 < (useCards.length - 1) ; karte1++){
			for (int karte2 = (karte1+1); karte2 < useCards.length ; karte2++){
				if((useCards[karte1]-1)/4 == (useCards[karte2]-1)/4){
					return true;
				}//Paare
			}//karte2
		}//karte1
		return false;
	}

hab den Fehler gefunden karte2 = karte1+1

Wegen karte als eigene Klasse, ich könnte es machen, aber ich weiß es nicht wie ich random machen kann, ich kenne halt nur int/double.

 

[Schumi]

Der müde Joe hatte da mal was geposted, fand ich sehr schön und verständlich: http://www.java-forum.org/java-basi...302-kartenspiel-programmieren.html#post526369