Zjemnění jmenných prostorů

+1

Zkusil jsem tam něco upravit.

Poznámky pro editory:

• Asi bude rozumné, když při přesunu zachováme na stránce třídám původní namespace. Tak bude poznat, co kam patřilo.
• Třídy i jmenné prostory jsou řazeny podle abecedy.
• Kontroverzní návrhy bude asi dobré nejdřív prodiskutovat na fóru.

V momentě, kdy se použije use, už na délce namespace přece tolik nezáleží.

Nilp napsal(a):

V momentě, kdy se použije use, už na délce namespace přece tolik nezáleží.

Otazka je proc vubec mit namespace (jemne deleny), kdyz se skrz „use“ okamzite odstrani na prvnim radku zdrojaku. Hrube deleny „Nette\“ lze pochopit jako obranu proti kolizim s jinou knihovnou ktera by mohla definovat stejne tridy, ale jakekoli dalsi deleni mi pak nedava smysl.

Kdyz se use nepouzije a pisou se NS vsude, pak zase na delce NS myslim zalezi, obecne plati ze kratsi kod se lip cte. Dobre pojmenovane NS muzou trochu prispet k citelnosti, ale vetsinu casu to mne osobne prijde jenom jako accident zbytecnost.

Honza Marek napsal(a):

Já vidím jmenné prostory jako docela příjemnou bariéru, která zajišťuje to, že při programování něčeho v jednom jmenném prostoru se absolutně nemusím starat o to, co je okolo.

V tom se jistě všichni shodneme. Jako samostatnou jednotku můžeme brát i celý framework, který může být umístěn v jednom prostoru bez dalšího rozlišení.

Vůbec se mi nezdá, že use ruší jmenné prostory.

Snad si tu nemusíme říkat, co dělá use. Ale chybí mi stále ten argument, v čem je lepší, že v klauzuli use použiju delší řetězec než dosud.

osobne by som navrhoval:

• Nette\Application\RoutingDebugger → Nette\Diagnostics
• Nette\Application\…RequestException → Nette\Web

Tak to teď čtu (je to dlouhé asi to budu muset dát na etapy)

• ArrayTools, …, Callback, String, Tools vs namespace Nette\Tools to mě trochu nesedí
• Nette\Diagnostics i když se tím zabrání vzniku Nette\Debuging\Debug tak pořád se mě Nette\Debuging líbí víc (Symfony\Component\HttpKernel\Debug\ExceptionListener)
• Nette\Bootstrap mě připadá zbytečný
• Nette\Application\Routing a Nette\Application\Responses no konečně
• zrušení Nette\Config je zajímavá myšlenka ale jsem tak 50:50
• Nette\Forms nevím teda jak přesně budou stavěné nové třídy ale taky bych byl pro oddělení jednotlivých prvků do samostatného namespace. možná pokud bude i více rendererů tak bych je hodil taky do stamostatného
• Nette\Latte vs. Nette\Template pořád si s tím nejsem jistý… :-/
• co takhle přejmenovat Nette\Web na Nette\Http nebo Nette\HttpKernel (Symfony\Component\HttpKernel\HttpKernel)
• neměl by být Presenter, Control a AppForm pod Nette\ComponentModel ?
• RoutingDebugger do Nette\Diagnostics
• PresenterComponentReflection do Nette\Reflection

ad Nette\Core vs Nette\Tools: v jednom by měly být základní knihovní třídy jako je NObject, ve druhém nástroje jako Image nebo Finder. U řady tříd ale sám nemám jasno

ad Nette\Diagnostics: kromě Debug bych sem chtěl zařadit Logger, profiler apod. Obecně jde o nástroje diagnostikující běh webu, nikoliv jen chyby.

ad PresenterComponentReflection, RoutingDebugger, DatabasePanel: tyhle třídy sice rozšiřují třídu z jiného prostoru/balíčku, ale přitom patří do balíčku, ve kterém jsou nyní zařazeny.

ad Presenter, Control a AppForm: tohle by mělo být v nějakém Nette\Application\Něco, ale zatím v tom nemám jasno.

ad Nette\Latte: tohle principiálně funguje i bez šablon

