comment faire fonctionner: ma_fonction("teste"); ?

Bonjour,

j'ai crée une fonction:

Code:

---

1 2 3 4 5 6 7

void ma_function(char *message) { char *cmd;   sprintf(cmd, "echo %s > log.txt", message); system(cmd); }

---

mais voilà, lorsque je l'appel, ça fonctionne pas:

Code:

---

ma_function("erreur dans la partie 1");

---

j'ai dû oublier quelque(s) chose(s)... :(

Code:

---

1 2 3 4 5 6 7

void ma_function(const char *message) { char *cmd;   sprintf(cmd, "echo %s > log.txt", message); system(cmd); }

---

il manqué le const char ;)

oky, bah merci quand même, je savais bien que j'avais oublié quelque chose ;)

Citation:

---

Envoyé par dtcSearch

Code:

---

1 2 3 4 5 6 7

void ma_function(const char *message) { char *cmd;   sprintf(cmd, "echo %s > log.txt", message); system(cmd); }

---

il manqué le const char ;)

oky, bah merci quand même, je savais bien que j'avais oublié quelque chose ;)

---

Je serais surpris que ça marche. cmd étant un pointeur non initialisé et sprintf copiant le résultat dans l'adresse de ce premier argument.

En C++, tu ferais :

Code:

---

1 2 3 4 5 6 7 8 9 10

#include <string> #include <sstream>   void ma_function(std::string const &message_) { std::ostringstream commande; commande << "echo "<<message_<<" > log.txt";   system(commande.str().c_str()); }

---

Pour des char (const)* et des sprintf voir le forum C.

Oui, désolé,

je me suis trompé de forum, ça aurai du être dans le forum C.

Et dans le poste au dessus, effectivement, c'est pas un pointeur qu'il faut pour cmd.

Merci

Code:

---

1 2 3 4 5 6 7

void ma_function(const char *message) { char cmd;   sprintf(cmd, "echo %s > log.txt", message); system(cmd); }

---

Citation:

---

Envoyé par dtcSearch

Oui, désolé,

je me suis trompé de forum, ça aurai du être dans le forum C.

Et dans le poste au dessus, effectivement, c'est pas un pointeur qu'il faut pour cmd.

Merci

Code:

---

1 2 3 4 5 6 7

void ma_function(const char *message) { char cmd;   sprintf(cmd, "echo %s > log.txt", message); system(cmd); }

---

---

Marche ce code la ?

- sprintf copie ton texte dans cmd
- mais cmd ici est 1 caratère (char)

Citation:

---

Envoyé par laflak

Marche ce code la ?

- sprintf copie ton texte dans cmd
- mais cmd ici est 1 caratère (char)

---

+1 .....

Clair, je me demande comment ca pète pas!

Faut initialiser cmd avec une taille convenable:

Code:

---

1 2 3 4 5 6 7 8

void ma_function(const char *message) { char cmd[1024]; // un chiffre assez grand pour contenir ta commande   sprintf(cmd, "echo %s > log.txt", message); system(cmd); }

---

Note qu'un truc beaucoup plus propre serait de calculer à la volée le nombre de caractères à allouer pour cmd...

Citation:

---

Envoyé par Gui13

Note qu'un truc beaucoup plus propre serait de calculer à la volée le nombre de caractères à allouer pour cmd...

---

La méthode "propre" serait justement :

Code:

---

1 2 3 4 5

char*cmd=(char*)malloc((strlen(message)+ strlen("echo > log.txt")+1)*sizeof(char)); /*Le premier strlen permet d'allouer pour la longueur de message,   Le deuxième pour la longueur de ton autre texte   et le +1 pour \0.*/

---

Après tu peux garder le reste de ton code.... Avec ce code tu peux avoir un nombre infini (en réalité sa dépend de ta mémoire) de caractères dans message. Plus propre encore serait de tester la valeur de retour de malloc...

Et encore plus propre: libérer la mémoire après l'avoir utilisée :)

désolé, mais on fait comment pour la libérer la mémoire après?

je vous remercie pour votre aide, mes messages étant assez court, je l'initialise à 1024 mais je garde en tête la méthode propre ;)

Pour libérer la mémoire, après ton

Code:

---

char* cmd=(char*)malloc((strlen(message) + strlen("echo > log.txt") + 1)*sizeof(char));

---

une fois que tu as fait tout ce que tu voulais avec ton cmd, tu fais:

Code:

---

free(cmd);

---

Et pour faire encore plus propre, il convient de tester si malloc() a bel et bien réussi à allouer la mémoire demandée:

Code:

---

1 2 3 4

if (cmd == NULL) { perror("Memoire non allouée"); }

---

Citation:

---

Envoyé par NoIdea

Code:

---

1 2	char*cmd=(char*)malloc((strlen(message)+ strlen("echo > log.txt")+1)*sizeof(char));

---

---

En complément des remarques précédentes :

• sizeof(char) vaut 1 par définition. Personnellement je recommande plutôt sizeof *cmd).
• En C, le cast du retour de malloc() n'est pas obligatoire.