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 ?

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

[jowo]

Bonjour,

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