Ayuda codigo c ++ estructura selectiva + repetitiva

[gleon_01]

Hola buenos días!

solicitando su ayuda, cualquier comentario es bueno

Estoy realizando un ejercicio, se trata de utilizar una estructura repetitiva do-while en un programa que ya tiene una estructura selectiva switch - case.

el asunto es que, el programa compila correctamente, pero no hace todo el proceso, cuando tiene que mostrar el resultado de las instrucciones, en este caso de la conversión de las monedas, se regresa al primer menu.

el proceso completo es: seleccionar que tipo de conversión deseamos realizar, seleccionarlo, que tipo de moneda deseamos convertir, seleccionamos, ingresamos la cantidad de moneda y da como resultado la conversión, si deseamos salir seleccionamos la opción 3.

dejo el código, gracias por sus comentarios.

Código (cpp) [Seleccionar] Expandir

/* conversor de monedas */
#include <cstdlib>
#include <iostream>
#include <stdlib.h>
#include <stdio.h>
#include <conio.h>


#define dolar1 18.34
#define euro1 19.40
#define yen1 0.1620
#define quetzal1 0.4218
#define libra1 25.34

#define pesos3 18.34
#define pesos4 19.40
#define pesos5 0.1620
#define pesos6 0.4218
#define pesos7 25.34

/* funciones para convertir de pesos a otra_moneda */
void dolar();
void peso();
void euro();
void yen();
void quetzal();
void libra();

/* funciones para convertir otra_moneda a pesos */
void dolar2();
void peso2();
void euro2();
void yen2();
void quetzal2();
void libra2();


int main()
{

int opc;

do{ /* estrcutura repetitiva, cuando hace esto, sigue funcionando */


system("cls"); /* primer menu de opciones */
int opc, opcion, numero;
using namespace std;
cout<<"Conversor de Monedas ACME"<<endl;
cout<<"Seleccione la opcion deseada"<<endl;
cout<<"1. para convertir de pesos a moneda extranjera"<<endl;
cout<<"2. para convertir de moneda extranjera a pesos"<<endl;
cout<<"3. para salir"<<endl;
cin>>opc;

system("cls");

/* inicia el anidamiento */

switch (opc)
{ /* primera opción de pesos a moneda extranjera */
case 1:
cout<<"Ha seleccionado convertir Pesos a Moneda Extranjera"<<endl;
cout<<"Seleccione la moneda extranjera a la cual desea convertir los pesos"<<endl;
cout<<"1.- Dolar\n2.- Euro\n3.- Yen\n4.- Quetzal\n5.- Libra\n"<<endl;
cin>>opcion;
system("cls");

if (opcion==1)
{dolar();
}

if (opcion==2)
{euro();
}

if (opcion==3)
{yen();
}

if (opcion==4)
{quetzal();
}

if (opcion==5)
{libra();
}
break;

case 2: /* segunda opción, de moneda extranjera a pesos */

cout<<"Ha seleccionado convertir Moneda Extranjera a Pesos"<<endl;
cout<<"Seleccione la Moneda Extranjera a convertir a Pesos "<<endl;
cout<<"1.- Dolar\n2.- Euro\n3.- Yen\n4.- Quetzal\n5.- Libra\n"<<endl;
cin>>opcion;
system("cls");

if (opcion==1)
{dolar2();
}

if (opcion==2)
{euro2();
}

if (opcion==3)
{yen2();
}

if (opcion==4)
{quetzal2();
}

if (opcion==5)
{libra2();
}
break;

case 3: /* tercera opción, salida */

cout<<"\n\nAdios\n"<<endl;
system("cls");


}
}
while (opc!=3);
return 0;
}




void dolar() /*conversion de pesos a dolar*/
{
using namespace std;
double npesos, dolar, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
dolar=pesos*dolar1;
system("cls");
cout<<"Usted tiene: "<<dolar<<" Dolares \n \n"<<endl;
}

void euro() /*conversion de pesos a euros*/
{
using namespace std;
double npesos, euro, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
euro=pesos*euro1;
system("cls");
cout<<"Usted tiene: "<<euro<<" Euros \n \n"<<endl;
}

void yen() /*conversion de pesos a yen*/
{
using namespace std;
double npesos, yen, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
yen=pesos*yen1;
system("cls");
cout<<"Usted tiene: "<<yen<<" yenes \n \n"<<endl;
}

void quetzal() /*conversion de pesos a quetzal*/
{
using namespace std;
double npesos, quetzal, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
quetzal=pesos*quetzal1;
system("cls");
cout<<"Usted tiene: "<<quetzal<<" quetzales \n \n"<<endl;
}

