Discussion :

[P'tite Nélodie]

bonjour,

dans l'un de mes projets en C , j'ai besoin de créer de nouveaux mots ( chaine de caractères) en concaténant un mot ancien avec une lettre d'un alphabet ( tableau de caractère).

pour cela j'ai stocké les lettres possibles pour la concaténation dans un tableau de caractères, et j'ai essayé de concaténer un mot ancien avec une case de ce tableau afin de créer un nouveau mot.
et voilà comment j'ai fait :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
char nouveau_mot[5];
char mot[2];
strcpy(nouveau_mot," ");
strcpy(mot,"aa");
char alphabet[4]={'a','c','t','g'};
int i;
for(i=0;i<4;i++){
     strcpy(nouveau_mot,mot);
     strcat(nouveau_mot,alphabet[i]);
}

Je n'ai pas d'erreur de compilation mais un joli warning qui me prévient que dans l'appel de strcat le second argument produit un pointeur à partir d'un integer sans cast. et par conséquent j'ai un "segmentation fault" à l'exécution

comment est ce que je peux faire pour utiliser mon tableau alphabet et ainsi créer correctement mes mots ?

Merci d'avance

[crocodilex]

[...]
Je n'ai pas d'erreur de compilation mais un joli warning qui me prévient que dans l'appel de strcat le second argument produit un pointeur à partir d'un integer sans cast. et par conséquent j'ai un "segmentation fault" à l'exécution
[...]

Il faut lire le manuel de plus près....en effet strcat attend une chaine de caractère en 2eme argument, alors que toi tu y mets un caractère.
Pour ce qui est du "segmentation fault", cela provient vraisemblablement de ces 2 lignes de codes :

Code :

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

1
2
3
4
5
6
 
[...]
strcpy(mot,"aa");
[...]
strcat(nouveau_mot,alphabet[i]);
[...]

Je te donne un indice : une chaine de caractère se termine toujours par le caractère de fin de chaine '\0'. Donc c'est une histoire de dépassement...

[P'tite Nélodie]

je crois que j'ai trouvé une solution pour mon ptit pb de concaténation , mais elle ne me plait pas tellement

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
 
char nouveau_mot[5];
char mot[2];
strcpy(nouveau_mot," ");
strcpy(mot,"aa\0");
char alphabet[4]={'a','c','t','g'};
char caractere[2];
caractere[0]="";
caractere[1]="\0";
int i;
for(i=0;i<4;i++){
     strcpy(nouveau_mot,mot);
     caractere[0]=alphabet[i];
     strcat(nouveau_mot,caractere);
}

Je n'ai plus d'erreurs ou de warning, mais je trouve que c'est pas très pratique de passer par une variable de "caractere"
Est ce que cette solution vous semble correcte?

[diogene]

Code :

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

1
2
caractere[0]="";
caractere[1]="\0";

Non, il suffit de (avec des ' simples)

Code :

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

caractere[1]='\0';

Code :

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

1
2
3
4
5
for(i=0;i<4;i++){
     strcpy(nouveau_mot,mot);
     caractere[0]=alphabet[i];
     strcat(nouveau_mot,caractere);
}

Tu réinitialises nouveau_mot à chaque fois ?

[P'tite Nélodie]

Avec le deuxième post ,j'ai pu éliminer la variable "caractere" en faisant de la manière suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
char nouveau_mot[5];
char mot[2];
strcpy(nouveau_mot,"\0");
strcpy(mot,"aa\0");
char alphabet[4]={'a','c','t','g'};
int taille_m;
int i;
for(i=0;i<4;i++){
     strcpy(nouveau_mot,mot);
     taille_m=strlen(nouveau_mot);
     nouveau_mot[taille_m]=alphabet[i];
     nouveau_mot[taille_m+1]='\0';
}

en fait si je voulais utiliser strcat, c'est que j'avais en mémoire d'autres langages de programmation où la manipulation des chaines de caractères est plus facile qu'en C et j'ai intuitivement adpaté cela au C avec strcat

merci pour ces conseils

je pense que mon code est correcte maintenant , non?

[P'tite Nélodie]

oki j'avais récticfié pour les " et '
j'avoue que je confond un peu leur utilisation :
- " c'est pour une chaine de caractère,
- ', c'est pour UN caractère
c'est bien ça ?

Et oui je réinitialise bien mon nouveau_mot à chaque fois afin de créer plusieurs mots selon la lettre ajoutée.

Peut etre que l'initialisation n'est pas au bon endroit c'est possible car le code que je poste sur le forum n'est qu'un extrait de mon véritable code que j'adapte de manière générale pour faciliter la compréhension ( simplification, changement du nom de variable, ...).

[diogene]

Remarques :
plus traditionnellement :

Code :

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

1
2
strcpy(nouveau_mot,"\0");
strcpy(mot,"aa\0");

devient :

Code :

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

1
2
strcpy(nouveau_mot,"");
strcpy(mot,"aa");

Code :

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

  taille_m=strlen(nouveau_mot);

a toujours la même valeur et doit donc sortie de la boucle for :

Code :

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

  taille_m=strlen(mot);

[P'tite Nélodie]

merci pour vos remarques et précieux conseils

[pirouli]

Bonjour,

Tu dit que mot a une taille de deux octets (ligne 2). Et tu copies 3 ou 4 octets dedans (ligne 3). Il y a donc un dépassement mémoire. Ceci peut avoir de graves conséquences ou n'en avoir pas du tout, ça dépend de ton compilateur.
Dans tous les cas, il faut corriger ça pour avoir un code valide :

Code :

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

1
2
 
char mot[4];

Sinon la solution de passer par une chaine intermédiaire pour pouvoir utiliser strcat n'est pas stupide. Mais peut être que le plus simple c'est de se passer de strcat...

Pirouli

[diogene]

Code :

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

  strcat(nouveau_mot,alphabet[i]);

strcat concatène deux chaînes de caractères (terminées comme d'habitude par '\0') mais ne peut ajouter un simple char comme alphabet[i] à une chaîne. Les deux arguments de strcat doivent être des pointeurs sur char. strcat considère le caractère alphabet[i] (un int) comme l'adresse d'une chaîne -> warning et plantage.
Pour ajouter un caractère à la suite d'une chaîne, il faut remplacer le '\0' terminal par le caractère et ajouter un '\0' ensuite (et que le tableau de destination soit assez grand). par exemple :

Code :

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

1
2
3
i = strlen(nouveau_mot);
nouveau_mot[i] = ...        /* le caractère à ajouter */
nouveau_mot[i+1] = '\0' ;