NOVINKA: Kurz kybernetické bezpečnosti s akreditací MŠMT, nyní již od 0 Kč. Staň se žádaným profesionálem. Zjisti více:
Avatar
Hacker Standa
Člen
Avatar
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
Avatar
Erik Šťastný
Člen
Avatar
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
Avatar
Hacker Standa
Člen
Avatar
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
Avatar
Erik Šťastný
Člen
Avatar
Odpovídá na Hacker Standa
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
Avatar
dez1nd
Člen
Avatar
dez1nd:10.11.2017 8:20
  1. První tři podmínky se dají spojit do jedné
if (volba == volba2){
vysledek = a;
}
  1. u měny bych nepoužíval slovo počet, ale spíš částka.
  2. 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
Avatar
David Novák
Tvůrce
Avatar
Odpovídá na Hacker Standa
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.