Discussion :

[Nelmo]

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 :

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
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 :

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

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 !

[hiko-seijuro]

tu as oubliés les accolades

c'est :

Code :

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

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

[ricky78]

bonjour

Fait attention à la portee de tes variables.

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

Cordialement

[Nelmo]

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

merci

[Nelmo]

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 :

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
 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 :

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

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 !

[hiko-seijuro]

ouaip je pense :d

[Nelmo]

Oki !

Merci bien en tout cas

[hiko-seijuro]

oublie pas le tag résolu

[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