Zkusil jsem ještě přidat jeden návrh, změnu Nette\Web na Nette\Http a odstranění prefixů Http u tříd jako HttpResponse nebo HttpRequest. Pro verzi 5.2 by se zachoval původní název.

Upravené jmenné prostory budou ve větvi namespaces

Co takhle ještě přejmenovat Nette\Web\IUser na Nette\Security\IUser? hodí se i v CLI.

David Grudl napsal(a):

ad Presenter, Control a AppForm: tohle by mělo být v nějakém Nette\Application\Něco, ale zatím v tom nemám jasno.

Nette\Application\UI ?

Interaction zní jako tahat kočku za ocas, to už radši ten Control

PetrP: Ano, na jmenné prostory (aka balíčky aka moduly) jsem pohlížel taky jen jako na způsob, jak se vyhnout jmenným kolizím. Ale to bychom je nevyužili jaksi OOPově – viz princip low coupling) a high cohesion).

Pokud se držíme zásady, že ve FW nejsou dvě třídy se stejným názvem, postrádá podle mě zjemňování NS smysl. Vlastní NS bych vyčlenil pouze na funkční celky, které je ev. opravdu možné používat samostatně jako Database, Forms, Templates. Rozhodně bych nezaváděl jmenné prostory Core, Utils, Iterators, DependencyInjection, Loaders, Localization, Reflection. Tedy proč psát NS, když už je „obsažen“ v názvu třídy – IMO by mělo být buď Nette\Loaders\Robot nebo Nette\RobotLoader.

Souhlasím s paranoiqem. Jediný opodstatněný namespace je \Nette. Zjemňování přidává práci, znepřehledňuje kód a hlavně se blbě pamatuje, jestli je Router v \Nette, \Nette\Routing. Víceménně taky dost pochybuju o tom, že budou mít jemné namespaces v Nette konečnou podobu. Vzhledem k bouřlivému vývoji za poslední dva roky, kdy je Nette veřejné, předpokládám rovněž bouřlivý vývoj dále. Vlastně když si tady čtu sálodlouhé příspěvky na téma: má mít Neon vlastní namespace, připadá mi to už trochu moc akademický a kavárenský kecání. Speciálně u Neonu to přece může bejt vlastní knihovna v libs ne? ;-) Ale to odbočuju. Nezamotávejte se v bludných kruzích namespaces, jeden jediný jmenný prostor přece stačí. Jestli má mít nějaká část vlastní, šup s ním z frameworku ven do vlastní knihovny. Ale abych při psaní řešil v jakym namespace tohodle frameworku to mám hledat? Občas potřebuju rychle upravit jednu dvě věci, přidat drobnej příkaz, ale nehodlám a nechci pátrat, kde je zrovna Nette\Config\Config, jestli se náhodou po akademický diskusi nepřesunul pod \Nette\ConfigurationEasyBaby…jak říká paranoiq, je to stále ještě směr KISS?

Zjemnění jmenných prostorů neznamená vždycky prodloužení názvu třídy. Naopak ten základní název třídy často bývá bez namespaců mnohem delší. Například takovému názvu Nette\Web\HttpUploadedFile moc prospělo, že se změnil na Nette\Http\UploadedFile. Souhlasím s tím, že název jmenného prostoru by se pokud možno neměl opakovat v části názvu třídy. Ale neřešil bych to zrušením jmenného prostoru.

Každopádně už mě ta diskuze o jmenných prostorech taky přestává bavit. Současný návrh je trochu konzistentnější než dřívější stav, kdy to, jestli je něco ve vlastním jmenném prostoru, záviselo pouze na historických důvodech. (I když se obávám, že se to může stát znovu.) David asi hned tak nepřijme můj styl práce se jmennými prostory, takže nedává smysl mrhat energii na další diskuzi a vysvětlování :)

Třeba se ty prefixy po plánovaném zjemnění jmenných prostorů ještě nakonec prosadí. ;)

potapnik:
připadá mi to už trochu moc akademický a kavárenský kecání

