Peut-on améliorer ce code ?

Version imprimable

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
if (argc<=1) {printf ("%s %s\n","Trop peu d'arguments","Syntaxe : camera.bin -i [image-sec] -d [nbr-jour]");exit(-1);}
        else if (argc >=6) printf("%s\n","Trop d'arguments");
        else if (argc ==5)
        {
 
                for (i=1;i<=4;i++)
                {
                        if  (!(strcmp(argv[i],"-i"))) 
                        {
                                param1 = atoi(argv[i+1]);
                                if ( (param1>=1) && (param1<=24) ) imsec=param1;
                                else 
                                {
                                        printf ("%s\n","valeur incorrecte de i (1-24)");
                                        exit (-1);
                                }
                        }
                        if  (!(strcmp(argv[i],"-d"))) 
                        {
                                param2 = atoi(argv[i+1]);
                                if ( (param2>=1) && (param2<=12) ) duree=param2;
                                else 
                                {
                                        printf ("%s\n","valeur incorrecte de d (1-12)");
                                        exit (-1);
                                }
                        }
 
 
                }
 
        }

Comme vous le voyez, je lance le programme camera de la façon suivante :

./ camera -i 10 -d 1

Cela correspond à une vidéo de 10 images par seconde pour une journée entière de film.

N'y a-t-il pas une manière plus élégante ( et plus fiable ) de tester les paramètres ?

18/02/2021, 07h14
WhiteCrow

Bonjour,

la gnu libc propose des parseurs de ligne de commande → Parsing Program Arguments. Si tu es sur une plateform *nix sans gnu libc tu as toujours l'option de la gnulib → GnuLib getopt Module.

Sans framework, vouloir parser la ligne de commande peut vite produire un code spaghetti innommable.
18/02/2021, 07h20
foetus
Je pense que ton code peut planter :mrgreen: :./camera 10 10 10 -i
Tu ne vérifies pas si argv[i+1] est valide :aie:

Après pour ta question, oui et non :mrgreen:
En gros, si tu veux faire du code sûr lorsque tu parses les arguments, il faut coder 1 mini machine à états en :
- catégorisant les bascules - les principales, les secondaires, les facultatives, les options (celles qui sont liées à 1 autre bascule), ...
- fixant 1 ordre précis. Souvent certaines bascules doivent êtres mises avant ou après (notamment les options) d'autres
C'est à peu près ce que font les bibliothèques spécifiques de parsage de paramètres.

Dans ton cas. Tu as 2 bascules -i et -d
- Si tu dis que -i est la première et -d la deuxième. Alors, argv[1] doit contenir "-i", argv[2] 1 valeur, argv[3] "-d" et argv[4] 1 valeur. Pas d'autre choix (donc argc doit valoir exactement 5)
- Si tu dis que -d est facultative. Alors, argv[1] doit contenir "-i", argv[2] 1 valeur. Et en fonction de argc (3 ou 5) tester argv[3] et argv[4]
- Si tu dis que tes 2 bascules sont interchangeables. Alors, argv[1] doit contenir soit "-i" soit "-d", et argv[3] l'autre bascule, argv[2] et argv[4] 1 valeur. Pas d'autre choix (donc argc doit valoir exactement 5)
- ...
18/02/2021, 18h12
frederic13870

Je ne connaissais pas !

getopt() à l'air cool.

Je vais l'utiliser.

Merci.

La doc en français http://manpagesfr.free.fr/man/man3/getopt.3.html

Citation:

Envoyé par WhiteCrow

Bonjour,

la gnu libc propose des parseurs de ligne de commande → Parsing Program Arguments. Si tu es sur une plateform *nix sans gnu libc tu as toujours l'option de la gnulib → GnuLib getopt Module.

Sans framework, vouloir parser la ligne de commande peut vite produire un code spaghetti innommable.

Une grosse optimisation à faire est d'intenter correctement le code pour le rendre lisible et maintenable ;)

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
if (argc <= 1) {
	printf("%s %s\n", "Trop peu d'arguments", "Syntaxe : camera.bin -i [image-sec] -d [nbr-jour]");
	exit(-1);
} else if (argc >= 6)
	printf("%s\n", "Trop d'arguments");
else if (argc == 5) {
 
	for (i = 1; i <= 4; i++) {
		if (!(strcmp(argv[i], "-i"))) {
			param1 = atoi(argv[i + 1]);
			if ((param1 >= 1) && (param1 <= 24))
				imsec = param1;
			else {
				printf("%s\n", "valeur incorrecte de i (1-24)");
				exit(-1);
			}
		}
		if (!(strcmp(argv[i], "-d"))) {
			param2 = atoi(argv[i + 1]);
			if ((param2 >= 1) && (param2 <= 12))
				duree = param2;
			else {
				printf("%s\n", "valeur incorrecte de d (1-12)");
				exit(-1);
			}
		}
	}
}

Ah ! v'là

J'ai suivi vos conseils et suis arrivé à quelque chose de bien plus propre :

.....

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 if (argc==1) printf ("camera -i [1-24] img/sec -d [1-7] nbr-jour(s)-record\n");
 
        while ((c = getopt (argc, argv, "i:d:")) != -1)
                switch (c)
                {
                         case 'i':ivalue = optarg;
                                  break;
                         case 'd':dvalue = optarg;
                                  break;                        
                         case '?':if (optopt == 'i')
                                  fprintf (stderr, "Option -%c a besoin d'un argument.\n", optopt);
                                  else if (optopt == 'd')
                                  fprintf (stderr, "Option -%c a besoin d'un argument.\n", optopt);                              
                                  else fprintf (stderr,"Option inconnue `\\x%x'.\n",optopt);
                                  return 1;
                         default:abort ();
                }
 
 
 
 
 
	if ((ivalue) && (dvalue))

....

ça fait plaisir !

je vais potasser la bibliothèque C Gnu, c'est plein de trésors !