malloc(); et visual studio

bonjour !

Encore une question sur une fonction en C!!

On vient de voir les pointeurs en cours, et chez moi, sous visual studio c++, lorsque j'essaye de compiler :

Code:

---

1 2 3 4 5 6 7

void main() { int *ptab; *ptab = (int *)malloc(sizeof(int)); //printf("%d\n",*(ptab+1)); system("pause"); }

---

j'ai l'erreur suivante: error C2440: '=' : cannot convert from 'int *' to 'int'
This conversion requires a reinterpret_cast, a C-style cast or function-style cast

Je n'arrive pas à comprendre l'erreur...

je viens de voir que si par contre je l'ecris comme ça:

Code:

---

1 2 3 4 5 6 7

void main() { int *ptab; ptab = (int *)malloc(sizeof(int)); //printf("%d\n",*(ptab+1)); system("pause"); }

---

La compilation ne trouve aucune erreur...

Citation:

---

Envoyé par Spike Spiegel

Code:

---

1 2 3 4 5 6 7

void main() { int *ptab; *ptab = (int *)malloc(sizeof(int)); <----------------- //printf("%d\n",*(ptab+1)); system("pause"); }

---

---

Sur la ligne flechée, tu déréférences ta variable ptab en mettant l'étoile devant. Tu désignes donc la valeur de ce pointeur, or malloc renvoie un pointeur.
Il faut donc enlever l'étoile.

*ptab = valeur de ton entier
ptab = adresse de ton entier

a+

Salut !

Rien à voir avec Visual Studio, c'est juste qu'il y a mélangeage de pinceau :)

int *ptab; est un pointeur sur un entier.
(int *)malloc(sizeof(int)); alloue de la mémoire, et te retourne l'adresse de celle-ci. C'est donc à ton pointeur (ptab) que tu dois l'affecter, comme tu l'as remarqué dans ton 2ème message :)

Comme le dit le compilateur, *ptab est un entier (tu déréférence le pointeur sur entier, donc tu obtiens un entier). Tu ne peux pas lui affecter une adresse mémoire ... Par contre, tu pourrais écrire:

Code:

---

*ptab = 1;

---

A condition bien sûr d'avoir alloué la mémoire avant ;)

merci mayhem.

Pourtant sur un cours de C présent sur en lien sur ce site, il y a ça comme exemple:

Code:

---

1 2 3 4 5 6 7

int *p = malloc(10*sizeof(int)); if(p) { p[0] = 5; printf("%d",p[0]); free(p); }

---

et ce code ne passe pas non plus.

Ah, j'ai été trop lent pour répondre :)
Euh, tu es sur que ce code ne compile pas ?

Code:

---

int *p = malloc(10*sizeof(int));

---

Sinon, cette fois ci il y a une étoile à gauche du malloc, car la déclaration du pointeur et l'affectation sont sur la même ligne: tu déclares un int*, et tu l'affectes par la même occasion

c'est sûr que ça ne passe pas...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#include<stdio.h> #include<stdlib.h>   void main() { int *p = malloc(10*sizeof(int)); if(p) { p[0] = 5; printf("%d",p[0]); free(p); } }

---

renvoi l'erreur: error C2440: 'initializing' : cannot convert from 'void *' to 'int *'
Conversion from 'void*' to pointer to non-'void' requires an explicit cast :roll:

Il me semble que certains compilateurs font le cast sur le retour de malloc implicitement en fonction du type de donnée auquel on les associe.

ainsi

Code:

---

int *ptab = malloc(sizeof(int))

---

ne passe pas sur Visual 6

alors que

Code:

---

int *ptab = (int *)malloc(sizeof(int))

---

passe puisque le retour du malloc est casté vers le bon type.

A confirmer, c'est peut-être d'anciens compilateurs qui acceptaient cette écriture...

sinon, merci je viens de comprendre que ces deux formes:

Code:

---

1 2 3 4

int *p = malloc(10*sizeof(int)) et int *p; p = malloc(10*sizeof(int));

---

veulent dire la même chose sauf que sur la deuxième on déclare et on allou la mémoire en une seule ligne :)

