Gra w warcaby: Nie błąd sprawdzanie poprawnie?

Przygotowuję grę Checkers do zadania. Cała sprawa przebiega tak, jak powinna, z wyjątkiem jednej dziwnej rzeczy. Oto moja płyta:Gra w warcaby: Nie błąd sprawdzanie poprawnie?

Checkers Board

przenieść dając wiersz źródłowy i kolumną, wówczas wiersz docelowy i kolumny.

move(int srcR, int srcC, int destR, int destC)

Mam wydrukować błąd, jeśli spróbuję przenieść element do nieprawidłowego miejsca (nie po skosie). Więc jeśli spróbuję przenieść element z 5 2 -> 4 2, otrzymuję komunikat o błędzie.

if(destR == srcR+1 || destR == srcR-1 && 
    destC == srcC+1 || destC == srcC-1){ 
     // code code code 
}else 
    message = "Invalid Move! Can only move diagonally one space.";

Dla większości rzeczy, to działa, ale gdy próbuję przenieść bezpośrednio w dół o jedno miejsce (na przykład 2 3 -> 3 3) to rusza kawałek i nie daje mi komunikat o błędzie.

Utknąłem! Jakieś pomysły, dlaczego tak się dzieje? W razie potrzeby mogę dodać więcej kodu.

Źródło

2013-08-05 alundy

Czy ważne ruchy zależą od tego, który kolor się porusza? – arshajii

@ErikPragt, który nigdy nie zostanie zatwierdzony, ponieważ destR nie może równać się srcR +/- 1 w tym samym czasie. – Deactivator2

desrR nie może być == srcR + 1 && srcR-1 –

Według logiki, jeśli

destC == srcC-1

jest prawdą, cała wyrażenie będzie prawdziwe. Musisz zwrócić uwagę na kolejność, w której Java ocenia operacje boolowskie ~ Dodaj() w razie potrzeby ~

Źródło

2013-08-05 15:22:38

Masz rację! Inny zestaw nawiasów poprawił go. Dzięki! – alundy

dziękuję za edycję @arshajii –

To jest dość proste, jak sądzę. Są tylko cztery dozwolone ruchy.

int deltaX = Math.abs(srcR - destR); 
int deltaY = Math.abs(srcC - destC); 
boolean validMove = deltaX == 1 && deltaY == 1;

Oczywiście ta kontrola umożliwia ruchy do tyłu. Ale kierunek ruchu w tył zależy od koloru gry, a promowanie utworu następuje poprzez dotarcie do drugiego końca.

Źródło

2013-08-05 15:18:18

+1 O wiele jaśniej jest o tym myśleć w kategoriach 'dx' i' dy'. – arshajii

Twoje wyrażenie logiczne jest nieprawidłowe.

zobacz Java operators precedence.

W razie wątpliwości zawsze używać nawiasów:

if((destR == srcR+1 || destR == srcR-1) && 
    (destC == srcC+1 || destC == srcC-1))

Źródło

2013-08-05 15:24:36 bjedrzejewski

Ups! Powinienem był to zauważyć. Dodatkowe nawiasy to naprawiły. Dzięki! – alundy

if(destR == srcR+1 || destR == srcR-1 && destC == srcC+1 || destC == srcC-1)

myślę, że trzeba pewne nawiasów tu, jak ta ocena jest trudne ponieważ starasz się porównać dwa wyrażenia naraz. Spróbuj

if((destR == srcR+1 || destR == srcR-1) && (destC == srcC+1 || destC == srcC-1))

ten sposób, jeśli (destR = srcR +/- 1) i (destC = SRCC +/- 1), ruch jest ważny.

Oryginalny sposób zadziałałby na niektóre czasu, ponieważ Boolean Java porównuje dwa wyrażenia lub wynik poprzedniego i następnego.

Źródło

2013-08-05 15:28:10 Deactivator2

if(destR == srcR+1 || destR == srcR-1 && destC == srcC+1 || destC == srcC-1)

odpowiada

if(destR == srcR+1 || (destR == srcR-1 && destC == srcC+1) || destC == srcC-1)

W drugim przykładzie:

srcR = 2 
srcC = 3 
destR = 3 
destC = 3

destr (= 3) jest równa srcR (= 2) + 1 =>destR == srcR+1 jest prawda => ocena (destR == srcR+1 || (destR == srcR-1 && destC == srcC+1) || destC == srcC-1) jest true.

Aby rozwiązać problem, trzeba zmienić swoje oświadczenie if do:

if((destR == srcR+1 || destR == srcR-1) && (destC == srcC+1 || destC == srcC-1))

Źródło

2013-08-05 15:31:33 mabbas

To wygląda dobrze. Właśnie zapomniałeś dodać nawias zamykający wokół instrukcji złożonej, jeśli. Tak więc powraca to prawda, kiedy nie powinno.

Źródło

2013-08-07 05:18:07 Jon

Gra w warcaby: Nie błąd sprawdzanie poprawnie?

Odpowiedz

Powiązane problemy