Discussion :

[parano]

Bonjour,

dans mon programme, j'ai l'erreur suivante :

Code :

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

1
2
raytrace.c: In function ‘makeCylinder’:
raytrace.c:49: erreur: syntax error before ‘)’ token

La fonction en question :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
cylinder* makeCylinder(GLfloat x, GLfloat y, GLfloat z, GLfloat r) {
  cylinder* cylinder;
  /* allocate memory */
  cylinder = (cylinder*) malloc(sizeof(cylinder));
 
  /* put stuff in it */
  cylinder->c = makePoint(x,y,z,1.0);   /* center */
  cylinder->r = r;   /* radius */
  cylinder->m = NULL;   /* material */
  return(cylinder);
}

Je ne vois pas très bien ou se situe l'erreur.
Quelqu'un pourrait-il m'indiquer ou se situe l'erreur.

Merci

[gl]

Bonjour,

dans mon programme, j'ai l'erreur suivante :

Code :

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

1
2
raytrace.c: In function ‘makeCylinder’:
raytrace.c:49: erreur: syntax error before ‘)’ token

Quelle est la ligne 49 ?

Sinon, les definitions de cylinder et GLfloat sont-elles bien visibles ? Tout les headers sont-ils bien inclus ?

[TabrisLeFol]

Dans un premier temps, je confirme le fait, implicitement dit, que ce genre d'erreur peut etre la consequence d'un erreur silencieuse, dans la definition par exemple.

Sinon juste a titre de curiosité,

cylinder* cylinder;

, ton nom de variable est le meme que ton type ?

[parano]

Dans un premier temps, je confirme le fait, implicitement dit, que ce genre d'erreur peut etre la consequence d'un erreur silencieuse, dans la definition par exemple.

Sinon juste a titre de curiosité,
, ton nom de variable est le meme que ton type ?

Le probleme venait de là.
Mais pourquoi ?

[TabrisLeFol]

Le compilateur definit variables et types par leur nom, et ne se sert pas du contexte pour les differencier, mais meme avec le contexte, ton malloc est ambigu.

Moi j'aurais fait

Code :

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

malloc(sizeof(*cylinder));

Pour dire le type pointé par le pointeur.

Mais on pourrait egalement reserver de la taille, seulement pour le pointeur... (type = void*)

Si jamais tu ne sais pas comment l'appeler, tu peux toujours prefixé ta variable par p pour expliciter que c'est un pointeur.

EDIT : Avec struct cylinder cylinder, par definition ce qui suit le struct est le nom d'un structure, il n'y a donc dans ce cas pas de probleme.

[parano]

Merci pour les précisions

[Emmanuel Delahaye]

La fonction en question :

En dehors de ce qui a déjà été signalé :

• un cast inutile
• il manque le test de la valeur retournée par malloc()

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
cylinder* makeCylinder(GLfloat x, GLfloat y, GLfloat z, GLfloat r) 
{
   cylinder* p_cylinder = malloc (sizeof *p_cylinder);
   if (p_cylinder != NULL)
   {
       /* put stuff in it */
       p_cylinder->c = makePoint (x, y, z, 1);   /* center */
       p_cylinder->r = r;   /* radius */
       p_cylinder->m = NULL;   /* material */
   }
    return cylinder;
}

L'utilisateur doit stocker l'adresse retournée dans un pointeur, tester sa validité et libérer le bloc pointé après usage. ('destructeur' recommandé)