Discussion :

[vincent.mbg]

Bonjour,

Je me demande si ma façon de procéder est correcte

J'ai un fichier couleur.h où je définie un nouveau type

Code :

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

1
2
 
typedef int Color[3] ;

puis j'inclue ce fichier dans le fichier couleur.c ci-dessous

Code :

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

1
2
3
4
5
6
 
#include "couleur.h"
 
const Color ROUGE = { 255, 0, 0 } ;
const Color NOIRE = { 0, 0, 0 } ;
const Color BLANC = { 255, 255, 255 } ;

puis j'inclue le fichier.c dans mon main.c par exemple

Code :

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

1
2
3
4
5
6
7
8
9
 
#include "color.c"
 
int 
main (int argc, char *argv[])
{
	printf( "r : %d, v : %d,  b : %d", NOIRE[0], NOIRE[1], NOIRE[2] ) ;
	return 0;
}

C'est le fait d'inclure un fichier .c que je trouve bizard mais je n'est pas trouvé d'autre solution

Qu'en pensé vous ?

[jabbounet]

C'est le fait d'inclure un fichier .c que je trouve bizard mais je n'est pas trouvé d'autre solution

Qu'en pensé vous ?

C'est effectivement une manière étrange de faire


pourquoi ne déclare tu pas tes constantes dans ton .h?

avec quelque chose du style:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#ifndef _COLOR_H_
#define _COLOR_H_
 
 
 
typedef int Color[3] ;
 
const Color ROUGE = { 255, 0, 0 } ;
const Color NOIRE = { 0, 0, 0 } ;
const Color BLANC = { 255, 255, 255 } ;
 
#endif

le #ifndef / #define sert à éviter les doubles inclusions de ton fichier.h

De plus si je ne me trompe pas des champs RVB vont de 0 à 255, donc tu peux les stocker sur des unsigned char.

[vincent.mbg]

pourquoi ne déclare tu pas tes constantes dans ton .h?

Je trouvai ça également bizard.
Un fichier .h n'est pas sensé contenir que des définitions ?

J'ai testais, ça marche bien
pour le makefile je fais juste ça ?

Code :

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

1
2
3
 
main : main.c color.h
	gcc -c main.c

Je fais pas de fichier .o avec mes couleurs ?

De plus si je ne me trompe pas des champs RVB vont de 0 à 255, donc tu peux les stocker sur des unsigned char.

bien vu !

Merci beaucoup pour tes réponses

[gl]

Effectivement, un fichier .h ne devrait contenir que des déclarations (donc pas de const Color ROUGE = { 255, 0, 0 } et on n'inclue pas de .c dans un autre .c.

La solution est de compiler les deux .c et de les lier ensemble.

[jabbounet]

Accéssoirement, tu peux aussi avoir un définition de Color plus complexe en utilisant un union avec quelque chose du style:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
typedef struct {
  unsigned red   :8;
  unsigned green :8;
  unsigned blue  :8;
} RgbColorCode ;
 
typedef unsigned char RgbColorArray[3];
 
typedef union {
  RgbColorCode  asRGB;
  RgBColorArray asArray;
} Color;

Cela te permettrai d'avoir des syntaxe plus sympas du genre:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
Color maCouleur;
 
maCouleur.asRgb.red   = 255;
maCouleur.asRgb.green = 0;
maCouleur.asRgb.blue  = 127;
 
printf("Couleur: asArray %c %c %c", 
		 maCouleur.asArray[0],
		 maCouleur.asArray[1],
		 maCouleur.asArray[2]);

Par contre comme je n'ai pas de compilateur sous la main, il est possible que je fasse quelques coquilles sur la syntaxe, mais l'esprit est là je pense.

De même si tu ne te sens pas encore à l'aise avec ce type de déclaration reste avec ton tableau et garde cette idée pour plus tard quand tu sera plus à l'aise avec le C.

http://publications.gbdirect.co.uk/c...bitfields.html
http://publications.gbdirect.co.uk/c...r6/unions.html

[vincent.mbg]

ok

Donc je reviens sur l'organisation du début sauf que dans main.c

J'ai mis

#include "couleur.h";

J'ai un makefile comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
all :: main
 
main : main.o color.o
	gcc main.o color.o -o main 
 
main.o : main.c
	gcc main.c -c
 
color.o : color.c color.h
	gcc color.c -c

mon fichier couleur.h

Code :

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

1
2
3
4
5
6
 
#ifndef _COLOR_H_
#define _COLOR_H_
typedef unsigned int Color[3] ;
Color NOIRE ; 
#endif

mon fichier couleur.c

Code :

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

1
2
3
4
5
6
 
#include "color.h"
 
NOIRE[0] = 0 ;
NOIRE[1] = 0 ;
NOIRE[2] = 0 ;

Mais ça marche pas

color.c:5: warning: data definition has no type or storage class
color.c:5: error: conflicting types for ‘NOIRE’
color.h:6: note: previous declaration of ‘NOIRE’ was here
color.c:5: error: invalid initializer
[...]

Et si je veux que NOIR existe dans main.c je suis bien obligé de le définir dans couleur.h
?

[ram-0000]

Ton point H définit la variable NOIRE, il suffit uniquement de la déclarer

Code :

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

1
2
3
4
5
#ifndef _COLOR_H_
#define _COLOR_H_
typedef unsigned int Color[3] ;
extern Color NOIRE ; 
#endif

[vincent.mbg]

C'est bon ça marche

Merci beaucoup.

Code :

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

1
2
3
4
5
6
// Color.h
extern const Color NOIRE ;

// Color.c
const Color NOIRE = { 255, 0, 0 } ;

Accéssoirement, tu peux aussi avoir un définition de Color plus complexe en utilisant un union avec quelque chose du style:

Sauf erreur de ma part, on peux utiliser que un seul membre d'un union à la fois. Si j'utilise un union, Je ne pourrais utiliser que asRGB ou asArray pour un type Color donné.

Ceci ne marchera pas.

Code :

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

1
2
3
 
maCouleur.asRgb.red   = 255;
printf( "red : %c", maCouleur.asArray[0] ) ;

[diogene]

Sauf erreur de ma part, on peux utiliser que un seul membre d'un union à la fois. Si j'utilise un union, Je ne pourrais utiliser que asRGB ou asArray pour un type Color donné.

Exact, la proposition de jabbounet est non portable (et inutile, Color est déja un tableau qu'il serait suffisant de déclarer comme tableau de unsigned char)

[jabbounet]

ah oui, j'avais oublié ces détails sur les unions. merci