[C::B-Linux] Erreur bizzare

**Goundy** · 03/04/2006, 13h53

Bonjour,

Un truc qui me gonfle, j'ai une erreur trés zarb regardez:
fonctions.c

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
#include "module.h"
 
void Sort_Table (CHAR tab [MAX_CHAR]){
   int  count1,
        count2;
   CHAR tmp;
 
     for (count1 = 0; count1 < MAX_CHAR; count1++)
       for (count2 = 0; count2 < (MAX_CHAR-1); count2++)
         if (tab[count2].occur > tab[(count2+1)].occur){
            tmp = tab[count2];
            tab[count2] = tab[(count2+1)];
            tab[(count2+1)] = tmp;
             }
    }

le type CHAR:

Code :

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

1
2
3
4
5
typedef struct{
 
   unsigned char   car;
   int           occur;
    }CHAR;

dans fonctions.h:

Code :

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

void Sort_Table (CHAR tab[MAX_CHAR]);

et le compilo me signal une erreur dans foncions.h

erreur: expected ')' before 'tab'

???
Merci

**niglo** · 03/04/2006, 13h59

Envoyé par Goundy

et le compilo me signal une erreur dans foncions.h

erreur: expected ')' before 'tab'

Effectivement, d'où peut provenir cette erreur ???
Encore un grand mystère de l'informatique.

Tu es sur que c'est sur cette partie de code ?
Le compilo ne donne pas de numéro de ligne ?

**Goundy** · 03/04/2006, 14h04

Yes, ligne 1 dans le fichier fonctions.h

**Emmanuel Delahaye** · 03/04/2006, 14h07

Envoyé par Goundy

le type CHAR:

Code :

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

1
2
3
4
5
typedef struct{
 
   unsigned char   car;
   int           occur;
    }CHAR;

dans fonctions.h:

Code :

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

void Sort_Table (CHAR tab[MAX_CHAR]);

et le compilo me signal une erreur dans foncions.h

erreur: expected ')' before 'tab'

Est-ce que CHAR est bien défini avant usage ?

**Goundy** · 03/04/2006, 14h11

Envoyé par Goundy

le type CHAR:

Code :

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

1
2
3
4
5
typedef struct{
 
   unsigned char   car;
   int           occur;
    }CHAR;

dans module.h

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
#ifndef H_MODULE
#define H_MODULE
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include "fonctions.h"
 
#define DONE          0
#define MAX_CHAR     255
 
typedef struct {
 
   unsigned char   car;
   int           occur;
    }CHAR;
 
#endif

**fearyourself** · 03/04/2006, 14h15

Envoyé par Emmanuel Delahaye

Est-ce que CHAR est bien défini avant usage ?

Je ne vois pas comment:

Question:

Envoyé par niglo

Effectivement, d'où peut provenir cette erreur ???
Encore un grand mystère de l'informatique.

Réponse:

Envoyé par Goundy

Yes, ligne 1 dans le fichier fonctions.h

**Goundy** · 03/04/2006, 14h17

NON!
Question:

Envoyé par niglo

Tu es sur que c'est sur cette partie de code ?
Le compilo ne donne pas de numéro de ligne ?

Réponse

Envoyé par Goundy

Yes, ligne 1 dans le fichier fonctions.h

**niglo** · 03/04/2006, 14h18

Attention, tu inclus fonctions.h dans module.h, alors que fonctions.h a besoin de connaitre la définition de CHAR !! Faudrait pas plutôt déclaré CHAR directement dans fonctions.h ???

Ca ne pourrait pas venir de la ?

**Emmanuel Delahaye** · 03/04/2006, 14h28

Envoyé par Goundy

le type CHAR:

Code :

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

1
2
3
4
5
typedef struct{
 
   unsigned char   car;
   int           occur;
    }CHAR;

dans module.h

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
#ifndef H_MODULE
#define H_MODULE
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include "fonctions.h"
 
#define DONE          0
#define MAX_CHAR     255
 
typedef struct {
 
   unsigned char   car;
   int           occur;
    }CHAR;
 
