Programa de divisas

[M-Stewart612]

Sucede que estoy realizando un programa de Divisas, el cual recibe la información de la cantidad a convertir, de diferentes monedas. Después debe imprimir el reporte total que obtuvo por cada moneda; pero sucede que el reporte final no lo imprime, solo da el pantallaso y regresa a la primera parte

¿Qué debo hacer para qué se imprima la información?

Código (cpp) [Seleccionar] Expandir

[/#include <iostream>
#include<stdlib.h>
#include<stdio.h>
#include <windows.h>
using namespace std;

/*USD= 7.72, EUR= 8.80, CAD= 5.89, GBP= 10.03, CHF= 7.71, AUD=5.58 , NZD= 5.13*/   

int main(){
string reporte,x;
int opcion, monto, i;

string mon[7];
double moneda[7],total;
total=0;
mon [0]="Dolar";
mon [1]="Euro";
mon [2]="Dolar Canadiense";
mon [3]="Libra";
mon [4]="Franco Suizo";
mon [5]="Dolar Australiano";
mon [6]="Dolar Neozelandes";
for(i=0;i<7;i++){
    moneda[i]=0;
   }
x="si";
   do{
    system("cls");
    total=0;
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
cout<<"\t \t \t  HAS INGRESADO AL REGISTRO DE DIVISAS";
cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n"; cout<<"\n";
    cout<<"Opciones: \n  \t 1. Dolar \n \t 2. Euro \n \t 3. Dolar Canadiense \n  \t 4. Libra \n \t 5. Franco Suizo \n \t 6. Dolar Australiano \n \t 7. Dolar Neozelandes"<<endl;
    cout<<"Ingrese el numero de la opcion que desea: "<<endl; cin>>opcion;
    switch (opcion){
    case 1:
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    cout<<"Opcion selecionada: DOLAR"<<endl;
    cout<<"Ingrese el monto que desea convertir: "; cin>>monto;
    moneda[0]=moneda[0]+monto;
    break;
cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";   
    case 2:
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    cout<<"Opcion selecionada: EURO"<<endl;
    cout<<"Ingrese el monto que desea convertir: "; cin>>monto;
    moneda[1]=moneda[1]+monto;
    break;
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    case 3:
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    cout<<"Opcion selecionada: DOLAR CANADIENSE"<<endl;
    cout<<"Ingrese el monto que desea convertir: "; cin>>monto;
    moneda[2]=moneda[2]+monto;
    break;
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    case 4:
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    cout<<"Opcion selecionada: LIBRA"<<endl;
    cout<<"Ingrese el monto que desea convertir: "; cin>>monto;
    moneda[3]=moneda[3]+monto;
    break;
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    case 5:
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    cout<<"Opcion selecionada: FRANCO SUIZO"<<endl;
    cout<<"Ingrese el monto que desea convertir: "; cin>>monto;
    moneda[4]=moneda[4]+monto;
    break;
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    case 6:
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    cout<<"Opcion selecionada: DOLAR AUSTRALIANO"<<endl;
    cout<<"Ingrese el monto que desea convertir: ";
    cin>>monto;
    moneda[5]=moneda[5]+monto;
    break;
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    case 7:
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    cout<<"Opcion selecionada: DOLAR NEOZELANDES"<<endl;
    cout<<"Ingrese el monto que desea convertir: ";
    cin>>monto;
    moneda[6]=moneda[6]+monto;
    break;
    cout<<endl<<"|------------------------------------------------------------------------------------------------|"; cout<<"\n";
    case 8 :{
          x="no";
    break;
    }
    default:
    cout<<"Por favor elija una opcion valida ";
    break;
   }
    cout<<"Desea ver el reporte final (si/no)";
    cin>>reporte;
    if (reporte=="si"){
   
/*IMPRIMIR EL REPORTE DE TRANSACCIONES*/

    for(i=0;i<7;i++){
    cout<<"moneda "<<mon[i]<<endl;
    cout<<"cantidad "<<moneda[i]<<endl;
    if (i==0){
    cout<<"cantidad en Quetzales "<<moneda[i]*7.72<<endl;
    total=total+(moneda[i]*7.72);
    }else if (i==1){
    cout<<"cantidad en Quetzales "<<moneda[i]*8.8<<endl;
    total=total+(moneda[i]*8.8);
    } else if (i==2){
    cout<<"cantidad en Quetzales "<<moneda[i]*5.89<<endl;
    total=total+(moneda[i]*5.89);
    }else if (i==3){
    cout<<"cantidad en Quetzales "<<moneda[i]*10.03<<endl;
    total=total+(moneda[i]*10.03);
    }else if (i==4){
    cout<<"cantidad en Quetzales "<<moneda[i]*7.7<<endl;
    total=total+(moneda[i]*7.7);
    }else if (i==5){
    cout<<"cantidad en Quetzales "<<moneda[i]*5.55<<endl;
    total=total+(moneda[i]*5.55);
    }else if (i==6){
    cout<<"cantidad en Quetzales "<<moneda[i]*5.13<<endl;
    total=total+(moneda[i]*5.13);
    }
   }
    cout<<"Total Invertido en Quetzales : "<<total<<endl;
   }
   }while(x=="si");
  }]

[elgilun]

Puedo comentarte un par de cosas que veo.

1) Deberías poder usar un debugger, que es una herramienta (normalmente disponible desde el mismo editor de código o IDE). Usando un debugger podrás poner "puntos de parada", o como se llamen en tu pueblo, y seguir el flujo de ejecución del código, paso a paso, con lo que podrás ver claramente qué es lo que hace que el programa se comporte como se comporta. Debugger, todo el mundo los usa. No puedes llamarte programador si no los usas. No hay excusas.