Citation:

---

le retour du malloc est casté.

---

désolè mais je ne comprends pas encore ce que signifie caster. :oops:

le cast te permet de convertir un type de donnée vers un autre explicitement.
Dans ton exemple, la fonction malloc renvoie un pointeur sur le type 'void' or tu lui affectes un pointeur vers un tableau d'entier.
Il faut donc convertir le retour du malloc vers le type dont tu as besoin, c'est à dire un pointeur vers un entier.
celà se fait en préfixant la fonction par le type voulu entre parenthèse :

(int *)malloc(sizeof(int))

Ceci va donc convertir le 'void *' retourné par la fonction malloc en 'int *'

Ah oui c'est vrai que des compilateurs font les difficiles :)
Sinon, "caster", c'est changer de type.
Par exemple:

Code:

---

1 2 3

float f = 2.653; int i = (int) f;

---

Ici tu "castes" f en entier, et i vaudra donc 2. Ici le cast n'était pas nécessaire, il aurait été fait par le compilateur (dans ce cas, on appelle ca un cast implicite).

Et donc ici, comme le disait Mayhem, malloc renvoie un (void*). C'est un type générique un peu particulier. Il faut donc le "caster" en ce que tu veux, donc dans ton cas un (int*).

Bon, comme je souhaiterai vraiment comprendre les pointeurs et rien oublier, vous allez me dire si c'est faux:

Code:

---

int *ptab;

---

On déclare un pointeur qui pointe vers un entier (qui contient l'adresse d'une variable qui contient un entier.

Code:

---

ptab = (int *)malloc(10*sizeof(int));

---

celle ligne signifie que l'on allou un espace mémoire de 10 fois 1 entier au pointeur ptab. C'est à dire, que le pointeur va contenir l'adresse mémoire de 10 entiers. (?)

Citation:

---

t donc ici, comme le disait Mayhem, malloc renvoie un (void*).

---

comprends pas pk?

Dans malloc(10*sizeof(int)) => 10*sizeof(int) n'est pas un entier?

Voilà :)

Enfin plus précisement:

Citation:

---

qui contient l'adresse d'une variable qui contient un entier.

---

je dirais "qui contiendra" puisque tant que tu ne l'as pas affecté, il contient une valeur quelconque, et pointe vers n'importe quoi ;)

Citation:

---

C'est à dire, que le pointeur va contenir l'adresse mémoire de 10 entiers.

---

en fait il pointe vers le 1er des 10, et il n'y a aucun moyen de savoir combien d'entiers sont dans cette suite de nombres, à part garder ce "10" quelquepart sous le coude (un #define, une constante, une variable, ...)

Citation:

---

Dans malloc(10*sizeof(int)) => 10*sizeof(int) n'est pas un entier?

---

Si si, il prend en parametre un entier: la taille de ce qu'il doit allouer. Par contre, il ne sait pas ce que tu vas faire de cette mémoire, donc il te renvoie un (void*) et c'est à toi de le convertir vers le type qui t'intéresse ...

Je sais, tu précises sizeof(int) donc il pourrait deviner tout seul que tu vas caster ça en (int*), mais c'est comme ça :)

Citation:

---

Envoyé par bigquick

en fait il pointe vers le 1er des 10, et il n'y a aucun moyen de savoir combien d'entiers sont dans cette suite de nombres, à part garder ce "10" quelquepart sous le coude (un #define, une constante, une variable, ...)

---

donc le pointeur va contenir l'adresse mémoire du 1er des 10?

Prend également soin de tester le retour de la fonction malloc pour savoir si celle-ci a échoué.
Elle renvoie la valeur NULL si le bloc mémoire n'ai pu être alloué (mémoire insuffisante ou autre problème) sinon elle retourne l'adresse du début du bloc alloué.

Citation:

---

Envoyé par bigquick

Si si, il prend en parametre un entier: la taille de ce qu'il doit allouer. Par contre, il ne sait pas ce que tu vas faire de cette mémoire, donc il te renvoie un (void*) et c'est à toi de le convertir vers le type qui t'intéresse ...

Je sais, tu précises sizeof(int) donc il pourrait deviner tout seul que tu vas caster ça en (int*), mais c'est comme ça Smile

---

LooL !

Ok merci j'ai l'impression d'y voir un peu plus clair.

exactement !

et tu peux "passer" au suivant en faisant

Code:

---

pointeur = pointeur + 1

---

, pour que ton pointeur contienne l'adresse du suivant

ou encore y accéder avec la notation "tableau"

Code:

---

pointeur[i]

---

où "i" est le numéro de l'élément qui t'intéresse :)

Citation:

---

Envoyé par Mayhem

Prend également soin de tester le retour de la fonction malloc pour savoir si celle-ci a échoué.

---

D'accord, mais comment on teste une fonction? :roll:

Citation:

---

Envoyé par bigquick

exactement !

et tu peux "passer" au suivant en faisant

Code:

---

pointeur = pointeur + 1

---

, pour que ton pointeur contienne l'adresse du suivant

ou encore y accéder avec la notation "tableau"

Code:

---

pointeur[i]

---

où "i" est le numéro de l'élément qui t'intéresse :)

