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