Discussion :

[Garwan50]

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 :

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
#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 !

[Neckara]

Bonjour,

Je pense que la FAQ pourrait te donner quelques pistes.

[Ehonn]

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/

[Garwan50]

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.

[ternel]

% 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)

[Garwan50]

Ah ok !

Donc dans:

Code :

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

(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 ^^

[ternel]

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 :

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

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

[Garwan50]

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 ?

[Neckara]

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 :

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

1
2
3
4
double rand_x(double x)
{
    return rand() * (x / RAND_MAX);
}

Qu'on peut aisément transformer en :

Code :

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

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

Ce qui peut donner :

Code :

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

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 ?

[ternel]

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)

[Garwan50]

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 :

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
#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)

[Neckara]

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 :

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

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

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
Je te laisse chercher un peu
Un indice : division d'entiers.

[Garwan50]

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 :

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

(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

[Neckara]

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)

[Garwan50]

Là je seche completement lol

[ternel]

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)

[takago]

pour un nombre aléatoire

rand()%t;

avec t la valeur max de ton rand

[therwald]

pour un nombre aléatoire

rand()%t;

avec t la valeur max de ton rand

As tu parcouru la thread ? Juste les premiers posts?