Discussion :

[dtcSearch]

Bonjour,

j'ai crée une fonction:

Code :

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

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 :

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

ma_function("erreur dans la partie 1");

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

[dtcSearch]

Code :

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

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

[3DArchi]

Code :

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

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 :

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

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.

[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 :

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

1
2
3
4
5
6
7
void ma_function(const char *message)
{
  char cmd;
 
  sprintf(cmd, "echo %s > log.txt", message);
  system(cmd);
}

[laflak]

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 :

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

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)

[flo291288]

Marche ce code la ?

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

+1 .....

[Gui13]

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

Faut initialiser cmd avec une taille convenable:

Code :

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

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...

[NoIdea]

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 :

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

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...

[Gui13]

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

[dtcSearch]

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

[mikhailo]

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

Code :

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

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 :

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

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 :

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

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

[gl]

Code :

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

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.