petit soucis de traitement sur les chaines de caracteres.

**Aleksis** · 03/08/2006, 16h01

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 :

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

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 :

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

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

**Médinoc** · 03/08/2006, 16h11

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 C (eh oui, c'est pas vraiment du C++) :

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

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

**superspag** · 03/08/2006, 16h13

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

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

tout un univer qui s'ouvre à toi

**roulious** · 03/08/2006, 16h20

Le même en utilisant std::string :

Code C++ :

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

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.

**Aleksis** · 03/08/2006, 16h26

Merci pour vos reponses les gars

je vais explorer tou ca

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

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

**roulious** · 03/08/2006, 16h30

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.