3

con unos amigos estábamos compitiendo para ver quién terminaba primero un programa para multiplicar todas las tablas de multiplicar del 1 al 10 (un poco extraño pero no teníamos otra idea), y yo hice lo propio en C++.

#include <iostream>

int main() {
  int v1 = 1, v2 = 1, limit = 10;
  for (; v1 <= (limit + 1); v1++) {
    if (v1 > limit) {
      v1 = 1;
      v2++;
      std::cout << "\n";
    }
    else if (v2 > limit) return 0;
    std::cout << v2 << "x" << v1 << " = " << int(v1 * v2) << "\n";
  }
}

Pero al compilar me doy cuenta que muestra una multiplicación extra.

Resultado

Supongo que el for loop es el problema, pero no sé qué tipo de solución tiene, por mi lo reescribiría pero quiero al menos encontrar el problema y que no me suceda lo mismo despues.

Gracias de antemano.

2 respuestas 2

3

Elimina el 'else' de

else if (v2 > limit)

Debería ser

if (v2 > limit) 

Para que así verifique el valor de v2 > 10 aun cuando v1 se reinicie a 1. Actualmente solo hace una de las dos cosas.

Es más, nunca v2 va a ser mayor que limit a menos que entre al if inicial (donde v2 es aumentado) por lo que una mejor solución sería solo verificarlo dentro de ese if:

#include <iostream>

int main() {
  int v1 = 1, v2 = 1, limit = 10;
  for (; v1 <= (limit + 1); v1++) {
    if (v1 > limit) {
      v1 = 1;
      v2++;
      std::cout << "\n";
      if (v2 > limit) return 0;
    }
  
    std::cout << v2 << "x" << v1 << " = " << int(v1 * v2) << "\n";
  }
}
1
  • Ha funcionado excelente, evidentemente no me había percatado de eso jajaja. Gracias por la solución.
    – nk125
    Commented el 14 oct. 2021 a las 23:07
3

Para evitar liarse con límites, lo más fácil es usar el for de rango:

int numeros[]{1,2,3,4,5,6,7,8,9,10};

for (const auto &a : numeros)
{
    for (const auto &b : numeros)
        std::cout << a << 'x' << b << " = " << a * b << '\n';
    std::cout << '\n';
}

Puedes ver el código funcionando en Try it online!. Si tienes alergia a escribir números, puedes pedirle al compilador que los escriba por ti:

std::vector<int> v(10);
std::iota(v.begin(), v.end(), 1);

for (const auto &a : v)
{
    for (const auto &b : v)
        std::cout << a << 'x' << b << " = " << a + b << '\n';
    std::cout << '\n';
}

Puedes ver el código funcionando en Try it online!.

2
  • El problema es que exactamente era lo que buscaba evitar, una lista super larga de int's en un array/vector, de todas formas es una buena solución. Gracias.
    – nk125
    Commented el 14 oct. 2021 a las 22:59
  • Si tienes alergia a escribir números, puedes pedirle al compilador que los escriba por ti. Gracias por la otra solución xD
    – nk125
    Commented el 15 oct. 2021 a las 19:42

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.