Exactly! Nemozem si pomoct ale pride mi, ze sa tu uz nejaku dobu dohadujeme o nepodstatnych veciach a rozoberaju sa uz len osobne preferencie diskutujucich (ja by som ale predsa len dal tak, to je istotne lepsie lebo to tak ja pouzivam apod). Zaujimavy nazor je tiez mat len jeden NS Nette. Mat len jediny NS je podla mna zase extrem z druheho konca (mat detailnucko rozdelene triedy podla toho ako cool su) ale z praktickeho hladiska je podla mna lepsie mat NS len na logicky samostatne vecsie celky a zo zvyskom nerobit cavyky (nerad by som bol aby to skoncilo ako v Jave kde co package to directory a clovek potom pise siahodlhe NS – nezabudat na fakt ze sucastne IDE nestihaju dobu a nedokazu doplnat use statemenst v PHP kode podla toho co pouzivam).

jtousek:
co se na GitHubu objevily aliasy pro všechny Nette třídy

Pokial viem tato featura bola revertnuta a stiahnuta tymto commitom:
https://github.com/…59a984d082ce

Honza Marek:
David asi hned tak nepřijme můj styl práce se jmennými prostory

Ehm, prepac, ale preco by mal? Je snad tvoj styl prace s NS ten jediny spravny a preto musi byt sucastou Nette?

Přesně jak říká redhead. Já už své řekl, snažil se to vysvětlit a David to myslím pochopil. To, že s tím třeba nesouhlasí je věc jiná.

Spíš mi přijde, že doba návrhů a věcného vysvětlování před 2 dny pominula a teď se tu akorát haní úvodní myšlenka a útočí na názory druhých argumenty typu „vždyť je to celé hloupost, stačí jeden ns“, případně „tvuj styl je jediny spravny a proto musi…?“. Možná je to jen můj pocit (hlavně ať se nenajde nějaký flame war master, co to bude zas zbytečně rozmazávat, to opravdu netřeba;-)

Mno každopádně to neva:-) Diskuse to byla zajímavá, hlavně na začátku, a teď už budu jen sledovat, jak to všechno dopadne:-)

Edit: já už oba thready beru jako jeden:-)

Editoval Ondřej Brejla (8. 2. 2011 23:24)

jtousek: A jak víš, že ji nikdo nepoužíval??? prosím tě, hlavně nedělej závěry podle sebe. já ji teď chvíli používal.

Ondřej Brejla napsal(a):

Spíš mi přijde, že doba návrhů a věcného vysvětlování před 2 dny pominula

(tím že do diskuze vstoupili ti, kteří nezastávají ten jediný správný názor)

a teď se tu akorát haní úvodní

(nezpochybnitelně správná)

myšlenka a útočí na názory druhých

prosím vysvětlete mě někdo, jaký je pro programátora konkrétní praktický přínos katalogizačních jmenných prostorů (těch co jen seskupují podobné třídy, které ale netvoří funkční celek. viz můj předchozí příspěvek). protože v předchozí debatě zazněl tak maximálně argument „nevadí mi, protože mi je napoví IDE“, což určitě nelze považovat za argument pro jejich zavedení

@Ondra: neber to prosím tak, že na tvůj názor útočím, nebo haním nějakou myšlenku ;-)

Vrtak: napsal jsem comment k danému commitu, ale to je jedno. Jde mi jen o to nedělat závěry, aniž by někdě byla anketa, či někdo měl (krom Davida) dostupné čísla o počtu downloadu jednotlivých verzí (5.2, prefixed, 5.3)

hrach napsal(a):

Vrtak: napsal jsem comment k danému commitu, ale to je jedno.

A byl jsi jedinný… Kdyby to lidem doopravdy trhalo žíly tak by se jich ozvalo více (ale spíše to bude tím že to nebyla stable verze).

Jde mi jen o to nedělat závěry, aniž by někdě byla anketa, či někdo měl (krom Davida) dostupné čísla o počtu downloadu jednotlivých verzí (5.2, prefixed, 5.3)

Pokud se někdo něco používá a ty mu to sebereš tak se ozve „kam to zmizlo?“. Viz nedávný Davidův experiment s odebráním prefixované verze Nette z download stránek. Kde se lidi začaly ozívat a tak byla navrácena zpět (to jasně ukazuje že anketa není vždy nutná).

Honza Marek napsal(a):

Mimochodem Netbeans už dnes umí generovat use.

Mně by se spíš hodilo, kdyby se při přidání use odstranily v kódu namespacy u příslušné třídy, tohle pokud vím nejde (?).