Simplifier ces IF imbriqués

Version imprimable

Je sais que ce n'est certainement pas la meilleure façon, mais pouvez vous me proposez une façon beaucoup plus simple.

Merci de m'aider :D

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
float CheckHand(int main, carte paquettxt[][5])
{
	int i=0;
	float tmp=0;
 
	//Ces if imbriqués regarde si la main est une Straight, Straight FLush ou Straight Royale
	if(paquettxt[main][i].valeur==paquettxt[main][i+1].valeur-1)
	{
		if(paquettxt[main][i].valeur==paquettxt[main][i+2].valeur-2)
		{
			if(paquettxt[main][i].valeur==paquettxt[main][i+3].valeur-3)
			{
				if(paquettxt[main][i].valeur==paquettxt[main][i+4].valeur-4)
				{
					cout<<"STRAIGHT ";
					tmp=20;
					if(paquettxt[main][i].sorte==paquettxt[main][i+1].sorte&&paquettxt[main][i].sorte==paquettxt[main][i+2].sorte&&paquettxt[main][i].sorte==paquettxt[main][i+3].sorte)
					{
						cout<<"FLUSH ";
						tmp=tmp+10;
						if(paquettxt[main][i].valeur==10&&paquettxt[main][i+1].valeur==11&&paquettxt[main][i+2].valeur==12&&paquettxt[main][i+3].valeur==13&&paquettxt[main][i+4].valeur==14)
						{
							cout<<"ROYALE";
							tmp=tmp+10;
						}
					}
				}
			}
		}
	}
 
	if(paquettxt[main][i].valeur==paquettxt[main][i+1].valeur&&paquettxt[main][i].valeur==paquettxt[main][i+2].valeur&&paquettxt[main][i].valeur==paquettxt[main][i+3].valeur)
	{
		cout<<"FOUR OF A KIND";
		tmp=25;
	}
 
	if(tmp<30)
	{
		if(paquettxt[main][i].sorte==paquettxt[main][i+1].sorte&&paquettxt[main][i].sorte==paquettxt[main][i+2].sorte&&paquettxt[main][i].sorte==paquettxt[main][i+3].sorte)
		{
			cout<<"FLUSH";
			tmp=15;
		}		
	}
 
	cout<<endl;
	return tmp;
}

Salut,

Pour les premiers tests, tu pourrais peut être envisager une boucle, quelque chose qui serait proche de

Code:

1
2
3
4
5
6
 
bool found{false};
for(int j = 1; i<=4;++i){
    found = paquettxt[main][i].valeur==paquettxt[main][i+j].valeur-j;
}
/* si found == true; tu es dans le bon */

Et pour la deuxième série de if, tu pourrais faire pareil ;)

Au final, cela donnerait quelque chose comme

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
bool found{false};
for(int j = 1; i<=4;++i){
    found = paquettxt[main][i].valeur==paquettxt[main][i+j].valeur-j;
}
if(found){
    bool sorteFound{false};
    cout<<"STRAIGHT ";
    tmp=20;
    for(int j = 1; i<=4;++i){
        sorteFound= paquettxt[main][i].sorte==paquettxt[main][i+j].sorte;
    }
    if(sorteFound){
        /* ce qui doit être fait si les deux booléens sont à true*/
    }
}

08/11/2017, 06h29
PichDereck

Merci beaucoup :)
08/11/2017, 09h32
Bousk

Franchement tu gagnerais surtout à nommer correctement et clairement tes trucs.
- Tu as le droit d'utiliser des espaces dans ton code
- Tu retournes un float qui ne contient que des valeurs entières (positives)
- Tu utilises un tableau rempli de valeurs magiques
- Ta fonction porte le nom bien générique CheckHand quand d'après tes commentaires elle ne vérifie que 4 mains possibles

Ajouter des classes rendraient déjà le tout moins brouillon, et on a inventé un truc appelé enum il y a pfiou années qui serait plus que bienvenu ici.

Sauf que, je me suis rendu compte cette nuit que je m'étais trompé dans la logique à suivre... Ce serait d'avantage quelque chose comme

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
bool found{true};
int j=1;
do
    found = paquettxt[main][i].valeur==paquettxt[main][i+j].valeur-j;
 
}while(found && j<4);
if(found && j ==4){
    bool sorteFound{true};
    cout<<"STRAIGHT ";
    tmp=20;
    j =1
    do{
        sorteFound= paquettxt[main][i].sorte==paquettxt[main][i+j].sorte;
        ++j;
    }while(sorteFound && j <4)
    if(sorteFound && j == 4){
        /* ce qui doit être fait si les deux booléens sont à true*/
    }
}

08/11/2017, 12h24
Bousk

Tu devrais avoir le même résultat en initialisant sorteFound à true et en utilisant un & binaire dans la boucle sorteFound &= paquettxt[main][i].sorte == paquettxt[main][i+j].sorte;, j'aime bien cette construction :zoubi:
Tu peux même "optimiser" le truc en changeant la condition du for pour j < 4 && sorteFound mais vu qu'on parle de 4 éléments à boucler, c'est un peu ridicule imo.