petit soucis de traitement sur les chaines de caracteres.

Bonjour,
j'ai ecris une petite fonction qui me permet de modifier une chaine de caractere. Cette chaine est un chemin (ex: C:\unRepertoire ), et la fonction me renvoi le repertoire courant ( dans ce cas: C: enfin dans la theorie).

Helas, ma fonction ne fonctionne que dans un cas.. mais je ne comprend pas pourquoi :/

je vous montre mon code..

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27

#include<iostream>   using namespace std;   void cdup(char* path){ char *tmp; int i; tmp = strrchr(path,'\\'); i = tmp-path; strncpy(tmp,path,i); strcpy(path, tmp); delete tmp; }   int main (){ char path[MAX_PATH] ="c:\\test\\qwerty"; char path1[MAX_PATH] ="c:\\test"; char path2[MAX_PATH] ="c:\\test\\vfdzgdagf\\qwerrrrrr\\aaaa\\qwwer"; cdup(path); cdup(path1); cdup(path2); cout << "path : " << path << endl; cout << "path1: " << path1 << endl; cout << "path2: " << path1 << endl; system("PAUSE"); return 1; }

---

les resultat:

Code:

---

1 2 3 4

path : c:\test path1: c:est path2: c:est Appuyez sur une touche pour continuer...

---

Si quelqu'un pouvait m'eclaircir les idees...

Merci :)

Si je comprends bien, ta fonction doit simplement tronquer la chaîne juste avant le dernier backslash... Tu te compliques la vie pour rien, mon ami:

Code:

---

1 2 3 4 5 6

void cdup(char * szPath) { char * const pBackSlash = strrchr(szPath, '\\'); if(pBackSlash) *pBackSlash = '\0'; }

---

PS: On peut savoir à quoi servait ton delete ? C'est un (mauvais) miracle que tu n'aies pas eu de crash...

vu de loin je dirai qu'il y a des erreur de maniement de pointeurs... qu'est ce que tu delete à la fin de ta fonction ? :?

bref, quoi qu'il en soit... tu es en c++ et tu ne profite pas du c++ :roll: c'est dommage en faite. Utilise des std::string pour manipuler des chaine de caractères.

std::string au lieu des char* => + de fonctionnalités et pas de problème d'allocation/desallocation

Edit : tu devrai lire ça http://c.developpez.com/faq/cpp/?page=strings

:faq: tout un univer qui s'ouvre à toi :mouarf:

Le même en utilisant std::string :

Code:

---

1 2 3 4 5 6

void cdup(std::string & path) { std::string::size_type const index = path.rfind('\\'); path = path.substr(0, index); }

---

Si c'est destiné à être autre chose qu'un exercice, regarde boost::filesystem, pour éviter de ré-inventer la roue.

Merci pour vos reponses les gars :) je vais explorer tou ca :P

pour le delete.. euh je pense que je devais pas etre bien reveille :roll:

sinon pour le PK j utilise des char* plutot que des string C juste que je travail avec les socket et G lu que pour faire un send(...) ou un recv(...) il demandait des char* et je pensais donc eviter des conversion en plus.. mais peut etre qu il serit mieux d utiliser des string et de les convertir en char* par la suite ??

Citation:

---

Envoyé par Aleksis

je travail avec les socket et G lu que pour faire un send(...) ou un recv(...) il demandait des char* et je pensais donc eviter des conversion en plus.. mais peut etre qu il serit mieux d utiliser des string et de les convertir en char* par la suite ??

---

Oui. C'est ce que fait la fonction membre std::string::c_str.

merci :)

je vais jeter un oeil sur la doc des string et tenter modif tout mon projet ki pour le moment utilise des char* et char[] dans tout les sens :?


Merci a vous pour votre aide :D

:resolu: ?

Citation:

---

Envoyé par Laurent Gomila

:resolu: ?

---

:oops: oups oui oui C resolu j'arrange ca :) :oops: