Discussion :

[senvedgi]

bonjour à tous!
La fonction FcStrAlz analyse une chaine "1234..".
Si un chiffre est présent devant la fonction un drapeau "0" empêche l'apparition d'une ligne.
Mais la valeur du drapeau ne sort pas de la fonction...

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
//analyse de chaine
#include <iostream>
#include <string>
using namespace std;
 
 
string FcStrAlz(string StrAlz,string StrItm)//fonction confrontant
{//Fc/déb 13/09/11                          //un verrou "1"..."n"
cout<<"-----------------------"<<endl;      //avec sa présence
     int taille = StrAlz.size ();           //dans la chaine StrAlz
     int t=0;                               //si verrou la ligne s'efface
     string ElmStrAlz,stp="0";              //si verrou absent ou ""
     cout<<"  /d/<"<<stp<<">"<<endl;        //la ligne apparaît
     while (t<taille)
          {ElmStrAlz=StrAlz.substr(t,1);
          t++;
          cout<<ElmStrAlz<<" ";
          if (ElmStrAlz==StrItm)
              {stp="0";break;}              //si 0 efface ligne
              else stp="1";                 //si 1 igne apparait
          }
          cout<<"/z/<"<<stp<<">"<<endl;
cout<<"------------------------"<<endl;
     return stp;
}//Fc/fin 13/09/11
 
 
//prg essai
string StrAlz="123456";//chaine des verrous possibles
string Alz,StrItm,stp;
int main ()
{
    cout<<"ei-<"<<stp<<">-"<<endl;                                    //état stp entrée int main
        StrItm="1";                                                   //verrou "1" actionnant la L39
        Alz=FcStrAlz(StrAlz,StrItm);
    cout<<"1ex-<"<<stp<<">-ex pourquoi TOUJOURS VIDE!"<<endl;         //état stp sortie de fonction
        if (stp=="0")                                                 //ligne actionnée par L36
        {cout<<"§1 effacé"<<endl;}
        if (stp=="1")                                                 //cette ligne fonctionne pas car L36 a valeur "1"
        {cout<<"§1 apparaît"<<endl;}
 
        //ici pas de verrou 3 installé(on pourrait mettre StrItm="";)
        Alz=FcStrAlz(StrAlz,StrItm);
    cout<<"3ex-<"<<stp<<">-ex pourquoi toujours vide!"<<endl;         //état stp sortie de fonction
        if (stp=="0")                                                 //ligne fonctionne pas car L44 sans valeur
        {cout<<"§3 effacé"<<endl;}
        if (stp=="1")                                                 //ligne fonctionne car L44 sans valeur
        {cout<<"§3 apparaît"<<endl;}
    cout<<"0 pour sortir de int main";int zer;cin>>zer;
}

Où est l'erreur?
Merci pour vos lumières.
Sen.

[bertry]

Mais la valeur du drapeau ne sort pas de la fonction...

La chaine stp globale n'est pas la chaine stp locale de ta fonction!!!!!!!
En revanche, ta fonction retourne bien le contenu de du stp local de la fonction et affecte cette valeur à Alz dans le main. La fonction semble, elle, bien fonctionner à première vue.

[senvedgi]

Merci.Je vais revoir la notion globale/locale.
Mais ne suis pas sûr de trouver.
Sen.

[Emmanuel Deloget]

Merci.Je vais revoir la notion globale/locale.
Mais ne suis pas sûr de trouver.
Sen.

1/ Toutes les variables déclarées dans des blocs {...} sont locales à ces blocs

2/ Toutes les variables déclarées en dehors de tout bloc sont dites "ayant la portée du non d'espace", c'est à dire qu'elle sont globales dans le nom d'espace (namespace). Si il n'y a pas de nom d'espace, elles sont globales au programme.

3/ une variable peut être cachée par une autre. Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
voif function(int param) // param a par exemple la valeur 3
{
  std::cout << "param = " << param << std::endl;
  int param = 4:
  std::cout << "param = " << param << std::endl;
  if (param != 3) {
    float param = 3.5f;
    std::cout << "param = " << param << std::endl;
  }
}

Dans ton code, tu définis une variable globale stp (en dehors de toute fonction ; ligne 31).

Puis, dans la fonction FcStrAlz (sérieux, il manque des touches à ton clavier ? Et pourquoi Fc ? Tu avais peur de confondre la fonction avec une livre de beurre fondu ?), tu redéfini une variable stp locale (ligne 13) qui cache la variable stp globale.

Du coup, les modifications de stp qui sont dans la fonction FcStrAlz ne modifie que la variable locale, et pas la variable globale.

[senvedgi]

beurre fondu:quand je crée une fonction elle commençe toujours par Fc(+)XYZ.Les strings associées sont voisines voires identiques à XYZ pour pouvoir m'y retrouver aujourd'hui ...et plus tard(smiley vieux).Homogénéité oblige.
Merci docteur pour le diagnostic.J'essaye de modifier astucieusement les déclarations de stp pour obtenir le résultat escompté et espère revenir bientôt pour présenter un travail fini Ou trouver une autre solution.
Sen.

[koala01]

Salut

beurre fondu:quand je crée une fonction elle commençe toujours par Fc(+)XYZ.Les strings associées sont voisines voires identiques à XYZ pour pouvoir m'y retrouver aujourd'hui ...et plus tard(smiley vieux).Homogénéité oblige.

