Concaténation string avec system("")

Bonjour,

Je fais une application en C++ et je n'arrive pas à concatener un system("") avec un string... je retourne la situation dans tout les sens mais pas moyen que ça fonctionne...

Code:

---

1 2 3 4 5 6

string chemin = system("pwd"); string chemin2 = "/test/test1.py";   cheminOK = chemin + chemin2;   system(cheminOK);

---

une idée ?

Bonjour,

Citation:

---

une idée ?

---

Oui, faire attention au message d'erreur, qui doit t'indiquer que system() attend un const char* et refuse donc un std::string.

utiliser string::c_str()

A la compilation j'ai bien l'erreur suivante :

Code:

---

"invalid conversion from 'int' to 'const char*'

---

à la ligne 1 donc je ne sais pas trop comment faire...

Quelle est le type de cheminOK ?

un string aussi mais le problème est que l'erreur est en ligne 1 au moment de la récupération dans le string. Du coup, je ne peux pas faire un c_str() au moment la récupération,si ?

system retourne un int (cf man de system).

Tu ne peux pas convertir un int en string il faut passer par une variable intermédiaire :
http://cpp.developpez.com/faq/cpp/?p...RINGS_numtostr

le problème est que ma commande "pwd" est une commande Linux qui renvoie un chemin actuel et non un int...

et si je fais :

Code:

---

1 2	int chemin = system("pwd"); cout << chemin << endl;

---

chemin = 0 alors que j'attends quelque chose du style : /home/quake/dev

Si je fais avec ostringstream comme vous me le dites, j'ai chemin = 0 aussi :(

Si tu veux récupérer le chemin actuel, il faut utiliser getenv

System est une fonction qui prend en paramètre une chaine de caractère et qui renvois le code de retour (un int) du shell lancé.

Avec getenv j'obtiens comme chemin :

Code:

---

/usr/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games

---

autrement dit pas du tout le chemin de mon répertoire courant/actuel.

Alors qu'avec system("pwd"); quand je le récupère en char j'ai bien le bon chemin même si après j'arrive pas le caster comme il faut.

et si au lieu d'utiliser system("pwd") qui est bien crade, tu utilisais un appel système qui te retourne le répertoire courant ?

getcwd

Citation:

---

Envoyé par quake3

Avec getenv j'obtiens comme chemin :

Code:

---

/usr/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games

---

---

Tu récupère le PATH et non le pwd, regarde le man de getenv.

Citation:

---

Envoyé par quake3

autrement dit pas du tout le chemin de mon répertoire courant/actuel.

Alors qu'avec system("pwd"); quand je le récupère en char j'ai bien le bon chemin même si après j'arrive pas le caster comme il faut.

---

Avec system tu vois le chemin courant sur l'entrée la sortie standard et non par le retour de la fonction system.EDIT : me suis légèrement trompé^^

Quelques liens qui t'intéresseront :
http://man.developpez.com/man3/getenv.3.php
http://man.developpez.com/man5/environ.5.php

Avec getcwd() ça fonctionne nickel pour récupérer le bon chemin. Merci ram-0000 :ccool:

Ci-joint le code pour aider d'autre personnes dans le futur :

Code:

---

1 2 3 4 5 6

char * pPath= new char[1024]; pPath = getcwd(pPath,50); if (pPath != NULL) printf("path is : %s", pPath); else printf("No pPath");

---

Sinon pour concaténer, j'ai toujours un des problèmes pour concaténer de façon à pouvoir exécuter mon test.py :

Code:

---

1 2 3

pPath = pPath + "/test/test.py";   system(pPath);

---

- inutile d'allouer dynamiquement un tableau à taille fixe
- sutout pour oublier de le libérer :mrgreen:
- à fortiori pour le remplacer la ligne d'après :aie:

Code:

---

1 2 3 4 5

char * pPath = getcwd(pPath,50); if (pPath != NULL) printf("path is : %s", pPath); else printf("No pPath");

---

devrait suffire. ;)

En effet :)

Je remarque qu'une personne a mis un vote négatif à environ toutes les personnes ayant essayée d'aider le PO.

J'ai relu assez vite les posts mais je ne vois pas d'erreurs ni de hors-sujet.

Dans l'éventualité où nous aurions dit des bêtises (ça peut arriver^^), est-ce que la personne à l'origine de ces votes pourrait nous dire où nous nous serions trompés?
Ainsi nous pourrions voir, comprendre puis apprendre de nos erreurs afin de ne plus les reproduire à l'avenir.

Heu ... ça c'est pas bon, ça va péter un jour ou l'autre.

Citation:

---

Envoyé par Bousk

Code:

---

char * pPath = getcwd(pPath,50);

