Discussion :

[freeman43]

salut a tous,
voila j essaye de faire un programme qui permet de convertir les lettre d une chaine de caractere en leur correspondance en decimal.
Ces chaine de caractere sont des pointeurs.
Cette convertion doit se faire dans un void.

donc voila un bout de programme,mais ca ne marche pas et je ne comprend pas pourqoi.

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
28
29
30
31
32
 
#include<stdio.h>
#include <stdlib.h>
#include <string.h>
void convertion(char* pt_X);
int res;
 
int main (void)
{
    char *pt_X=(char*)malloc(15*sizeof(char*));
    char *pt_nom1=(char*)malloc(15*sizeof(char*));
    printf("quelle est votre nom:\n");
    gets(pt_nom1);
    pt_X=pt_nom1;
    convertion( pt_X);
 
free(pt_nom1);
free(pt_X);
 
    getch();
 
}
void convertion(char* pt_X)
{
    int i=0;
    //printf("coucou");
    for(i=0;*(pt_X+i)!='\0';i++)
    {
                      res='*(pt_X+i)';
                      printf("a:%d",res);
    }
}

Merci.

[bountykiler]

Bon, déjà pour commencer ton code n'est pas du C++ mais du C tout court.
Si tu veux coder correctement en C++, commence par utiliser la class string par exemple plutot que des char*.
Ensuite dans ton code, je vois ceci:

Code :

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

1
2
3
4
5
 
    pt_X=pt_nom1;
    //.. 
    free(pt_nom1);
    free(pt_X);

Petite question: Ton appli ne plante pas à la ligne free(pt_X)???
Parce que ce que tu fais, c'est initialiser un pointeur de char, ensuite tu fais une copie de ce pointeur. Donc pt_X et pt_nom1 pointe tout deux vers la même adresse, et donc tu libère 2 fois la même zonz mémoire. (Je ne parle même pas du fait que tu dois avoir un memory-leak...)
Si maintenant tu veux recopier la chaine de caractère pointée par pt_nom1 plutot que le pointeur lui-même, utilise plutôt strcpy, tu verra ça devrait déjà aller beaucoup mieux.