Discussion :

[Student_Master]

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 :

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

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.

[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.

[Médinoc]

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.

[Student_Master]

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

Code :

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

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++;
    }

[RPGamer]

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 :

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

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

On obtient bien un angle modulo de π/2.

[mintho carmo]

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

[Disixlis]

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 :

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

1
2
 
int borne_maximale = 90;

devient ceci :

Code :

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

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 :

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

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 :

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

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

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

[Pyramidev]

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

Code :

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

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

[Disixlis]

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