Contenu d'une variable qui disparait :/

Version imprimable

Bonjour :)

Comme l'indique le titre, j'ai un petit soucis de vairable.. 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
 
void Server::Rename(char* fileName, SOCKET& soc){
  Commande* com;
  HANDLE hFile;
  com = new Commande("","");       
  cout << "FILENAME1 : " << fileName <<endl;
  if ((hFile = CreateFile(fileName,
                          GENERIC_WRITE,
                          0,
                          NULL,
                          OPEN_EXISTING,
                          0,
                          NULL)) == INVALID_HANDLE_VALUE){
    sendReply(550,soc);
    return;
  }
  CloseHandle(hFile);  
  sendReply(351,soc);
  cout << "FILENAME1 : " << fileName <<endl;
}

Code:

1
2
3
4
5
6
7
8
9
 
void Server::sendReply(int code, SOCKET& soc){
  char tmp[65535]= "";
  int nbcar;
  strcat(tmp," ");     
  strcat(tmp,getMsgReply(code));
  nbcar = send(soc,tmp,strlen(tmp),0);                  
  cout << "réponse '"<< tmp << "' envoyee"<<endl;
}

Après l'éxécution de la commande : sendReply(351,soc);, le contenu de mon char* fileName disparait.. je ne comprend pas trop pourquoi :cry: si quelqu'un pouvait m'éclaircir les idées...

merci ;)

PS: je travail sous Windows avec Dev-c++ 4.9.9.2

22/05/2006, 15h36
b Oo

Salut Aleksis,
je voudrais savoir, tu es sûr que sendReply(351,soc); est exécutée, car tu fais un return; dans le if donc si tu passes la fonction se termine.
Le mieux est peut-être de faire un cout après la métode rename.
Tu peux me dire comment tu as initialisé ton char * ?

b Oo
22/05/2006, 15h43
Aleksis
SAlut b Oo;)
alors mon char est initialiser a l'aide d'un constructeur.
Code:

1 2 3 4 5 Commande::Commande(char* commande, char* parametre){ cmd = commande; param = parametre; }
jusque là, aucun soucis, je peu utiliser tout ça sans problème ( j'ai fais des methode get).

une fois ma structure crée, je pass le champs param en paramètre
Code:

1 2 3 4 if (strcmp(com->getCmd() ,"RNFR") == 0){ Rename(com->getParam(),soc); }
la j'arrive donc dans ma fonction avec ma variable bien initialisé à la bonne valeur.

Et oui je suis sur que sendReply(351,soc); est exécutécar je fais un cout juste avant et un autre juste après et les 2 s'affiche. Juste que sur le 2eme ma variable fileName ne contient plus rien :s
22/05/2006, 15h55
b Oo

Tu ne peux pas modifier ce char * (car tu n'alloues pas de mémoire pour la chaine) donc met le en const char *.
Si cela n'a pas résolu le problème, l'erreur doit venir d'ailleurs (tu dois modifier ton pointeur).

b Oo
22/05/2006, 16h04
Aleksis

j'ai suivi ton conseil, j'ai mi ma var en const.. mais hélas.. le problème est toujours la :/ merci quand même :)
22/05/2006, 16h12
b Oo

Tu peux me donner le code de CreateFile(...), car cela vient peut-être de là.

b Oo
22/05/2006, 16h29
Laurent Gomila

CreateFile est une fonction de l'API Win32.

A part ça, en ne stockant que des pointeurs vers les mêmes chaînes constantes, tu t'exposes sans doute à ce genre de problème.

Essaye de stocker des copies de tes chaînes, et utilise std::string pour ça (voir la FAQ).
22/05/2006, 16h35
b Oo
Bon c'est quand même bizarre, il faudrait avoir un peu plus de code pour voir d'où cela vient. Sinon essayes :
Code:

1 2 3 4 void Server::Rename(const char * const fileName, SOCKET& soc){ ... }
Si tu n'as pas d'erreur de compilation, et si ce n'est pas affiché, cela m'étonne beaucoup que l'erreur vienne du code posté (plutôt à l'appel de la fonction).

b Oo
30/05/2006, 14h03
Aleksis

Citation:

Envoyé par Laurent Gomila

CreateFile est une fonction de l'API Win32.

A part ça, en ne stockant que des pointeurs vers les mêmes chaînes constantes, tu t'exposes sans doute à ce genre de problème.

Essaye de stocker des copies de tes chaînes, et utilise std::string pour ça (voir la FAQ).

il m'a fallu un peu de temps avant de m'y remettre.. j'ai fait ce que tu m'as propose, ca a marcher :) mais j aimerais quand meme savoir pourquoi le contenu disparait :/
30/05/2006, 14h13
Médinoc

À mon avis, tu y verras plus clair si tu rajoutes des const partout où il le faut... Tu verras sans doute qu'une chaîne supposée constante se prend une écriture là où ça fait mal...

(ou bien, tu peux aussi avoir un dépassement de buffer si tu as deux tableaux de caractères qui se suivent, et que suite à une erreur, tu écris le \0 non pas à la fin d'un buffer, mais au début du suivant...)

Code:

1 2 3 4 5 6 7 8 9 void Server::sendReply(int code, SOCKET& soc){ char tmp[65535]= ""; int nbcar; strcat(tmp," "); strcat(tmp,getMsgReply(code)); nbcar = send(soc,tmp,strlen(tmp),0); cout << "réponse '"<< tmp << "' envoyee"<<endl; }
voila :) j'ai trouvé le pourquoi du comment :) appareemnt dans ma fonction sendReply, la décalration de ma var tmp écrasserait ma valeur

donc en déclarant cette var en static

Code:

char tmp[65535]= "";

ça marche très bien :)

en tout cas merci a tous pour vos différentes solution proposées :)