Où est l'abbhération informatique sur ces ***** de pointeurs

Question

Salut &#224; tous et &#224; toutes, 
Je ne sais pas comment j'aurais du nommer le topique mais en fait pouvez vous me dire o&#249; est l'abbh&#233;ration informatique l&#224; dedans : 
 
#include <stdio.h> 
 
void CopieChaine(char * pdest, char * psrc) 
{ 
 while(*psrc != '\0') // Tant que le caractere de la chaine source n'est pas 
      // un ensemble vide 
 { 
  *pdest = *psrc; // On met la valeur de psrc dans pdest 
  *psrc++; // On incremente psrc 
  *pdest++; // On incremente pdest 
 } 
  
 if(*psrc == '\0') 
  pdest[*psrc] = '\0';  
} 
 
int main() 
{ 
 char source[512]/*="Un beau dimanche"*/; 
 char *destination; 
 char indic='o'; 
 
 do  
 {  
  // on rentre la chaine  
  printf("Veuillez rentrer une chaine de caractere :
");  
  gets(source); 
   
  printf("%s
", source); 
 
  CopieChaine(destination, source); 
   
  // on affiche la source et la destination 
  printf("Chaine source = %s", source); 
  printf("
Chaine destination = %s
", destination); 
 
  printf("'q' pour quitter
");  
  scanf("%c", &indic); 
   
 }while(indic != 'q');  
 
 return 0;  
} 
 
Mon prof pr&#233;tend que ce qui est &#233;crit en rouge ne devrait th&#233;oriquement pas march&#233;, car il nous a dit que *chose renvoie en fait une adresse donc dans cet exemple eh ben , je devrais l'avoir dans l'os, mais si vous compilez ce programme, miracle, il marche &#224; merveille...  et si je remplace *pdest++ et *psrc++ par pdest++ par psrc++, mon programme ne fonctionne plus
 
Pouvez vous m'expliquer en d&#233;tail ces quelques lignes. Merci...

wxccxw · Answer

pas obliger de faire 3 topics pour poser ta question !
ps : Rien est en rouge :)

cs_bouba · Answer

Y'a aucun problème au niveau de la compilation à mon avis. Par contre c'est stupide de faire
  *psrc++; // On incremente psrc
  *pdest++; // On incremente pdest
Ce que l'on veut ici c'est une copie de chaîne
donc 
  psrc++; 
  pdest++;
Sinon, c'est balot, le programme ne plante pas mais aucune copie n'est faite, seule la valeure '\0'-1 sera copiée dans *pdest, les pointeurs ne serait pas incrémentée, pour sortir de la boucle il faudrait attendre que psrc[0] vale '\0'.
Puis la condition
 if(*psrc == '\0')
est inutile puisque l'on sorte du while avec pour valeur de *psrc 0. Mettre directement
pdest[*psrc] = '\0';  
 
Voilà, pas grand chose à dire de plus.

Bouba.

MasterShadows · Answer

Y'a 3 topics parce mon pc a plant&#233;.
Mon programme fonctionne car quand j'&#233;cris coucou, il me renvoy&#233; bien en sortie coucou.
En fait, on pouvait remplacer &#231;a :

while(*psrc != '\0') // Tant que le caractere de la chaine source n'est pas 
      // un ensemble vide 
 { 
  *pdest = *psrc; // On met la valeur de psrc dans pdest 
  *psrc++; // On incremente psrc 
  *pdest++; // On incremente pdest 
 } 

par &#231;a :
while(*psrc) // Tant que le caractere de la chaine source n'est pas 
      // un ensemble vide 
 { 
  *pdest = *psrc; // On met la valeur de psrc dans pdest 
  psrc++; // On incremente psrc 
  pdest++; // On incremente pdest 
 } 

ou encore
while(*pdest++ == *psrc++) 

Enfin, marci quand meme

cs_louis14 · Answer

ce qui m'&#233;tonne le plus dans ce code , c'est qu'il n'y a pas de r&#233;servation de m&#233;moire pour la destination. d&#233;j&#224; &#224; la d&#233;claration 
char* destination = 0;
en rentrant dans la fonction :
-si destination est diff&#233;rent de 0 on d&#233;salloue la m&#233;moire et on refait une allocation de la taille de source. puis on recopie.

louis14

Où est l'abbhération informatique sur ces ***** de pointeurs

4 réponses

Votre réponse