Discussion :

[F.R.E.D]

Bonjour,

Je programme avec MPLAB un robot mobile (e-puck) avec le compilateur C30.

J'utilise les bibliothèques de ce compilateur (math.h, stdlib.h) pour générer un nombre aléatoire.

Voici une fonction :

Code :

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

1
2
3
4
5
6
7
8
int NombreAleatoire(int a, int b)
{
	float x;
	x = rand()/RAND_MAX; // x dans [0,1]
	x = x*(b-a); // x dans [0,b-a]
	x = x + a; // x dans [a,b]
	return floor(x);
}

Je n'ai hélas aucun moyen d'afficher des résultats dans une console (du moins je ne pense pas), mais quand le robot bouge tout se passe comme si cette fonction me renvoyait toujours l'entier a.

D'où peut venir le problème?

[Invité(e)]

Bonjour,

Code :

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

1
2
3
4
5
6
7
8
int NombreAleatoire(int a, int b)
{
	float x;
	x = rand()/RAND_MAX; /* x à 0 */
	x = x*(b-a); /* x à 0 */
	x = x + a; /* x à a */
	return floor(x);
}

Ce code pose problème :

Code :

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

x = rand()/RAND_MAX;

Est toujours égal à 0.

Pourquoi ? rand() et RAND_MAX sont deux entiers, leur division est donc une division entière, comme 0 <= rand <= RAND_MAX, cela donne toujours zero.

Solution :

Code :

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

1
2
3
4
5
6
7
8
int NombreAleatoire(int a, int b)
{
	float x = rand(); /* x dans 0 RAND_MAX*/
	x /= RAND_MAX; /* x dans [0.f, 1.f] */
	x *= (b-a); /* x dans [0.f,(float)(b-a)] */
	x += a; /* x dans [(float)a, (float)b] */
	return floor(x);
}

[Emmanuel Delahaye]

Bonjour,

Je programme avec MPLAB un robot mobile (e-puck) avec le compilateur C30.

J'utilise les bibliothèques de ce compilateur (math.h, stdlib.h) pour générer un nombre aléatoire.

Voici une fonction :

Code :

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

1
2
3
4
5
6
7
8
int NombreAleatoire(int a, int b)
{
	float x;
	x = rand()/RAND_MAX; // x dans [0,1]
	x = x*(b-a); // x dans [0,b-a]
	x = x + a; // x dans [a,b]
	return floor(x);
}

Je n'ai hélas aucun moyen d'afficher des résultats dans une console (du moins je ne pense pas), mais quand le robot bouge tout se passe comme si cette fonction me renvoyait toujours l'entier a.

D'où peut venir le problème?

En dehors de la correction déjà citée, la formule est fausse. Il manque un +1 :

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
 
#include <stdlib.h>
#include <stdio.h>
 
int NombreAleatoire (int a, int b)
{
   double x = rand ();
   x /= RAND_MAX;               /* x dans [0,1] */
   x *= (b - a + 1);            /* x dans [0,b-a] */
   x += a;                      /* x dans [a,b] */
   return x;
}
 
int main (void)
{
   int i;
 
   for (i = 0; i < 160; i++)
   {
      int n = NombreAleatoire (1, 10);
      printf ("%4d", n);
   }
   return 0;
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
   1   6   2   9   6   5   4   9   9   8   2   9   8   6   4   1   1   4   2   2
  10   5   2   1   1   4   6   6   7   7   2   7   5   4   1   7   8   9   6   4
   9   8  10  10   6   2   5   3   9   3   8   9  10  10   7   4   3   3   9   1
   4   1   7   1   1  10   3   3   6   7   9   8   5   3   8   5   5  10   8   2
   6   4   8   7   6   4   2   3   5   9   6  10   8   4   2   7   5   1   7   6
   2  10   2  10   7   4   5   1  10   7   2   9   9   6   2   2   9   5   2   6
   8   5   3   6   7   8   8   5   2   4   9   1   6   7   5   2  10  10   6   4
   5   4   9   4   5   3  10   3   8   6   2   8   9   4   6   9   1  10   6   1
 
Process returned 0 (0x0)   execution time : 0.080 s
Press any key to continue.

On a des 10. Sans le '+1', on a pas de 10 :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
   1   6   2   8   6   5   4   9   8   7   2   8   7   5   3   1   1   4   2   2
   9   5   2   1   1   4   5   6   6   6   2   6   5   4   1   6   8   8   5   3
   8   7   9   9   5   2   5   3   8   2   8   8   9   9   6   4   3   3   8   1
   4   1   7   1   1   9   3   3   6   7   8   7   5   2   7   5   5   9   7   1
   6   4   7   6   6   4   2   3   4   8   5   9   7   4   2   6   5   1   7   5
   2   9   2   9   7   3   4   1   9   7   2   8   8   6   2   2   8   5   2   5
   7   4   3   6   7   7   7   5   2   4   8   1   5   6   4   1   9   9   5   4
   5   4   8   3   5   3   9   3   7   6   2   7   8   4   5   9   1   9   6   1
 
Process returned 0 (0x0)   execution time : 0.091 s
Press any key to continue.

Détails et expérimentations :

http://emmanuel-delahaye.developpez.com/notes.htm#rand

Faut pas hésiter à faire des tests unitaires sur les fonctions. Installe Code::Blocks sur ton PC...

Quand je développais en C pour des cartes électroniques avec 68k ou Power PC (32-bit big endian), j'écrivais et testais quasiment tout le code sur PC avec Borland C++ 3.1 (x86 16-bit little endian). Ça apprend à écrire du code portable !