Discussion :

[Devely]

Bonjour,

Je suis en train de recoder la fonction strcat de la lib string.h
Après avoir écrit mon code, vérifié et revérifié, mon programme SegFault.
Pouvez vous m'aider ?

Voici le code :

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
char	my_putchar(char c)
{
  write(1, &c, 1);
}
 
char	my_putstr(char *str)
{
  char i;
 
  i = 0;
  while(str[i] != '\0')
    {
      my_putchar(str[i]);
      i = i + 1;
    }
  return (0);
}
 
int	my_strlen(char *str)
{
  int	i;
 
  i = 0;
  while(str[i] != '\0')
    {
      i = i + 1;
    }
  return (i);
}
 
char	*my_strcat(char *dest,char *src)
{
  int i = my_strlen(dest); 
  int j = 0; 
 
  while (src[j] != '\0') 
    { 
      dest[i] = src[j];
      i++;
      j++; 
    } 
  dest[i + j] = '\0'; 
 
  return (dest); 
}
 
int	main()
{
  char	*src;
  char	*dest;
 
  src = "sal";
  dest = "tro";
  my_strcat(dest, src);
  return (0);
}

Cordialement.

[Obsidian]

Bonjour,

• Ligne 42, c'est dest[i] et pas dest[i+j] ;
• Juste pour le principe, on net met pas de parenthèses à l'argument de return : ce n'est pas une fonction.

Mais surtout :

• Si tu veux concaténer des chaînes, il faut s'assurer que tu disposes de l'espace suffisant après la chaîne de destination et, pour cela, il faut le réserver à l'avance. Dans ta fonction main, non seulement tu ne le fais pas, mais tu essaies de concaténer des chaînes constantes. Le programme va donc essayer d'aller concaténer ta chaîne source dans un espace en lecture seule.

[EpiTouille]

Bonjour,

Je crois que je ne le dirais jamais assez : VALGRIND

C'est outil est magique car il détecte les segfaults à la ligne près, surtout dans un code comme le tien.

Et puis bonne nouvelle, il est installé par default sur les dumps Epitech (ne nie pas )

[Médinoc]

Comme l'a dit Obsidian, ton main est pourri:

Code C :

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

1
2
3
4
5
6
7
8
9
10
int	main()
{
  char	*src;
  char	*dest;
 
  src = "sal";  /*Les chaînes littérales sont dans une mémoire en lecture seule*/
  dest = "tro"; /*Un gcc bien configuré te le dirait: -Wwrite-strings*/
  my_strcat(dest, src); /*Boum! Tu cherches à écrire dans la mémoire read-only après "tro"*/
  return (0);
}

Bien sûr, la correction est simple (char dest[7] = "tro";). Mais l'important, c'est de la comprendre. Et d'ajouter l'option -Wwrite-strings pour gcc, afin de ne pas répéter la même erreur.

PS: Les nombres magiques, c'est le mal. Utilise STDOUT_FILENO au lieu de 1 pour le descripteur de la sortie standard.

[Devely]

Merci pour vos réponses les gars !

j'ai corrigé les l'erreur a la ligne 42. Mais ca ne marche toujours pas même avec un malloc dans le main. Je pensais utiliser la fonction strdup dans ma fonction car elle fait un malloc. Bonne idée ??

EpiTouille : Non nos dumps sont pourris et il n'y a pas de debugger... Tu a reconnu la NORME hein ?

[Devely]

Ou pouvez vous me montrer comment utiliser malloc dans ma fonction main svp ?

[LittleWhite]

Tout est dit ici, sur le malloc : http://rperrot.developpez.com/articles/c/allocationC/

Et puis : http://c.developpez.com/cours/bernar...gne/node89.php

[Obsidian]

Tu a reconnu la NORME hein ?

Hélas oui…

[Médinoc]

strdup() ne marchera pas, puisque pour concaténer il faut réserver assez pour les longueurs des deux chaînes, et strdup() ne réserve que pour la chaîne qu'il duplique.

[EpiTouille]

Et il y a valgrind installé sur les dumps sauf si tu as un dump différent des autres, mais permet moi d'en douter. (gdb est aussi installé par default)