Variable globale initialisée aléatoirement

Version imprimable

10/02/2008, 01h10
Leole

Variable globale initialisée aléatoirement

Bonsoir , j'aimerai connaître l'astuce pour éviter d'avoir cette erreur :
" error: initializer element is not constant " lorsque je déclare une variable globale que j'initialise grâce au retour d'une fonction random.

merci :)

[edit]

exemple:

int n = rand();
int tab[n];

int main()
{
...
}
10/02/2008, 03h05
smpinf
Citation:

Envoyé par Leole

int n = rand();
int tab[n];

int main()
{
...
}

suivant ta conception:
Code:

1 2 3 4 5 6 7 8 #include <stdio.h> #include <stdlib.h> #define n rand() int main() { int tab[n]; ...... }
sinon voici un lien qui va surement te servir: http://c.developpez.com/faq/?page=no..._random_bornes;)
10/02/2008, 08h41
nicolas.sitbon
Citation:
Envoyé par smpinf

suivant ta conception:

Code:

1 2 3 4 5 6 7 8 #include <stdio.h> #include <stdlib.h> #define n rand() int main() { int tab[n]; ...... }
Attention l'utilisation des VLAs nécessite un compilateur C99 qui les supporte ce qui n'est ni le cas pour GCC ni pour VC; à ma connaissance, pour windows il n' a que Comeau qui supporte les VLAs et sous unix, il y a Sunstudio 12, j'imagine qu'il y en d'autre mais je ne les connaît pas.
Cordialement.
10/02/2008, 10h35
Sve@r
Citation:

Envoyé par Leole

Bonsoir , j'aimerai connaître l'astuce pour éviter d'avoir cette erreur :
" error: initializer element is not constant " lorsque je déclare une variable globale que j'initialise grâce au retour d'une fonction random.

merci :)

[edit]

exemple:

int n = rand();
int tab[n];

int main()
{
...
}

Ben l'astuce c'est de ne pas tenter cette hérésie que de définir un tableau d'éléments dont on ne connait pas le nombre d'éléments à l'avance. Ce n'est pas dans la philosophie des tableaux...
Code:

1 2 3 4 5 6 7 8 9 int n = rand(); int *tab; int main() { tab=malloc(n * sizeof(int)); ... free(tab); }
PS: Les globales c'est pas bien
10/02/2008, 12h30
smpinf
Citation:
Envoyé par Sve@r

Ben l'astuce c'est de ne pas tenter cette hérésie que de définir un tableau d'éléments dont on ne connaît pas le nombre d'éléments à l'avance. Ce n'est pas dans la philosophie des tableaux...

Code:

1 2 3 4 5 6 7 8 9 int n = rand(); int *tab; int main() { tab=malloc(n * sizeof(int)); ... free(tab); }

PS: Les globales c'est pas bien
bonne solution!!
sauf que pour le "int n=rand()" je crois que ca va lui générer la même faute de type " error: initializer element is not constant " donc il vaut mieux la déclarer dans le main().
smpinf
10/02/2008, 13h42
Leole
Bonjour ! Merci pour vos réponses :)
J'ai changé un poil ma conception , Je déclare toujours ma variable globale mais j'ai abandonné l'idée du tableau.

Ainsi j'ai 2 problèmes depuis ce matin apparement de "portée":

exemple:

ex1.c
Code:

1 2 3 4 5 6 7 int x; int main() { x = rand(); ... }
ex2.c
Code:

1 2 3 4 5 6 7 8 9 10 11 extern int x; struct a { int tab[x]; // error: variably modified ‘tab’ at file scope }; int main() { //utilisation de la variable X // error: undefined reference to `x' }
Quelqu'un pourrait m'apporter plus de précisions sur ces erreurs , sur la portée etc , merci :)

Citation:
Envoyé par Leole
ex1.c
Code:

1 2 3 4 5 6 7 int x; int main() { x = rand(); ... }
Pour ce code je ne vois pas de problème!!
ex2.c
Code:

1 2 3 4 5 6 7 8 9 10 extern int x; struct a { int tab[x]; //c'est quoi la valeur de x???!! }; int main() { .... }

10/02/2008, 14h10
Médinoc

Si on ne compte pas les VLA (Variable-Length Array) du C99, la taille d'un tableau doit forcément être une constante connue à la compilation.
C'est également le cas pour les différents case d'un switch.

Citation:

Envoyé par Leole

Bonjour ! Merci pour vos réponses :)
J'ai changé un poil ma conception , Je déclare toujours ma variable globale mais j'ai abandonné l'idée du tableau.

