Discussion :

[casier]

bonjour,

j'ai beau réfléchir ou essayer des choses comme un bourrin sans réfléchir, ou encore chercher (et trouver) sur le net comment on utilise strcat, y'a pas, je reste bloqué depuis vraiment longtemps, donc je me permets de vous soumettre mon problème, qui est très simple: définir deux chaines de caractères et les concaténer (j'ai honte)....

je sais comment on fait (en théorie), je sais que strcat est de type char *, qu'elle prend un char *s1 et une chaine constate s2 et qu'il renvoie s1 à laquelle on a ajouté s2, mais malgrè ça, j'essai et mon programme C plante lamentablement à cette opération....Et pourtant comme je l'ai dit, j'ai tout essayé (enfin, il me semble !), des pointeurs, des pas pointeurs, de tout, rien à faire, soit ça marche pas, soit j'ai un warning "assing arg 1 of `strcat' makes pointer from integer without a cast ", soit "assignment makes integer from pointer without a cast ", soit ça plante le prog...

Quelqu'un peut-il me montrer un exemple de deux lignes qui fonctionne que je vois où peut bien être mon problème ?

merci beaucoup

[Franck.H]

Le mieux serait encore que tu nous montre ton code en question avec en plus tes déclaration/allocations de pointeurs, tes tableaux ... la totale quoi. Ta chaîne cible est au moins assez grande pour accueillir la seconde chaîne ?

[jowo]

Bonjour,

Montre-nous ton code et on te dira où se trouve ton problème.

[casier]

ben disons que comme j'ai essayé 30000 choses, je sais plus quoi vous soumettre ...
Le truc le plus bête que j'ai envie de faire est:

Code :

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

1
2
3
4
5
6
7
 
char *s1;
char *s2;
char *s3;
s1="toto";
s2="titi";
s3=strcat(&s1,&s2);

Je sais bien qu'il faut que s3 soit assez grande pour acceuillir s1+s2, j'ai essayé de définir des char s1[100] etc, mais là je peux même pas faire une affectation du genre s1="toto", ça veut pas...j'ai essayé tout ce que j'ai pu, mais à chaque fois j'ai soit des erreurs (comme j'ai dit dans mon premier post), soit ça plante le prog).

edit: le code que je vous ai soumis ne plante pas mais me sort un warning sur les types des arguments s1 et s2 et le résultat est n'importe quoi

[Franck.H]

Je sais bien qu'il faut que s3 soit assez grande pour acceuillir s1+s2

Heu ouais mais tu sais peut-être pas qu'il faut allouer un espace assez grand (longueur de s1+s2+1) à s3 pour pouvoir accueillir la concatenation ! D'ailleurs s3 doit également être le premier argument de strcat !

Voir : strcat pour lplus d'informations sur la fonction !


De plus:

Code :

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

s3=strcat(&s1,&s2);

vu que tu utilises déjà des pointeurs, il faut passer le pointeur et non pas leur adresse !

[casier]

si, j'avais cru comprendre qu'il fallait un +1 pour le caractère de fin, mais là j'ai fait large: de taille 100 alors que s1+s2 est plus petit.
J'ai essayé de faire s2=strcat(&s2,&s1), mais ça marche pas non plus, ça plante pas mais le résultat est débile, c'est pas du tout ce qu'on peut attendre...

