const char+string+const char == error. // Probleme de sytaxe ou de compilateur?

Bonjour,
je fourni a une fonction une chaine de caractère (une url) en paramètre, qui doit être sous forme de char*.

J'ai deux location, donc deux ip, donc dans mon code j'ai placer

Code:

---

fonction("http:\\"+location+"\xxx")

---

C'est alors que code block avec MinGW me sort:

Code:

---

Warning: cannot pass objects of non-POD type `struct std::basic_string<char, std::char_traits<char>, std::allocator<char> >' through `...'; call will abort at runtime|

---

Et effectivement, ca passe pas.
Alors qu'il me semble que sous dev-cpp je faisais la même chose et que ca marcher tres bien.
J'ai un peus jongler avec les type de données, même que totalement désespère j'y suis aller a coup de c_str(), mais ca ne voulait même plus se compiler. (normale :D)

Quelqu'un pourrait m'éclairer.

Merci

Code:

---

1 2	std::string prout("http://" + location + "\xFF"); //Remplace FF par quelque chose en hexadécimal fonction(prout.c_str());

---

Citation:

---

Envoyé par raphamil

Code:

---

1 2	std::string prout("http://" + location + "\xFF"); //Remplace FF par quelque chose en hexadécimal fonction(prout.c_str());

---

---

C'est pas mieux tu ajoutes toujours des pointeurs
Tu peux faire ça par exemple:

Code:

---

1 2 3 4

std::string prout("http://"); prout += location; prout += ...; fonction(prout.c_str());

---

Le message d'erreur est pourtant clair.
Tu passes un objet de type std::string à une fonction à nombre d'arguments variable, ce qui n'est pas autorisé (en effet, on ne peut passer que des PODs à ce genre de fonctions).

Citation:

---

Envoyé par Twindruff

C'est pas mieux tu ajoutes toujours des pointeurs
Tu peux faire ça par exemple:

Code:

---

1 2 3 4

std::string prout("http://"); prout += location; prout += ...; fonction(prout.c_str());

---

---

Et alors ?
En quoi ce n'est pas bien ?

Ben faire la somme de deux adresses absolues ça n'a pas de sens.
Exemple:

Code:

---

1 2 3 4 5 6 7

{ const char *a = "abc"; const char *b = "def";   const char *c = a + b; }

---

c ne pointe pas vers une chaine qui serait "abcdef", mais c contient la somme des deux adresse contenues dans a et b. Donc quelque chose d'inconnu. Le + c'est bien la somme et pas la concaténation.

Si vous lui expliquez comme ça, il va rien comprendre le pauvre…

Version courte :

Pour pouvoir concaténer des chaines de caractères, il faut t'assurer que le premier opérande soit de type std::string. Pour cela, tu peux effectuer une conversion explicite en appelant le constructeur de std::string qui prend un const char* :

Code:

---

url = std::string("http://") + location + "/xxx";

---

En effet, la concaténation de chaines à l'aide de l'opérateur + ne fonctionne pas avec les chaines littérales seules (de type const char*).
Faire cela revient à additionner des pointeurs, ce qui n'est pas du tout ce que tu cherches à faire.

L'opérateur + est surchargé pour le type std::string, ce qui te permet de concaténer facilement des chaines de ce type. La seule condition, c'est que le premier opérande soit de type std::string.

C'est donc un problème de sémantique (pas syntaxe) et non de compilateur.




Ensuite, pour convertir en const char* :

Code:

---

fonction(url.c_str());

---

Autre chose : tu as des « \ » dans tes chaines de caractères, alors que les url contiennent des « / » uniquement.
Attention, en C (et dans tous les langages de la même famille) « \ » est un caractère spécial dans les chaines littérales.

lol Florian GOO,
je retien .c_str();

Je devait etre fatiguer, car je savais que ca passerai, mais je ne sais plus trop pourquoi je ne voulais pas declarer de nouvelle variable.

[comme je ne peu pas avoir de changement de location en cours d'execution >
1 choix de location
2 string a = "http:\\"+location] //déclaration avant le main, ou en def
3 loc = location+ce_dont_g_besoin_sur_le_moment
4 fonction(param, ..., loc.c_str() );

oui, ce qui m'enbaiter c'ete d'avoir une ligne de plus pour chaque url apeller.

Citation:

---

Envoyé par Florian Goo

L'opérateur + est surchargé pour le type std::string, ce qui te permet de concaténer facilement des chaines de ce type. La seule condition, c'est que le premier opérande soit de type std::string.

---

Le premier opérande d'un opérateur peut ne pas être un objet s'il est définit par une fonction (pas par une méthode). Tu peux donc rajouter:

Code:

---

1 2 3 4 5 6 7 8 9

string operator + (const char* c, const string& s) { return string(c) + s; }   int main() { string s = "bar"; string t = "foo " + s; // ok }

---

l'operateur + marche tres bien, cessez de l'embrouiller

raphamil avait donné tout de suite la bonne réponse!!

Heu non ça compile pas ça hein

Code:

---

1 2	char *location = "abc"; std::string prout("http://" + location + "\xFF");

---

Certes, mais ce code fonctionne parfaitement :

Code:

---

1 2	string location = "developpez.net"; string s = "http://" + location + "/forums";

---

le warning n'est pas lié à ça, mais à autre chose. Il passe une std::string à une fonction de type printf (cf message de loufoque).

Ça marche si location est un string mais pas si c'est un char*. Les messages précédents ne disent pas son type je crois :(