[SOLUCIONADO] Funciones c++

[HelThunk]

Buenas, estaba haciendo unos ejercicios para ir mejorando en c++,. y me he topado con un problema, he intentado resolver este usando funciones (que es lo que todos recomendáis para no hacer un código largo), sabría resolver este problema usando otros métodos (if, else,swich...) pero me gustaría aprender a usar funciones.. les muestro mi cogido, ¿podríais echarme una mano?

Código C++:

#include <iostream>
//23. Programa que muestre un menú de múltiplos (De 2,3,5,7,11).
 
//Funcion multiplo de X num;
void multiple(int a){
 
    int i;
    for (i = 0; i < 61; i++){
        if (i%a == 0){
            std::cout << i <<std::endl;
        }
    }
    std::cin.get();
}
 
//Funcion principal;
int main(){
    std::cout << "Bienvenido, eliga la opcion que mas se acomode a sus necesidads" << std::endl << std::endl;;
    std::cout << "[1]Multiplos de 2" << std::endl;
    std::cout << "[2]Multiplos de 3" << std::endl;
    std::cout << "[3]Multiplos de 5" << std::endl;
    std::cout << "[4]Multiplos de 7" << std::endl;
    std::cout << "[5]Multiplos de 11" << std::endl;
    //damos valor a 'a';
    int a;
    std::cin >> a;
    //si introduce 5, 4, 3, 2, 1;
    if (a < 6 && a > 0){
        void multiple(int a);//NO ME DEVUELVE EL VALOR, HE PROBADO CON INT Y TAMPOCO
    }
    //si introduce un valor no valido;
    else{
        std::cout << "Introduzca un numero del 1 al 5" << std::endl;
        
    }
    //terminamos el proceso
    std::cin.get();
    return 0;
}
 

[ecfisa]

Hola.

Lo que sucede es que envias a la función la misma variable que usas para las opciones y los valores no son concordantes: 1 -> 2, 2 -> 3, 3 -> 5, 4 -> 7, 5 -> 11.

Una solución es usar un arreglo que contenga los valores a enviar para hallar los múltiplos:

Código C++:

