Discussion :

[bouritosse]

Bonjour,

J'ai un petit problème , c'est à dire que je récupére via la ligne de commande argv[] des infos du type :
-i fichier.ppm

Et je prend en compte le cas ou l'utilisateur n'écrit rien, le problème c'est que
s'il écrit rien, je voudrais que l'utilisateur puisse réecrire le argv vide (d'ou mon scanf) le gros problème c'est qu'il est null et je ne vois pas comment faire ...

Je vous remercie d'avance !

Code :

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

1
2
3
4
5
6
7
8
9
 
  //Appel du programme sans argument
    if(argv[1]="null" )
    {
    printf("example of command:\n \t -i picture.ppm pak.ppm\n ");
    printf("It copies and paste the input file to the one in the output\n");
    argc=2;
    scanf("%s",argv[1]);//segFault car, de base le tableau de string est null
     }

[gangsoleil]

Bonjour,

Tu cherches a travailler sur argv directement, mais il te sera beaucoup plus simple de travailler sur des variables internes que tu maitrises entierement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
char mes_args[TAILLE_MAX];
 
Si je lis un argument valide 
{
   strncpy (mes_args, argv[1], taille_de_mes_args);
}
sinon
{
  printf ("blabla");
  /* lire la ligne de commande de maniere securisee */
}
 
suite du programme qui bosse sur mes_args[]

Des entrees solides en C (c'est a dire pas scanf, qui est bien mais pas pour ca en depit de ce qui est enseigne dans pas mal de cours) : https://c.developpez.com/cours/

[Bktero]

Bonjour,

Pour commencer, ton test if(argv[1]="null" ) m'étonne. Si tu veux savoir si l'utilisateur a saisie des paramètres, tu devrais tester la valeur de argc.

Si je devais faire un tel mécanisme (ce que j'approuve moyennement, voir plus loin), ma première idée serait de faire quelque chose comme cela :

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
#include <assert.h>
#include <stdlib.h>
#include <stdio.h>
 
int main(int argc, char* argv[])
{
    puts("DEBUT");
    for(int a = 0; a < argc; a++)
    {
        printf("Argument %d: [%s]\n", a, argv[a]);
    }
 
    if(argv[argc] == NULL)
        puts("OK");
    else
        puts("Not null");
 
    if(argc == 1)
    {
        argc = 2;
 
        // Recuperation avec fgets() plutot que scanf()
        char* buffer = malloc(100); // voir si cela te suffit
        assert(buffer != NULL);
        puts("Saisir:");
        fgets(buffer, 100, stdin); // il reste un \n a la fin si saisie OK. A gerer.
        argv[1] = buffer;
    }
 
    puts("FIN");
    for(int a = 0; a < argc; a++)
    {
        printf("Argument %d: [%s]\n", a, argv[a]);
    }
 
    if(argv[argc] == NULL)
        puts("OK");
    else
        puts("Not null");
}

argv est un tableau de pointeur, alors mettons un pointeur dans la tableau.

Le soucis ici est que le argv[argc] ne vaut plus NULL. Il faut donc bien faire attention dans la suite du code. Pourquoi ne pas faire le classique appel à la méthode usage() pour demander à l'utilisateur de relancer son programme avec le bon nombre d'argument ?

Pourquoi ne pas copier le contenu de argv[] dans des variables à toi et si avec argv[] ne contient pas ce que tu veux, alors faire une saisie directement dans les bonnes variables ? Cette technique est sans doute bien meilleure. En effet, ici, je ne peux rajouter qu'une seule chaine dans argv. Si tu en as 2 ou plus, c'est mort. Voici de manière sommaire ce que tu pourrais avoir :

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
int main(int argc, char* argv[])
{
    char *un = NULL;
    char *deux = NULL;
 
    if(argc != 3)
    {
        un = malloc(100);
        deux = malloc(100);
 
        puts("Saisir un:");
        fgets(un, 100, stdin);
 
        puts("Saisir deux:");
        fgets(deux, 100, stdin);
    }
    else
    {
        un = argv[1];
        deux = argv[2];
    }
 
    printf("Un = [%s] ; deux = [%s]", un, deux);
}

EDIT : bon, gangsoleil et moi sommes sur la même longueur d'onde ^^

[bouritosse]

J'ai changé ma méthode en celle de Bktero, pour l'instant je n'ai prévu qu'un seul cas, celui ou l'utilisateur ne rentre rien dans les arguments.

Le seul truc bizarre c'est que maintenant que j'ai remplacé argv par tab, j'ai un segfault qui d'après GDB vient de ma fonction "cpt" et je n'arrive pas a comprendre pourquoi ... (alors que ça marché avec argv)

La première étape de mon programme est de faire un simple copier coller, puis dés que ce sera fait il me restera plus qu'à faire la même opération mais sur plusieurs fichiers avec quelques commandes.

Je vous remercie d'avance pour votre aide !

Le main:

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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <stddef.h>
 
#include "Structure.h"
 
//Aide mémoire to myself: -to stock[marchandise]
//			  -to store[computer science]
 
int main(int argc,char *argv[])
{
    Picture Im;
    int i=0,j=0;
    int count;
    char **tab;
   // char temp[6][12];
 
 
    /**************************************************/    
    /*********Appel du programme sans argument*********/
    if(argc == 1)
    {
    printf("example of command:\n \t -i picture.ppm\n ");
    printf("It copies and paste the input file to the one in the output\n");
	printf("how many words would like to write ? ex:-i = one word\n");
	scanf("%i",&count);
	tab=createTable(count, 20);
 
	for(i=0;i<count;i++)//on rentre autant de variable qu'on veut
	{
	printf("please enter new variable:\n");
	scanf("%s",tab[i]);
	}
    }//Nb: il faut un tableau dynamique a deux dimmension qui permet de rentrer autant de mots qu'on veut
 
/**************************************************/    
/*********Appel du programme avec argument********/
 
    if(argc >1)//l'utilisateur a écrit dans argv (à terminer)
	{
		for(i=0;i<argc;i++)
		{
		//tab[i]=argv[i+1];//argv[0] adresse réservé
		}
	}     
 
    for(;;)
    {
   	 if(tab[0][0]=='-')
   	 {
 
    	    if(strcmp(tab[0],"-?")==0)
    	    {
    	    printf("example of command:\n \t -i picture.ppm pak.ppm\n ");
    	    printf("It copies and paste the input file to the one in the output\n");
   	     }
 
    	    else if(strcmp(tab[0],"--help")==0)
            printf("need mega help ?\n");
/*
        else if(strcmp(argv[1],"-g")==0)
	printf("work in progress ...");
 
        else if(strcmp(argv[1],"-h")==0)
	printf("work in progress ...");
 
        else if(strcmp(argv[1],"-e")==0)
	printf("work in progress ...");
 
        else if(strcmp(argv[1],"-d")==0)
	printf("work in progress ...");
 
        else if(strcmp(argv[1],"-b")==0)
	printf("work in progress ...");
 
        else if(strcmp(argv[1],"-c")==0)
	printf("work in progress ...");
 
        else if(strcmp(argv[1],"-p")==0)
	printf("work in progress ...");
*/	
        	else if(strcmp(tab[0],"-i")==0)
		{
		InpFile(&Im,tab);
		cptChar(&Im);
		}
 
        	else if(strcmp(tab[0],"-o")==0)
        	{
        	OutFile(&Im,tab);
		StoreMemory(&Im);
       		CopyFile(&Im);
       		printf("copy succeeded\n\n");
        	}
	/*
        else if(strcmp(argv[1],"-li")==0)
	printf("work in progress ...");
 
        else if(strcmp(argv[1],"-lo")==0)
	printf("work in progress ...");
	*/
 
       		fflush(stdin);
    	  	freeTable(tab);
       		printf("how many words would like to write ? ex:-i = one word\n");
		scanf("%i",&count);
		tab=createTable(count, 12);
		for(i=0;i<count;i++)//on rentre autant de variable qu'on veut
			{
			printf("please enter new variable:\n");
			scanf("%s",tab[i]);
			}
 
         }//fin du if tab[0]
 
     else {
		fflush(stdin);
    	  	freeTable(tab);
       		printf("how many words would like to write ? ex:-i = one word\n");
		scanf("%i",&count);
		tab=createTable(count, 12);
		for(i=0;i<count;i++)//on rentre autant de variable qu'on veut
			{
			printf("please enter new variable:\n");
			scanf("%s",tab[i]);
			}
    	   }
 
    }	
	return 0;//The programm quit normally	
}
 
 
/******Function***************/
void InpFile(Picture *Im1,char **arguments)
//That function catch up the input file name and move it to the structure
{
 
    FILE *pictureI=NULL;
	int i;
 
	for(i=0;arguments[1][i]!='\0';i++)
	{
	Im1->InpName[i]=arguments[1][i];
		printf("%c",arguments[1][i]);
	//tab[1]=inputFile
	}
 
    pictureI= fopen(Im1->InpName,"rb");
    		if(pictureI== NULL)
    		perror(Im1->InpName);
 //If the inputPicture is null,it displays an error
    		else {printf("ok\n");}
 //However, if it is ok, it displays "ok"      
        close(pictureI);
}
 
void OutFile(Picture *Im1,char **arguments1)
//That function catch up the input file name and move it to the structure
{
	FILE *pictureO=NULL;
	int i;
 
	for(i=0;arguments1[1][i]!='\0';i++)
	{
	Im1->OutName[i]=arguments1[1][i];
	//argv[3]=Output File
	}
 
    pictureO= fopen(Im1->OutName,"rb");
 
        close(pictureO);
}
 
char *cptChar(Picture *Im2)
//CptChar is used as to count the number of character in 
//the input file
{
	FILE *pictureI=NULL;
	int temp;
 
      pictureI= fopen(Im2->InpName,"rb");
      if(pictureI== NULL)
    		perror(Im2->InpName);
 
      while(fgetc(pictureI) != EOF)
      Im2->cpt++;
//Cpt is incremented while the file is not at the end
 
	close(pictureI);
}
 
void StoreMemory(Picture *Im3)
//StoreMemory is used for putting in memory the input File
{
 
    FILE *pictureI=NULL;
 
    char tab[Im3->cpt+1];
    int i=0;
 
 
     pictureI= fopen(Im3->InpName,"rb");
    		if(pictureI== NULL)
    		perror(Im3->InpName);
 
              	Im3->tabPicture = malloc( Im3->cpt * sizeof(Im3->cpt) );
 
                 while( fgets(tab,Im3->cpt,pictureI) != NULL)
   		 {
    		strcat(Im3->tabPicture,tab);
//Each line of the input file is store on "tab", then concenatancé
//with Im3->tabPicture
    		}
 
    close(pictureI); 
}
 
void CopyFile(Picture *Im3)
//That function copy from the memory to the Output file
{
    FILE *pictureI=NULL;
    FILE *pictureO=NULL;
 
    char tab[Im3->cpt+1];
    int i=0;
 
     pictureI= fopen(Im3->InpName,"rb");
    		if(pictureI== NULL)
    		perror(Im3->InpName);
 
      pictureO = fopen(Im3->OutName,"wb");
                if (pictureO == NULL)
                perror(Im3->OutName);
 
    for(i=0;i<Im3->cpt;i++)
    {
    fputc(Im3->tabPicture[i],pictureO);
    //Paste the "memory tab" to the output file
    }
    free(Im3->tabPicture);
 
     //system("cd Projet/livrable_1/%s gqview ", Im3->OutName );
	close(pictureI);
	close(pictureO);
}
 
char **createTable(int nbLin, int nbCol){
	int i=0;
 
	char **tableau = (int **)malloc(sizeof(int*)*nbLin);
	char *tableau2 = (int *)malloc(sizeof(int)*nbCol*nbLin);
	for(i = 0 ; i < nbLin ; i++){
		tableau[i] = &tableau2[i*nbCol];
	}
	return tableau;
}
void freeTable(char **tableau){
	free(tableau[0]);
	free(tableau);
}

Function.h

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
 
/********Constant**************/
#define TAILLE_MAX 22
 
/*******Struct_1***************/
typedef struct{
	 char InpName[TAILLE_MAX];//Store the input file name
	 char OutName[TAILLE_MAX];//Store the output file name
	unsigned int cpt;//Store the number of character in the input file
	char* tabPicture;//This is the input file in memory
}Picture;
 
/*********Prototypes***********/
void InpFile(Picture *Im1,char **arguments);
void OutFile(Picture *Im1,char **arguments);
char *cptChar(Picture *Im2);
void CopyFile(Picture *Im3);
void StoreMemory(Picture *Im3);
char **createTable(int nbLin, int nbCol);
void freeTable(char **tableau);