Discussion :

[michaeljeru]

Bonjour,
J'ai un algorithme, si je le simplifie, ça donne :

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
 
//init
char*	prefixe;
char code_tp[5 + 1];
char code_tb[5 + 1];
 
EXEC SQL 
			SELECT h_code_tp, h_code_tb
			INTO	:code_tp, :code_tb
			FROM matable
			WHERE ...
		;
 
if(strcmp(code_tp,"") != 0) {
			prefixe= "TP ";
			strcat(prefixe,code_tp);       *
}
else if(strcmp(code_tb,"") != 0) {
			prefixe= "TB ";
			strcat(prefixe,code_tb);
}

Le code, en l'état actuel, comporte manifestement un problème, qui se situe sur la ligne du strcat (*). J'ai inséré des traces tout autour avant et après, et la dernière trace qui sort est celle juste avant le strcat. Après, plus rien, le programme plante.
Je ne comprends pas quel est le pb, ni pourquoi il survient là.
Merci pour votre aide.

[Invité]

Bonjour,

Seuls 4 caractères sont alloués à prefixe.
En faisant strcat, le programme essaye d'écrire dans des zones mémoires non allouées.

Il faudrait faire quelque-chose come ça:

Code :

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

1
2
3
4
prefixe = new char[9]; // Edit pour en dessous: désolé pour l'erreur
(...)
strcpy(prefixe, "TP ");
strcat(prefixe, code_tp);

ou bien

Code :

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

1
2
3
prefixe = new char[9];
(...)
sprintf(prefixe,"TP %s",code_tp);

[Elijha]

Bonjour,

Ta variable prefixe est définie en tant que pointeur sur une chaine de caractères. Si tu veux lui attribuer une chaine il faut faire un malloc du nombre d'octets dont tu as besoins + 1 pour le \0 final. Sans oublier bien-sur de libérer l'espace mémoire allouée.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
char *prefixe = NULL ;
 
if(strcmp(code_tp,"")!= 0) {
    prefixe = (char*)malloc(3+strlen(code_tp)+1) ;  // "TP " + "nombre d'octets de code_tp" + '\0'
    if(prefixe!=NULL) {
        sprintf(prefixe, "TP %s", code_tp) ;
        // Utilisation de prefixe
        { ... }
        free(prefixe) ;     // Libération de la mémoire précédemment allouée.
        prefixe = NULL ;    // RAZ de prefixe
    }
}

[edit] : J'avais oublié de mettre à NULL prefixe après la libération de la mémoire.

[jeroman]

Il faudrait faire quelque-chose come ça:

Code :

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

1
2
3
4
prefixe = new char[9];
(...)
strcpy(prefixe, "TP ");
strcat(prefixe, code_tp);

ou bien

Code :

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

1
2
3
prefixe = new char[9];
(...)
sprintf(prefixe,"TP %s",code_tp);

Pas en C !

[michaeljeru]

Merci pour vos réponses, j'ai finalement utilisé la fonction sprintf, qui m'a paru plus claire à utiliser que strcat.
Et ça marche.
Effectivement, le = new char[10], ce n'est pas du C !
Je clos le topic.