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


Zobrazeno 10 zpráv z 10.
K personalizaci obsahu a reklam, poskytování funkcí sociálních médií a analýze naší návštěvnosti využíváme soubory cookie. Informace o tom, jak náš web používáte, sdílíme se svými partnery pro sociální média, inzerci a analýzy. Partneři tyto údaje mohou zkombinovat s dalšími informacemi, které jste jim poskytli nebo které získali v důsledku toho, že používáte jejich služby.
Používáme nezbytné cookies pro fungování webu a s tvým souhlasem také analytické a marketingové cookies.
Zajišťují základní funkce, bezpečnost a služby, které sis vyžádal. Nelze je vypnout.
| Služba | Poskytovatel | Účel | Cookies a úložiště | Doba uložení |
|---|---|---|---|---|
| ITnetwork | ITnetwork | Provoz webu, relace, přihlášení a uložení nastavení cookies. | PHPSESSID, auth_token, sid, itn_consent_impression, __Host-itn_consent | Relace až 1 rok |
| Google Tag Manager | Správa značek a načítání měřicích nástrojů webu. | Žádné | Neukládá se | |
| Google Fonts | Načítání typografie webu. | Úložiště řízené poskytovatelem | Dle podmínek poskytovatele | |
| Google Hosted Libraries | Načítání potřebných knihoven a stylů webu. | Úložiště řízené poskytovatelem | Dle podmínek poskytovatele | |
| Google reCAPTCHA | Ochrana formulářů a webu před zneužitím. | _GRECAPTCHA, rc::a, rc::b, rc::c, rc::f | Relace až 180 dní | |
| YouTube | Přehrávání vloženého video obsahu. | localStorage, IndexedDB; cookies after playback interaction | Relace až trvalé úložiště | |
| Vimeo | Vimeo | Přehrávání vloženého video obsahu. | __cf_bm, _cfuvid, vuid, localStorage, IndexedDB | Relace až 2 roky |
| Facebook Login | Meta | Přihlášení pomocí účtu třetí strany. | Úložiště řízené poskytovatelem | Relace až 1 rok |
| GoPay | GoPay | Zpracování uživatelem vyžádané platby. | Úložiště řízené poskytovatelem | Dle podmínek poskytovatele |
Pomáhají nám porozumět používání webu a zlepšovat ho.
| Služba | Poskytovatel | Účel | Cookies a úložiště | Doba uložení |
|---|---|---|---|---|
| Google Analytics 4 | Měření návštěvnosti a používání webu. | _ga, _ga_* | Až 2 roky | |
| Microsoft Clarity | Microsoft | Měření návštěvnosti a používání webu. | _clck, _clsk, _cltk | Relace až 1 rok |
Slouží k měření kampaní, personalizaci reklamy a marketingové komunikaci.
| Služba | Poskytovatel | Účel | Cookies a úložiště | Doba uložení |
|---|---|---|---|---|
| Google Ads | Měření kampaní, reklama a remarketing. | _gcl_au, _gcl_ls | Relace až 90 dní | |
| Meta Pixel | Meta | Měření kampaní, reklama a remarketing. | _fbp, _fbc, localStorage | Až 90 dní |
| Sklik | Seznam.cz | Měření kampaní, reklama a remarketing. | retargeting, sid, szn:* | Relace až trvalé úložiště |
| LinkedIn Insight | Měření kampaní, reklama a remarketing. | bcookie, li_gc, lidc, __cf_bm | Relace až 1 rok | |
| Ecomail | Ecomail.cz | Měření kampaní, reklama a remarketing. | ecmid, Úložiště řízené poskytovatelem | Dle podmínek poskytovatele |
| Atribuce kampaní ITnetwork | ITnetwork | Přiřazení návštěvy a objednávky ke kampani. | campaign_clid[*], user_session_context | Až 1 rok |
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.