typedef, pointeurs de int et assignment makes pointer ...

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : typedef, pointeurs de int et assignment makes pointer ...

xav14

C'est sûrement très c*n mais j'ai ce petit bout de code :

Code :

typedef int *CensusCode ;
CensusCode newCensusCode() {
CensusCode cc;
if ((cc=malloc(sizeof(int)*censusSize)==NULL)) {
printf("Census : newCensusCode : memory allocation failure\n" );
return NULL;
}
}

qui me renvoie cette erreur :

census.c:17: warning: assignment makes pointer from integer without a cast

ca vient de quoi ?

Publicité

Elmoricq

Modérateur

A priori, je dirais que tu as oublié #include <stdlib.h> et le compilateur pense que malloc() retourne un int.

Ou alors c'est autre chose. Tu compiles avec quoi au fait ?

xav14

gcc 3.4.2. sous mingw

stdlib est bien incluse

je teste sur un autre gcc au cas ou [:spamafote]

xav14

et je compile avec ca :

gcc -c census.c -Wall -Werror -ansi -pedantic

Elmoricq

Modérateur

xav14 a écrit :

je teste sur un autre gcc au cas ou [:spamafote]

Pas la peine.

Sinon, censusSize c'est défini comment ?

Et essaie en remplaçant :

if ((cc=malloc(sizeof(int)*censusSize)==NULL))

par

if ((cc=malloc(sizeof(int)*censusSize))==NULL)

EDIT :
Tu peux même faire plus propre avec :

cc = malloc(censusSize * sizeof *cc);
if ( cc == NULL ) {
...

Message édité par Elmoricq le 23-08-2005 à 16:40:05

xav14

ha ben forcément [:wam]

Message édité par xav14 le 23-08-2005 à 16:40:07

olivthill

D'habitude, on met la taille de l'autre côté

if ((cc=malloc(censusSize*sizeof(int))==NULL)) {

et le problème devait disparaitre.

L'explication est un peu subtile. C'est une question de priorité des opérateurs.
Le compilateur croit que *censusSize est un pointeur, au lieu d'être une multiplication. Donc, il faut mettre des parenthèses, ou peut-être simplement un expace après l'étoile pour éviter la confusion.

xav14

non mais c'est juste que j'avais pas vu que je fermais pas les parenthèses au bon endroit
loin de moi l'idée de vouloir écrire x=y==z

matafan

olivthill a écrit :

D'habitude, on met la taille de l'autre côté

if ((cc=malloc(censusSize*sizeof(int))==NULL)) {

et le problème devait disparaitre.

L'explication est un peu subtile.

Elle est surtout fausse.

olivthill

Moi-aussi, j'ai ps vu le plcement incorrect de la parenthèse fermante, mais sinon il me semble que mon explication est correcte. C'est facile de dire c'est faux sans dire pourquoi. Pourrais-tu développer pour que j'apprenne quelque chose ? En changeant de côté, le compilateur ne va pas considérer que "*sizeof(int)" est un pointeur car il sait que "sizeof(int)" n'est pas un nom de variable, et donc il n'y a pas de confusion possible pour lui.

Publicité

Elmoricq

Modérateur

olivthill a écrit :

Sauf que la précédence ne s'applique par définition qu'entre plusieurs opérateurs.
Là il n'y en a qu'un seul, et qui plus est binaire. Pas de confusion possible.

La précédence s'applique plutôt dans des cas comme celui-ci, par exemple :

machin * *bidule

(*bidule sera évalué avant la multiplication)

Emmanuel Delahaye

C is a sharp tool

xav14 a écrit :

Code :

if ((cc=malloc(sizeof(int)*censusSize)==NULL)) {

qui me renvoie cette erreur :

census.c:17: warning: assignment makes pointer from integer without a cast

Parenthèses mal placées :

if ((cc=malloc(sizeof(int)*censusSize))==NULL) {

Pour éviter ça :

#include <stdlib.h>
#include <stdio.h>

typedef int *CensusCode;

CensusCode newCensusCode (size_t censusSize)
{
CensusCode cc = malloc (sizeof (int) * censusSize);

if (cc == NULL)
{
printf ("Census : newCensusCode : memory allocation failure\n" );
}

return cc;
}

int main (void)
{
CensusCode c = newCensusCode (10);
if (c != NULL)
{
free (c), c = NULL;
}

return 0;
}

Je déconseille formellement l'usage du type 'pointeur' et l'usage qui en est fait. En effet, si la définition change, il faut aussi modifier le malloc(), ce qui est une absurdité. Le bon sens commande de faire ceci :

#include <stdlib.h>
#include <stdio.h>

typedef int CensusCode;

CensusCode *newCensusCode (size_t censusSize)
{
CensusCode *cc = malloc (sizeof *cc * censusSize);

if (cc == NULL)
{
printf ("Census : newCensusCode : memory allocation failure\n" );
}

return cc;
}

int main (void)
{
CensusCode *c = newCensusCode (10);

if (c != NULL)
{
free (c), c = NULL;
}

return 0;
}

http://mapage.noos.fr/emdel/notes.htm#malloc

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

xav14

:jap: pour le mini-cours

FORUM HardWare.fr

Programmation

typedef, pointeurs de int et assignment makes pointer ...

Sujets relatifs
[MSSQL] Pointer vers une autre BDD	Pointer sur un exe avec flash (supérieur au 5)
les pointeurs , les tableaux ??? je mis perd	Bug sur une affectation "Java null pointer exeption"
pb avec typedef	les pointeurs - petite aide !
A propos des pointeurs.[Débutant]	Toujours constructeur, pointeurs, agrégation
java et pointeurs sur fonction	[c++] classe et typedef
Plus de sujets relatifs à : typedef, pointeurs de int et assignment makes pointer ...

Page générée en 0.051 secondes