void libra() /*conversion de pesos a libras*/
{
using namespace std;
double npesos, libra, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
libra=pesos*libra1;
system("cls");
cout<<"Usted tiene: "<<libra<<" libras \n \n"<<endl;
}

/* para el caso 2, convertir de moneda extranjera a pesos */

void dolar2() /*conversion de dolar a pesos*/
{
using namespace std;
double npesos, dolar, pesos;
cout<<"Ingrese la cantidad en Dolares"<<endl;
cin>>dolar;
pesos=dolar/pesos3;
system("cls");
cout<<"Usted tiene: "<<pesos<<" Pesos \n \n"<<endl;
}

void euro2() /*conversion de Euros a Pesos*/
{
using namespace std;
double npesos, euro, pesos;
cout<<"Ingrese la cantidad en Euros"<<endl;
cin>>euro;
pesos=euro*pesos4;
system("cls");
cout<<"Usted tiene: "<<pesos<<" Peros \n \n"<<endl;
}

void yen2() /*conversion de Yenes a Pesos*/
{
using namespace std;
double npesos, yen, pesos;
cout<<"Ingrese la cantidad en Yenes"<<endl;
cin>>yen;
pesos=yen/pesos5;
system("cls");
cout<<"Usted tiene: "<<pesos<<" Pesos \n \n"<<endl;
}

void quetzal2() /*conversion de Quetzales a Pesos*/
{
using namespace std;
double npesos, quetzal, pesos;
cout<<"Ingrese la cantidad en Quetzales"<<endl;
cin>>quetzal;
pesos=quetzal*pesos6;
system("cls");
cout<<"Usted tiene: "<<quetzal<<" quetzales \n \n"<<endl;
}

void libra2() /*conversion de Libras a pesos*/
{
using namespace std;
double npesos, libra, pesos;
cout<<"Ingrese la cantidad en Libras"<<endl;
cin>>libra;
pesos=libra/pesos7;
system("cls");
cout<<"Usted tiene: "<<pesos<<" Pesos \n \n"<<endl;

}

/* final*/

Mod: Usa etiquetas GeSHi al publicar código

[class_OpenGL]

Yo creo que el problema es que cuando llamas por primera vez a cin (cuando pides la opción de si convertir de moneda extranjera a pesos y viceversa), el búfer manejado por cin se queda con datos "basura". Entonces, tendrías que limpiar esa basura después de llamar a cin. Hay diversas formas, pero yo te digo esta:

Código (cpp) [Seleccionar] Expandir

std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

Esa línea de código limpiaría el búfer. Para usarla tienes que incluir <limits>

[gleon_01]

hola!

gracias por el aporte, lo voy a considerar para ver que tal.

saludos!

[gleon_01]

Hola!

ya di con la solución, parece que tenia algunos system ("cls") de más, por eso se bugeaba el programa.

pongo el código por si le es útil a alguien, gracias por el aporte: class_OpenGL

tiene diferente orden al posteado por primera vez, es para hacerlo más practico, y de una vez modifique algunas cositas para hacerlo más sencillo de manejar.

Código (cpp) [Seleccionar] Expandir

/* conversor de monedas */

#include <iostream>
#include <stdlib.h>
#include <stdio.h>
#include <conio.h>


#define dolar1 18.34
#define euro1 19.40
#define yen1 0.1620
#define quetzal1 0.4218
#define libra1 25.34

#define pesos3 18.34
#define pesos4 19.40
#define pesos5 0.1620
#define pesos6 0.4218
#define pesos7 25.34

/* funciones para convertir de pesos a otra_moneda */
void dolar();
void peso();
void euro();
void yen();
void quetzal();
void libra();

/* funciones para convertir otra_moneda a pesos */
void dolar2();
void peso2();
void euro2();
void yen2();
void quetzal2();
void libra2();


//funciones de voida

void dolar() /*conversion de pesos a dolar*/
{  
using namespace std;
double npesos, dolar, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
dolar=pesos*dolar1;
system("cls");
cout<<"Usted tiene: "<<dolar<<" Dolares \n \n"<<endl;

}

void euro() /*conversion de pesos a euros*/
{
using namespace std;
double npesos, euro, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
euro=pesos*euro1;
system("cls");
cout<<"Usted tiene: "<<euro<<" Euros \n \n"<<endl;

}

void yen() /*conversion de pesos a yen*/
{
using namespace std;
double npesos, yen, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
yen=pesos*yen1;
system("cls");
cout<<"Usted tiene: "<<yen<<" yenes \n \n"<<endl;

}

