Discussion :

[Vinch006]

Bonjour,

j'ai un petit problème. Je reçois des variables unes par unes que je stocke chaque fois dans la variable y01. Mon but est de stocker alors dans une variable appelée valueoutf1, le maximum de toutes les valeurs que y01 a pris.

Pour cela, j'ai écrit cette fonction:

if (y01>valueoutf1)
{valueoutf1=y01;
}
else
{valueoutf1=valueoutf1;
}

Mais je m'aperçois que valueoutf1 reste constant et ne s'adapte pas au maximum.
Y a-t-il une faute dans mon code?

[zooro]

Pour cela, j'ai écrit cette fonction:

Code :

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

1
2
3
4
5
6
if (y01>valueoutf1)
{valueoutf1=y01;
}
else
{valueoutf1=valueoutf1; 
}

Salut,

La partie "else" ne sert à rien ici.

Tu dois initialiser "valueoutf1" à la valeur minimale, par exemple 0 si y01 est un entier positif. Sinon, tu risques de ne jamais vérifier la condition "y01>valueoutf1".

Edit:

[igor24]

bonsoir

else
{valueoutf1=valueoutf1;
}

Superflu.

Peut être que le problème se trouve dans l'initialisation de valueoutf1 ou dans les valeurs que tu reçois dans y01.

[Emmanuel Delahaye]

j'ai un petit problème. Je reçois des variables unes par unes que je stocke chaque fois dans la variable y01. Mon but est de stocker alors dans une variable appelée valueoutf1, le maximum de toutes les valeurs que y01 a pris.

Pour cela, j'ai écrit cette fonction:

if (y01>valueoutf1)
{valueoutf1=y01;
}
else
{valueoutf1=valueoutf1;
}

Mais je m'aperçois que valueoutf1 reste constant et ne s'adapte pas au maximum.
Y a-t-il une faute dans mon code?

Déjà, il est horriblement mal présenté...

Code :

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

1
2
3
4
5
6
7
8
9
 
   if (y01 > valueoutf1)
   {
      valueoutf1 = y01;
   }
   else
   { 
      valueoutf1 = valueoutf1; 
   }

Ensuite, il est inutilement compliqué :

Code :

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

1
2
3
4
5
 
   if (y01 > valueoutf1)
   {
      valueoutf1 = y01;
   }

Enfin, tel quel, il semble correct, mais tout dépend de la valeur initiale de 'valueoutf1' , qui entre nous, pourrait s'appeler 'max', ce qui rendrai le code lisible...

[fearyourself]

Personnellement, ce qui me fait peur c'est 3 choses :

- Quelle est la valeur de valeuroutf1 de départ ?
- Comment prouver que c'est pas plus grand que le max que tu cherches ?
- Comment tu déclares ta fonction et cette variable ?

Jc

[josse95]

C'est ce qui s'appelle enfoncer le clou ! Et pourtant, zooro est arrivé ...