pb concaténation d'une chaine avec un caractère

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:

---

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

Citation:

---

Envoyé par P'tite Nélodie

[...]
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:

---

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...

Code:

---

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:

---

1 2 3

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

---

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

Code:

---

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?

Code:

---

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

---

Non, il suffit de (avec des ' simples)

Code:

---

caractere[1]='\0';

---

Code:

---

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 ?

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

Code:

---

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 :oops:

merci pour ces conseils

je pense que mon code est correcte maintenant , non?

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, ...).

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:

---

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

Remarques :
plus traditionnellement :

Code:

---

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

---

devient :

Code:

---

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

---

Code:

---

taille_m=strlen(nouveau_mot);

---

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

Code:

---

taille_m=strlen(mot);

---

merci pour vos remarques et précieux conseils