Discussion :

[guy777]

Bon je suis totalement débutant en C, j'arrive au chapitre des pointeurs et malgré mes recherches, je ne comprends pas tout

Dans tous les exemples que j'ai lus, quand on veut déclarer un pointeur sur un entier (par exemple), on fait :

Code :

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

int *var;

et si j'ai bien compris
var est une adresse
*var est le contenu à l'adresse var

C'est ce que me dis tout le monde et tous les bouquins. Pour illustrer cet exemple, je me suis fait un petit prog :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#include <stdio.h>
 
int main(void) { 
    int* b;
 
    b = 10;
    b++;
 
    printf("Voici les valeurs de a et b : %i", b);
 
    return 0 ;
}

Là, je me suis dit que b ne vaudrait pas 11 au final, et j'ai bien eu raison car 14 (en fait 10 + 4 puisqu'on a décalé le pointeur de 4 octets).

Eh ben, là où je ne comprends pas c'est lorsque je fait :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#include <stdio.h>
 
int main(void) { 
    int* b;
 
    *b = 10;
 
    printf("Voici les valeurs de a et b : %i", b);
 
    return 0 ;
}

Le compilateur ne dis rien et compile normalement, et à l'execution, le prog plante en me mettant le contenu des différents registres du processeur. Pourtant les bouquins disent que *b = 10 fonctionne, et bah moi pas et je pige pas...

Merci d'avance

[Franck.H]

Code :

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

int *var;

et si j'ai bien compris
var est une adresse
*var est le contenu à l'adresse var
[CODE]

On va dire que var "contient" une adresse, ni plus ni moins. D'ailleurs, lorsque tu déclares un pointeur, pense à au moins toujours l'initialiser avec NULL comme:

Code :

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

int *var = NULL;

...et à l'execution, le prog plante en me mettant le contenu des différents registres du processeur. Pourtant les bouquins disent que *b = 10 fonctionne, et bah moi pas et je pige pas...

Normal, tu essayes d'affecter une valeur à une zone mémoire non allouée !
Si tu veux pouvoir lui donner la valeur 10, il faut tout d'abord allouer la zone mémoire, par exemple (pour une seule valeur numérique):

Code :

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

1
2
3
4
5
6
7
 
var = malloc (sizeof (*var));
 
if (var == NULL)
{
   /* Erreur d'allocation, gerer l'erreur ici */
}

Mais attention ... toujours penser à vérer les retour de malloc comme l'exemple ci-dessus ! Aussi, il ne faut jamais oublier de libérer les allocation dynamique avec la fonction free !


Ton second code corrigé serais:

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
#include <stdio.h>
#include <stdlib.h>

int main(void) {
    int* b = NULL;

    b = malloc (sizeof (*b));
    *b = 10;

    printf("Voici les valeurs de a et b : %i", *b);

    free (b);
    b = NULL;

    return 0 ;
}

Attention à:

Code :

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

printf("Voici les valeurs de a et b : %i", b);

c'est pas vrai, ici tu n'affiches pas la valeur de b mais plutôt son adresse, si tel est ce que tu souhaite, tu peux utiliser ce modificateur:
%p et tu peux faire ainsi:

Code :

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

printf("Adresse de b : %p", b);

[guy777]

Effectivement cela marche beaucoup mieux maintenant

Merci pour cette réponse rapide

[fearyourself]

Ce serait bien de vérifier le retour de malloc comme tu le dis et j'ai une préférence pour le free/affectation sur la même ligne...

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
#include <stdio.h>
#include <stdlib.h>

int main(void) {
    int* b = NULL;

    b = malloc (sizeof (*b));

    if (var == NULL)
     {
     /* Erreur d'allocation, gerer l'erreur ici */
     return EXIT_FAILURE;
     }

    *b = 10;

    printf("Voici les valeurs de a et b : %i", *b);

    free (b) , b = NULL;

    return EXIT_SUCCESS ;
}

Jc

[crocodilex]

Foutu copier/coller.....

Ce serait bien de vérifier le retour de malloc comme tu le dis et j'ai une préférence pour le free/affectation sur la même ligne...

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
#include <stdio.h>
#include <stdlib.h>

int main(void) {
    int* b = NULL;

    b = malloc (sizeof (*b));

    if (b == NULL)  /* if (var == NULL) */
     {
     /* Erreur d'allocation, gerer l'erreur ici */
     return EXIT_FAILURE;
     }

    *b = 10;

    printf("Voici les valeurs de a et b : %i", *b);

    free (b) , b = NULL;

    return EXIT_SUCCESS ;
}

Jc