Discussion :

[petdelascar]

Quelqu'un pourait il m'expliquer le pb suivant?

J'ai 2 pgs typiquement identiques à la variance près que dans l'un je déclare certaines variables extern dans mon .h comme suit:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
extern double son[NL];
extern double son_prec[NC];
 
double syn_frame[NL];
extern double syn_frame_prec[NC];
 
double syn[NL];
 
extern double syn_sin[NL];
double syn_sin1[NC];
double syn_sin2[NC];

Alors que dans mon second programme, je ne les déclare en extern que dans le .c situé tout en haut du code comme suit:

.h:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
double son[NL];
double son_prec[NC];
 
double syn_frame[NL];
double syn_frame_prec[NC];
 
double syn[NL];
 
double syn_sin[NL];
double syn_sin1[NC];
double syn_sin2[NC];

.c:

Code :

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

1
2
 
extern double son[NL], son_prec[NC], syn_frame_prec[NC], syn_sin[NL];

Le problème étant que seule la première version fonctionne correctement.
Pourquoi, comment y remédier pour faire fonctionner correctement mon second sans déclarer les variables en extern dans mon .h ?

Merci

[Skyounet]

A mon avis c'est parce que tu inclus ton fichier .h plusieurs fois donc tu a des redefinitions de tes variables globales.

Donc pour remedier à ca tu vires toutes les variables globales dans le .h tu les declare une et une seule fois de le fichier qui contient le main par exemple, et dans les autres .c qui ont besoin des ces variables globales tu met un extern au debut du fichier.

[Emmanuel Delahaye]

J'ai 2 pgs typiquement identiques à la variance près que dans l'un je déclare certaines variables extern dans mon .h comme suit:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
extern double son[NL];
extern double son_prec[NC];
 
double syn_frame[NL];
extern double syn_frame_prec[NC];
 
double syn[NL];
 
extern double syn_sin[NL];
double syn_sin1[NC];
double syn_sin2[NC];

Horribles erreurs de conception. On ne définit pas de variables dans un .h.

http://emmanuel-delahaye.developpez....s.htm#globales

Revoit ta conception selon les regles d'usage et tu verras que tes problèmes se corrigeront tout seuls...

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

[petdelascar]

J'ai fait comme me préconise skyrunner mais ça ne marche toujours pas.
Je ne comprend vraiment pas pourquoi...

[petdelascar]

Horribles erreurs de conception. On ne définit pas de variables dans un .h.

Je sais bien, mais pourquoi mon programme fonctionne t il bien dans ce cas?

[Thierry Chappuis]

Code :

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

Je sais bien, mais pourquoi mon programme fonctionne t il bien dans ce cas?

Le monde est plein d'exemples de code mal conçus qui fonctionnent. Ce n'est pas une pratique recommandable pour autant (maintenance difficile, debuggage difficile, extensibilité et réutilisabilité du code quasi nulle, etc.).

Meilleures salutations

Thierry

[petdelascar]

ça me permettrait de faire fonctionner mon code si je trouvais

[loufoque]

Horribles erreurs de conception. On ne définit pas de variables dans un .h.

Si, ça se fait un peu quand même.
Je sais que dans la partie unicode de glib par exemple, y'a plein de tables définies dans des .h.

[Emmanuel Delahaye]

Horribles erreurs de conception. On ne définit pas de variables dans un .h.

Si, ça se fait un peu quand même.
Je sais que dans la partie unicode de glib par exemple, y'a plein de tables définies dans des .h.

• Ca reste à prouver
  Si c'est avéré
  [list:8ed47458f3]
• Tout le monde fait des erreurs
• Ca reste quand même une erreur de conception.

Fin si[/list:u:8ed47458f3]

[ToTo13]

Bonjour,

on peut définir des variables dans un *.h, mais on va rapidement etre confronté à de tres nombreux problèmes (j'en ai fait l'expérience).
La meilleure solution est de déclarer TOUTES les variables dans les *.c et d'utiliser "extern" dans les fichiers où elles sont utilisées.

[gege2061]

Horribles erreurs de conception. On ne définit pas de variables dans un .h.

Si, ça se fait un peu quand même.
Je sais que dans la partie unicode de glib par exemple, y'a plein de tables définies dans des .h.

Dans gunicode.h ? La seule ligne qui aurait pu déclarer une variable est :

Code :

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

1
2
3
/* Array of skip-bytes-per-initial character.
 */
GLIB_VAR const gchar * const g_utf8_skip;

Cependant :

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
/* We prefix variable declarations so they can
 * properly get exported in windows dlls.
 */
#ifndef GLIB_VAR
#  ifdef G_PLATFORM_WIN32
#    ifdef GLIB_STATIC_COMPILATION
#      define GLIB_VAR extern
#    else /* !GLIB_STATIC_COMPILATION */
#      ifdef GLIB_COMPILATION
#        ifdef DLL_EXPORT
#          define GLIB_VAR __declspec(dllexport)
#        else /* !DLL_EXPORT */
#          define GLIB_VAR extern
#        endif /* !DLL_EXPORT */
#      else /* !GLIB_COMPILATION */
#        define GLIB_VAR extern __declspec(dllimport)
#      endif /* !GLIB_COMPILATION */
#    endif /* !GLIB_STATIC_COMPILATION */
#  else /* !G_PLATFORM_WIN32 */
#    define GLIB_VAR extern
#  endif /* !G_PLATFORM_WIN32 */
#endif /* GLIB_VAR */
 
#endif /* __G_TYPES_H__ */

Donc il s'agit bien d'une variable extern.

La glib a la réputation d'être bien écrite, ce n'est pas pour rien.

[Thierry Chappuis]

Si, ça se fait un peu quand même.
Je sais que dans la partie unicode de glib par exemple, y'a plein de tables définies dans des .h.

Comme je l'ai dis, le monde est rempli de lignes de code douteuses. C'est une raison de plus pour se former (et se discipliner) à écrire du code propre et bien conçu.

ça me permettrait de faire fonctionner mon code si je trouvais

On appelle ça de la "bidouille"! C'est difficile à maintenir, pas portable et en tous les cas ce n'est pas une PRATIQUE RECOMMENDABLE.

Salutations

Thierry