lorsque je ne mets pas le & (je suis d'accord avec toi: il ne le faut pas...), le programme plante cash, c'est pour ça que j'ai essayé un peu tout et n'importe quoi

[Franck.H]

si, j'avais cru comprendre qu'il fallait un +1 pour le caractère de fin, mais là j'ai fait large: de taille 100 alors que s1+s2 est plus petit.
J'ai essayé de faire s2=strcat(&s2,&s1), mais ça marche pas non plus, ça plante pas mais le résultat est débile, c'est pas du tout ce qu'on peut attendre...

Voici un exemple vite fait qui fonctionne:

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
 
int main (void)
{
   char * s1 = "toto";
   char * s2 = "titi";
   char * s3 = NULL;
 
 
   s3 = malloc (strlen (s1) + strlen (s2) + 1);
 
   if (s3 != NULL)
   {
      strcpy (s3, s1);
      strcat (s3, s2);
   }
 
   printf ("%s\n", s3);
   free (s3);
   s3 = NULL;
 
 
   return EXIT_SUCCESS;
}

Je pense que là tu as tout en main pour réussir

[Emmanuel Delahaye]

le code que je vous ai soumis ne plante pas mais me sort un warning sur les types des arguments s1 et s2 et le résultat est n'importe quoi

Normal. Il invoque plusieurs comportements indéfinis.

strcat(), c'est pas bien difficile. Le but est "d'ajouter" une chaine à une chaine existante. Il faut simplement connaitre les contraintes du C, à savoir :
- Une chaine est un tableau de char initialisé avec une séquence de caractères terminée par un 0.
- une chaine dites 'littérale' n'est pas modifiable.

le premier paramètre de strchar() est l'adresse du premier élément du tableau de char de destination. A l'évidence, il doit être modifiable. de plus, comme on ajoute à de l'existant, il faut qu'il y ait un existant, c'est à dire une chaine initiale (même vide) dans le tableau de destination. A l'évidence, il faut aussi qu'il y a ait de la place pour le suite.

le deuxième paramètre est l'adresse du premier élément de la chaine à ajouter. Celle-ci n'est que lue, elle peut donc être non-modifiable.

Tu n'as plus qu'à écrire le code.

[Emmanuel Delahaye]

Code :

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

1
2
 
   s3 = malloc (strlen (s1) + strlen (s2) + 1);

Je pense que là tu as tout en main pour réussir

Pas si sûr !

malloc(), c'est bien, mais il ne sait pas utiliser strcat() et tu lui rajoutes un concept et des fonctions en plus ? Pas d'accord. Restons en aux tableaux de char pour le moment.

[Franck.H]

Pas si sûr !

malloc(), c'est bien, mais il ne sait pas utiliser strcat() et tu lui rajoutes un concept et des fonctions en plus ? Pas d'accord. Restons en aux tableaux de char pour le moment.

Mouais c'est pas si faut

[casier]

Merci pour toutes ces réponses. Voilà quelque chose que j'avais essayé et qui ne marche pas: le prog plante.

Code :

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

1
2
3
4
5
6
 
char *s1;
char *s2;
s1="titi";
s2="toto";
s1=strcat(s1,s2);

Le seul truc que je vois est que s1 n'a pas la taille recquise (je ne les précise pas). Mais problème, pour définir une taille à s1, si je fais un truc du genre:

Code :

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

1
2
3
4
5
6
 
char *s1[100];
char *s2;
s1="titi";
s2="toto";
s1=strcat(s1,s2);

l'affectation de s1 me sort un erreur, qui me parait logique puique un truc correct serait s1[0]="toto", s1 serait un tableau de chaine de caractères, ce que je ne veux pas...bref, j'y arrive toujours pas...

[Franck.H]

Merci pour toutes ces réponses. Voilà quelque chose que j'avais essayé et qui ne marche pas: le prog plante.

Code :

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

1
2
3
4
5
6
 
char *s1;
char *s2;
s1="titi";
s2="toto";
s1=strcat(s1,s2);

Comme Emmanuel l'a déjà dit, la chaîne dest (premier argument de strcat) doit être une chaîne modifiable alors que là s1 dans ton exemple est une chaîne statique donc non modifiable.

Tu as le choix entre un tableau statique ou un tableau dynamique (comme montré dans mon exemple). Pour un tableau statique, tu peut faire:

Code :

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

1
2
3
4
5
 
/*
 * Tableau de 10 char rempli avec des zeros.
 */
char dest [10] = {0};

Ceci ne peut fonctionner

Code :

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

char *s1[100];

Ici tu a créé un tableau de 100 pointeurs sur char ... c'est encore pire

[casier]

disons que je me doute que c'est une histoire de chaine qui a la mauvaise taille, mais chaque fois que j'essai de définir une taille à s1 (après tout, je peux en faire une chaine statique mais de grande taille, ça devrait marcher), et bien je peux plus rien en faire...(comme lorsqu'il y avait le * avec le tableau)

Par exemple:

Code :

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

1
2
3
 
char s1 [100]={0};
s1="toto";

me sort un erreur ! "incompatible types in assignment". C'est pour ça que j'ai abandonné cette idée, car seule une affectation du genre s1[0]="toto" passe à la compil mais plante mon prog (et puis de toute façon ça m'aurait pas intéressé ). Je suis toujours coincé...

[Franck.H]

disons que je me doute que c'est une histoire de chaine qui a la mauvaise taille, mais chaque fois que j'essai de définir une taille à s1 (après tout, je peux en faire une chaine statique mais de grande taille, ça devrait marcher), et bien je peux plus rien en faire...(comme lorsqu'il y avait le * avec le tableau)

Par exemple:

Code :

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

1
2
3
 
char s1 [100]={0};
s1="toto";

me sort un erreur ! "incompatible types in assignment".

Normal, ici tu disposes d'un tableau (d'ailleur pour ton exemple de départ, une taille de 10 suffit amplement ) donc maintenant si tu veut le remplir, il te faut au moins user de la fonction strcpy ce qui nous donne:

Code :

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

strcpy (s1, "toto");

mais

Code :

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

strcat (s1, "toto");

fonctionne également du moment que le tableau est initialisé comme ici par exemple, avec des zéros


Si tu veux vraiment remplir le tableau avec une permière chaîne sans fonctions, tu peux mais à ce prix:

Code :

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

1
2
3
4
5
s1 [0] = 't';
s1 [1] = 'o';
s1 [2] = 't';
s1 [3] = 'o';
s1 [4] = '\O';

Ce qui n'est pas très souhaitable

[Emmanuel Delahaye]

Le seul truc que je vois est que s1 n'a pas la taille recquise (je ne les précise pas). Mais problème, pour définir une taille à s1, si je fais un truc du genre:

Code :

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

1
2
3
4
5
6
 
char *s1[100];
char *s2;
s1="titi";
s2="toto";
s1=strcat(s1,s2);

l'affectation de s1 me sort un erreur<...>

Evidemment, pourquoi définis tu une tableau de 100 pointeurs sur char ? Je t'ai pourtant indiqué ce qu'il fallait faire...

- Une chaine est un tableau de char <...>

Tu trouves que ça ressemble à un tableau de char?

Code :

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

char *s1[100];

Alors c'est quoi le problème ?

• J'écris mal ?
• J'écris trop petit ?
• Tu ne m'a pas lu ?
• Tu ne comprends pas ce que j'ai écrit ?
• Tu penses à autre chose ?
• Tu fais n'importe quoi au hasard jusqu'à ce que ça marche ?

Je bien aider, mais pas n'importe comment...

[casier]

Y'a pas de manipulation plus aisée des chaines de caractères ?! Ouch, dis donc, de souvenir, en C++ c'est archi aisé comparé à ça ! Ca veut dire que, comme on sait qu'une chaine de char est un tableau de char, il le voit vraiment comme ça et qu'on est obligé de le manipuler comme tel ou utiliser strcopy à chaque fois, effectivement il est normal que ce que je faisais ne marche pas !

En tout cas ça marche maintenant, merci à tous pour votre aide !

[Franck.H]

Y'a pas de manipulation plus aisée des chaines de caractères ?! Ouch, dis donc, de souvenir, en C++ c'est archi aisé comparé à ça !

Je ne vois vraiment rien de compliqué

Ca veut dire que, comme on sait qu'une chaine de char est un tableau de char, il le voit vraiment comme ça et qu'on est obligé de le manipuler comme tel ou utiliser strcopy à chaque fois, effectivement il est normal que ce que je faisais ne marche pas !

Oui en C une chaîne n'est autre qu'un tableau de char, c'est comme ca et pas autrement !

[Emmanuel Delahaye]

Code :

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

1
2
3
 
char s1 [100]={0};
s1="toto";

me sort un erreur ! "incompatible types in assignment". C'est pour ça que j'ai abandonné cette idée,

Franchement, tu crois que le C, ça se devine en faisant n'importe quoi au hasard ? Tu n'as donc pas de livre de C, de cours, de tuto ?

Tes questions sont complètement basiques et couvertes par n'importe quel livre ou tutoriel de C. Il n'en manque pas sur ce site...

Il faut cesser de bricoler et se mettre au travail avec un livre ou un cours sérieux.

[casier]

? Mais je ne fais pas un procès au langage C ! Je dis juste que de souvenir, en C++, ces manips n'étaient pas nécessaires, rien de plus, j'ai jamais dit que c'était compliqué, simplement je m'en rappelait pas du tout et je comprends pourquoi je n'ai même pas pensé que mes problèmes pouvaient venir de là .

Je ne bricole pas comme ça au hasard non plus, j'ai déjà fait un paquet d'applis bien pratiques en C qui sont relativement évoluée quand même, simplement les chaines de char j'avais pas eu à effectuer des manips de ce genre dessus depuis un moment, donc j'ai oublié quelques bases, désolé (puis ça faisait un moment que j'avais pas retouché au C). Bon là, c'est vrai que j'en avait tellement marre que comme je l'ai dit "j'ai essayé 3000 trucs" dont je savais raisonnablement que ça pouvait pas marcher, ce qui explique ta remarque emmanuel, mais c'était plus du désespoir qu'autre chose, et sincèrement, c'est pas faute d'avoir cherché mon erreur sur le net...simplement je cherchais pas du bon coté.

Pour finir, si vous relisez bien, je n'ai pas posé la quesiton de savoir si les char étaient des tableaux de char, je l'ai "affirmé"

En tout cas merci, ça fonctionne parfaitement et je peux enfin continuer mon prog.

[Franck.H]

Oublie pas de passer ce topic en