remplir aléatoirement un tableau

Bonjour,

Je cherche une fonction SIMPLE qui me permette de remplir un tableau (1d sur cet exemple) de valeurs binaires prises au hasard. De toutes évidences je n'utilise pas la fonction rand correctement, puisqu'avec le code qui suit les éléments de mon tableau sont tous nuls ... Ou alors la fonction rand n'est pas capable de réaliser ce que je souhaite ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 #include <stdio.h> #include <stdlib.h> main() { int *tab, i; tab=(int *)malloc(50*sizeof(int)); for (i=0; i<50; i++) tab[i]=rand()/RAND_MAX; for (i=0; i<50; i++) printf("%d ", tab[i]); }

parce que tu n'as pas initialisé le générateur de nombres pseudo-aléatoires.

ça aussi c'est dans la faq

Bonjour,

Les nombres aléatoires en C

Bon courrage ;)

Citation:

Envoyé par khayyam90

parce que tu n'as pas initialisé le générateur de nombres pseudo-aléatoires.

Ce n'est pas une obligation, par contre la division entière pas RAND_MAX ce n'est surement pas la bonne méthode.

Indépendemment de ce qui a déjà été dit, ton code est plein d'erreurs :

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
28
29
30
31
32
33
34
35
36
#include <stdio.h>
#include <stdlib.h>
 
// Comme tu n'utilises pas les arguments de la ligne de commande
// Le prototype de main à utiliser est
int main(void)
{
      int *tab, i;
 
      // En C, il n'est pas nécessaire de caster le retour de malloc
      // C'est même déconseillé !!
      // tab=(int *)malloc(50*sizeof(int));
      tab=malloc(50*sizeof(int));
      // si tu as un message à la Compil, c'est que tu compiles avec le
      // compilateur C++, l'extension du source est .cpp ou .C
 
      // Il faut tester le retour de malloc
      if (tab == NULL)
      {
         fprintf(stderr, "Pb malloc\n");
         return 0;
      }
 
      for (i=0; i<50; i++)
            // ici la division te donne en général 0 puisque rand() est inférieur à RAND_MAX
            tab[i]=rand()/RAND_MAX;
 
      for (i=0; i<50; i++)
            printf("%d ", tab[i]);
 
      // il ne faut pas oublier de libérer la mémoire allouée
      free(tab);
 
      // on retourne une valeur au système
      return 0;
}

Citation:

Envoyé par gege2061

Bonjour,

Les nombres aléatoires en C

Bon courrage ;)

Merci 8-)

Avec ce code j'obtiens un résultat apparemment satisfaisant :
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 #include <stdio.h> #include <stdlib.h> int my_rand (void) { static int first = 0; if (first == 0) { srand (time (NULL)); first = 1; } return (rand ()); } main() { int *tab, i; tab=(int *)malloc(50*sizeof(int)); for (i=0; i<50; i++) tab[i]=(int)((float)my_rand() / RAND_MAX * (2)); }
Citation:

Envoyé par Trap D

ton code est plein d'erreurs

C'est l'avis des puristes, pas le mien. Ca n'est pas non plus l'avis du compilateur, ni de celui qui se contente de faire un programme qui fonctionne, dans la plupart des cas. A partir de là, je te remercie de tes remarques, que je connaissais déjà, mais je ne fais pas dans la dentelle 8-)

Citation:

Envoyé par babar56

C'est l'avis des puristes, pas le mien. Ca n'est pas non plus l'avis du compilateur, ni de celui qui se contente de faire un programme qui fonctionne, dans la plupart des cas. A partir de là, je te remercie de tes remarques, que je connaissais déjà, mais je ne fais pas dans la dentelle 8-)

Libre à toi de programmer ainsi, c'est ton problème si tu te satisfais de la médiocrité. Au moins apprend à régler tes outils, en l'occurence ton compilateur.

Citation:

Envoyé par babar56

Ca n'est pas non plus l'avis du compilateur

Je sais plus dans quelle signature j'ai vu un truc du genre "un programme qui compile n'est pas un programme qui marche"

Citation:

Envoyé par babar56

mais je ne fais pas dans la dentelle 8-)

Dommage

Cher Trap D. Toi tu apportes des remarques inutiles qui visent essentiellement à flatter ton égaux, et moi j'apporte un code opérationnel. Deux conceptions de la vie radicalement opposées. Et si tu t'exprimais sur le sujet de la discussion pour changer ? En attendant, je préfère te laisser parler dans le vent.

Citation:

Envoyé par Gruik

Dommage

Je ne suis pas informaticien ! Ce n'est pas mon métier, en l'occurence c'est surtout un passe-temps.

Si t'as pas envie d'etre rigoureux, fais autre chose que du C

Citation:

Envoyé par Gruik

Si t'as pas envie d'etre rigoureux, fais autre chose que du C

