Discussion :

[pierro91]

Bonjour bonjour à tous....

J'ai cherché pendant pas mal de temps alors je viens ici par dépit... (ewi ça me frustre de pas trouver comme ça :B)
Bref donc en fait je voudrais exécuter mon programme avec un argument et récupérer cet argument pour l'utiliser dans une fonction

au début j'avais fait ça

Code :

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

1
2
3
4
5
6
int main ( int argc , char * argv[] )
{
   string monArgument;
   FILE *monFichier = fopen(monArgument, "w"); 
 
}

mais ça j'obtiens une erreur de conversion je ne sais plus laquelle

Ensuite j'ai essayé

Code :

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

1
2
3
4
5
6
int main ( int argc , char * argv[] )
{
    string str = argv[1];
    const char *ptr = str.c_str() ;
    printf(ptr);
}

je n'ai plus d'erreur mais rien ne s'affiche dans mon printf(ptr); donc inutilisable dans ma fonction

Quelqu'un saurait m'aiguiller gentiment ? :B

[Neckara]

Bonjour,

Tu ne vérifie pas que tu ais le bon nombre d'arguments.

Pour l'affichage en C++ tu as :

Code :

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

std:cout << ptr << std::endl;

Essaye d'afficher argv[1] pour être sûr que ton problème ne vienne pas tout simplement du fait que tu ne passes pas d'arguments en paramètre de ton exécutable.

EDIT : chez moi ton code marche, mais tu devrais rajouter un retour à la ligne après ton affichage.

[pierro91]

Hmmph bonjour Neckara et merci de m'avoir répondu : )

M'en fait c'était pour un autre problème car ceci était un exemple pour pouvoir chercher dans mon code pourquoi ça marche pas. J'ai peut être avancé depuis hier mais je suis toujours bloqué X_X

voici mon code

Code :

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

1
2
3
4
5
6
7
 
if(argv != NULL)
{
     string argument;
     argument = argv[1];
     curl_easy_setopt(maSession, CURLOPT_URL, argument.c_str());
}

Et la j'obtiens une erreur console : segmentation fault
J'ai compris que c'était une erreur de pointeur et donc une erreur de construction de code
Si tu sais m'orienter cela me permettrait beaucoup d'avance :p

[LittleWhite]

Bonjour,

Votre utilisation de argv n'est pas efficace.
argv n'est jamais nul en C/C++ car il contient toujours le nom du programme dans argv[0].
Il faut utiliser argc, pour connnaître le nombre d'arguments (qui est au minimum de 1, comme je l'ai dit).
Ici, vous devez vérifier s'il y a bien un argument en plus du nom du programme, soit, si argc == 2.
Pour cette raison :

./programme	argument
argc=2
argv[0] = "./programme"	argv[1] = "argument"

Une fois votre vérification de argc faite, vous pouvez utiliser tranquillement argv[], sans avoir de crainte que cela plante.

[pierro91]

Ah oui en effet, c'est ce que j'avais avant... je ne sais pas pourquoi j'ai changé pour ça. Merci : )

Code :

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

1
2
3
4
5
6
7
 
if(argc > 1)
{
        string argument;
        argument = argv[1];
        curl_easy_setopt(maSession, CURLOPT_URL, argument.c_str());
}

Bon petit à petit j'avance. Mais cela ne marche toujours pas. Cela devrait normalement fonctionner ou alors ai-je fait une erreur à la fin de

Code :

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

curl_easy_setopt(maSession, CURLOPT_URL, argument.c_str());

?

[r0d]

Quel est l'erreur que tu obtiens?

Note: tu peux écrire directement:

Code :

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

string argument = argv[1];

Dans les faits ça ne change rien, mais ça te fais une ligne de code en moins, c'est toujours ça de pris.

[r0d]

Bonjour,

argv ne peut pas être null, car il contient au moins un élément: le path de l'exécutable. Donc ton test if(argv != NULL) est inutile.
En revanche, il faut que tu vérifie que le nombre d'argument est correct, parce que lorsque tu essaie d'accéder à argv[1], si argv n'a qu'un seul élément, alors ça va effectivement générer une segfault.

edit: doublon avec LittleWhite (nous disons la même chose)