Discussion :

[salseropom]

Bonjour, je définis un type comme suit :

fichier main.h :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#include <pthread.h> /* pthread functions and data structures */
#include "vector.h"
#include "nvector_serial.h"
#include "engine.h"
#include "mechanism.h"
#include "task.h"
 
typedef void (*pfunOutputs)(double,N_Vector *,unsigned int,Engine *,Mechanism *,
char *NameFileOutput[],Vector *,pthread_t *,WriteHardDisk *);

et j'obtiens l'erreur suivante :

Code :

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

1
2
3
 
../Headers/Myheaders/main.h:34: error: erreur de syntaxe before "WriteHardDisk"
../Headers/Myheaders/main.h:34: attention : function declaration isn't a prototype

Dans mon main j'ai les choses suivantes :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
  pthread_t * task=malloc(N*sizeof(*task));
  if(task==NULL)
  {
    MEMERROR;
    exit(EXIT_FAILURE);
  }
 
  WriteHardDisk * whd=malloc(N*sizeof(*whd));
  if(whd==NULL)
  {
    MEMERROR;
    exit(EXIT_FAILURE);
  }

D'où pourait provenir cette erreur ?
Merci

[Médinoc]

Tu es sûr de ton coup, pour "Vector" avec une majuscule, alors que tu as N_Vector à coté?

[salseropom]

j'ai trouvé mon erreur : il suffisait que je mette le typedef dans mon fichier main.c et là ça marche... ceci dit, j'aimerais bien savoir pourquoi (j'avais bien fait #include "main.h" dans mon main.c)

[Médinoc]

Est-ce que tu incluais autre chose que main.h dans ton main.c ?

[salseropom]

oui bien sûr

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
23
24
25
26
 
#include<stdio.h>
#include<stdlib.h>
#include<math.h>
#include<string.h>
#include<errno.h>
#include<time.h>
#include <pthread.h> /* pthread functions and data structures */
 
#include "vector.h"
#include "matrix.h"
#include "functions.h"
#include "userdata.h"
#include "species.h"
#include "engine.h"
#include "mechanism.h"
#include "temporary.h"
#include "inidata.h"
#include "mystring.h"
#include "solver.h"
#include "randompulling.h"
#include "randinicond.h"
#include "main.h"
#include "inimass.h"
#include "task.h"
#include "distribution.h"

[Médinoc]

Code :

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

1
2
3
#include "inimass.h"
#include "task.h"
#include "distribution.h"

Un ou plusieurs de ces trois fichiers d'en-tête contient les déclarations dont ton typedef a besoin, ou bien inclut le(s) fichier(s) d'en-tête les contenant...

[salseropom]

bien vu !
Merci.

[Emmanuel Delahaye]

bien vu !
Merci.

règles d'or des fichiers d'en-tête :

• Un fichier d'en-tête doit être protégé contre les inclusions multiples.
• Un fichier d'en-tête doit être autonome. Pour s'en assurer, lors des tests de l'implémentation correspondante, le placer en 1er include et ajouter ce qui manque dedans.
• Un fichier d'en-tête ne doit n'inclure que le strict nécessaire à son autonomie.

main.h est inutile. Il apporte une fausse facilité, une abstraction inutile (il n'y a pas d'objet 'main' et le module main.c étant de plus haut niveau, i ne peut rien exporter).

Il n'est pas inutile de voir la liste des fichiers d'en-tête, ça permet une vérification visuelle assez rassurante...

Le code doit être 'beau'...