Discussion :

[akrogames]

Bonjour,

J'essaye de fusionner deux chaines de caractères sans jamais y parvenir même en regardant la documentation. Il me retourne une erreur de segmentation.

Voici mon code C:

Code :

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

1
2
3
4
5
6
7
8
    char* tmp;
char* engine;
 
    tmp = (char*)malloc(100*sizeof(char));
    engine = (char*)malloc(1024*sizeof(char));
    engine = "<?xml version=\"1.0\" encoding=\"UTF-8\"?>";
    sprintf(tmp,"<plateau></plateau>\n");
    engine = strcpy(engine, tmp);

J'ai une erreur de segmentation au niveau du strcpy et je ne comprend pas pourquoi. J'ai essayé d'utiliser strcat avec le même résultat. Pouvez-vous m'aider ?

Merci d'avance
Bonne journée

[Flow_75]

Salut,
déja, pour remplir une chaine de caractères, tu peux pas simplement faire

Code :

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

engine = "<?xml version=\"1.0\" encoding=\"UTF-8\"?>";

Il faut faire :

Code :

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

strcpy(engine, "<?xml version=\"1.0\" encoding=\"UTF-8\"?>");

Ensuite tu pourras faire ton strcat

Code :

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

strcat(engine, tmp);

[matafan]

Ta ligne :

Code :

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

engine = "<?xml version=\"1.0\" encoding=\"UTF-8\"?>";

ne fait pas ce que tu penses qu'elle fait. Elle ne copie la chaine de droite dans le buffer alloué à la ligne précédente ; elle fait simplement pointer le pointeur "engine" vers la chaine statique "<?xml version=\"1.0\" encoding=\"UTF-8\"?>". Donc après cette instruction le buffer alloué à la ligne précédente est perdu, et engine pointe vers une zone mémoire que tu n'as pas le droit de modifier.

Le strcpy() qui suit va donc écrire des données dans une zone que tu n'as pas le droit de modifier.

La solution a été donnée par Flob91.

[akrogames]

Merci à vous deux pour ces précisions.

Comment je peux faire alors pour faire un code plus jolie comme en java par exemple :

En java :

Code :

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

1
2
3
4
 
String engine = "TEST";
engine += "allo";
engine += "coucou";

Ce qui donne :
TESTallocoucou

Je sais pas si c'est possible en C...

[matafan]

Ce n'est pas possible en C.

[pfeuh]

Salut,

En C c'est une usine à gaz, c'est faisable mais ça n'a pas beaucoup d'intérêt. Il faut clairement changer d'habitudes de programmation, car si on peut facilement passer un bout de soft de C vers n'importe quel langage "évolué", l'inverse est beaucoup moins évident.

A+

Pfeuh

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
#include <stdio.h>
#include <string.h>
 
char* addStrings(char* str1, char* str2, char* str3)
{
    size_t size = strlen(str1) + strlen(str2) + strlen(str3) + 1;
    char* ret_str = malloc(size);
    if(ret_str == NULL)
    {
        return NULL;
    }
    strcpy(ret_str, str1);
    strcpy(&ret_str[strlen(str1)], str2);
    strcpy(&ret_str[strlen(str1) + strlen(str2)], str3);
    return ret_str;
}
 
int main(int argc, char **argv)
{
    char* str1 = "TEST";
    char* str2 = "allo";
    char* str3 = "coucou";
    char* mystring = addStrings(str1, str2, str3);
 
    if(mystring == NULL)
    {
        return 1;
    }
    printf("%s\n", mystring);
    free(mystring);
    return 0;
}

[ram-0000]

et pourquoi pas

Code :

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

1
2
    strcat(ret_str, str2);
    strcat(ret_str, str3);

à la place de

Code :

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

1
2
    strcpy(&ret_str[strlen(str1)], str2);
    strcpy(&ret_str[strlen(str1) + strlen(str2)], str3);

cela me parait plus lisible

[pfeuh]

cela me parait plus lisible

Ça l'est, c'est juste que je ne me rappelais plus de cette fonction