Discussion :

[kase74]

Bonjour,
voici mon 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
static bool ExtractVariableName (const char *Source, char* &DestName)
{
    // The function look if the Source is a variable name and if it is, it extract just the name and stock it in DestName
    // A variable in the document is specify with its name between brackets '[' and ']'
    // If it is effectively a variable it return TRUE else it return FALSE
 
    DestName = NULL;
    if ((Source[0] == '[') and (Source[strlen(Source)-1]==']'))
    {
        const char* i = &Source[1];
        char VariableName[strlen(i)];
        strncpy(VariableName, i,strlen(i)-1);
        VariableName[strlen(i)-1]='\0';
        DestName = VariableName;
        return true;
    }
    return false;
}

Au retour de cette fonction, j'ai des soucis : la variable contient bien la bonne valeur, mais cette valeur est modifiée ensuite depuis un autre traitement qui n'a rien à voir.
Comment écrire correctement cette fonction ?
Merci d'avance pour vos réponses.

[ram-0000]

Normal, tu fais "DestName = VariableName" alors que "VariableName" est une variable sur la pile. En fin de fonction, ta pile est détuite donc ta variable "DestName" pointe sur n'importe quoi.

[Mat.M]

Ram-0000 a raison tu affectes à DestName une variable locale à la fonction.

Il faut que tu rappelles strcnpy ou strcpy:
strcpy(DestName , VariableName)

En C++ privilégier plutot std::string dans la mesure du possible plutot que char* ça évite les erreurs et c'est plus souple

[kase74]

Merci pour vos réponses aussi rapide.
Décidemment j'ai du mal avec les char*

j'ai réecrit ma fonction comme ceci :

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
static bool ExtractVariableName (const char *Source, char* DestName)
{
    // The function look if the Source is a variable name and if it is, it extract just the name wich is stocked in DestName
    // A variable in the document is specify with its name between brackets '[' and ']'
    // If it is effectively a variable it return TRUE else it return FALSE
 
    DestName = "";
    if ((Source[0] == '[') and (Source[strlen(Source)-1]==']'))
    {
        const char* i = &Source[1];
        strncpy(DestName, i,strlen(Source)-2);
        return true;
    }
    return false;
}

Ca plante au niveau de strncpy. Je pense que c'est parceque la taille de DestName = 0. Je pensais que strncpy se chargait du redimensionnement de la variable destination mais ce n'est apparemment pas le cas.
Comment faire ce redimensionnement ?
Merci.

PS : je suis obligé d'utiliser les char* dans ce cas. Sinon, j'utilise au maximum les string et effectivement, j'ai beaucoup moins de soucis.

[ram-0000]

On est dans un forum C++ alors fait un truc genre

Code :

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

static bool ExtractVariableName (const std::string & Source, std::string & DestName)

tu auras moins de difficultés

[kase74]

Effectivement, je vais poster dans le forum C.
Merci encore pour vos réponses.

[ram-0000]

Post verrouillé, la suite ici : Modif d'une variable char* dans une fonction dans le fourm C.