---

Ok!! j'ai compris ça alors! 8) C'est moi ou c'est dur au début pour tout le monde???

eh bien si elle te renvoie NULL tu sais qu'il y a une erreur

Code:

---

1 2 3 4 5 6

int * ptab; if( (ptab = (int *)malloc(sizeof(int)) ) == NULL) //problème d'allocation mémoire else //tout s'est bien passé, on continue...

---

Citation:

---

Prend également soin de tester le retour de la fonction malloc pour savoir si celle-ci a échoué.

---

En effet, très important si tu ne veux pas passer 15mn à chercher ailleurs alors que c'est juste le malloc qui a échoué :)

Tu peux tester, comme dans l'exemple que tu as posté, la valeur du pointeur. En cas d'echec, malloc retourne NULL, et donc donc pointeur contiendra NULL

Code:

---

1 2 3 4

if (p != NULL) { // tout va bien }

---

Et ouai, au début c'est dur, mais après t'y penses plus ;)

Merci à vous deux! :wink:

Je vais me lancer sur l'exo que le prof nous à demander de faire, et je penses m'en sortir, enfin j'éspère!

Allez A bientôt je penses :P

A bientôt, et bon courage :wink:

autre chose, chaque bloc alloué doit être restitué au système.
malloc va de paire avec free qui s'occupe de libérer la mémoire.
A la fin de ton application, il faut donc libérer ton tableau dynamique d'entier

Code:

---

1 2	free(ptab);

---

Citation:

---

Envoyé par Mayhem

Il me semble que certains compilateurs font le cast sur le retour de malloc implicitement en fonction du type de donnée auquel on les associe.

---

