conditions qui ne marchent pas

Version imprimable

salut tout le monde .
voila j'ai un fais une petite fonction pour mon jeu(tetris) qui permet de supprimer les lignes completes et de calculer le score .voila le code:

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
int nb=1;
for(int j=NB_BLOCS_HAUTEUR-1;j>0;j--)
{   
    bool complet = true;
    for(int i=0;i<NB_BLOCS_LARGEUR;i++)
    {  
        if(carte[i][j] == 0)
            complet = false;
    }
    if(complet == true)
    {
        nb ++;
        compteur+=100;
        FSOUND_PlaySound(FSOUND_FREE, casse);// Pour generer un son pendant la suppression d'un ligne
        for(int k=j;k>0;k--)
        {               
            for(int i=0;i<NB_BLOCS_LARGEUR;i++)
            carte[i][k] = carte[i][k-1];
        }
        for(int i=0;i<NB_BLOCS_LARGEUR;i++)
        {
            carte[i][0] = 0;
        }
    }
}
if( nb == 4)
{
    FSOUND_PlaySound(FSOUND_FREE, yeah1);
    compteur+=400;
}
if(nb == 3)
{
    FSOUND_PlaySound(FSOUND_FREE, yeah);
    compteur+=300;
}

vous pouvez remarquez que je veux produire des sons selon le nombre de ligne s supprimer a la fois et attribuer des scores bonus aussi.
le probleme que ca marche pas : j'ai pas encore fais un affichage des score mais j'entend le son correspondant a 3 lignes meme si je fais 4 !!

09/02/2008, 10h41
koala01
Salut,

Es tu sur de ne pas plutôt entendre le son qui doit représenter quatre lignes quand tu en supprimes trois, et de ne pas avoir de son quand tu en supprime 4 :question:

En effet, tu met nb à 1 au départ, ce qui fait que nb vaudra
- 2 si ton code trouve une ligne
- 3 s'il en trouve 2
- 4 s'il en trouve 3
- ...
Vu qu'il est incrémenté si complet est vrai ;)

En plus, j'aurais plutôt recours à un if else if ou mieux, à un switch... case, si tu prévois de rajouter un son si 5 puis 6 puis... lignes sont effacées en même temps :p
09/02/2008, 10h56
guitariste

j'affirme que c'est bizar mais je suis sur que c'est le son de 3 lignes que j'entend meme si je fais 4 .
Encore plus si j'initialise nb a 0 j'entend plus rien si je fais 3 ou meme 4 !!!!

c'est vraiment bizar ce truc :

j'ai changé les conditions pour essayer:

nb=0
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 switch( nb ) { case 1 : FSOUND_PlaySound(FSOUND_FREE, yeah1); compteur+=400; break; case 2: FSOUND_PlaySound(FSOUND_FREE, yeah); compteur+=300; break; }
donc yeah1 pour une ligne
yeah pour deux lignes

voila les resultats

si je fais une ligne : yeah1 ( tout va bien )
si je fais deux lignes : 2 yeah1 8O
si je fais 3 lignes : yeah1 et yeah 8O
si je fais 4 lignes: 2 yeah1 et yeah 8O

Prends l'habitude de faire un scope pour chaque case, sinon un jour tu vas avoir une erreur de déclaration multiple et tu ne sauras même pas pourquoi.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 switch( nb ) { case 1 : { FSOUND_PlaySound(FSOUND_FREE, yeah1); compteur+=400; break; } case 2: { FSOUND_PlaySound(FSOUND_FREE, yeah); compteur+=300; break; } }
Pour ton problème je ne sais pas.

09/02/2008, 14h04
screetch
il y a une erreur dans ta boucle qui fait que tu ne detectes qu'une ligne sur deux. a la frame suivante tu vas detecter une autre ligne. un truc dans ce gout la.

si j'ai bien compris :
- tu pars du bas de l'ecran.
- tu regardes si la ligne en cours est pleine.
- si oui :
- a partir de la ligne en cours et jusqu'au haut de l'ecran, tu remplaces la ligne par sa suivante

PROBLEME :
tu vas remplacer la ligne en cours par la ligne suivante, puis passer a la ligne suivante qui a ete remplacée par la ligne encore suivante. en d'autres termes, tu vas sauter la ligne suivante.
Code:

1 2 3 int nb=1; for(int j=0;j<NB_BLOCS_HAUTEUR;j++) .......
Pour eviter cela, pars du haut de l'ecran, comme cela tu remplaceras la ligne en cours par une que tu as deja vue, et donc a plus le bug.

Citation:

Envoyé par NiamorH

Prends l'habitude de faire un scope pour chaque case, sinon un jour tu vas avoir une erreur de déclaration multiple et tu ne sauras même pas pourquoi.

Je n'ai jamais créé de scope pour mes différents case, et je n'ai jamais eu de problème à ce sujet...

Maintenant, il est vrai que j'évite les déclarations au sein d'un case ;)

guitariste>> c'est donc qu'il te faut revoir la logique de calcul pour nb ...

J'aurais presque l'impression qu'il y a une boucle qui est effectuée quelque part (ou que ce n'est pas le seul endroit où tu demande de jouer ces sons...)

Mais le problème semble se situer en dehors du code que tu nous as fourni, car il n'y a, a priori, aucune raison que cela se passe comme tu le décris :P

Ceci dit, et parce que j'ai (enfin) pris la peine de me plonger intensément dans ton code, je me demande si tu n'aurais pas intérêt à remplacer ta boucle
Code:

1 2 3 4 5 for(int i=0;i<NB_BLOCS_LARGEUR;i++) { if(carte[i][j] == 0) complet = false; }
par une boucle du genre de
Code:

1 2 3 4 5 6 while(i<NB_BLOCS_LARGEUR && complet) { if(carte[i][j]==0) complet=false; ++i; }
La raison en est simple: dans le premier cas, tu devra d'office tester toutes les cases de la ligne, y compris si l'une des toutes première est vide (ce qui fait d'office que la ligne est vide), alors que, dans le second cas, tu ne testera les cases de ta ligne QUE... si tu as encore une possibilité que ta ligne soit complète ;) :P

Le gain de temps sera d'autant plus important que la première case vide apparaitra tôt ;)

J'ai en outre l'impression que ta matrice est construite sur le modèle colone ligne (que le premier indice est celui... de la colone :P)

De manière général, une matrice est toujours construite sur le modèle inverse: le premier indice donné est celui... de la ligne, et le second celui de la colone.

La raison en est simple: tu obtiendra le même nombre total d'éléments (ici la possibilité de placer 18 caractères) si tu crées une matrice sous la forme de

Code:

char mat[3][6];

que si tu la crée sous la forme de

Code:

char mat[6][3];

Sauf que, dans le premier cas, il s'agira de pouvoir placer 3 chaines de 6 caractères (dont le '\0' si c'est une chaine "C style")
alors que dans le deuxième cas, il s'agira de pouvoir placer... 6 chaines de trois caractères (dont le '\0' si c'est une chaine "C style")

En plus, il apparait dans ton code que tu travaille régulièrement à faire bouger toute une ligne (c'est à dire l'ensemble des NB_BLOCS_LARGEUR blocs qui composent la ligne ;))

Le fait de "changer ton fusil d'épaule" et de travailler de manière classique sur les matrices (en modifiant le code partout où c'est nécessaire) te permettrait avantageusement de remplacer le tronçon de code
Code:

1 2 3 4 5 6 7 8 9 for(int k=j;k>0;k--) { for(int i=0;i<NB_BLOCS_LARGEUR;i++) carte[i][k] = carte[i][k-1]; } for(int i=0;i<NB_BLOCS_LARGEUR;i++) { carte[i][0] = 0; }
par un tout simple
Code:

1 2 3 for(int k=j;l>0;--k) memcpy(carte[k-1],carte[k],NB_BLOCS_LARGEUR * sizeof( le_type_utilise_pour_carte); memcpy(carte[j],0,NB_BLOCS_LARGEUR * sizeof( le_type_utilise_pour_carte);
qui sera bien plus rapide à l'exécution (et qui est tout de suite plus "compact", sans pour autant perdre en lisiblité ;))

09/02/2008, 14h54
HanLee

std::copy plutôt, pour ne pas avoir à se trimballer le sizeof à chaque fois.