Discussion :

[Maskeren]

Bonjour, je débute tout juste en algorithmique (sur CodeBlocks) dans le cadre de mes études supérieures, et je suis confronté à un petit problème que je ne parviens pas à résoudre seul.
J'ai d'abord demandé à mon professeur d'algo qui, pressé d'aller se chercher son café, m'a balancé une réponse à la va-vite, et s'est éclipsé avant que je puisse lui dire que j'avais déjà essayé, et que ça ne marchait pas.

Il s'agit d'une boucle qui se présente sous la forme suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if ((ItemMap[X][Y])!= RIEN) //Si on vérifie la présence de l'objet//
        {
            for(space=0; space<10; space++) //Parcourir l'ensemble de l'inventaire contenant 10 poches//
            {
                if (Inventaire[space] == RIEN) //Si une poche est libre, alors//
                {
                    (Inventaire[space])=(ItemMap[X][Y]); //On prend l'objet pour le mettre dans la poche d'inventaire//
                    (ItemMap[X][Y])=RIEN; //On le retire de la map//
                    break; //Et on sort de la boucle//
                }
            }
            printf("Votre inventaire est plein."); //Cette partie ne devrait être atteinte que si la condition n'est vérifiée pour aucune poche//
        }
        break;

Le problème c'est que "Votre inventaire est plein." s'affiche à chaque fois.
Il s'agit en fait d'un projet de rogue-like, qui n'en est qu'à ses débuts.

J'espère avoir créé ce sujet dans la bonne section du forum, dans le cas contraire dites-le moi, je l’effacerais.

[Invité]

Bonjour,

Ton printf avec "Votre inventaire est plein" est en dehors de la boucle.
Tu rentres dans la boucle et effectue tes opérations. Une fois sortis de celle-ci (par un break ou normalement), le programme continue le cheminement normal des opérations et passe donc sur le printf, et l'affiche donc.

Tu pourrais d'ailleurs réécrire ta boucle bien plus simplement.

Code :

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

1
2
3
4
5
6
7
8
9
10
space = 0
while(space < 10 &&  Inventaire[space] != RIEN)  // Recherche d'une poche vide
    space++
 
if(space < 10){ // Si space est inférieur à 10; signifie qu'une poche vide a été trouvé
    // traitement
} 
else { // Sinon, aucune poche vide rencontré
   // printf poche plein
}

Je vois un "break" qui traine en dessous de ta boucle... Rassure moi, ce n'est pas là pour sortir de ton if ?

[Maskeren]

Je viens de de rajouter, un if (space>9) après ma boucle for, ça marche impec' merci.

Je vois un "break" qui traine en dessous de ta boucle... Rassure moi, ce n'est pas là pour sortir de ton if ?

Non non, le break en toute fin est là pour le switch/case qui n'appairait pas dans la portion de code que j'ai postée, je te rassure.
En tout cas, merci beaucoup pour ta réponse rapide !

[Invité]

Je t'en prie.
N'oublie pas de marquer le sujet comme [résolu].

[Sve@r]

Bonjour

Tu pourrais d'ailleurs réécrire ta boucle bien plus simplement.

Code :

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

1
2
3
space = 0;
while(space < 10 &&  Inventaire[space] != RIEN)  // Recherche d'une poche vide
    space++;

Et encore plus simplement...

Code c :

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

for (space = 0; space < 10 &&  Inventaire[space] != RIEN; space++);

Je vois un "break" qui traine en dessous de ta boucle... Rassure moi, ce n'est pas là pour sortir de ton if ?

Oui, c'est pour sortir de la fameuse "boucle if"

Je viens de de rajouter, un if (space>9) après ma boucle for, ça marche impec' merci.

Et ça ne te semblait pas plus évident et plus logique d'écrire if (space == 10) ???