#endif

Donc, il est défini après ! Il faut cesser de bricoler et commencer à travailler avec méthode.

La regle de conception est simple :

Un header doit être 'autonome'. Tout ce qu'il utilise doit avoir été défini avant, soit par définition directe, soit par inclusion. Ni plus ni moins.

Ici, je conseille :

Code :

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

1
2
3
4
5
6
#ifndef H_MODULE
#define H_MODULE
 
#define DONE  0
 
#endif

Ce header séparé est-il vraiment utile ?

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
 
#ifndef H_FONCTIONS
#define H_FONCTIONS
 
#define MAX_CHAR     255
 
typedef struct 
{
   unsigned char   car;
   int occur;
}
CHAR;
 
void Sort_Table (CHAR tab[MAX_CHAR]);
#endif /* guard */

J'ai quand même passé du temps à expliquer tout ça sur mon site, c'est un peu dommage de ne pas suivre les conseils qui y sont données...

http://emmanuel-delahaye.developpez....ganiser_source
http://emmanuel-delahaye.developpez.....htm#organiser

et ne me dit pas que c'est une découverrte, depuis le temps que tu postes ici...

Je rappelle que les headers ne sont pas là pour 'faciliter le codage' en incluant des choses qu'on veut cacher, mais pour fournir l'exact interface nécessaire à l'utilisation des fonctions et objets publics.

Ici, on a

Code :

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

void Sort_Table (CHAR tab[MAX_CHAR]);

donc, on doit avoir la définition de CHAR et celle de MAX_CHAR.

En général, se sont précisément ces définitions communes qui font que des fonctions vont naturellement se regrouper dans des headers et modules spécialisés. Il y a tout simplement un 'lien organique' entre elles, qui est le type. Les fonctions vont tout simplememnt manipuler des objets de type CHAR. Il serait donc naturel que le header (et le fichier d'implémentation correspondant) s'appelle char.h / char.c.

Pour une approche sytématique :

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

**fearyourself** · 03/04/2006, 14h28

Envoyé par Goundy

NON!
Question:

Envoyé par niglo

Tu es sur que c'est sur cette partie de code ?
Le compilo ne donne pas de numéro de ligne ?

Réponse

Envoyé par Goundy

Yes, ligne 1 dans le fichier fonctions.h

C'est guère mieux, finalement ton fichier fonctions.h ressemble à ceci:

Code :

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

1
2
 
void Sort_Table (CHAR tab[MAX_CHAR]);

Comment veux-tu que dans le fichier fonctions.h, le compilateur connaisse le type CHAR?

Comme Emmanuel a dit dans un autre post:
(ici)

Un header doit être 'autonome'. Il doit contenir/inclure tout ce qu'il lui est nécessaire. Ni plus, ni moins.

Donc il faut changer ta façon de faire tes .h...

Jc

**DaZumba** · 03/04/2006, 14h29

Il faut faire attention aux header.
Tu as un module.h qui, apparemment, definis le type CHAR (module.h, c'est pas super transparent comme non).
Donc

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#ifndef H_MODULE
#define H_MODULE
 
#define DONE                  0
#define MAX_CHAR     255
 
typedef struct 
{
   unsigned char   car;
   int           occur;
} CHAR; 
#endif

Tu inclues cet header dans tous les sources et headers qui ont besoin du type CHAR.
Notamment dans fonctions.h (ou functions.h). module.h est un 'header de base'. Il n'a rien a inclure.

**niglo** · 03/04/2006, 14h35

Envoyé par niglo

Attention, tu inclus fonctions.h dans module.h, alors que fonctions.h a besoin de connaitre la définition de CHAR !! Faudrait pas plutôt déclaré CHAR directement dans fonctions.h ???

Ca ne pourrait pas venir de la ?

D'après ce que dis tout le monde (en expliquant bien mieux que moi

), l'erreur vient effectivement de là. Désolé de ne pas expliquer aussi bien que les autres, je ne suis pas encore un gourou du C ni de dvp