card1과 card2의 순서 때문에
if문이 복잡해지잖아.
가장 간단하게 해결할 방법은
card1과 card2의 숫자를 정렬해서 쓰는 거야.
예를 들면

int a = -1;
int b = -1;

if(card1 < card2){
a = card1; b = card2;
}
else {
a = card2; b= card1;
}

이렇게 하고

if( a == 3 && b == 8 ) { .... }

요로케하면 좀 편하지

검사하는 로직과 검사 내용을 분리할 수 있는 방법은 많은데~ 그건 다른 갤러가 설명해 줄 거야~