Pourquoi préciser du C ?

Pas la peine d'essayer de programmer dans ce cas, l'absence de rigueur conduit tôt ou tard dans un mur, très vite d'ailleurs en général.
Citation:
Envoyé par babar56
Avec ce code j'obtiens un résultat apparemment satisfaisant :
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 #include <stdio.h> #include <stdlib.h> int my_rand (void) { static int first = 0; if (first == 0) { srand (time (NULL)); first = 1; } return (rand ()); } main() { int *tab, i; tab=(int *)malloc(50*sizeof(int)); for (i=0; i<50; i++) tab[i]=(int)((float)my_rand() / RAND_MAX * (2)); }
Au minimum, libère la mémoire réservée par malloc.

Ah bein c'était toi la signature ^^

Bonjour,

Citation:

Envoyé par Gruik

Ah bein c'était toi la signature ^^

Ben oui, j'en ai vu tellement qui me disent "ça compile sans erreur ni warning, donc c'est bon", et qui se demandent pourquoi pourquoi ça ne marche pas...

Citation:

Envoyé par babar56

Avec ce code j'obtiens un résultat apparemment satisfaisant:

Les appararences sont trompeuses :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : D:\dev\ed02\cb\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:5: warning: no previous prototype for 'my_rand'
main.c: In function `my_rand':
main.c:10: error: implicit declaration of function `time'
main.c:10: warning: nested extern declaration of `time'
main.c:10: warning: passing arg 1 of `srand' as unsigned due to prototype
main.c: At top level:
main.c:16: warning: return type defaults to `int'
main.c: In function `main_':
main.c:23: warning: cast does not match function type
main.c:24:2: warning: no newline at end of file
Process terminated with status 1 (0 minutes, 0 seconds)
1 errors, 6 warnings

Ton code commenté et corrigé.

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
 
#include <stdio.h>
#include <stdlib.h>
 
/* -ed- pour time() */
#include <time.h>
 
/* -ed- fonction non exportee : ajoute 'static' */
static int my_rand (void)
{
   /* -ed-
   static int first = 0;
   fonctionnement correct, mais semantique illogique... */
   static int first = 1;
 
   if (first)
   {
      /* -ed- srand() attend un unsigned int
      srand (time (NULL));
      */
      srand ((unsigned) time (NULL));
      first = 0;
   }
   return (rand ());
}
 
/* -ed-
main()
 
manque int (obligatoire en C99) */
int main()
{
 
   /* -ed-
      int *tab, i;
      tab = (int *)malloc(50 * sizeof(int));
 
      cast inutile
      codage inutilement complexe
      malloc() peut echouer
      le bon type pour un indice est size_t
      manque la liberation
      la taille devrait etre sauverardee...
   */
   size_t nb_elem = 50;
   int *tab = malloc (nb_elem * sizeof * tab);
 
   if (tab != NULL)
   {
      size_t i;
      for (i = 0; i < nb_elem; i++)
      {
         /* -ed- le type naturel des flottants est double
               tab[i] = (int)((float)my_rand() / RAND_MAX * (2));
         */
         tab[i] = (int)((double)my_rand() / RAND_MAX * (2));
      }
 
      /* -ed- ajoute visualisation */
      i = 0;
      while (i < nb_elem)
      {
         printf ("%2d", tab[i]);
         i++;
         if ((i % 16) == 0)
         {
            printf ("\n");
         }
      }
      printf ("\n");
 
      free (tab) , tab = NULL;
   }
 
   /* -ed- obligatoire en C90 */
   return 0;
}

Code:

1
2
3
4
5
6
7
 
 0 0 0 0 1 0 0 1 1 0 0 1 0 0 0 1
 1 0 1 1 1 1 0 1 1 0 0 1 0 0 0 0
 1 0 1 1 1 0 0 1 1 0 0 1 0 0 0 1
 0 0
 
Press ENTER to continue.

Citation:

Envoyé par gege2061

Bonjour,

Les nombres aléatoires en C

Bon courrage ;)

A les memes problemes que la proposition de la FAQ.

http://www.bourguet.org/cpp/random.pdf

Citation:

Envoyé par Emmanuel Delahaye

Ton code commenté et corrigé.

OK merci, je lirais tout ça ce soir.

J'avais effectivement précisé que ça fonctionnait en apparence seulement 8-)

Citation:
Envoyé par Trap D
Code:

1 2 3 4 5 // Comme tu n'utilises pas les arguments de la ligne de commande // Le prototype de main à utiliser est int main(void) {
Qu'est ce que cela change si l'on met "int main (int argc, char *argv[])" ?

Citation:

Envoyé par Dud225

Qu'est ce que cela change si l'on met "int main (int argc, char *argv[])" ?

Ca fait râler les compilateurs (variables non utliisées). Comme une alternative existe, autant l'utiliser.