---

---

Ça, c'est mieux :

Code:

---

1 2 3

char pPath[PATH_MAX]; char *ptr = getcwd(pPath, sizeof(pPath)); if(ptr ...

---

J'ai lu le man vite fait, n'est-il pas plus simple de faire :

Code:

---

1 2	char * ptr = getcwd(NULL, 0);

---

Car si on renseigne le pointeur (!= NULL) ou si on renseigne la taille (> 0), il faudra que la taille allouée soit suffisante et dans le cas où ce n'est pas le cas, il faut faire un realloc puis rappeler la fonction getcwd.

Alors que là, getcwd se charge de tout pour nous.

/!\ bien penser à faire le free.

Je n'ai pas regardé le manuel au moment de ma réponse et l'ai parcouru qu'en diagonale maintenant.
Je pense que la solution de Neckara serait la meilleure (en tous cas celle que je trouve la plus simple), en faisant attention au free, mais semble dépendant de la libc utilisée:

Citation:

---

Comme extension de la norme POSIX.1-2001, la version Linux (libc4, libc5, glibc) de getcwd() alloue le tampon dynamiquement, en utilisant malloc(3), si buf est NULL lors de l'appel. Dans ce cas, le tampon alloué a la longueur size à moins que size soit égal à zéro, auquel cas buf est alloué avec la taille nécessaire. Il est possible (et même fortement conseillé) de libérer le tampon avec free(3) lorsqu'il a été obtenu ainsi.

---

http://manpagesfr.free.fr/man/man3/getcwd.3.html

Salut,

Par rapport à la demande originel de monsieur, je voulais préciser que la fonction system ne renvoie pas le résultat de la commande... Généralement, mais comme dit précédemment c'est bien crade, les gens qui utilise system de cette manière redirige dans un fichier le résultat de la commande et lise le fichier pour récupérer le résultat.

De cette façon :

Code:

---

system("pwd > mon_chemin.txt");

---

Mais comme tu es quelqu'un de propre, et que tu veux un programme pas trop lent, utilise les méthodes décrites si avant ;)

Ok c'est noté, merci beaucoup à vous tous pour vos précieux conseils :ccool:

Pour la concaténation char + string qui puisse fonctionner avec ma commande system() (cette fois-ci pour lancer le fichier test.py), j'ai fait ça mais je sais que ça ne fonctionne pas:

Code:

---

1 2 3 4 5 6 7

string cheminTest = "/test/test.py";   pPath = pPath + cheminTest; // Bien sure il y a une erreur ici mais je n'arrive pas à m'en dépatouiller en castant dans tout les sens.   system(pPath);

---

Un idée simple et précise ? Ça doit être pourtant plutôt simple à faire mais le c++ me rebute un peu (c'est aussi pour cela que je souhaite lancer des scripts en python !!!)

Comme l'as dis bousk dans sa première réponse, system n'accepte que des char *, si tu veux utiliser un string il faut donc faire : maString.c_str() ce qui retourne un char *.
De plus, il faut aussi savoir que les chaînes de caractères (char *) ne peuvent pas être comparé (on compare les pointeurs et non la chaîne) et ne peuvent pas utiliser l'opérateur + et = (actions effectuées sur le pointeur et non sur la chaîne). C'est donc pour cela qu'on a créé la classe std::string.


Donc, si tu veux concaténé un char * avec un string, le plus simple est de faire :

Code:

---

1 2 3 4 5

std::string instruction(pPath); instruction += cheminTest;   system(instruction.c_str());

---

Citation:

---

Envoyé par quake3

j'ai fait ça mais je sais que ça ne fonctionne pas:

Code:

---

1 2 3 4

string cheminTest = "/test/test.py";   pPath = pPath + cheminTest;

---

---

Quel est le type de pPath ?

J'ai l'impression que tu additionnes des torchons et des assiettes.

PS : les messages d'erreur du compilateur ne sont pas faits pour emm... le monde, parfois (souvent), ils sont utiles.

Citation:

---

Envoyé par quake3

en castant dans tout les sens.

---

Quand on commence à caster, c'est que :

• soit on sait parfaitement ce que l'on fait et on veut enlever un warning du compilateur,
• soit on ne sait pas ce qu'il se passe et on se dit que si on enlève l'insulte du compilateur, cela enlèvera l'erreur de conception. L'expérience a prouvé que cela ne marche pas.

Choisis ton camps :)

@Neckara, Merci beaucoup, ça fonctionne plutôt bien ! :ccool:

@ram-0000 en effet vous avez raison sur toute la ligne, les torchons et les serviettes ne se mélangent pas !

Dans tous les cas, un énorme merci à vous tous pour votre aide plus que précieuse !