Diskuze: Měnová kalkulačka V1

Naučíme tě pracovat na home-office.

Zjistit více...

V předchozím kvízu, Online test znalostí C++, jsme si ověřili nabyté zkušenosti z kurzu.

Aktivity

Hacker Standa

Člen

Hacker Standa:9.11.2017 17:20

Dobrý dej, rád bych představil můj první program a chtěl bych se zeptat na názor případně jak program vylepšit nebo co dál dělat abych se posunul.Nejlépe něco k využití.
http://leteckaposta.cz/679417498

Odpovědět

9.11.2017 17:20

Erik Šťastný

Člen

Erik Šťastný:9.11.2017 17:32

Bez zdrojáků ti k tomu moc lidí na programátorské stránce neřekne, nicméně chtělo by to obohatit o aktuální kurz který si budeš stahovat z kurzu ČNB Jestli to na tebe není zatím moc těžké, je to asi hlavní podstatné vylepšení pro měnovou kalkulačku.

Nahoru Odpovědět

9.11.2017 17:32

Hacker Standa

Člen

Hacker Standa:9.11.2017 19:57

#include <iostream>

using namespace std;

int main()
{
cout << "Vítejte v menove kalkulacce" << endl;
cout << "Zadejte menu:" << endl;
cout << "1 - koruna" << endl;
cout << "2 - euro" << endl;
cout << "3 - dolar" << endl;
int volba;
cin >> volba;
cout << "Zadejte pocet:" << endl;
int a;
cin >> a;
cout << "Zadejte druhou menu:" << endl;
cout << "1 - koruna" << endl;
cout << "2 - euro" << endl;
cout << "3 - dolar" << endl;
int volba2;
cin >> volba2;

float vysledek = 0;
if ((volba == 1) && (volba2 == 1)){
vysledek = a;
}
else if ((volba == 2) && (volba2 == 2)){
vysledek = a;
}
else if ((volba == 3) && (volba2 == 3)){
vysledek = a;
}
else if ((volba == 1) && (volba2 == 2)){
vysledek = a / 25.5504132;
}
else if ((volba == 1) && (volba2 == 3)){
vysledek = a * 0.045375;
}

else if ((volba == 2) && (volba2 == 1)){
vysledek = a * 25.5504132;
}

else if ((volba == 2) && (volba2 == 3)){
vysledek = a * 1.15935;}

else if ((volba == 3) && (volba2 == 1)){
vysledek = a * 22.0385675;}

else if ((volba == 3) && (volba2 == 2)){
vysledek = a * 0.862552292;}

cout << vysledek;
cout << "\n" << endl;
main();

}

Nahoru Odpovědět

9.11.2017 19:57

Erik Šťastný

Člen

Erik Šťastný:9.11.2017 21:22

Používej tag pro kód, hrozně se to čte

Máš tam několik zbytečně složitých podmínek, začal bych tím jak to zjednodušit.

Nahoru Odpovědět

9.11.2017 21:22

dez1nd

Člen

dez1nd:10.11.2017 8:20

První tři podmínky se dají spojit do jedné

if (volba == volba2){
vysledek = a;
}

u měny bych nepoužíval slovo počet, ale spíš částka.
kurzy bych uložil předem do proměnné, budou se ti lépe měnit z jednoho místa než na 6ti různých místech.

Nahoru Odpovědět

10.11.2017 8:20

David Novák

Tvůrce

David Novák:10.11.2017 10:51

Překlop tu if hrůzu aspoň do switche.. a ve volba == 1 použij konstanty/enum namísto běžných čísel (takhle je to strašně nepřehledné)..

Příklad s konstantami:

#define KORUNA 1
#define EURO 2
#define DOLAR 3

...

if (volba == KORUNA ...

Další problém je, že nekontroluješ vstupy.. Co když uživatel zadá 4?

Nahoru Odpovědět

10.11.2017 10:51

Chyba je mezi klávesnicí a židlí.

Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 6 zpráv z 6.