template fonction aléatoire

Bonjour,

je cherche à réaliser une fonction qui genererait un nombre aléatoire entre un minimum et un maximum, j'ai coder ca mais ca me donne une erreur

invalid operands of types 'int' and 'double' to binary 'operator%'|

Est ce qu'il faut surcharger l'opérateur % ? Si oui comment faire ?

Voici le code

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

#include<iostream> #include<vector> #include <ctime> #include <cstdlib>   using namespace std;   template<typename T> T aleatoire(T minn, T maxx) { return (minn + (rand () % (maxx-minn+1))); }   int main() { srand(time(0)); int minimum(5); int maximum(10); double minimumd(1); double maximumd(100);   cout << aleatoire(minimum, maximum) << aleatoire(minimumd, maximumd) << endl; }

---

Merci d'avance !

Bonjour,

Je pense que la FAQ pourrait te donner quelques pistes.

Si ce n'est pas dans un but pédagogique, tourne toi vers std::random (C++11) ou boost::random.
Voici un post qui pourrait t'aider :
http://www.developpez.net/forums/d13...chier-lecture/
Et voici uniform distribution de la std (tu pourrais t'inspirer de l'interface à défaut de l'utiliser)
http://www.cplusplus.com/reference/r..._distribution/

Salut Neckara et merci de ta réponse, cependant c'est un lien vers la faq C, et quand je regarde et que j'essaye de l'appliquer à mon code, je ne vois pas l'erreur que je j'ai commise.

Salut Ehonn, une fois de plus merci, j'essaye ce code pour m'entrainer à mieux comprendre les template ^^, mais je vais quand même voir

Merci à vous encore.

% est le modulo, il n'est défini que pour un opérande de droite entier.

avec T=double, l'opérande droit n'est plus entier.

essaie une conversion, ou interdit le double (et le float)

Ah ok !

Donc dans:

Code:

---

(minn + (rand () % (maxx-minn+1)));

---

je dois convertir le resultat de maxx-minn+1 en int ?
Si oui comment faire ? un cast quelque chose non ? Je me souviens plus ^^

caster? par un simple C-cast tel que (int) valeur ou mieux un cast C++, probablement le static_cast<int>(valeur).

Il y a une autre variante possible

Code:

---

1 2 3 4

template<typename T> T aleatoire(T base, int ecart_max) { return base + rand () % (1 + ecart_max); }

---

Cela dit, tu devrais regarder du côté de boost/C++11 pour la génération propre, efficace et avec une bonne qualité d'aléatoire

Super, là ca marche, j'ai définit une variable double valant 100.24 que j'ai mise en deuxième argument de la fonction, et ca a fonctionné, alors qu'il s'agit d'un double, elle est convertit automatiquement ?

Sauf que cette solution va tronquer le double, ainsi si ton maximum est 4.5, ton maximum sera en fait 4.

La FAQ montre un code pour généré un nombre aléatoire entre 0 et X :

Code:

---

1 2 3 4

double rand_x(double x) { return rand() * (x / RAND_MAX); }

---

Qu'on peut aisément transformer en :

Code:

---

1 2 3 4

template<typename T> T rand_x(T x) { return rand() * (x / RAND_MAX); }

---

Ce qui peut donner :

Code:

---

1 2 3 4

template<typename T> T rand_x(T base, T ecart) { return base + rand() * (ecart / RAND_MAX); }

---

Est-ce que ceci ne marcherait pas ?

oui, parce que tu affectes un double à un int, explicitement.


maxx-minn était du type T, "T-1" est encore un T, et base%(T) n'est valable que pour T un type entier. (ou pour % redéfini pour le type de base et pour T)

In function 'int main()':|
22|error: no matching function for call to 'aleatoire(double&, int&)'|
22|note: candidate is:|
8|note: template<class T> T aleatoire(T, T)|
8|note: template argument deduction/substitution failed:|
22|note: deduced conflicting types for parameter 'T' ('double' and 'int')|

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

#include<iostream> #include<vector> #include <ctime> #include <cstdlib>   using namespace std;   template<typename T> T aleatoire(T minn, T maxx) { return minn + rand() * (minn / RAND_MAX); }     int main() { srand(time(0)); int minimum(5); int maximum(10); double minimumd(1.54); double maximumd(100.24);   cout << aleatoire(minimum, maximum) << " " << aleatoire(minimumd,maximum) << endl; }

---

Autre chose étrange, quand je remplace le minimumd(valant 1.24) de aleatoire(minimumd,maximum), par minimum(valant 5), la console m'affiche systematiquement le chiffre 5, mais tout le temps (j'ai essayé 40 fois pour voir si c'était pas juste de la chance)

Pour ton premier message d'erreur, le problème vient juste du fait que le compilateur ne sait pas s'il doit utiliser un int ou un double, il faut donc lui dire explicitement ce qu'on veut :

Code:

---

1 2 3 4

aleatoire<int>(5.5, 4); aleatoire<double>(5.5, 4); aleatoire(5.5, 4.0); aleatoire(5, 4);

---

Citation:

---

Autre chose étrange, quand je remplace le minimumd(valant 1.24) de aleatoire(minimumd,maximum), par minimum(valant 5), la console m'affiche systematiquement le chiffre 5, mais tout le temps (j'ai essayé 40 fois pour voir si c'était pas juste de la chance)

---

Oui, il y avait un piège :aie:
Je te laisse chercher un peu ;)
Un indice : division d'entiers.

je viens de me rendre compte que le deuxième argument était inutile.
Egalement, pour le double ca fonctionne, mais le chiffre ne varie pas beaucoup. ca change qu'apres 3 chiffre apres la virgule.

Pour la fonction avec la division d'entier, je cherche toujours :)

En décomposant le calcul:

Code:

---

(5 + rand()) * (5 / RAND_MAX)

---

(j'ai pris 5 mais c'est juste un exemple)

5+ rand = 18406 (par exemple)
5/RAND_MAX) = 0 !

Donc maintenant, je n'obtiens plus 5, mais 0 à chaque fois

Cette entrée de la FAQ pourrait t'aider.

Sinon ce n'est pas : (5 + rand()) * (5 / RAND_MAX) mais 5 + rand() * (5 / RAND_MAX)

Là je seche completement lol

La division entre entier est la division euclidienne, qui a pour résultat un entier.
Elle possède un reste.

Pour tous entiers p et q, p = (p/q)*q + p%q.
5 = (5/2)*2 + 5%2 = 2*2 + 1
5 = (5/10)*10 + 5%10 = 0*10 + 5

RAND_MAX est plus grand que 5, donc 5/RAND_MAX = 0.

La solution est de forcer la division à être un nombre flottant, en convertissant l'un de ses arguments en float ou double (le 5, de préférence)

pour un nombre aléatoire

rand()%t;

avec t la valeur max de ton rand

Citation:

---

Envoyé par takago

pour un nombre aléatoire

rand()%t;

avec t la valeur max de ton rand

---

As tu parcouru la thread ? Juste les premiers posts?