Skip to main content
4 de 4
replaced http://es.stackoverflow.com/ with https://es.stackoverflow.com/

A mi me compila perfectamente. ###Problema. Se queda en un bucle infinito porque los for no hacen avanzar la variable de control numero ###Propuesta. Cambia la instrucción de avance del bucle for por numero += 2:

for(int numero = 50; numero<=60; numero +=2)

...

for(int numero = 51; numero<60; numero +=2)

###Posible mejora. Puedes hacerlo todo en un solo bucle:

for (int numero = 50; numero <=60; ++numero)
{
    (numero % 2 ? sumaimpares : sumapares) += numero;
}

El truco usado en el código anterior es la combinación del operador ternario (?:) con el operador de incremento-asignación (+=) y la conversión implícita de entero a booleano.

El operador ternario (?:) evalúa una expresión y devuelve lo que haya tras el interrogante (?) en caso de que la expresión sea verdadera o lo que haya tras los dos puntos (:) en caso de que sea falsa, lo que no suele saberse es que el operador ternario funciona tanto como valor izquierdo como valor derecho (LValue y RValue1).

En el ejemplo anterior lo usamos de valor izquierdo, dicho valor será sumaimpares si la expresión numero % 2 es verdadera y sumapares en caso de ser falsa. Pero el tipo de numero % 2 es entero ¿Cómo se evalúa como si fuese booleano?

Todo valor numérico se convierte implícitamente a booleano siguiendo la lógica:

  • Falso si es cero.
  • Verdadero si no es cero.

Así pues, la expresión numero % 2 devuelve 1 si el valor no es divisible por 2 (el residuo de dividir un número impar por dos es uno) y 0 si el valor es divisible por 2. Dicho de otra manera: La expresión numero % 2 evaluada como booleana es true si el número es impar y false si el número es par.

Por lo tanto, la expresión (numero % 2 ? sumaimpares : sumapares) será sumaimpares para cada numero impar y sumapares para cada numero par, a una u otra variable se le aplica la operación de incremento-asignación con la variable numero como valor derecho. A grandes rasgos el código sería equivalente a:

for (int numero = 50; numero <=60; ++numero)
{
    int *a_quien_sumo = nullptr;
    if ((numero % 2) != 0)
    {
        a_quien_sumo = &sumaimpares;
    }
    else
    {
        a_quien_sumo = &sumapares;
    }
    (*a_quien_sumo) = (*a_quien_sumo) + numero;
}

###A tener en cuenta. En general, no se aconseja usar using namespace std;, consulta esta pregunta para conocer el por qué.

Por otro lado, el uso de std::endl puede derivar en problemas de rendimiento porque vacía el búfer; aunque en un programa tan sencillo como el tuyo no supondrá ningún problema. Consulta esta pregunta para tener más detalles.


1Puedes consultar más sobre LValue y RValue en esta pregunta.

PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86