...
int main() {
  int mul[5] = {2, 3, 5, 7, 11}; 
  std::cout << "Bienvenido, eliga la opcion que mas se acomode a sus necesidads" << std::endl << std::endl;;
  std::cout << "[1]Multiplos de 2" << std::endl;
  std::cout << "[2]Multiplos de 3" << std::endl;
  std::cout << "[3]Multiplos de 5" << std::endl;
  std::cout << "[4]Multiplos de 7" << std::endl;
  std::cout << "[5]Multiplos de 11" << std::endl;
 
  int a;
  std::cin >> a;
  
  if ( a < 6 && a > 0 )  {
    std::cout << "Multiplos de " << mul[a-1] << std::endl;
    std::cout << "---------------------" << std::endl;
    multiple ( mul[a-1] );
  }
  else 
  ...
 

Saludos.

[HelThunk]

Iniciado por ecfisa

Hola.

Lo que sucede es que envias a la función la misma variable que usas para las opciones y los valores no son concordantes: 1 -> 2, 2 -> 3, 3 -> 5, 4 -> 7, 5 -> 11.

Una solución es usar un arreglo que contenga los valores a enviar para hallar los múltiplos:

Código C++:

...
int main() {
  int mul[5] = {2, 3, 5, 7, 11}; 
  std::cout << "Bienvenido, eliga la opcion que mas se acomode a sus necesidads" << std::endl << std::endl;;
  std::cout << "[1]Multiplos de 2" << std::endl;
  std::cout << "[2]Multiplos de 3" << std::endl;
  std::cout << "[3]Multiplos de 5" << std::endl;
  std::cout << "[4]Multiplos de 7" << std::endl;
  std::cout << "[5]Multiplos de 11" << std::endl;
 
  int a;
  std::cin >> a;
  
  if ( a < 6 && a > 0 )  {
    std::cout << "Multiplos de " << mul[a-1] << std::endl;
    std::cout << "---------------------" << std::endl;
    multiple ( mul[a-1] );
  }
  else 
  ...
 

Saludos.

He probado lo que me has dicho, y me surgen unas dudas (y también algunos errores);

Código C++:

int mul[5] = {2, 3, 5, 7, 11};
 

¿Con esto le decimos a la variable mul que solo puede aceptar valores de '2', '3', '5', '7' y '11'?

Código C++:

std::cout << "Multiplos de " << mul[a-1] << std::endl;
 

¿Por qué le restamos a la 'a' 1?

Código C++:

multiple ( mul[a-1] );
 

Aquí también volvemos a restarle a mul[a-1]..


*Al compilar y ejecutar hay un error, cuando estamos haciendo la operación, si pongo 2 me da múltiplos de 3, si pongo 3 me da múltiplos de 5, si pongo 5 me da múltiplos de 11.. (si pongo en vez de restar -1 que reste -2, al principio va bien hasta los múltiples de 5, a partir de ahí ya falla..)

*Me puedes explicar como funciona lo de int NombreVariable[Xnum] = {X resultados};

*Cabe añadir que me hubiese gustado también el poder aprender a usar las funciones..

[ecfisa]

Hola.

Código C++:

int mul[5] = {2, 3, 5, 7, 11};
 

¿Con esto le decimos a la variable mul que solo puede aceptar valores de '2', '3', '5', '7' y '11'?

No, en el arreglo mul, almacenamos valores cuyos índices guardan relación con lo expresado en el menu.

Código C++:

std::cout << "Multiplos de " << mul[a-1] << std::endl;
 

¿Por qué le restamos a la 'a' 1?

Por que los arreglos son basados en índice 0 ( es decir v[0] = 2, v[1] = 3, ... )

... si pongo 2 me da múltiplos de 3, si pongo 3 me da múltiplos de 5, si pongo 5 me da múltiplos de 11...

Pero, ¿ No es eso lo que indican las opciones de tu menú ? :

Código C++:

std::cout << "[1]Multiplos de 2" << std::endl;
  std::cout << "[2]Multiplos de 3" << std::endl;
  std::cout << "[3]Multiplos de 5" << std::endl;
  std::cout << "[4]Multiplos de 7" << std::endl;
  std::cout << "[5]Multiplos de 11" << std::endl;
 

Es decir, si alguien lee ese menú y desea obtener multiplos de 7 sin dudas presionará el 4. Si el valor del menu se correspondiera con el número a obtener el múltiplo, el arreglo sería innecesario.

Me puedes explicar como funciona lo de int NombreVariable[Xnum] = {X resultados};

Un arreglo es un conjunto de valores del mismo tipo a los cuales se les puede hacer referencia mediante un nombre y un índice.
En la declaración del arreglo se lo puede inicializar mediante un '=' seguido de los valores de su tipo, encerrados entre llaves y separados por comas.

Al compilar y ejecutar hay un error

Por si te sirve, hice esta prueba respetando básicamente tu código y funciona sin error:

Código C++:

#include <iostream>
 
using namespace std;
 
void multiple(int a) {
  for (int i = 2; i < 61; i++)
    if (i % a == 0)
      cout << i << "  ";
}
 
int main() {
  int mul[5] = {2, 3, 5, 7, 11};
  cout << "Bienvenido, eliga la opcion que mas se acomode a sus necesidads" << endl << endl;;
  cout << "[1]Multiplos de 2" << endl;
  cout << "[2]Multiplos de 3" << endl;
  cout << "[3]Multiplos de 5" << endl;
  cout << "[4]Multiplos de 7" << endl;
  cout << "[5]Multiplos de 11" << endl;
 
  int a;
  cin >> a;
 
  if (a < 6 && a > 0) {
    cout << "Multiplos de " << mul[a-1] << endl;
    cout << "---------------------" << endl;
    multiple(mul[a-1]);
  }
  else
    cout << "Introduzca un numero del 1 al 5" << endl;
 
  cin.ignore();
  cin.get();
  return 0;
}
 

Saludos.

[HelThunk]

Vale, creo que lo he entendido, pero.. ¿y si quiere que al poner un valor fuera del arreglo se iniciase de nuevo el proceso?, ¿qué le puedo poner al else?

También me podrías decir que es lo que hace exactamente el multiple dentro de la funcion?, es decir, cuando pones lo de;

Código C++:

multiple(mul[a-1]);
 

Lo que estas diciendo es que en

Código C++:

void multiple(int x){
 
    int i;
    for (i = 0; i < 61; i++){
        if (i%x == 0){
            std::cout << i <<std::endl;
        }
    }
    std::cin.get();
}
 

x sea igual a mul[a-1] no?

[lareto]

En la línea

Código:

void multiple(int a);//NO ME DEVUELVE EL VALOR, HE PROBADO CON INT Y TAMPOCO

Estás declarando una nueva función multiple que toma un int como parámetro y devuelve void.
Para invocar la función de arriba debes poner

Código:

multiple(a); // AHORA SÍ.

Sugerencia: hay usados por menos de 20 dólares...
http://www.amazon.com/dp/0321992784/...ackoverfl08-20

[ecfisa]

Hola.

Iniciado por HelThunk

...
También me podrías decir que es lo que hace exactamente el multiple dentro de la funcion?, es decir, cuando pones lo de;

Código C++:

multiple(mul[a-1]);
 

Lo que estas diciendo es que en

Código C++:

void multiple(int x){
 
    int i;
    for (i = 0; i < 61; i++){
        if (i%x == 0){
            std::cout << i <<std::endl;
        }
    }
    std::cin.get();
}
 

x sea igual a mul[a-1] no?

Así es. x es un parámetro por valor, de tipo int que admite valores de dicho tipo. Cuando recibe como argumento mul[a-1], el valor de x es igual al valor de mul[a-1].

Saludos.