A zászlóváltozók abszolút rosszak-e? [zárt]

Hatalmas funkcióblokk a szaga , nem a zászlók. Ha az 5. sorban állítja be a jelzőt, akkor csak a 354. sorban ellenőrizze, hogy ez rossz-e. Ha rosszul állítja be a 8. sort, és a 10. sorban található, akkor ez rendben van. Ezenkívül egy kódblokkonként egy vagy két zászló rendben van, a függvényekben 300 zászló rossz.

Általában megjelöl teljesen helyettesíthető a stratégia mintájának valamilyen ízével, egy stratégia megvalósításával a zászló minden lehetséges értékénél. Ez sokkal könnyebbé teszi az új viselkedés hozzáadását.

Teljesítménykritikus helyzetekben az indirekció költsége esetleg felszínre kerül, és egyértelmű zászlókká teszi a dekonstrukciót. Ennek ellenére nehezen emlékszem egyetlen esetre, amikor ezt valóban meg kellett tennem.

Nem, A zászlók nem rosszak vagy gonoszok, amelyeket mindenáron el kell távolítani.

Fontolja meg a Java “s Pattern.compile (String regex, int zászlók) hívás. Ez egy hagyományos bitmaszk és működik. Pillantás a java konstansaira , és bárhol is látható egy csomó 2 ⁿ , tudod, hogy vannak zászlók.

Ideális felújított világban ehelyett egy EnumSet használna, ahol az állandók ehelyett értékek egy enumban, és ahogy a dokumentáció olvasható:

Ennek az osztálynak a tér- és időteljesítményének elég jónak kell lennie ahhoz, hogy lehetővé tegye a hagyományos int-alapú “bit zászlók” magas színvonalú, típusbiztonságos alternatívájának használatát.

Egy tökéletes világban ez a Pattern.compile hívás Pattern.compile(String regex, EnumSet<PatternFlagEnum> flags) lesz.

Mind azt mondta, még mindig zászlók.Sokkal könnyebb dolgozni a Pattern.compile("foo", Pattern.CASE_INSENSTIVE | Pattern.MULTILINE) vel, mint Pattern.compile("foo", new PatternFlags().caseInsenstive().multiline()) vagy valamilyen más stílussal, amikor megpróbáljuk megtenni azt, ami a zászlók valójában, és jó.

A zászlók gyakran láthatók, amikor rendszerszintű dolgokkal dolgoznak. Ha valamihez kapcsolódik az operációs rendszer szintjén, akkor valószínűleg van valahol egy jelző – legyen az egy folyamat visszatérési értéke, vagy egy fájl engedélyei, vagy egy socket megnyitásának zászlói. Ha megpróbálja visszafejteni ezeket az eseteket egy boszorkányüldözés során egy észlelt kódszag ellen, akkor valószínűleg rosszabb a kód, mint ha elfogadnák és megértenék a zászlót.

A probléma akkor fordul elő, amikor az emberek visszaélnek a zászlókkal, ha egymáshoz dobják őket, és frankenflag halmaz létrehozása mindenféle nem kapcsolódó zászlóból, vagy megpróbálja használni őket ott, ahol egyáltalán nem jelennek meg.

Feltételezem, hogy a metódus aláírásokon belüli zászlókról beszélünk.

Egyetlen zászló használata elég rossz.

Ez semmit sem jelent kollégáinak abban az időben, amikor látják. Meg kell nézniük a módszer forráskódját annak megállapítása érdekében, hogy mit csinál. Valószínűleg ugyanabban a helyzetben leszel néhány hónappal a sorban, amikor elfelejted, hogy mi a módszered.

A zászló átadása a metódusnak általában azt jelenti, hogy a metódus több dologért felelős. A módszer belsejében valószínűleg egy egyszerű ellenőrzést végez a következő sorokon:

if (flag) DoFlagSet(); else DoFlagNotSet(); 

Ez az aggodalmak gyenge elkülönítése, és általában megtalálhatja a módját.

Általában két külön módszerem van:

public void DoFlagSet() { } public void DoFlagNotSet() { } 

Ennek több értelme lesz az Ön által megoldott problémára alkalmazható metódusokkal.

Több zászló átadása kétszer olyan rossz. Ha valóban több zászlót kell átadnia, akkor fontolja meg, hogy ezeket egy osztályba foglalja. Akkor is ugyanazzal a problémával fog szembesülni, mivel a módszer valószínűleg több dolgot is végrehajt.

Jelzők és a legtöbb temp változó erős szagú. A legvalószínűbb, hogy ezeket át lehet alakítani és lekérdezési módszerekkel helyettesíteni.

Felülvizsgált:

Az állapot kifejezésekor a jelzőket és a temp változókat át kell dolgozni a lekérdezési módszerekre. Az állapotértékeket (logikai értékeket, intseket és más primitíveket) szinte mindig el kell rejteni a megvalósítás részleteinek részeként.

A vezérléshez, az útválasztáshoz és az általános programfolyamathoz használt jelölők szintén jelezhetik lehetőség a vezérlő struktúrák egyes szakaszainak átalakítására külön stratégiákba vagy gyárakba, vagy bármilyen helyzeti helyzetbe, amelyek továbbra is használják a lekérdezési módszereket.

A kérdésből azt feltételeztem, hogy a minőségbiztosítás zászlós (globális) változókat jelent, és nem egy függvényparaméter bitjeit.

Vannak helyzetek ahol nincs sok más lehetősége. Például operációs rendszer nélkül ki kell értékelnie a megszakításokat.Ha egy megszakítás nagyon gyakran bekövetkezik, és nincs ideje hosszas értékelésre az ISR-ben, akkor nem csak megengedett, de néha a legjobb gyakorlat is, hogy csak néhány globális zászlót állítson be az ISR-be (minél kevesebb időt töltsön el) az ISR-ben), és értékelje ezeket a zászlókat a fő hurokban.

Nem hiszem, hogy bármi abszolút rossz a programozásban, valaha.

Van még egy olyan helyzet, amikor a zászlók rendben lehetnek, ami még nem kerültek itt említésre …

Fontolja meg a lezárások használatát ebben a Javascript-részletben:

exports.isPostDraft = function ( post, draftTag ) { var isDraft = false; if (post.tags) post.tags.forEach(function(tag){ if (tag === draftTag) isDraft = true; }); return isDraft; } 

A belső funkció átadásra kerül az “Array.forEach” fájlra, nem lehet egyszerűen “true” értékre térni.

Ezért az államot kint kell tartani egy zászlóval.