Diskuze: Kritika kódu
V předchozím kvízu, Online test znalostí C++, jsme si ověřili nabyté zkušenosti z kurzu.
Tvůrce
Zobrazeno 10 zpráv z 10.
//= Settings::TRACKING_CODE_B ?> //= Settings::TRACKING_CODE ?>
V předchozím kvízu, Online test znalostí C++, jsme si ověřili nabyté zkušenosti z kurzu.
Rozhodně lepší než můj. Dle mého názoru je to vypadá docela k světu.
Používáš tam několikrát typ std::array<int, 12>, možná by stálo za to si jej speciálně pojmenovat.
Do funkce vycetka předáváš druhý parametr hodnotou, ale asi by byla lepší konstantní reference (const array<int, 12> &). Ušetříš tak jedno kopírování toho pole. samozřejmě za předpokladu, že se jedná o pole konstant.
Jestli to chápu dobře, tak program nebude zrovna nadšený, když místo částky zadáš něco, co číslem třeba vůbec není. Měl bys ověrovat, že uživatel zadal správný vstup předtím, než tento vstup použiješ.
Víc mě k tomu při rychlém pohledu nenapadá (resp. asi ano, ale jedná se o hodně subjektivní věci, které souvisí s mým stylem programování).
Jinak pokud chceš, aby program byl rychlý, jakože opravdu při svém výpočtu využíval na maximum procesor a jiná zařízení, s vysokou pravděpodobností nebude hezky vypadat.
Mně osobně tam na první pohled nejvíc vadí ty globální proměnné.
Mně osobně tam na první pohled nejvíc vadí ty globální proměnné.
Je fakt, že by všechny mohly být lokální (pokud už by musely existovat), ale ten program mi přišel tak malý, že jsem se na toto vůbec nedíval.
Zeptám se laicky, Co znamená atoi ma 25 řádku? V knížkách jsem to neviděl
atoi je Cčková funkce, která převádí ANSI řetězec (char *) na celé číslo. Její název lze považovat za zkratku z "ANSI TO Integer".
Dokumentace k ní:
http://www.cplusplus.com/…stdlib/atoi/
Pretože
#include "stdafx.h"
includuje štandartné knižnice
Zobrazeno 10 zpráv z 10.