2)

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

O tu programa es C++ (<iostream>) o es C (<stdlib.h>); no puede ser las dos cosas. No puede, no, nunca. ¿Te han dicho que no sé qué sobre C/C++? Bueno, no, es mentira, o C o C++, decídete.

3)

/*USD= 7.72, EUR= 8.80, CAD= 5.89, GBP= 10.03, CHF= 7.71, AUD=5.58 , NZD= 5.13*/

Si tienes estas cosas ya definidas desde el principio, podrías declararlos como macros (lenguaje C) o como constantes (C++), pero recuerda decidirte primero si tu programa va a ser C o C++.

4) defines primero

double moneda[7]

y luego usas mon[0], mon[1], ... ? (¿necesitas una secretaria para esas tareas menores?)

5) if ... else if... else if... else... con pruebas de condición sobre enteros, debes usar switch / case.

Creo que ya corrigiendo estas cositas ya quedará.

[AlbertoBSD]

5) if ... else if... else if... else... con pruebas de condición sobre enteros, debes usar switch / case.

Estaba por comentarle cosas similares, pero me ganaste.

respecto al codigo tiene fallas e ineficiencias por ejemplo el siguiente codigo hace varias comprofaciones inecesarias.

Código [Seleccionar] Expandir


  for(i=0;i<7;i++){
  cout<<"moneda "<<mon[i]<<endl;
  cout<<"cantidad "<<moneda[i]<<endl;
  if (i==0){
  cout<<"cantidad en Quetzales "<<moneda[i]*7.72<<endl;
  total=total+(moneda[i]*7.72);
  }else if (i==1){
  cout<<"cantidad en Quetzales "<<moneda[i]*8.8<<endl;
  total=total+(moneda[i]*8.8);
  } else if (i==2){
  cout<<"cantidad en Quetzales "<<moneda[i]*5.89<<endl;
  total=total+(moneda[i]*5.89);
  }else if (i==3){
  cout<<"cantidad en Quetzales "<<moneda[i]*10.03<<endl;
  total=total+(moneda[i]*10.03);
  }else if (i==4){
  cout<<"cantidad en Quetzales "<<moneda[i]*7.7<<endl;
  total=total+(moneda[i]*7.7);
  }else if (i==5){
  cout<<"cantidad en Quetzales "<<moneda[i]*5.55<<endl;
  total=total+(moneda[i]*5.55);
  }else if (i==6){
  cout<<"cantidad en Quetzales "<<moneda[i]*5.13<<endl;
  total=total+(moneda[i]*5.13);
  }
  }


Siendo que todo eso esta dentro de un for realmente i solo puede tener un valor a la vez por lo cual podrias poner TODO FUERA DEL FOR y ahorrarte las evaluaciones de la siguiente manera.

Código [Seleccionar] Expandir


cout<<"moneda "<<mon[0]<<endl;
cout<<"cantidad "<<moneda[0]<<endl;
cout<<"cantidad en Quetzales "<<moneda[0]*7.72<<endl;
total=total+(moneda[0]*7.72);

cout<<"moneda "<<mon[1]<<endl;
cout<<"cantidad "<<moneda[1]<<endl;
cout<<"cantidad en Quetzales "<<moneda[1]*8.8<<endl;
total=total+(moneda[1]*8.8);


cout<<"moneda "<<mon[2]<<endl;
cout<<"cantidad "<<moneda[2]<<endl;
cout<<"cantidad en Quetzales "<<moneda[2]*5.89<<endl;
total=total+(moneda[2]*5.89);

cout<<"moneda "<<mon[3]<<endl;
cout<<"cantidad "<<moneda[3]<<endl;
cout<<"cantidad en Quetzales "<<moneda[3]*10.03<<endl;
total=total+(moneda[3]*10.03);

cout<<"moneda "<<mon[4]<<endl;
cout<<"cantidad "<<moneda[4]<<endl;
cout<<"cantidad en Quetzales "<<moneda[4]*7.7<<endl;
total=total+(moneda[4]*7.7);

cout<<"moneda "<<mon[5]<<endl;
cout<<"cantidad "<<moneda[5]<<endl;
cout<<"cantidad en Quetzales "<<moneda[5]*5.55<<endl;
total=total+(moneda[5]*5.55);

cout<<"moneda "<<mon[6]<<endl;
cout<<"cantidad "<<moneda[6]<<endl;
cout<<"cantidad en Quetzales "<<moneda[6]*5.13<<endl;
total=total+(moneda[6]*5.13);


O dentro del FOR pero ahorrandote los IF de igualmanera

Código (c) [Seleccionar] Expandir


float fix[7] = {7.72,8.80,5.89,10.03,7.71,5.58,5.13};

for(i=0;i<7;i++){
cout<<"moneda "<<mon[i]<<endl;
cout<<"cantidad "<<moneda[i]<<endl;
cout<<"cantidad en Quetzales "<<moneda[i]*fix[i]<<endl;
total=total+(moneda[i]*fix[i]);
}


Este ultimo es el mas eficiente a mi manera de ver las cosas, sobre el por que no los imprime seria cuestion de que me expliques como  es que funciona esta parte del codigo. Yo se como funciona, pero si no entienda esa parte no sabras realmente que estas haciendo

Código [Seleccionar] Expandir


if (reporte=="si"){


[M-Stewart612]

Muchísimas gracias a ambos, me sirvieron mucho sus observaciones para reparar los errores del programa