Problème switch case

Version imprimable

bonjour ( et oui, encore lui lol )
Le code qui suit devrait me permettre de tester la valeur de retour d'une fonction et de réaliser l'action souhaité grace à un swith case !

or, Monsieur le compilateur me jete comme un mal propre lors de la compilation.

voici 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
void CRecupValideesDlg::OnActualiser2() 
{
 char CMD[] = {"\r"};// variable stockant la commande. parametre de la fonction de sandrine
 int erreur;   // variable stockant le code d'erreur retourner par la fct récupération
 
 // fonction de Récupartion
 // à implémenter
 erreur = 0;   // affectation d'une valeur pour tester le switch case
 
 switch (erreur)
 {
  case 0 : // ce qui doit ce passer
   // lecture du fichier contenant le nom des fichiers :
   char buff[200];
   ifstream infile("liste"); // ouverture du fichier liste.txt
   //while(!infile.eof()) si plusieurs ligne dans le fichier, faire une boucle
   infile.getline(buff, sizeof(buff)); //lecture d'une ligne complete
   infile.close();
 
   // affichage du fichier dans la zone d'édition
   CString s(buff), s2("il y a ");   
   s.Replace(":"," fichiers présents :\r\n\r\n");  // change les ":" par la chaine avec un retour chariot + nvlle ligne
   s.Replace(",","\r\n");  // change les "," par des retour chariot + nvlle ligne
   s2 += s;
   m_edit.SetWindowText(s2); // affiche le contenu de s
   break;
 
  case 1 : 
   CString s(buff);
   m_edit.SetWindowText(s2);
   break;
 }
}

Si je réalise le test en mettant case 1 en commentaire, tout ce passe bien, il réagit au changement de valeur de la variable erreur.
Mais, dès que je tente de compiler avec du code dans le case 1, il me jete en me disant :

Code:

1
2
3
4
5
6
7
8
9
10
11
 
Compiling...
RecupValideesDlg.cpp
E:\Projet\C++\aquarium\RecupValideesDlg.cpp(84) : error C2360: initialization of 's2' is skipped by 'case' label
        E:\Projet\C++\aquarium\RecupValideesDlg.cpp(77) : see declaration of 's2'
E:\Projet\C++\aquarium\RecupValideesDlg.cpp(84) : error C2360: initialization of 's' is skipped by 'case' label
        E:\Projet\C++\aquarium\RecupValideesDlg.cpp(77) : see declaration of 's'
E:\Projet\C++\aquarium\RecupValideesDlg.cpp(84) : error C2360: initialization of 'infile' is skipped by 'case' label
        E:\Projet\C++\aquarium\RecupValideesDlg.cpp(69) : see declaration of 'infile'
E:\Projet\C++\aquarium\RecupValideesDlg.cpp(85) : error C2086: 's' : redefinition
Error executing cl.exe.

Si vous aviez des ptits trucs a me donner ... lol je serai le plus heureux !

tu as oubliés les accolades

c'est :

Code:

1
2
3
4
5
6
7
8
9
switch (erreur)
{ 
case 0: 
    {
        ... 
        break;
     }
...
}

04/05/2006, 14h44
ricky78

bonjour

Fait attention à la portee de tes variables.

Declare tes CString au debut de ta fonction et initialise les pendant tes CASE

Cordialement
04/05/2006, 14h57
Nelmo

d'accord, je vais de ce pas mettre en place vos recommendations !

merci

Alors vous aviez tous les deux raisons, il me manquait les parenthèses :?
* demande une corde pour ce pendre *

De plus, mes variables était mal déclarées ....

Voici le code final ;)

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
 switch (erreur)
 {
  case 0 : // ce qui doit ce passer
   {
   // lecture du fichier contenant le nom des fichiers :
   char buff[200];
   ifstream infile("liste"); // ouverture du fichier liste.txt
   //while(!infile.eof()) si plusieurs ligne dans le fichier, faire une boucle
   infile.getline(buff, sizeof(buff)); //lecture d'une ligne complete
   infile.close();
 
   // affichage du fichier dans la zone d'édition
   CString s(buff), s2("il y a ");   
   s.Replace(":"," fichiers présents :\r\n\r\n");  // change les ":" par la chaine avec un retour chariot + nvlle ligne
   s.Replace(",","\r\n");  // change les "," par des retour chariot + nvlle ligne
   s2 += s;
   m_edit.SetWindowText(s2); // affiche le contenu de s
   break;
   }
  case 1 : 
   {
   char buff[200];
   CString s(buff);
   m_edit.SetWindowText(s);
   break;
   }
 }
}

Sa fonctionne !

Ps : si je déclare mes variable au début, sa va devenir comme cela ?

Code:

1
2
3
4
5
6
7
 
void CRecupValideesDlg::OnActualiser2() 
{
 char CMD[] = {"\r"};// variable stockant la commande. parametre de la fonction de sandrine
 int erreur;   // variable stockant le code d'erreur retourner par la fct récupération
 char buff[200];
CString s(buff), s2("il y a ");

Merci de votre aide !

04/05/2006, 15h10
hiko-seijuro

ouaip je pense :d
04/05/2006, 15h10
Nelmo

Oki !

Merci bien en tout cas ;)
04/05/2006, 15h12
hiko-seijuro

oublie pas le tag résolu
04/05/2006, 15h17
Nelmo

Pour le coup, j'ai fait le test et donc lorsque je déclare

CString s(buff),s2("il y a"), sa déconne, je doit au moins les déclaré dans le case .... certainement à cause de Replace ....

Voili ...
En tout cas, merci sa me fait apprendre des choses ;)

allez, maintenant je vais me prendre la tête pour que le case m'ouvre une boite de dialogue :aie: