Discussion :

[toutyx]

Bonjour,

voila mon problème :

j'ai une fonction qui prend en paramètre un const char * chemin.
en début de la fonction je copie ce chemin dans une variable char * chemin_temp;
jusque là aucun soucis ...
j'affiche chemin et chemin_temp ils sont identiques.

dans mon programme je modifie chemin_temp pour une raison x, et la survient une chose magique mon chemin est aussi modifié...

Or jusqu'a présent une variable définit en const ne se modifié pas !!!
Est ce quelqu'un sait pourquoi j'ai cette incohérence??

voici mon bout de code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void traiter_fichier(const char * chemin){
     //declaration des variables
     char chemin_temp[255];
 
     memcpy(chemin_temp,chemin,strlen(chemin)+1) ;
     cout<<"chemin : "<<chemin<<endl;
     system("pause");
     cout<<"chemin temp avant modif : " <<chemin_temp<<endl;
     strcpy(chemin_temp,"toto");
     cout<<"chemin apres modif de chemin temp\n"<<chemin<<endl; //chemin modifié
     system("pause"); 
 }

voici ce que j'ai dans ma console dos :

chemin :C:\\Documents and Settings\\Antony\\Bureau\\Test\\doc1.txt
Appuyez sur une touche pour continuer...
chemin temp avant modif : C:\\Documents and Settings\\Antony\\Bureau\\Test\\doc1.txt
chemin apres modif de chemin temp
C:\\Documents and Settings\\Antony\\Bureau\\Test\\doc1.♠
Appuyez sur une touche pour continuer...

je ne sais pas par quelle magie j'ai mon const modifié et surtout il sort d'où ce pique???

Je développe sous Dev-C++ version 4.9.9.2
J'ai déjà essayé d'utiliser strcpy, strdup pour copier le chemin, sans succès toujours la même incohérence...

[Rocklee.gt]

Salut,

Effectivement c'est bizarre.
Mais je viens d'essayer ton bout de code que j'ai mis dans un main et je n'ai eu aucun problème.
Voila le code que j'ai tapé:

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
28
29
#include<iostream>
#include<fstream>
#include<string>
 
void traiter_fichier(const char * chemin);
 
using namespace std;
 
int main(int argc, char *argv){
    //initialisation du chemin
    char * chemin = "C:\\Documents and Settings\\Antony\\Bureau\\Test\\doc1.txt";
 
    //passage de chemin par adresse
    traiter_fichier(chemin);
    system("pause");
}
 
void traiter_fichier(const char * chemin){
     //declaration des variables
     char chemin_temp[255];
 
     memcpy(chemin_temp,chemin,strlen(chemin)+1) ;
     cout<<"chemin : "<<chemin<<endl;
     system("pause");
     cout<<"chemin temp avant modif : " <<chemin_temp<<endl;
     strcpy(chemin_temp,"toto");
     cout<<"chemin apres modif de chemin temp\n"<<chemin<<endl; //chemin modifié
     system("pause"); 
 }

Le résultat que j'obtiens donne ceci:

chemin :C:\\Documents and Settings\\Antony\\Bureau\\Test\\doc1.txt
Appuyez sur une touche pour continuer...
chemin temp avant modif : C:\\Documents and Settings\\Antony\\Bureau\\Test\\doc1.txt
chemin apres modif de chemin temp
C:\\Documents and Settings\\Antony\\Bureau\\Test\\doc1.txt
Appuyez sur une touche pour continuer...
Appuyez sur une touche pour continuer...

Voila j'espère que cela peut t'aider

[nicroman]

Tu ne fais aucun test concernant la longueur du chemin initial (en particulier si celui-ci fait + de 255 charactères).
Pour info, sous windows, il existe un define MAX_PATH qui est la longueur max d'un chemin... et il n'est pas de 255 !

[ram-0000]

Tu ne fais aucun test concernant la longueur du chemin initial (en particulier si celui-ci fait + de 255 charactères).
Pour info, sous windows, il existe un define MAX_PATH qui est la longueur max d'un chemin... et il n'est pas de 255 !

Tu as tout à fait raison mais dans ce cas précis, ce n'est pas le problème.

[Médinoc]

Je ne vois aucune erreur dans la fonction elle-même (à part le manque de contrôle du buffer).
Peux-tu montrer le code qui appelle cette fonction ?
Typiquement, comment est définie la valeur passé en paramètre?

[toutyx]

Merci pour vos réponses,

voici le code qui appelle cette fonction :

Code :

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

1
2
3
4
int main(int argc, char *argv){
   traiter_fichier(argv[1]);
   return 0;
}

je ne comprends toujours pas...
Est ce que cela peut venir de l'environnement de développement??

[toutyx]

Tu ne fais aucun test concernant la longueur du chemin initial (en particulier si celui-ci fait + de 255 charactères).
Pour info, sous windows, il existe un define MAX_PATH qui est la longueur max d'un chemin... et il n'est pas de 255 !

oui je sais bien, mais bon à terme je ne voudrais pas passer par ce genre d'astuce, j'aimerais bien comprendre pourquoi en un const se modifie...

[ram-0000]

Si ton programme est aussi simple que celui que tu exposes, il ne reste plus que le bug dans la librairie C de ton compilateur. J'ai un peu de mal à y croire parce que sur des fonctions aussi simples et triviales, cela se saurait mais bon !!!.

En désespoir de cause, as tu regardé du côté du code assembleur généré ? Je sais, c'est pas très cool à lire mais il faut bien faire avancer le chmilblick...

[toutyx]

Si ton programme est aussi simple que celui que tu exposes, il ne reste plus que le bug dans la librairie C de ton compilateur. J'ai un peu de mal à y croire parce que sur des fonctions aussi simples et triviales, cela se saurait mais bon !!!.

En désespoir de cause, as tu regardé du côté du code assembleur généré ? Je sais, c'est pas très cool à lire mais il faut bien faire avancer le chmilblick...

non je ne me suis pas encore aventuré dedans...

par contre j'avance petit à petit car ce j'ai fait c'est au lieu de passer un char * en parametre de la fonction je passe un string que je convertit après en char * avec comme dans la FAQ un buffer.
Je me suis rendu compte que le passage en string -> char * ne pose pas de problème par contre le passage entre char * -> string c'était pas la même chose... il me modifie la fin de ma chaine... en me mettant des caractères illogique (trèfle, pique,...)

(ndlr comme s'il voulait jouer aux cartes...)
(ok je sors !)