Discussion :

[Joseph J]

Bonjour
Je suis étudiant en informatique et dans l'optique de travailler le c++ j'ai voulu coder un jeu de carte.
Mon probleme est que j'ai beau suivre des tutoriels sur le rand() je n'arrrive pas à faire une fonction que fonctionne.

voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#include <iostream>
#include <time.h>
#include <stdio.h>
#include <stdlib.h>
using namespace std;
#include "fonction_carte.h"
 
 
int aléatoire(int a, int b)
{
	return rand() / RAND_MAX * (b - a) + a;
}

cette fonction me retourne toujours 0.

Merci d'avance pour vos conseils.

[Bousk]

rand retourne un int, RAND_MAX est un int qui sera < au retour de rand (il me semble que RAND_MAX est inatteignable mais je n'en ai pas la certitude).
Donc la division entière sera toujours 0.
Du coup ta fonction elle retournera toujours a.

[Joseph J]

J'ai essayer de remplacer la ligne

Code :

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

return rand() /RAND_MAX * (b - a) + a;

par la ligne

Code :

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

return rand() / (b - a) + a;

Dans ce cas la valeur retournée sortait de mon interval (a;b)

puis par

Code :

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

return rand() % (b - a) + a;

Et maintenant la fonction me retourne toujours 2.

avec un intervale [0;3].

[Invité]

Bonsoir,

La fonction rand() est un reliquat du C à ne plus utiliser.
C++11 a apporté bien mieux : std::uniform_int_distribution.

Également :

Code :

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

1
2
3
#include <time.h>
#include <stdio.h>
#include <stdlib.h>

sont des entêtes C et pas C++ qui supprime le .h et ajoute un « c » devant.

using namespace std; est également une très mauvaise pratique.