Discussion :

[pierre267]

Bonjour,

J'ai un problème avec ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#include <stdio.h>
 
#define str char *
 
int main(int k, const str argv[])
{
	argv += k;
	puts(* argv);
	while(* argv) puts(* argv --);
	return 0;
}

qui me donne l'erreur suivante :

Code :

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

Erreur de segmentation (core dumped)

Pourriez-vous m'expliquer d'où vient cet erreur (je souhaite faite afficher les mots à l'envers).

Merci d'avance !
Cordialement

[diogene]

k donne le nombre d'arguments passés au programme (k=1...).
argv[] est un tableau de k+1 pointeurs dont le dernier, argv[k], est NULL. argv += k place argv sur ce dernier pointeur et *argv == NULL et puts(NULL) peut poser problème suivant l'implémentation.
De plus, on ne rentre donc pas dans la boucle while().

Mettre argv += k-1 résoud ce problème, et on liste bien les paramètres avec la boucle while, mais on va planter ensuite parce que le test d'arrêt while(* argv) est faux : il n'y a aucune raison d'avoir un pointeur NULL en tête du tableau argv.

On peut envisager

Code :

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

1
2
3
4
5
6
int main(int k, const str argv[])
{
	while(k!=1) // si on ne veut pas l'affichage de argv[0]
              puts(argv[--k]);
	return 0;
}

[Franck.H]

et bienvenue sur nos forums !

Moi personnellement, j'éviterais ce genre d'obscurcissement :

Code :

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

1
2
3
#define str char *
 
int main(int k, const str argv[])

Ca peut induire en erreur, j'avais même pas regardé ta constante plus haut que la signature de ton main m'avais laissé perplexe

Les signatures originales étant:

Code :

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

1
2
3
int main (void)
int main (int argc, char ** argv)
int main (int argc, char * argv [])

Voilà, autant rester dans les standard pour éviter toutes confusions. Moi j'aime quand on respecte les choses comme elles doivent l'être

[pierre267]

Suite à vos réponses, pourriez-vous m'indiquer pourquoi dans ce code j'obtiens le résultat voulut mais suivit encore de l'erreur "Erreur de segmentation (core dumped)" :

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
#include <stdio.h>
 
void scan_r (char *mot);
 
int main(int argc, char *argv[])
{
	int k=1; //et non 0 afin de retirer le nom du fichier
	while(argv)
	{
		scan_r(argv[k++]);
	}
	return 0;
}
 
void scan_r (char *mot)
{
	while (*mot)
	{
		printf("%c\n", *mot++);
	}
}

Merci d'avance !

[droggo]

Loa,

Ta boucle

Code :

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

	while(argv)

plantera forcément, puisque argv est toujours vrai.

Tu as oublié d'indiquer argv[k]

C'est typiquement le genre d'erreur qu'on fait souvent, et qui est parfois difficile à repérer par l'auteur du code (on croit toujours ne pas avoir fait d'erreur).

[Sve@r]

Code c :

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

1
2
3
4
5
6
 
	int k=1; //et non 0 afin de retirer le nom du fichier
	while(argv)
	{
		scan_r(argv[k++]);
	}

Salut
Pour ce genre de boucle, tu as le for() qui est fait pour ça...

Code c :

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

1
2
3
	int k;
	for (k=1; k < argc /* ou argv[k] != NULL */; k++)
		scan_r(argv[k]);

Et éventuellement tu peux remplacer argv[k] (calculé à chaque fois) par un pointeur directement positionné sur le bon argument

Code c :

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

1
2
3
	char **pt_arg;
	for (pt_arg=argv + 1; *pt_arg != NULL; pt_arg++)
		scan_r(*pt_arg);