Discussion :

[bradley vier]

Bonsoir,

Lorsque je fais tourner le bout de code ci-dessous, j'ai quelques problèmes avec des caractères non désirés

Code :

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

1
2
3
4
5
6
7
8
9
10
 
char *commande2 = new char[200];
	strcat (commande2, "cp ");
	strcat (commande2, nomfichier_me_temp.data());
	strcat (commande2, " ");
	strcat (commande2, nomfichier_me.data());
	cout << "\nCOMMANDE 2" << endl;
	cout << commande2 << endl;
	etat = system(commande2);
	delete [] commande2;

et j'obtiens :

üÀ#@üÀ#@cp /home/essai-tmp /home/essai

J'ai essayé de remplir le tableau de caractères blancs et ca fonctionne mais n'existe-t-il pas une solution plus élégante?

Merci d'avance

[piff62]

et avec

Code :

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

1
2
 
 strcpy (commande2, "cp ");

au lieu de

Code :

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

1
2
 
 strcat (commande2, "cp ");

Ca change quelque chose ? car la premiere fois je sais pas a quoi tu concatene ton truc

[Troll]

Je suppose que tes nomfichier_me* sont des AnsiString.
Dans ce cas, je te propose de continuer avec les AnsiString.

Code :

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

1
2
3
4
5
6
7
8
 
AnsiString commande2;
commade2.printf("cp %s %s", nomfichier_me_temp, nomfichier_me);
 
cout << "\nCOMMANDE 2" << endl;
cout << commande2.c_str() << endl;
 
etat = system(commande2.c_str());

Ha non, je me suis trompé de forum, ça doit plutôt être des string
Oublie !

[Laurent Gomila]

Effectivement, strcat concatène une chaine à la fin d'une autre. Donc si ta chaine c'est n'importe quoi, ça concatenera n'importe ou (au 1er \0 trouvé en l'occurence).

http://c.developpez.com/faq/cpp/?page=strings

Code :

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

1
2
3
4
#include <string>
 
std::string commande2 = "cp" + nomfichier_me_temp + " " + nomfichier_me;
etat = system(commande2.c_str());

[piff62]

Effectivement, strcat concatène une chaine à la fin d'une autre. Donc si ta chaine c'est n'importe quoi, ça concatenera n'importe ou (au 1er \0 trouvé en l'occurence).

Comme quoi je ne dit pas que des conneries lol

[rolkA]

Code :

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

1
2
3
4
5
6
7
8
 
AnsiString commande2;
commade2.printf("cp %s %s", nomfichier_me_temp, nomfichier_me);
 
cout << "\nCOMMANDE 2" << endl;
cout << commande2.c_str() << endl;
 
etat = system(commande2.c_str());

Sauf qu'AnsiString c'est pas du C++ standard, c'est une classe de la VCL de C++Builder, introduite pour l'équivalence avec les chaînes AnsiString de Delphi.

[Edit]Ok j'avais pas vu l'edit ^^, fait pendant que je postais :p[/edit]

[bradley vier]

En utilisant std::string cela fonctionne

Merci beaucoup !

[narsi1]

Bonjour

J'ai essayer cette façon de faire et ça ne fonctionne pas... pouvez-vous m'aider s'il vous plaît
voici le code C(++):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#include <iostream>
#include <stdlib.h>
#include <string.h>
#include <string>
 
int main(){
  int h=12, min=00, sec=00;  //j'impose l'heure à MIDI
 
  std::string commande = "TIME " + h + ":" + min + ":" + sec;
  cout << commande << endl; //je veux afficher TIME 12:00:00
 
  system("PAUSE");	
  return 0;
  exit(0);
}

le compilateur n'accepte pas cette ligne :

Code :

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

std::string commande = "TIME " + h + ":" + min + ":" + sec;

et je sais pas pourquoi...

[Balise [code] ajouté par netah_25 - merci d'y penser a l'avenir]

[Laurent Gomila]

Il faut que le premier terme de la concaténation soit un string.

Code :

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

std::string commande = std::string("TIME ") + h + ":" + min + ":" + sec;

[narsi1]

Merci pour ta réponse, mais même avec la modif le compi ne l'accepte pas...
je suppose plutôt que ça provient du mélange types : string avec int...
ce n'est juste qu'une supposition, je peux me tromper.

[Laurent Gomila]

Hmm en effet, j'avais pas fait gaffe. Je croyais que tu étais resté sur l'exemple précédent qui ne comportait que des chaînes. Il faudra ici passer par un ostringstream.

Code :

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

1
2
3
4
5
#include <sstream>
 
std::ostringstream oss;
oss << "TIME " << h << ":" << min << ":" << sec;
std::string commande = oss.str();

[narsi1]

J'te remercie de tout coeur !!!!
ça roule, impeccable !!