Discussion :

[salseropom]

Bonjour, je suis en train de faire un programme qui résoud une équation différentielle. Toutes les conditions initiales se trouvent dans un fichier input.txt et ma fonction main lit ce fichier. Jusque là tout se passe bien. Sauf que maintenant je veux donner le choix à l'utilisateur d'appeler le nom de son fichier comme il le veut et non plus input.txt. Je voudrais aussi que le nom par défaut soit input.txt. J'ai donc essayé de donner un argument à ma fonction main mais j'ai qq soucis. Voici un extrait de mon 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
 
int main(int argc, char * argv[])
{
  blabla
 
 if(argc==1)
   char * inputfile=strdup("input.txt"); /* le nom par defaut */
 else
   char * inputfile=strdup(argv[1]);
 
 Read(inputfile,&a,&b,&c);
 
 free(inputfile);
 blabla
 
return 0;
}

Et là j'ai une erreur qui est

main.c:139: error: erreur de syntaxe before "char"
main.c:138: attention : empty body in an if-statement
main.c:140: error: erreur de syntaxe before "else"
main.c:141: error: `__len' undeclared (first use in this function)
main.c:141: error: (Each undeclared identifier is reported only once
main.c:141: error: for each function it appears in.)

Savez-vous pourquoi ?

Merci.

[Luther13]

Perso,
je declarerai

Code :

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

 char * inputfile ;

avant le if

//#include de strdup

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
int main(int argc, char * argv[]) 
{ 
  blabla 
 
char * inputfile;
 
 if(argc==1) 
   inputfile=strdup("input.txt"); /* le nom par defaut */ 
 else 
   inputfile=strdup(argv[1]); 
 
 Read(inputfile,&a,&b,&c);  //a, b, c ??
 
 free(inputfile); 
 blabla 
 
return 0; 
}

[salseropom]

Merci ça marche !

[Emmanuel Delahaye]

Voici mon code

C'est une façon inutilement complexe d'écrire :

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
 
int main (int argc, char * argv[])
{
   /* le nom par defaut */
   char * inputfile = "input.txt"; 
 
   if (argc==2)
   {
      inputfile=argv[1];
   }
 
   Read (inputfile, &a, &b, &c);
 
   return 0;
}

C'était quoi déjà la question ?

[salseropom]

Salut, mon pb était que je déclarais deux fois char * inputfile. Mais maintenant c'est réglé.
Merci.

[Emmanuel Delahaye]

Salut, mon pb était que je déclarais deux fois char * inputfile. Mais maintenant c'est réglé.
Merci.

OK. Tu as bien noté qu'ici, le strdup() était inutile. (Les paramètrtes de la ligne de commande ont une durée de vie égale à celle du programme, c'est garanti par la définition du langage C.)

[Médinoc]

Salut, mon pb était que je déclarais deux fois char * inputfile. Mais maintenant c'est réglé.
Merci.

OK. Tu as bien noté qu'ici, le strdup() était inutile. (Les paramètrtes de la ligne de commande ont une durée de vie égale à celle du programme, c'est garanti par la définition du langage C.)

Est-ce qu'ils sont aussi garantis disponibles en écriture ?

[Emmanuel Delahaye]

<paramètres de la ligne de commande> Est-ce qu'ils sont aussi garantis disponibles en écriture ?

Oui, mais il est considéré comme 'bad practice' de modifier les paramètres de la ligne de commande.

[Médinoc]

Merci, c'est bien ce qui me semblait.
Donc, vivent les const char * (et les strdup() quand il s'agit de modifier)

[gege2061]

(et les strdup() quand il s'agit de modifier)

man strdup[/url]]SVID 3, BSD 4.3. strndup(), strdupa(), et strndupa() sont des extensions GNU.

Aller histoire de grossir les pages sources :

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
#include <string.h>
 
char *str_dup (const char *src)
{
  char *dest = NULL;
 
  if (src)
  {
    dest =  malloc ((strlen (src) + 1) * sizeof (*src));
    if (dest)
    {
      strcpy (dest, src);
    }
  }
  return dest;
}

[Médinoc]

SVID 3, BSD 4.3. strndup(), strdupa(), et strndupa() sont des extensions GNU.

Mais ça veut dire que strdup() sans rien d'autre est posix, non? (Et implémenté par windows)