Problème de menu

Bonjour à tous,

Alors voilà j'ai un petit soucis. J'essaye de créer un programme qui déplace un paramètre sur une chaine qui fait deux fois la longueur de ce paramètre. Et à chaque déplacement de la taille d'un espace la première lettre du paramètre doit passer en dernière.

Ex:

hello
elloh
llohe
lohel
ohell
hello

Bien entendu le déplacement s'effectue sur une seule ligne. Bref pour l'instant je cherche déjà à gérer le déplacement du paramètre. Pour faire en sorte que le mot se déplace sur deux fois sa longueur. Mais à chaque fois que je compile et que j’exécute ça m'affiche segmentation fault. Je n'arrive pas à en comprendre la raison.

Le programme est le suivant (juste pour mettre le déplacement du mot):

Code:

---

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

#include <stdlib.h>   void main(int argc, char **argv) { int lengh; char *str; int lengh2;   lengh = (2 * my_strlen(argv[1])); lengh2 = (my_strlen(argv[1])); str = malloc(lengh); my_movement(str, lengh, lengh2); }   int my_movement(char *str, int lengh, int lengh2) { int i;   i = 0; while(i + lengh2 < lengh) { my_putchar(' '); i++; my_putstr(str); } }   void my_putstr(char *str) { while (*str != '\n') { my_putchar(*str); str++; } }

---

Sachant que le my_putchar est forcément bon.

Quelqu'un aurait-il une idée de mon erreur ? merci d'avance.

- Le contenu de la mémoire allouée (str) n'a pas été initialisé

- Dans l'allocation, tenir compte du '\0' terminal si my_strlen() ne le fait pas (Si c'est un clone de strlen, le '\0' n'est pas compté).

- Dans my_putstr() le test à '\n' est bizarre. Pourquoi pas tester avec '\0' ?

- main() renvoie un int, pas void, dans un environnement d'exécution sous OS

Tout d'abord merci pour les erreurs soulevées. Si tu me le permets je vais reprendre tes commentaires avec les observations que je viens d'en faire.

Citation:

---

Envoyé par diogene

- Le contenu de la mémoire allouée (str) n'a pas été initialisé

---

On initialise bien sa longueur avec le malloc, il est initialisé à 0 par défaut non ?
Autrement si j'initialise le contenue de la mémoire en faisant un :

'' str = argv[1] " afin d'initialiser dès le début avec le paramètre que je souhaite est-ce correcte ?

Citation:

---

Envoyé par diogene

- Dans l'allocation, tenir compte du '\0' terminal si my_strlen() ne le fait pas (Si c'est un clone de strlen, le '\0' n'est pas compté).

---

Donc il faudrait pour l'allocation mémoire faire 2 * longueur de argv + 1 ? ( à moins que j'ai mal compris.)

Citation:

---

Envoyé par diogene

- Dans my_putstr() le test à '\n' est bizarre. Pourquoi pas tester avec '\0' ?

---

Une erreur de ma part le test à 0 était en effet plus logique.

Citation:

---

Envoyé par diogene

- main() renvoie un int, pas void, dans un environnement d'exécution sous OS

---

J'ai changé ça mais ça ne change absolument rien ...

Citation:

---

Autrement si j'initialise le contenue de la mémoire en faisant un :

'' str = argv[1] " afin d'initialiser dès le début avec le paramètre que je souhaite est-ce correcte ?

---

Non. Une chaine de caractères se copie avec strcpy() ou analogue. Sinon, tu copie des pointeurs, pas le contenu pointé.

Citation:

---

Donc il faudrait pour l'allocation mémoire faire 2 * longueur de argv + 1 ? ( à moins que j'ai mal compris.)

---

Oui

Citation:

---

J'ai changé ça mais ça ne change absolument rien ...

---

Non, c'était juste une remarque.