Discussion :

[student_]

Bonsoir,

j'ai fait un petit programme permettant de stocker les arguments qu'on entre dans une liste chainee, ca fontionne plutot bien, mais je veux faire une deuxieme fonction qui la parcourt en affichant chaque maillon, et la je plante... Ce qui est normal, mais je ne vois pas comment faire autrement (on m'a dit au'il etait plus simple de l'afficher a l'envers mais ca ne m'a pas beaucoup aide), 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
void            my_print_list(t_list **params)
{
  t_list        *moove;

  moove = *params;
  while (moove != 0)
    {
      putstr(moove->arg);  SEGMENTATION FAULT 
      moove = moove->next;
    }
}

void            my_params_in_list(t_list **params, char **av)
{
  int           y;
  int           x;
  t_list        *new;
  t_list        *moove;

  y = 0;
  while (av[y])
    {
      new = malloc(sizeof(*new));
      new->arg = malloc((my_strlen(av[y]) + 1) * (sizeof(new->arg)));
      new->next = 0;
      x = 0;
      while (av[y][x])
        {
          new->arg[x] = av[y][x];
          x++;
        }
      if (*params == 0)
        *params = new;
      else if (*params != 0)
        {
          moove = *params;
          while (moove->next != 0)
            moove = moove->next;
          moove->next = new;
        }
      y++;
    }
}

int     main(int ac, char **av)
{
  t_list        *params;

  params = malloc(sizeof(*params));
  my_params_in_list(&params, av);
  my_print_list(&params);
}

Merci de me donner un petit coup de main ! Bonne soiree.

[Pouet_forever]

On utilise généralement (toujours ?) NULL pour vérifier si un pointeur est nul et pas 0

Dans cette ligne là :

Code :

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

new->arg = malloc((my_strlen(av[y]) + 1) * (sizeof(new->arg)));

sizeof(new->arg) vaudra la taille d'un pointeur mais comme toi tu ne veux que la taille d'un char (pour la chaîne de caractère) ça ne va pas

Au lieux de faire une boucle tu peux utiliser 'strcpy' :

Code :

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

1
2
3
4
5
        while (av[y][x])
        {
          new->arg[x] = av[y][x];
          x++;
        }

Ton segfault doit venir de la boucle décrite juste au dessus. Tu copies toute la chaîne mais tu ne mets pas le '\0' de fin de chaîne (pourtant tu l'as alloué !).

Dans ta fonction my_print_list tu n'as pas besoin de passer l'adresse de ta liste car tu ne modifies pas sa valeur.

@+

[student_]

Pour les listes chainees on m'a dit d'utiliser 0 et non NULL...


Ok ! j'avais bien vue que mon malloc était trop grand mais je comprenais pas pourquoi... Donc si je fais:

Code :

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

new->arg = malloc((my_strlen(av[y]) + 1) * (sizeof(*arg)));

Ca me ferra bien un malloc de type char* ?


Pour le '\0', quel boulet.... C'est le genre d'erreur d'inattention que je détecte pas avant d'avoir lâcher un peu mon code, merci pour le gain de temps !

Quelques secondes plus tard...

Je segfault toujours au meme endroit, pourtant j'utilise strcpy et je suppose qu'il copie egalement le '\0' de la chaine.

Au cas ou, voici le nouveau 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
 
void            my_print_list(t_list *params)
{
  t_list        *moove;
 
  moove = params;
  while (moove != 0)
    {
      my_putstr(moove->arg);
      moove = moove->next;
    }
}
 
 
void            my_params_in_list(t_list **params, char **av)
{
  int           y;
  t_list        *new;
  t_list        *moove;
 
  y = 0;
  while (av[y])
    {
      new = malloc(sizeof(*new));
      new->arg = malloc((my_strlen(av[y]) + 1) * (sizeof(new->arg)));
      new->next = 0;
      strcpy(new->arg, av[y]);
      if (*params == 0)
        *params = new;
      else if (*params != 0)
        {
          moove = *params;
          while (moove->next != 0)
            moove = moove->next;
          moove->next = new;
        }
      y++;
      my_putstr(new->arg); /*ici ca me print les arguments comme il faut*/
      my_putstr("\n");
    }
}
 
int     main(int ac, char **av)
{
  t_list        *params;
 
  params = malloc(sizeof(*params));
  my_params_in_list(&params, av);
  my_print_list(params);
}

[Pouet_forever]

Tu peux faire comme ça :

Code :

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

new->arg = malloc((my_strlen(av[y]) + 1) * (sizeof(*new->arg)));

Essaye ce code et dis moi si ça segfault encore :

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
void            my_print_list(t_list *params)
{
	t_list        *moove;
 
	moove = params;
	while (moove != NULL)
    {
		puts(moove->arg);
		moove = moove->next;
    }
}
 
void            my_params_in_list(t_list **params, char **av)
{
	int           y;
	t_list        *new;
	t_list        *moove;
 
	y = 0;
	while (av[y] != NULL)
    {
		new = malloc(sizeof(*new));
		new->arg = malloc((strlen(av[y]) + 1) * (sizeof(*new->arg)));
		new->next = NULL;
 
		strcpy(new->arg, av[y]);
 
		if (*params == NULL)
			*params = new;
		else
        {
			moove = *params;
			while (moove->next != NULL)
				moove = moove->next;
			moove->next = new;
        }
		y++;
    }
}
 
int     main(int ac, char **av)
{
	t_list        *params = NULL;
 
	my_params_in_list(&params, av);
	my_print_list(params);
 
	return EXIT_SUCCESS;
}

[student_]

Ca fonctionne... Je reve ou tu as simplement mit "NULL" partout a la place de mes 0 ??^^

Je vois que tu as initialiser le pointeur sur structure dans le main, je pense que mon segmentationfault venait d'ici, enfin j'espère, car si c'est a cause des 0 je serai traumatise.

En tout cas merci beaucoup pour l'efficacité des aides que j'ai reçut !

Bonne soirée a tous !

(Je viens de tester en remplaçant tous les NULL par des 0 et ca fonctionne aussi même si c'est vrai que des pointeurs pointes sur NULL et non 0 en théorie...)

[diogene]

L'erreur était effectivement à cause de la non initialisation du pointeur sur structure dans le main.

NULL peut être défini comme 0 ou comme (void*) 0, mais il est préférable de mettre NULL plutôt que 0 pour des pointeurs pour une question de clarté du code (on voit immédiatement que la grandeur est un pointeur et non une grandeur numérique)