Nombre aléatoire type "float" entre deux bornes

Bonjour,

Est ce que vous pouvez me corriger le message d'erreur suivant : invalid operands of types ‘int’ and ‘float’ to binary ‘operator%’.
Le code :

Code:

---

1 2 3 4 5 6 7 8 9

float borne_maximale = 0.9; float borne_minimale = 0.7; float proba[10]; cpmt =0; while (cpmt<10) { proba[cpmt]=rand()%(borne_maximale - borne_minimale) + borne_minimale; cpmt++; }

---

Merci d'avance.

L'opérateur % (modulo) n'existe pas sur les décimaux.
x%0.2, cela ne veut rien dire.
x%1.0 non plus.

double rand1_0 = rand() / (double)RAND_MAX; te donnera un nombre à virgule flottante entre 0.0 et 1.0 (diviser par (double)(RAND_MAX+1) permet d'exclure la valeur 1.0).
À partir de ça, tu devais sans peine obtenir la plage de valeurs que tu veux.

J'ai résolu mon problème, je vous remercie.

Code:

---

1 2 3 4 5 6 7 8 9 10

int borne_maximale = 90; int borne_minimale = 70; float proba[10]; cpmt =0; while (cpmt<10) { int bigratio =rand()%(borne_maximale - borne_minimale) + borne_minimale; proba[cpmt] = bigratio/100.0; cpmt++; }

---

Citation:

---

Envoyé par bacelar

L'opérateur % (modulo) n'existe pas sur les décimaux.
x%0.2, cela ne veut rien dire.
x%1.0 non plus.

---

L'opérateur non mais la fonction si. Par exemple pour obtenir un angle modulo 2π, on peut faire ceci:

Code:

---

1 2	float angle = 5*M_PI/2; float modulo = std::fmod(angle, 2*M_PI);

---

On obtient bien un angle modulo de π/2.

Ou std::remainder, qui est une autre facon de calculer le reste de la divison. Cf http://en.cppreference.com/w/cpp/numeric/math/remainder

Nous sommes sur la session C++ donc rand() n'a rien à faire ici :s et le reste du code est C-style ou c++oldschool x)
Utilise <random> :)
En c++ on utilise les {} pour init donc ceci :

Code:

---

1 2	int borne_maximale = 90;

---

devient ceci :

Code:

---

int borne_maximale{90};

---

de plus tu devrais utilisé unsigned int qui est plus approprié
A la place de ton while une bonne boucle for serait plus adapté :

Code:

---

for( size_t cmpt{0}; cmpt < 10; ++cmpt ) {}

---

En c++ on utilise std::array pour les tableau statique.
Et tes deux variables mini et maxi seront inutile car il te suffit de créer ton générateur avec une distribution :

Code:

---

std::uniform_int_distribution<int> distribution(70,90);

---

Bref corrige ton code pour le mettre à niveau, c'est plus propre :)

En utilisant <random>, le modèle de classe le plus approprié dans ce contexte est uniform_real_distribution :

Code:

---

1 2 3 4 5

std::default_random_engine generator; std::uniform_real_distribution<float> distribution(0.7f, 0.9f); std::array<float, 10> proba; for(float& p : proba) p = distribution(generator);

---

J'ai pas chercher plus loin il avait mi du int mais oui ce serai encore mieux de tout mettre en float :) par contre j'aurais laisser l'ancien for car vu le code, ça m'étonnerais qu'il utilise c++11, mais bon tant qu'a faire bien, autant le faire jusqu'au bout