Non, le probleme est que Spike Spiegel compile en mode C++. Comme le typage est plus fort en C++, le cast implicite d'un pointeur void est interdit. Par contre, en C, c'est permis (il est même recommandé de ne pas caster le retour de malloc, qui cache l'erreur de la non-inclusion de stdlib).
Donc:
- s'assurer qu'on compile en C (extension du fichier .c et non .cpp)
- inclure stdlib.h

tout juste dazumba, je compile en C++ en fait, car l'extension du fichier est cpp. ^^


juste une question encore. Dans cette fonction je ne trouve pas pourquoi ca bloque... je n'ai aucune erreur mais juste après la saisie du premier nombre j'ai un message d'erreur du debuger...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

void saisir(int taille) { int i; int *ptab= (int *)malloc(taille * sizeof(int)); if(ptab == NULL) printf("\n\n ALLOCATION ECHOUEE \n\n"); else printf("\n Allocation reussie \n");   for(i=0;i<taille;i++) { printf("\nEntrez un entier: "); scanf("%d",*(ptab+i)); printf("%d",*(ptab+i)); } }

---

Citation:

---

Envoyé par Spike Spiegel

tout juste dazumba, je compile en C++ en fait, car l'extension du fichier est cpp.

---

Dans ce cas, le bon forum est C++... mais tu vas te faire critiquer car tu programmes "à la C".

Citation:

---

juste une question encore. Dans cette fonction je ne trouve pas pourquoi ca bloque... je n'ai aucune erreur mais juste après la saisie du premier nombre j'ai un message d'erreur du debuger...

---

Encore une confusion entre pointeur et valeur pointée. scanf() attend l'adresse d'une variable, et tu lui donnes l'adresse d'un pointeur.

Code:

---

scanf("%d", ptab+i);

---

ou

Code:

---

scanf("%d", &ptab[i]);

---

Eh bien, il s'en passe des choses pendant ma sieste!

Citation:

---

Envoyé par Spike Spiegel

Encore une question sur une fonction en C!!

On vient de voir les pointeurs en cours, et chez moi, sous visual studio c++, lorsque j'essaye de compiler :

Code:

---

1 2 3 4 5 6 7

void main() { int *ptab; *ptab = (int *)malloc(sizeof(int)); //printf("%d\n",*(ptab+1)); system("pause"); }

---

j'ai l'erreur suivante: error C2440: '=' : cannot convert from 'int *' to 'int'
This conversion requires a reinterpret_cast, a C-style cast or function-style cast

---

Tu n'invoques pas le bon compilateur. Assure toi que tu appelles bien le compilateur C :

• extension .c et non .C ni .cpp
• configuration de ton IDE

Néanmoins, ton code est incorrect et invoque de mauvaises pratiques :

• main() retourne int. Toujours.
  
  http://emmanuel-delahaye.developpez....s.htm#typemain
  
• manque le prototype pour malloc () (<stdlib.h>)
• '*ptab' est un int. De plus, ptab n'étant pas encore initialisé, le déréférencer invoque un comportement indéfini. Tu veux probablement dire 'ptab'.
• la valeur retournée n'est pas testée
• la mémoire allouée n'est pas libérée

http://emmanuel-delahaye.developpez....tes.htm#malloc

Ceci est correct :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

#include <stdlib.h>   int main (void) { int *ptab = malloc (sizeof *ptab);   if (ptab != NULL) { /* utilisation de ptab */ free (ptab), ptab = NULL; } return 0; }

---

Citation:

---

Envoyé par Spike Spiegel

Bon, comme je souhaiterai vraiment comprendre les pointeurs et rien oublier, vous allez me dire si c'est faux:

Code:

---

int *ptab;

---

On déclare un pointeur qui pointe vers un entier (qui contient l'adresse d'une variable qui contient un entier.

---

Oui.

Citation:

---

Code:

---

ptab = (int *)malloc(10*sizeof(int));

---

celle ligne signifie que l'on alloue un espace mémoire de 10 fois 1 entier

---

Oui (aux fautes de frappes près...)

Citation:

---

au pointeur ptab.

---

Non.

Citation:

---

C'est à dire, que le pointeur va contenir l'adresse mémoire de 10 entiers. (?)

---

Non. Je reformule (et j'utilise une meilleure présentation) :

Code:

---

int *ptab = malloc (10 * sizeof *ptab);

---

Cette ligne signifie que l'on alloue un bloc mémoire de la taille d'un tableau de 10 int et que l'adresse de ce bloc est stockée dans le pointeur ptab.

On peut ensuite utiliser le tableau alloué gràce au pointeur et à la notation indicée.

euh désole dazumba: je voulais dire que je programme en C mais que le non de fichier étant en cpp, je compile avec le mauvais compilateur ^^

Citation:

---

Encore une confusion entre pointeur et valeur pointée. scanf() attend l'adresse d'une variable, et tu lui donnes l'adresse d'un pointeur

---

si je lui donné l'adresse d'un pointeur ça serait plutôt &ptab plutôt que *ptab non?
lorsque l'on fait *ptab=2; on affecte pas à la première "case" du tableau l'entier 2. Si oui, je comprends tjrs pas pk le code ne marche pas.

Emmanuel: merci pour toutes ces précisions.

Citation:

---

Envoyé par bigquick

Par contre, il ne sait pas ce que tu vas faire de cette mémoire, donc il te renvoie un (void*) et c'est à toi de le convertir vers le type qui t'intéresse ...

Je sais, tu précises sizeof(int) donc il pourrait deviner tout seul que tu vas caster ça en (int*), mais c'est comme ça :)

---

Pas vraiment. Mais je rappelle que le cast n'est pas obligatoire en C, qui garantit la conversion implicite void* -> T* et T* -> void *.

Evidemment, si on se trompe de compilateur, c'est autre chose.... Pour éviter ça, dans mes codes source,

http://emmanuel-delahaye.developpez.com/clib.htm

il y a ça :

Code:

---

1 2 3 4

#ifdef __cplusplus #error This source file is not C++ but rather C. Please use a C-compiler #endif

---

Non c'est bon j'ai compris!! je viens de faire le rapport entre scanf() et son utilisation avec les pointeurs.

comme l'a dis Dazumba, scanf attend une adresse de variable, donc ptab+i ici.

C'était pourtant logique... :roll:

bon, j'ai encore un soucis.

Code:

---

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 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81

# ifdef __cplusplus #error This source file is not C++ but rather C. Please use a C-compiler #endif   #include<stdio.h> #include<stdlib.h> #include<malloc.h>   void saisir(int **t,int n) { *t = (int *)malloc(n * sizeof(int)); int *ptr; if(t != NULL) { for(ptr=*t;ptr<*t+n;ptr++) { printf("\nEntrez un entier: "); scanf("%d",ptr); } } else printf("\n\n ALLOCATION ECHOUEE \n\n"); system("pause"); }   void afficher(int *t,int n) { int *ptr; for(ptr=t;ptr<t+n;ptr++) { printf(" %d ",*ptr); if(ptr!=t+n-1) { printf("-"); } } printf("\n"); system("pause"); }   void main() { int choix, nb; int *ptab; do { system("cls"); printf("\n ___ Menu ___\n"); printf("\n 1 : Saisir le tableau"); printf("\n 2 : afficher"); printf("\n 3 : trier"); printf("\n 4 : recherche"); printf("\n 5 : rotation"); printf("\n 0 : occurence"); printf("\n Votre choix : "); scanf("%d",&choix);   switch(choix) { case 1: printf("Indiquez la taille du tableau: "); scanf("%d",&nb); saisir(&ptab,nb); break; case 2: printf("Le tableau que vous avez entrez est: "); afficher(ptab,nb); break; case 3: /*trier(&ptab,nb);*/ break; case 4: break; case 5: break; case 0: printf(" \n\n Fin du programme \n\n"); system("pause"); free(ptab); break; default: printf(" \n Erreur de frappe. Veuillez rechoisir: \n"); break; } }while(choix!=0);   }

---

Il y 5 minutes, j'ai essayé ce code, il fonctionnait.

J'ai tout fermé (workspace etc...) et tout réouvert et depuis, ça ne foncitonne plus.

error C2065: 'ptr' : undeclared identifier
et
error C2143: syntax error : missing ';' before 'type'

bon en fait ça ne fonctionne pas lorsque je le compile avec le compilateur C mais avec le C++ ça marche... Alors là je suis paumé. :cry:

Code:

---

1 2 3 4 5 6

void saisir(int **t,int n) { *t = (int *)malloc(n * sizeof(int)); int *ptr; if(t != NULL) {

---

Tu dois déclarer tes variables en début de bloc (sauf en C99 où ce n'est pas obligatoire)

Code:

---

1 2 3 4

{ int *ptr; *t = (int *)malloc(n * sizeof(int)); if(t != NULL)

---

Bonjour

je vais m'y mettre aussi :
Attention au prototype de main
En C c'est :

Code:

---

1 2 3

int main (void) ou int main(int argc, char *argv[])

---

et en principe à la fin du main on a return 0;

Autre chose, je n'ai pas vu de remarque au sujet de scanf, j'ai lu trop vite, ou on est fatigué sur le forum ??

merci les gens! la remarque qui concerne le scanf est dans la page deux je crois.

Sinon, je n'ai pas bien saisie l'utilisation du **t. je suppose que c'est pareil que *(*t) donc ce serait un pointeur d'un pointeur ?!

si quelqu'un peut m'éclairer sur ce sujet je suis preneur!!