Discussion :

[KevinDA]

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
#include <iostream>
#include <sstream>
#include <windows.h>
 
using namespace std;
int main(){	
int id_menu;
string choix;
cout << "MENU" << endl;
cout << " " << endl;
cout << "Votre choix" << endl;
cout << " " << endl;
cout << "1" << endl;
cout << "2" << endl;
cout << "3" << endl;
cout << "4" << endl;
cout << "" << endl;
cin >> id_menu;	
if(id_menu == 1){
string choix = "Vous avez choisi 1";	
}
if(id_menu == 2){
string choix = "Vous avez choisi 2";
}
if(id_menu == 3){
string choix = "Vous avez choisi 3";
}
if(id_menu == 4){
string choix = "Vous avez choisi 4";
}
cout << choix << endl;
system("pause");
return 0;
}

Bonsoir, par rapport à un choix dans un menu j'arrive pas à l'afficher par la suite. Pourquoi ?

[Médinoc]

Tu dois déclarer choix hors des if pour pouvoir l'utiliser après.

[dalfab]

Dans tous tes if tu as défini une nouvelle variable choix, il ne faut pas déclarer une nouvelle mais modifier la valeur de la variable choix définie ligne 9.

Code :

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

1
2
3
if(id_menu == 1){
string choix = "Vous avez choisi 1";  // nouvelle variable choix
}                                     // la variable disparait à la fin du bloc

Il faut plutôt écrire :

Code :

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

1
2
3
if(id_menu == 1){
    choix = "Vous avez choisi 1";  // la variable choix reçoit une nouvelle valeur
}

[KevinDA]

Tu dois déclarer choix hors des if pour pouvoir l'utiliser après.

Je comprends pas j'ai pas envoyé de message en privé...

[Bousk]

C'est encore plus vicieux, et je serais surpris qu'au moins un warning ne ressorte pas. Si ce n'est pour variable déjà existante, au moins pour variable inutile.
choix existe en haut du main, puis est redéclarée dans chaque scope de if, donc on change une variable locale inutile, puis forcément à la fin on se retrouve avec un truc vide vu qu'on l'a jamais modifié.

[KevinDA]

Merci dalfab, j'avais pas codé en C++ depuis un moment. J'avais oublié ce détail.
Merci encore

[Médinoc]

Je comprends pas j'ai pas envoyé de message en privé...

C'est juste ma signature, ça n'était pas adressé à toi en particulier.
Disons qu'il y a eu une période où avoir ce message dans ma signature était nécessaire.