Ainsi j'ai 2 problèmes depuis ce matin apparement de "portée":

exemple:

ex1.c

Code:

1
2
3
4
5
6
7
 
int x;
int main()
{
    x = rand();
    ...
}

ex2.c

Code:

1
2
3
4
5
6
7
8
9
10
11
 
extern int x;
struct a
{
   int tab[x];   // error: variably modified ‘tab’ at file scope
};
int main()
{
   //utilisation de la variable X
  // error: undefined reference to `x'
}

Quelqu'un pourrait m'apporter plus de précisions sur ces erreurs , sur la portée etc , merci :)

On te dit qu'il ne faut pas utiliser les VLA, surtout qu'il y a des règles d'usages assez précises que visiblement tu ne connais pas. Tu peux utliser un poihnteur et un mécanisle de 'constructeur' qui va créer le tableau à une taille pseudo aléatoire. Il faut penser à sauvegarder la taille...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
#ifndef H_ED_TAB_20080210141731
#define H_ED_TAB_20080210141731
 
/* tab.h */
 
#include <stddef.h>
 
struct tab
{
   int *a;
   size_t n;
};
 
struct tab *tab_create (int max);
void tab_delete (struct tab *self);
 
#endif                          /* guard */
 
/* Guards added by GUARD (c) ED 2000-2005 Jan 17 2005 Ver. 1.7 */

avec

Code:

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
27
 
/* tab.c */
#include "tab.h"
 
/* http://delahaye.emmanuel.free.fr/clib */
#include "ed/inc/random.h"
 
struct tab *tab_create (int max)
{
   struct tab*self = malloc (sizeof *self);
   if (self != NULL)
   {
      self->n = 1 + random (max-1); /* 1 a max-1 */
      self->a = malloc (sizeof self->a * self->n);
   }
   return self;
}
 
void tab_delete (struct tab *self)
{
   if (self != NULL)
   {
      free (self->a), self->a = NULL;
      self->n = 0;
      free (self), self = NULL;
   }
}

et le test :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
/* http://delahaye.emmanuel.free.fr/clib */
#include "ed/inc/random.h"
#include "ed/inc/prt.h"
 
#include <stdio.h>
 
#include "tab.h"
 
int main (void)
{
   randomize ();
 
   {
      struct tab *t = tab_create (100);
      if (t != NULL)
      {
         PRT_I (t->n);
 
         tab_delete (t), t = NULL;
      }
   }
   return 0;
}

10/02/2008, 14h32
Leole

Merci pour ces précisions , j'ai finalement opté pour une solution plus simple pour le tableau , je lui ai mis une taille fixe :)
Cependant j'ai toujours l'erreur suivante pour la variable globale :
"undefined reference to 'x' ", dois-je rajouter une dépendance vers le fichier qui a initialisé x dans le makefile lorsque je compile mon 2e fichier qui utilise cette variable aussi ?

merci d'avance
10/02/2008, 14h36
Emmanuel Delahaye

Citation:

Envoyé par Leole

Cependant j'ai toujours l'erreur suivante pour la variable globale :
"undefined reference to 'x' ", dois-je rajouter une dépendance vers le fichier qui a initialisé x dans le makefile lorsque je compile mon 2e fichier qui utilise cette variable aussi ?

Si tu tiens vraiment à utiliser une globale (déconseillé), fait le selon les règles de l'art :

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

Il faut un .h

Citation:

Envoyé par Emmanuel Delahaye

Si tu tiens vraiment à utiliser une globale (déconseillé), fait le selon les règles de l'art :

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

Il faut un .h

Oui j'avais lu ça il y a qq temps déjà mais je ne pensais pas cela utile pour un si petit projet , j'avais essayé mais une erreur similaire restait , je vais retenter plus proprement , j'édite quand c'est fait :)

[EDIT] : toujours le problème de référence non définie :

global.h
Code:

1 2 3 4 5 6 7 8 9 10 #ifndef _GLOBAL_H_ #define _GLOBAL_H_ #include <time.h> int my_rand(); extern int x; #endif
global.c
Code:

1 2 3 4 5 6 7 8 #include "global.h" int x =my_rand(); int my_rand() { // ... }
appli.c
Code:

1 2 3 4 5 6 7 #include "global.h" int main(void) { // chaque utilisation de la variable externe x provoque une erreur : "undefined reference to 'x'" }
J'ai fais quoi de mal ? Merci :)

[EDIT 2]
Avant je faisais :
Code:

1 2 3 4 5 6 cc -c global.c cc -o global global.o ./global cc -c appli.c cc -o appli appli.o
j'ai rajouté l'objet global.o au link
Code:

1 2 cc -o appli appli.o global.o
Je n'ai plus les messages d'erreur : undefined reference to 'x' mais un joli :
"multiple definition of `main'"

global.c est devenu :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 #include "global.h" int x; /*Si x est initialisée hors du main avec le retour de my_rand() -> ERROR : initializer element is not constant */ int main(void) { x = my_rand(); return 0; } int my_rand() { //... }
Merci de votre aide :)

10/02/2008, 16h41
Sve@r
Citation:
Envoyé par Leole

global.h

Code:

1 2 3 4 5 6 7 8 9 10 #ifndef _GLOBAL_H_ #define _GLOBAL_H_ #include <time.h> int my_rand(); extern int x; #endif
ENLEVE VITE ce underscore en début du token "GLOBAL_H" parce que si Emmanuel le voit avant moi, il va te découper en lamelles !!!
Sinon c'est bon
Citation:
Envoyé par Leole

global.c

Code:

1 2 3 4 5 6 7 8 #include "global.h" int x =my_rand(); int my_rand() { // ... }
Là c'est pas bon. T'as pas le droit d'initialiser une variable avec autre chose qu'une constante. Et my_rand() c'est pas une constante. Et personnellement si je devais faire une globale je la définirais dans le main qui est généralement considéré comme un peu "à part" du reste...
Citation:
Envoyé par Leole

appli.c

Code:

1 2 3 4 5 6 7 #include "global.h" int main(void) { // chaque utilisation de la variable externe x provoque une erreur : "undefined reference to 'x'" }
Ouiap. T'as bien linké le global.o et appli.o dans ta compilation ???

Autre chose => Il est d'usage qu'une fonction déclare toutes ses variables qu'elle va utiliser. Même si ces variables sont globales ben on peut quand-même les déclarer dans le corps avec "extern"...

Voici comment j'aurais fait le truc

global.h
Code:

1 2 3 4 5 6 7 8 9 10 #ifndef GLOBAL_H_ #define GLOBAL_H_ #include <time.h> int my_rand(); extern int x; #endif // GLOBAL_H_
my_rand.c
Code:

1 2 3 4 5 6 #include "global.h" int my_rand() { // ... }
appli.c
Code:

1 2 3 4 5 6 7 8 9 #include "global.h" int x; int main(void) { extern int x; x=my_rand(); ... }
Compilation
gcc -c my_rand.c
gcc -c appli.c
gcc appli.o my_rand.o -o appli
10/02/2008, 17h50
Leole
Merci ! cela compile :) je viens de tester un peu , une question demeure non résolue pour moi , je pensais que ce serait plus simple mais bon , en fait j'ai besoin de cette variable globale dans 2 fichiers donc j'ai 2 fichiers , on va dire appli.c et appli2.c .
Dans appli.c , je fais ainsi donc :
Code:

1 2 3 4 5 6 7 8 9 10 #include "global.h" int x; int main() { extern int x; x = my_rand(); printf("x = %d",x); // affiche par exemple 2. //... }
Et je m'attendais à avoir la meme valeur dans appli2.c vu que c'est global mais apparement non , j'ai 0 .
appli2.c:
Code:

1 2 3 4 5 6 7 8 9 #include "global.h" int x; int main() { extern int x; printf("x = %d",x); // affiche 0 //... }
J'en ai effet besoin de la meme valeur dans appli2.c , comment faire ?
merci :)

Citation:
Envoyé par Leole
Oui j'avais lu ça il y a qq temps déjà mais je ne pensais pas cela utile pour un si petit projet , j'avais essayé mais une erreur similaire restait , je vais retenter plus proprement , j'édite quand c'est fait :)

[EDIT] : toujours le problème de référence non définie :

global.h
Code:

1 2 3 4 5 6 7 8 9 10 #ifndef _GLOBAL_H_ #define _GLOBAL_H_ #include <time.h> int my_rand(); extern int x; #endif
global.c
Code:

1 2 3 4 5 6 7 8 #include "global.h" int x =my_rand(); int my_rand() { // ... }
appli.c
Code:

1 2 3 4 5 6 7 #include "global.h" int main(void) { // chaque utilisation de la variable externe x provoque une erreur : "undefined reference to 'x'" }
J'ai fais quoi de mal ? Merci :)

[EDIT 2]
Avant je faisais :
Code:

1 2 3 4 5 6 cc -c global.c cc -o global global.o ./global cc -c appli.c cc -o appli appli.o
j'ai rajouté l'objet global.o au link
Code:

1 2 cc -o appli appli.o global.o
Je n'ai plus les messages d'erreur : undefined reference to 'x' mais un joli :
"multiple definition of `main'"

global.c est devenu :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 #include "global.h" int x; /*Si x est initialisée hors du main avec le retour de my_rand() -> ERROR : initializer element is not constant */ int main(void) { x = my_rand(); return 0; } int my_rand() { //... }
Dans global.c, on se tue à te dire qu'une globale peut pas être initialisée par un appel de fonction. Il fait une constante. Il faut donc changer de méthode et créer une fonction d'initialisation qui va mettre la bonne valeur.
Code:

1 2 3 4 5 6 7 8 #include "global.h" int G_x; void x_init(void) { G_x = my_rand(); }
mais c'est de la programmation de barbare que je ne recommande pas du tout. j'ai indiqué une alternative correcte.
avec :
Code:

1 2 3 4 5 /* global.h */ int G_x; void x_init(void);
Tu as bien fait un projet avec le source qui contient main et global.c ?

Ca devrai ressembler à ça :
Code:

1 2 3 4 cc -c global.c cc -c appli.c cc -o appli appli.o global.o

10/02/2008, 18h18
souviron34
z'êtes bien compliqué....

De la difficulté d'employer correctement des globales INTER-modules... D'où DU CONSEIL DE NE PAS UTILISER DE GLOBALES INTER-MODULES...

global.h :
Code:

1 2 3 4 5 6 7 8 9 #ifndef GLOBAL_H_ #define GLOBAL_H_ #include <time.h> extern int my_rand(); #endif
global.c
Code:

1 2 3 4 5 6 7 8 9 #include "global.h" int my_rand() { // ... return result ; }
appli.c
Code:

1 2 3 4 5 6 7 8 9 10 #include <stdio.h> #include "global.h" int main() { printf("x = %d",my_rand()); // affiche par exemple 2. //... }
MAINTENANT UN GLOBAL N'EST PAS GLOBAL ENTRE 2 EXECUTABLES .................

Entre différents modules d'un MEME exécutable oui.... 1 SEUL MAIN........
10/02/2008, 18h25
Emmanuel Delahaye

Citation:

Envoyé par souviron34

z'êtes bien compliqué....

De la difficulté d'employer correctement des globales INTER-modules... D'où DU CONSEIL DE NE PAS UTILISER DE GLOBALES INTER-MODULES...

Oin tu as raison. N'utilisant pas (ou peu) les variables à instances uniques, cette solution simple ne m'est pas venue à l'esprit...

bah c'est juste parce que tu vois tellement d'horreurs ...

et si vraiment on tient à avoir une VARIABLE globale, c'est kifkif..

global.h :

Code:

1
2
3
4
5
6
7
8
9
10
 
#ifndef GLOBAL_H_
#define GLOBAL_H_
 
#include <time.h>
 
extern int my_rand();
extern int MonX ;
 
#endif

global.c :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
#include "global.h"
 
int MonX ;
 
int my_rand()
{
	// ...
 
   MonX = result ;  
   return result  ;
}

appli.c

Code:

1
2
3
4
5
6
7
8
9
10
 
#include <stdio.h>
#include "global.h"
 
int main()
{
 
  printf("fct = %d MonX = %d",my_rand(),MonX); // affiche par exemple 2.
  //...
}

10/02/2008, 21h23
Leole

Bonsoir :) Merci de votre aide à tous , cela fonctionne avec la première solution de souviron , en effet pas besoin d'une variable globale :aie:

bonne soirée !
11/02/2008, 21h48
Sve@r

Citation:

Envoyé par Leole

Bonsoir :) Merci de votre aide à tous , cela fonctionne avec la première solution de souviron , en effet pas besoin d'une variable globale :aie:

bonne soirée !

Dans un truc bien construit, on n'a quasiment jamais besoin de variables globales (j'ai dit quasiment parce qu'il y a quand-même des outils comme signal() qui ont besoin de globales)...
12/02/2008, 17h25
souviron34

Citation:

Envoyé par Sve@r

Dans un truc bien construit, on n'a quasiment jamais besoin de variables globales (j'ai dit quasiment parce qu'il y a quand-même des outils comme signal() qui ont besoin de globales)...

et encore... si le design est bien concu, ce ne sont que des varaibles globales AU FICHIER...

On repete encore une fois : il est FORMELLEMENT DECONSEILLE d'utiliser des variables globales entre plusieurs fichiers.... Des methodes Get ou Set doivent servir..