A vrai dire, je ne suis pas partisan de cette notation (qui se rapproche de la notation hongroise) simplement parce que:

• On se doute qu'une fonction est une fonction, et non une livre de beurre fondu ( )
• il est beaucoup plus facile de s'y retrouver lorsque les fonctions, variables et types ont des noms clairs et explicites, et, surtout pas, des abréviations

Le nom de tes fonctions, types et variable ne coute rien en terme de performance:
une fonction nommée fcMachChose ne prendra pas plus d'espace en mémoire que la même fonction nommé faitMachinChoseAvecCeci, et, l'auto complétion aidant, cela ne te prendra même pas plus de temps pour écrire l'invocation de la deuxième que pour le faire pour la première.

Par contre, la fonction "faitMachinChoseAvecCeci" sera beaucoup plus facile à comprendre lorsque l'on relira le code ( et tu n'es pas sans savoir qu'un code est beaucoup plus souvent lu / étudié qu'il n'est écrit / compilé ), ne serait ce que parce qu'on ne doit pas, en plus de tout le reste, se souvenir que l'on utilise "classiquement" str pour parler de chaine de caractères ou que Alz correspond à quelque chose de particulier (à quoi ca correspond, d'ailleurs )

[bertry]

J'ajoute une précision :

Si tu souhaite effectivement que la variable stp modifiée par ta fonction soit la variable globale stp il suffit de déclarer la variable globale stp avant ta fonction FcStrAlz et de ne pas redéclarer stp dans cette fonction.
Voici une version ou la fonction modifie la variable globale:

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
 
//analyse de chaine
#include <iostream>
#include <string>
using namespace std;
 
//prg essai
//Déclarér les variables globales
string StrAlz="123456";//chaine des verrous possibles
string Alz,StrItm,stp;
 
string FcStrAlz(string StrAlz,string StrItm)//fonction confrontant
{//Fc/déb 13/09/11                 //un verrou "1"..."n"
cout<<"-----------------------"<<endl;//avec sa présence
     int taille = StrAlz.size ();   //dans la chaine StrAlz si verrou la ligne s'efface
     int t=0;						//si verrou absent ou "" la ligne apparaît
     string ElmStrAlz;
	 stp="0";//Ne pas redéclarer stp sinon elle devient locale ( seulement l'initialiser )
     cout<<"  /d/<"<<stp<<">"<<endl;
     while (t<taille){
		 ElmStrAlz=StrAlz.substr(t,1);
         t++;
         cout<<ElmStrAlz<<" ";
         if (ElmStrAlz==StrItm){
			 stp="0";
			 break;
		 }              //si 0 efface ligne
         else
			 stp="1";                 //si 1 igne apparait
	 }
     cout<<"/z/<"<<stp<<">"<<endl;
	 cout<<"------------------------"<<endl;
     return stp;
}//Fc/fin 13/09/11
 
int main ()
{
    cout<<"ei-<"<<stp<<">-"<<endl;                                    //état stp entrée int main
    StrItm="1";                                                   //verrou "1" actionnant la L39
    Alz=FcStrAlz(StrAlz,StrItm);
 
    cout<<"1ex-<"<<stp<<">-ex pourquoi TOUJOURS VIDE!"<<endl;         //état stp sortie de fonction
    if (stp=="0"){                                              //ligne actionnée par L36
        cout<<"§1 effacé"<<endl;
	}
	if (stp=="1"){                                                 //cette ligne fonctionne pas car L36 a valeur "1"
		cout<<"§1 apparaît"<<endl;
	}
 
    //ici pas de verrou 3 installé(on pourrait mettre StrItm="";)
    Alz=FcStrAlz(StrAlz,StrItm);
    cout<<"3ex-<"<<stp<<">-ex pourquoi toujours vide!"<<endl;         //état stp sortie de fonction
	if (stp=="0"){                                                 //ligne fonctionne pas car L44 sans valeur
		cout<<"§3 effacé"<<endl;
	}
	if (stp=="1"){                                                 //ligne fonctionne car L44 sans valeur
        cout<<"§3 apparaît"<<endl;
	}
    cout<<"0 pour sortir de int main";int zer;cin>>zer;
}

[senvedgi]

Merci:çà marche!
Ce qui me chagrine c'est l'obligation de déclarer la variable stp avant la FonCtion.La déclaration de stp "fait donc partie de la fonction"..
D'autre part si ma fonction fonctionne en temps que telle l'utilisation de la chaine de verrous possible(StrAlz...123..) est mal foutue.Je vais revoir la conception de mon effacement sélectif de lignes de menu avant un switch(nn).
En attendant merci pour la leçon globale/locale avec l'exemple fourni.

Sen.

[bertry]

Si je peux me permettre, l'utilisation de variables globales est fortement déconseillée ( car n'importe qui peut y accéder ) , il est toujours préférable d'utiliser des variables locales ( comme tu l'as fais dans ton premier programme ) et de récupérer la valeur souhaitée par retour ( comme tu l'avais fait; je me répète )

Comme je vois que tu est débutant, je te conseille cette excellente lecture : Penser en C++ disponible gratuitement sur ce site