Lepšie nedefinovať takto štruktúry v C.

4.8.2020 21:46 | C

#include <stdio.h>

static struct {
	int day;
	int month;
	int year;
} date = {
	4, 8, 2020
};

int main(){
	printf("year: %d\n", date.year);
	return 0;
}

       

Komentáře

[Michal Kubeček]

Asi by měla následovat anketa, co tím autor vlastně myslel. :-)

• nekombinovat deklaraci struktury jako typu s deklarací proměnné toho typu
• nepoužívat anonymní initializátory prvků struktury
• nepoužívat signed proměnné pro hodnoty, které jsou z definice unsigned
• místo lidové tvořivosti používat existující knihovní datové typy
• …

[samalama]

ci problem nebude prave v tom, ze autor nemysli...

[Žufánek]

nepoužívat signed proměnné pro hodnoty, které jsou z definice unsigned

Tak tady to fakt hrozně vadí, přetečení iminentní.

[Michal Kubeček]

Ono nemusí být iminentní. Schválně si zkuste bez hledání po webu nebo nahlédnutí do specifikace odpovědět, jak přesně se chová "X < Y" nebo "A ? X : Y", je-li X signed a Y unsigned nebo naopak, případně je-li to ještě zpestřeno různou velikostí těch proměnných. Podobně bohorovný přístup často končí bezpečnostními chybami.

[xxx]

No nevim. Spis jsou to takova tri stadia.

Programator:

1. pouziva vsude int

2. mixuje signed/unsigned int ruznych velikosti

3. po mnoha chybach zpusobenych s/u int ruznych velikosti a nastudovani, jak to funguej, opet pouziva pouze int

[Michal Kubeček]

Do fáze 3 se ale naštěstí nedostanou všichni.

[z_sk]

nekombinovat deklaraci struktury jako typu s deklarací proměnné toho typu

👍

[MakeIranBombedAgain❗]

všecko co se jako zkompiluje je správně :D :D ;D ;D

[Heron]

Mě nejvíc dráždí to, že jsou to inty a že je to inicializované přímo. Datum nejsou tři inty. Datum je složitější datová struktura a ne všechny hodnoty těchto tří polí tvoří validní datum. Ve skutečnosti drtivá většina hodnot není validní datum.

[Martin Tůma]

A jak by sis jako v C představoval datovou strukturu, která by sama o sobě řešila validitu dat? Jako jednoduchá datová reprezentace datumu je to naprosto ok, pro "bezpečnou" práci s tou strukturou pak můžeš mít nadefinované funkce, které dělají potřebné kontroly.

Co se týče té "přímé inicializace", tak tohle se v C zcela běžně záměrně používá, pokuď se definovaná struktura (běžně se spíš jedná o pole struktur) použije v daném modulu přesně 1x.

[Michal Kubeček]

"Zcela běžně se používá" spousta věcí, které bych za hodné následování neoznačil ani omylem.

[Martin Tůma]

Měl jsem na mysli "běžně používá v nějaké situaci". Přirovnal bych to k použití GOTO. To taky obecně není úplně doporučovaný pattern, ale jako error handling je to zcela běžná (a preferovaná) technika. Samozřejmě, někdo to může vidět jinak, ostatně stejně jako u toho GOTO...

[Odin1918]

Nejlepsi by bylo vykaslat se na 50 let stary jazyk a pouzit treba neco objektoveho mladsi 30ti let.