Esta condición:
0.25<r<0.5
No se evalúa como estás pensando, sino de la siguiente manera:
- Se evalúa
0.25<r
. Esta operación devolverá dos posibles valores: 1
(true) y 0
(false). En tu caso, r=0.26
, luego devolverá 1
.
- Se evalúa
1<0.5
. El 1
sale del punto anterior. En este caso la operación es falsa, luego la condición no se cumple
- Entonces se evaluará el tercer condicional:
0.5<r<0.75
. Volvemos a repetir el proceso: 0.5<0.26 == 0
y 0<0.75 == 1
. Por eso se está ejecutando la tercera condición en vez de la segunda
Para concatenar comparaciones debes usar los operadores OR ||
y AND &&
de la siguiente manera:
if (0.25 < r && r < 0.5)
En cualquier caso, nota que estás usando if else
, es decir, esta estructura de control ejecuta únicamente uno de los dos códigos (el que esté en el if
o el que esté en el else
, pero nunca ambos a la vez). Es decir, no hace falta evaluar varias veces lo mismo.
Si en el if
ya has comprobado que r<0.25
, es totalmente innecesario hacer esta evaluación nuevamente en los else
. El código te queda más limpio, legible y seguro:
if (r<0.5)
{
// ...
}
else if (r<0.5) // damos por hecho que r>=0.25
{
// ...
}
else if (r<0.75) // damos por hecho que r>=0.5
{
// ...
}
Por otro lado, date cuenta que estás usando tipos decimales. Hay que tener especial cuidado al comparar números con decimales ya que su representación no es exacta. Para más información te sugiero revisar esta otra pregunta: ¿Por qué mis programas no pueden hacer cálculos aritméticos correctamente?