Discussion :

[Thordax]

Hello tout le monde, lorsque j'effectue une compilation du code suivant, sous Solaris :

Code :

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

1
2
3
4
5
6
7
char* Fichier_analyse::stringtochar(std::string entree)
{   //CONVERTIT STRING EN CHAR
    size_t size = entree.size() + 1;
    char  *buffer = new char[ size ];
    strncpy( buffer, entree.c_str(), size );
    return(buffer);
}

J'obtiens le warning suivant, apres compilation :

line 78: Warning: buffer hides Fichier_analyse::buffer.

Je crois que le "buffer" créé dans la méthode est susceptible de rentrer en conflit avec le buffer créé dans la classe Fichier_analyse.

Est-ce effectivement le cas ?

[Matthieu Brucher]

Non, il indique simplement que ta variable locale cache ta variable de classe.

[Médinoc]

Et comme cette fonction ne semble avoir aucun besoin des variables de classe, je pense que tu as deux solutions:

1. Sortie ta fonction de la classe
2. La déclarer static (si ça ne corrige pas le warning, il faudra quand même changer le nom de variable)

[Thordax]

Et comme cette fonction ne semble avoir aucun besoin des variables de classe, je pense que tu as deux solutions:

1. Sortie ta fonction de la classe
2. La déclarer static (si ça ne corrige pas le warning, il faudra quand même changer le nom de variable)

Eh bien, justement, la méthode est indépendante de toute classe (pas de préfixe maclasse::ma_methode)

Pour la déclarer Static, je dois faire un

Code :

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

1
2
3
4
static ma_methode{
...
...
}

à l'extérieur de toute classe ?

[Matthieu Brucher]

Si c'est en dehors de toute classe, c'est une fonction libre normale, static sert dans une classe.

[Thordax]

Si c'est en dehors de toute classe, c'est une fonction libre normale, static sert dans une classe.

Alors je peux tout simplement rennomer le membre privé de la classe (ex : buffer -> buffer_m) ?

[Médinoc]

Edit: un joli double-post croisé...

Eh bien, justement, la méthode est indépendante de toute classe (pas de préfixe maclasse::ma_methode)

Code de Thordax :

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

char* Fichier_analyse::stringtochar(std::string entree)

À moins que Fichier_analyse soit un namespace et Fichier_analyse::buffer une variable globale ?
Une variable globale en C++ alors qu'on a les singletons et les variables membres static ?

Pour la déclarer Static, je dois faire un

Code :

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

1
2
3
4
static ma_methode{
...
...
}

à l'extérieur de toute classe ?

Non, sauf si tu veux qu'elle soit limimitée au fichier source.
(Et sorti d'une classe, je n'appelle plus cela une méthode: C'est une fonction, tout simplement.
Et j'ai déjà tord d'appeler cela une méthode, car en C++, on dit "Fonction membre"; mais "méthode" est plus court)

[Thordax]

Edit: un joli double-post croisé...

Code de Thordax :

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

char* Fichier_analyse::stringtochar(std::string entree)

À moins que Fichier_analyse soit un namespace et Fichier_analyse::buffer une variable globale ?
Une variable globale en C++ alors qu'on a les singletons et les variables membres static ?

Effectivement, si je lis mal mon code, aussi ...

Non, sauf si tu veux qu'elle soit limimitée au fichier source.
(Et sorti d'une classe, je n'appelle plus cela une méthode: C'est une fonction, tout simplement.
Et j'ai déjà tord d'appeler cela une méthode, car en C++, on dit "Fonction membre"; mais "méthode" est plus court)

Donc, je dois la créer statique dans une classe ?

[Médinoc]

Ou en faire une fonction hors-classe non-static.

[Matthieu Brucher]

Oui, si elle a un rapport avec une classe. Si elle n'a aucune rapport avec aucune classe, c'est une fonction libre dans un namespace, c'est tout.