Discussion :

[ceimoi]

Bonjour,

J'ai le code suivant

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
#include <stdio.h>
#include <stdlib.h>
 
int main (void)
{
  int x=10;
  int *px;
 
  px=malloc(sizeof(int));
  if ( px == NULL )
     return 0;
 
  x=10;
  px=&x;
 
  printf (" la valeur est %d\n",*px);
 
  free(px);
 
}

Ca compile puis quand j'exécute, j'ai bien le résultat qui s'affiche (10), mais j'ai aussi l'erreur suivante qui s'affiche *** Error in `./test': free(): invalid pointer: 0x00007ffdf42e0748 ***
Aborted

Question 1: Vous savez m'expliquer d'où vient l'erreur ? Pourquoi j'ai un problème en libérant mon pointeur ?
Question 2: si je voulais afficher l'adresse de x (ca n'a pas forcément d'intéret, c'est juste comme ça) - c'est stocker dans px pour pouvoir pointer sur x, mais je n'arrive pas l'afficher - comment faire ?

Merci pour votre aide

[Sun.Wu.Kong]

Le malloc et le free ne serve a rien.

Le malloc te crée un int dans la memoire. Le free libère ce int... sauf que tu liberes x et non le inr crée par malloc

[Sun.Wu.Kong]

Quand je dit ne servent à rien, c'est dans ton cas

[Sun.Wu.Kong]

Pour la question 2, printf de px devrait afgicher l'adresse de x

[ceimoi]

Bonjour,
- pour la question 1 : le malloc et free ne servent à rien parce que je travaille sur un int que j'ai déclaré avant c'est ça? Si j'avais utilisé uniquement des pointeur, il aurait fallu que j'utilise malloc & free ?
- pour la question 2 : si je fais printf (" px est %d\n",px), ca me fait une erreur de compilation "error: format specifies type 'int' but the argument has type 'int *' [-Werror,-Wformat]". Ma syntaxe est mauvaise ?

[sambia39]

Bonjour,

Bonjour,

J'ai le code suivant

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

int main (void)
{
  int x=10;
  int *px;
  
  px=malloc(sizeof(int));
  if ( px == NULL )
     return 0;
  
  x=10;
  px=&x;
  
  printf (" la valeur est %d\n",*px);

  free(px);

}

Ca compile puis quand j'exécute, j'ai bien le résultat qui s'affiche (10), mais j'ai aussi l'erreur suivante qui s'affiche *** Error in `./test': free(): invalid pointer: 0x00007ffdf42e0748 ***
Aborted

Question 1: Vous savez m'expliquer d'où vient l'erreur ? Pourquoi j'ai un problème en libérant mon pointeur ?
Question 2: si je voulais afficher l'adresse de x (ca n'a pas forcément d'intéret, c'est juste comme ça) - c'est stocker dans px pour pouvoir pointer sur x, mais je n'arrive pas l'afficher - comment faire ?

Merci pour votre aide

Votre erreur, viens du fait que vous avez attribué à la variable pointeur "px" une autre adresse que celle qui a été allouer et comme votre pointeur pointe sur une autre adresse la fonction "free()" ne peut libérer l’espace mémoire assigné (car l'objet pointé n'est pas alloué) générant ainsi une erreur de trap(6) EXC_CRASH (SIGABRT) de plus la mémoire allouée précédemment ne étant pas libérée entraînent en conséquence la où des pertes de mémoire irrécupérables (le programme crée des fuites de mémoires). Pour corriger le problème, il faut déréférencé le pointeur pour qu’il puisse contenir la valeur de la variable "X" comme ceci: *px = x

Plus de précision: La manipulation des pointeurs se fait de la façon suivante:
L’opérateur ‘&' permet d’obtenir l’adresse d’une variable de type type exemple: int *p = &x.L’opérateur ‘&' ne peut être appliqué qu’a des variables donc, des objets qui se trouvent uniquement dans la mémoire interne et ne peut pas être utilisés à des variables déclarées avec les mots-clés « register » ou des expressions.
L’opérateur ‘*’ permet d’accéder à la valeur de l’objet pointé en clair au contenu de l’adresse. Ce procédé est appelé déréférencement de pointeur Exemple:

Code C :

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

1
2
3
4
 
    int x = 10;
    int *p = &x;
    *p = 19; // change la valeur de x.

Il est toutefois conseillé de faire attention car, un pointeur est une variable comme toutes les autres donc un pointeur doit être initialisé a « NULL » en attente de données a pointé car, si vous ne le faites pas elle contiendra une valeur aléatoire (sauf cas de static) donc vers une donnée en mémoire quelconque que vous n’avez pas le droit d’accès.

à l'avenir il faut êtres très vigilants à la manipulation des pointeurs et voici d'autre d'informations qui vous seront très utiles
à bientôt

[ceimoi]

Merci Sambia39 de la réponse détaillée.

Est ce que le code suivant :

Code :

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

1
2
3
4
5
6
 
  int x=10;
  int *px;
  x=10;
  px=&x;  
  printf (" la valeur est %d\n",*px);

est une utilisation "non conforme" où j'utilise un pointeur que je ne déréférence pas? L'utilisation correcte est avec *px=x ?

Pour ma 2ieme question : si jamais je veux afficher l'adresse de x - comment faire?
printf (" la valeur est %d\n",px) ne fonctionne pas car %d attend un int et pas un int* comme px

[dalfab]

Code :

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

1
2
3
4
5
6
 
  int x=10;
  int *px;
  x=10;
  px=&x;  
  printf (" la valeur est %d\n",*px);

est une utilisation "non conforme" où j'utilise un pointeur que je ne déréférence pas? L'utilisation correcte est avec *px=x ?

C'est conforme, le pointeur pointe quelque part (sur x), et il est déréférencé au moment du printf par *px, et il n'y a ni allocation ni dés-allocation dynamique.
après avoir affecté px, on peut utiliser *px pour lire ou écrire, on pourrait ensuite effectuer *px = x + 1 pour lire 11 dans x.

Pour ma 2ieme question : si jamais je veux afficher l'adresse de x - comment faire?
printf (" la valeur est %d\n",px) ne fonctionne pas car %d attend un int et pas un int* comme px

En effet il faut faire printf(" la valeur du pointeur %n\n" , px ).

Correctif, merci Sambia, il faut utiliser %p pour le format des pointeurs, surtout pas %n qui sert à tout autre chose.

[sambia39]

Bonjour,
Attention, l'instruction proposé est incorrect

printf(" la valeur du pointeur %n\n" , px ).

Pour affiché l'adresse du pointeur ou d'une variable on utilise le spécificateur de format "%p"
fprintf("Adresses\t:%p\n", (void*)&x);
fprintf("Adresses\t;%p\n", px);
A bientôt

[ceimoi]

Merci à tous