void quetzal() /*conversion de pesos a quetzal*/
{
using namespace std;
double npesos, quetzal, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
quetzal=pesos*quetzal1;
system("cls");
cout<<"Usted tiene: "<<quetzal<<" quetzales \n \n"<<endl;

}

void libra() /*conversion de pesos a libras*/
{
using namespace std;
double npesos, libra, pesos;
cout<<"Ingrese la cantidad en Pesos"<<endl;
cin>>pesos;
libra=pesos*libra1;
system("cls");
cout<<"Usted tiene: "<<libra<<" libras \n \n"<<endl;

}

/* para el caso 2, convertir de moneda extranjera a pesos */

void dolar2() /*conversion de dolar a pesos*/
{  
using namespace std;
double npesos, dolar, pesos;
cout<<"Ingrese la cantidad en Dolares"<<endl;
cin>>dolar;
pesos=dolar/pesos3;
system("cls");
cout<<"Usted tiene: "<<pesos<<" Pesos \n \n"<<endl;

}

void euro2() /*conversion de Euros a Pesos*/
{
using namespace std;
double npesos, euro, pesos;
cout<<"Ingrese la cantidad en Euros"<<endl;
cin>>euro;
pesos=euro*pesos4;
system("cls");
cout<<"Usted tiene: "<<pesos<<" Peros \n \n"<<endl;

}

void yen2() /*conversion de Yenes a Pesos*/
{
using namespace std;
double npesos, yen, pesos;
cout<<"Ingrese la cantidad en Yenes"<<endl;
cin>>yen;
pesos=yen/pesos5;
system("cls");
cout<<"Usted tiene: "<<pesos<<" Pesos \n \n"<<endl;

}

void quetzal2() /*conversion de Quetzales a Pesos*/
{
using namespace std;
double npesos, quetzal, pesos;
cout<<"Ingrese la cantidad en Quetzales"<<endl;
cin>>quetzal;
pesos=quetzal*pesos6;
system("cls");
cout<<"Usted tiene: "<<quetzal<<" quetzales \n \n"<<endl;

}

void libra2() /*conversion de Libras a pesos*/
{
using namespace std;
double npesos, libra, pesos;
cout<<"Ingrese la cantidad en Libras"<<endl;
cin>>libra;
pesos=libra/pesos7;
system("cls");
cout<<"Usted tiene: "<<pesos<<" Pesos \n \n"<<endl;

}



int main()
{

int opc;
char letra;

do{ /* estrcutura repetitiva, cuando hace esto, sigue funcionando */


/* primer menu de opciones */
int opc, opcion, numero;

using namespace std;
cout<<"Conversor de Monedas ACME"<<endl;
cout<<"Seleccione la opcion deseada"<<endl;
cout<<"a. para convertir de pesos a moneda extranjera"<<endl;
cout<<"b. para convertir de moneda extranjera a pesos"<<endl;
cout<<"q. para salir"<<endl;
cin>>letra;
system("cls");



    /* inicia el anidamiento */
   
switch (letra)
{ /* primera opción de pesos a moneda extranjera */
case 'A':
case 'a':
cout<<"Ha seleccionado convertir Pesos a Moneda Extranjera"<<endl;
cout<<"Seleccione la moneda extranjera a la cual desea convertir los pesos"<<endl;
cout<<"1.- Dolar\n2.- Euro\n3.- Yen\n4.- Quetzal\n5.- Libra\n"<<endl;
cin>>opcion;
system("cls");

if (opcion==1)
{dolar();
}

if (opcion==2)
{euro();
}

if (opcion==3)
{yen();
}

if (opcion==4)
{quetzal();
}

if (opcion==5)
{libra();
}
break;


case 'B':
case 'b': /* segunda opción, de moneda extranjera a pesos */

cout<<"Ha seleccionado convertir Moneda Extranjera a Pesos"<<endl;
cout<<"Seleccione la Moneda Extranjera a convertir a Pesos "<<endl;
cout<<"1.- Dolar\n2.- Euro\n3.- Yen\n4.- Quetzal\n5.- Libra\n"<<endl;
cin>>opcion;
system("cls");

if (opcion==1)
{dolar2();
}

if (opcion==2)
{euro2();
}

if (opcion==3)
{yen2();
}

if (opcion==4)
{quetzal2();
}

if (opcion==5)
{libra2();
}
break;

case 'Q':
case 'q':  /* tercera opción, salida */
cout<<"\n\nAdios\n"<<endl;
  break;
}
}
while (!((letra == 'Q')||(letra == 'q')));

}