Discussion :

[CyberTwister]

Bonjour

Je suis encor un débutant en Java, et je suis certains que mon code peu être simplifié :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
switch(c.mMapId) 
{
case 1 : min=1;  
max=NbMap[1]; break;
case 2 : min=NbMap[1]+1;  
max=NbMap[1]+NbMap[2]; break;
case 3 : min=NbMap[1]+NbMap[2]+1;  
max=NbMap[1]+NbMap[2]+NbMap[3]; break;
case 4 : min=NbMap[1]+NbMap[2]+NbMap[3]+1;  
max=NbMap[1]+NbMap[2]+NbMap[3]+NbMap[4]; break;
case 5 : min=NbMap[1]+NbMap[2]+NbMap[3]+NbMap[4]+1;  
max=NbMap[1]+NbMap[2]+NbMap[3]+NbMap[4]+NbMap[5]; break;
case 6 : min=NbMap[1]+NbMap[2]+NbMap[3]+NbMap[4]+NbMap[5]+1;  
max=NbMap[1]+NbMap[2]+NbMap[3]+NbMap[4]+NbMap[5]+NbMap[6]; break;
}

Voila, ça me parai assez lourd, il faut que j'aille jusqu'à 35 ^^
Si vous pouviez me dire comment simplifier ce ptit bout de code un peu trop répétitif, ça serait gentil.
Bonne journée

[zo1odMSQr5Fy]

Un for me semble une bonne idée pour ton petit bout de code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
int min =1,max=Nbmap[1];
 
for(int i =2;i<=c.mMapId;i++)
{
min += Nbmap[i];
max+= Nbmap[i];
}

Je dirais que ça donnerais quelque chose comme ça

[CyberTwister]

Merci beaucoup TheBear, c'est parfait