Zjemnění jmenných prostorů

Neměl bych problém, je pravda, že jsou třídy rozděleny moc obecně ale jsem spokojen se současným stavem

Už jsem si zvykl, kde které třídy hledat, ale nijak zásadně proti nejsem. S čistými Nette třídami totiž vlastně ani moc do styku nepřicházím. AppForm, Control a Presenter mám obalené vlastním BaseXX a když už potřebuju volat něco z vnitřností, tak mi namespace doplní IDE.

Zrovna jsem si v práci položil naprosto odlišnou otázku jak udělat ve firemním projektu jmenné prostory hrubější. Je fakt že aktuální stav v Nette by se dal malinkdo zjemnit s tím souhlasím ale pak by mohly vznikat zajímavé „fullclass“ kde se jedno slovo vyskytuje zbytečně vícekrát. A to protože Nette 2.0 je kompatabilní s non-namespace takže by se nemohli zkrátit názvy tříd. Nedávno jsem přemýšlel nad Nette\Application\AppForm že by se mě víc líbilo Nette\Application\Form ale narazil jsem přesně na tento problém proto jsem to ani nikam nepsal jako návrh.

Editoval vrtak-cz (14. 11. 2010 13:09)

@DG: Od toho je přece klauzule use.

+1

podobnými změnami jsem si vždycky slušně zavařil :) přemýšlíte nad tím, že jsou lidé/firmy co mají na nette postaveno „stovky“, myslím to vážně, stovky projektů (velkých, malých, to je jedno)?

moc hezká změna, která dle mého nepřináší nic nového ani lepšího (nijak zvlášť mi to nepomůže orientovat se v nette), která by mne stála mooooc hodně peněz, davide raději ti zaplatím dovolenou v karibiku než tohle (ber to s nadsázkou, s ambiente samozřejmě počítám, ale čekám, kdy už konečně vyjde to nette :) a changelog!)

• 1

připojuju s k Jan Tvrdík ukažte návrh, možná to až tak super nebude a ta změna opravdu nic nepřinese, ale třeba se pletu, každopádně plýtvat energií se dá i na jiné méně nepodstatné věci

Na tom argumentu je špatné „jen“ to, že IDEčko, o kterém mluvím, jsem si vytáhl z křišťálové koule. V javě nebo .netu si balíčků nebo jmenných prostorů ani nevšimneš, právě kvůli dobré podpoře IDE.

mcmatak napsal(a):
podobnými změnami jsem si vždycky slušně zavařil :) přemýšlíte nad tím, že jsou lidé/firmy co mají na nette postaveno „stovky“, myslím to vážně, stovky projektů (velkých, malých, to je jedno)?
moc hezká změna, která dle mého nepřináší nic nového ani lepšího (nijak zvlášť mi to nepomůže orientovat se v nette), která by mne stála mooooc hodně peněz, davide raději ti zaplatím dovolenou v karibiku než tohle (ber to s nadsázkou, s ambiente samozřejmě počítám, ale čekám, kdy už konečně vyjde to nette :) a changelog!)

• 1

připojuju s k Jan Tvrdík ukažte návrh, možná to až tak super nebude a ta změna opravdu nic nepřinese, ale třeba se pletu, každopádně plýtvat energií se dá i na jiné méně nepodstatné věci

Mělo by být možné napsat skript, co by změnu označení NS provedl napříč projektem za nás, ne? Ne že by to úplně anulovalo tenhle názor, ale z nemalé části by to řešilo (i když samozřejmě někdo by to musel napsat)

Editoval Nox (15. 11. 2010 19:37)

Změna se tyka jen Nette 2.0, tedy nebranim se tomu a mozna to i uvitam.

Neberu argument, ze hromada firem ma na Nette hromadu projektu. Stare projekty – stare Nette. Takze jsem pro vsechny BC zmeny.

Jsem také pro zjemnění. Třeba responses, form controly a AppForm → Form jsou kandidáty. :)

Honza Marek napsal(a):

anebo naopak… kdo je potřebuje zpětně kompatibilní, může použít také class_alias

Tak tak.

Bylo by dobré sjednotit množné a jednotné číslo u namespaců. Protože např. Nette\Applications nedává smysl, přikláněl bych se k jednotnému (Nette\Form).

Namespace Nette

Vyčlenit Nette\DependencyInjection:

• AmbiguousServiceException
• Context (nebo budoucí jiný název)
• IContext
• asi další výjimky (ServiceNejdeVyrobitException, …)

Vyčlenit Nette\Debug:

• Debug
• DebugPanel
• IDebugPanel
• RoutingDebugger (z Nette\Application)

Vyčlenit Nette\ComponentModel nebo Nette\Component:

• Component
• ComponentContainer
• RecursiveComponentIterator

Vyčlenit Nette\Helper nebo Nette\Tool:

• ArrayTools
• Paginator
• RegexpException
• SafeStream
• String
• přimlouval bych se za zavedení třídy Regexp
• Tools
• Html (z Nette\Web)

Vyčlenit Nette\Image:

• Image
• ImageMagick
• budoucí ImageException ;)

Vyčlenit Nette\Neon:

• Neon
• NeonException

Vyčlenit Nette\Bullshit:

• Callback

Vyčlenit Nette\Environment nebo možná nechat nebo je nechat umřít s příchodem DI:

• Configurator
• Environment

Vyčlenit Nette\Finder:

• Finder

Vyčlenit Nette\Internatialization nebo jak se to píše:

• ITranslator
• …

Nechat:

• Framework
• Object
• FreezableObject
• ObjectMixin
• IFreezable

Nevím nebo neznám:

• ArrayList
• CallbackFilterIterator
• GenericRecursiveIterator
• RecursiveCallbackFilterIterator
• InstanceFilterIterator
• SmartCachingIterator
• Tokenizer
• TokenizerException

Namespace Nette\Application

Vyčlenit Nette\Application\Routing:

• CliRouter
• MultiRouter
• Route
• SimpleRouter
• IRouter

Vyčlenit Nette\Application\Response:

• DownloadResponse
• ForwardingResponse
• JsonResponse
• RedirectingResponse (RedirectResponse?)
• RenderResponse
• IPresenterResponse
• tady bych i sundal ten suffix Response z názvů tříd

Nechat:

• AbortException
• AppForm
• Application
• ApplicationException
• BadRequestException
• BadSignalException
• Control
• ForbiddenRequestException
• InvalidLinkException
• InvalidPresenterException
• Presenter
• PresenterComponent
• PresenterLoader
• Link
• PresenterRequest
• IPartiallyRenderable
• IPresenter
• IPresenterLoader
• IRenderable
• ISignalReceiver
• IStatePersistent
• kdyby tady někdo měl nějaké světoborné myšlenky, uvítám diskuzi

Přesunout jinam:

• RoutingDebugger → Nette\Debug

Nette\Caching

nechat

Nette\Forms

Přejmenovat na Form a dál ať to vymyslí někdo jinej.

Nette\Loaders

Jednotné číslo…

Nette\Mail

nechat

Nette\Reflection

nechat

Nette\Security

nechat

Nette\Templates

Jednotné číslo Nette\Template.

Vyčlenit Nette\Template\LatteFilter, jinak nechat.

Nette\Web

Vykopnout Nette\Web\Html do Nette\Helper či Nette\Tool, jinak nechat.

Editoval Honza Marek (20. 11. 2010 18:05)

Souhlasím, jen bych možná ještě přejmenoval Nette\Caching na Nette\Cache, když už jsme u toho sjednocování…

Nechcete to dát na Google Wave? Dá se v tom líp orientovat a mj. je tam i plugin pro hlasování.

Souhlasím s Honzou Markem, až na:

Bylo by dobré sjednotit množné a jednotné číslo u namespaců. Protože např. Nette\Applications nedává smysl, přikláněl bych se k jednotnému (Nette\Form).

Třeba .NET Framework to taky má všude jinak a já třeba nevím jak vy, ale na Nette\Tool\Tools bych si fakt těžko zvykal. Tady by se hodilo to hlasovátko, co je na Google Wave.

Vyčlenit Nette\Helper nebo Nette\Tool:

Tady bych byl pro množné číslo a možná bych použil raději třeba Utils (Nette\Tools\Tools vs Nette\Utils\Tools).

přimlouval bych se za zavedení třídy Regexp

S tím nesouhlasím, vlastně ani David.

Vyčlenit Nette\Bullshit:

• Callback

(Asi jde o vtip.) Tohle bych možná nechal v Nette, ale klidně by to mohlo taky být v Nette\Utils.

Vyčlenit Nette\Finder:

• Finder

Tohle by taky mohlo být v utilitách.

Vyčlenit Nette\Internatialization nebo jak se to píše:

• ITranslator
• …

Správně je to Internationalization. Možná bych raději Globalization (právě kvůli té délce).

• ArrayList

Nette\Collection (možná Collections z historických důvodů, ale když je tam teď už jenom jedna třída… :-))

• CallbackFilterIterator
• GenericRecursiveIterator
• RecursiveCallbackFilterIterator
• InstanceFilterIterator
• SmartCachingIterator

Nette\Iterators (tady bych byl taky pro plurál)

• Tokenizer
• TokenizerException

Nette\Tokenizer

Namespace Nette\Application

Vyčlenit Nette\Application\Routing:
Vyčlenit Nette\Application\Response:

Víc bych to uvítal tak, jak je akt. adresářová struktura (Routers, Responses), ale Routing ani Response se nebráním.

• DownloadResponse
• ForwardingResponse
• JsonResponse
• RedirectingResponse (RedirectResponse?)
• RenderResponse
• IPresenterResponse
• tady bych i sundal ten suffix Response z názvů tříd

Tady bych byl pro sjednocení (ForwardResponse a RedirectResponse). Suffixy bych nechal, bylo by tam rozhraní IPresenter a to by se pletlo s Nette\Application\IPresenter.

Nette\Caching

Možná spíš ta Nette\Cache (jak navrhuje Lopata).

Nette\Forms

Přejmenovat na Form a dál ať to vymyslí někdo jinej.

Nette\Forms\Form mi připadá OK, udělal bych to podle adresářové struktury (rozdělení na Controls a Renderers).

Nette\Loaders

Jednotné číslo…

Tady bych taky nechal plurál.

Nette\Templates

Jednotné číslo Nette\Template.

Tady taky.

Na první pohled souhlasím s @jansfabik. Na ten druhý bych si musel udělat víc času.

$I18N++;

Když si to takhle rozepíšu:

• Nette\I18n\ITranslator
• Nette\L10n\ITranslator
• Nette\Globalization\ITranslator
• Nette\Localization\ITranslator
• Nette\Internationalization\ITranslator

Tak nejlepší se mi zdá Localization.

to v PHP 5.2 nejde

(Předpokládal jsem přechod z PHP 5.2 na 5.3 + to, co říká Majkl578.)

To nešlo kompatibilně řešit ani předtím.

Nette\I18n\ITranslator, Nette\L10n\ITranslator asi není moc intuitivní. Nette\Localization\ITranslator vypadá mnohem lépe.

Naštěstí to ale budeš potřebovat psát málokdy + situace s IDE už je výrazně lepší než když PHP 5.3 vyšlo.

když už jsme u těch aliasů. klidně je možné ihned vyhodit z distribuce non-prefixed verzi pro 5.2. kompatibilita jde vyřešit jedním souborem s aliasy tříd

Současný stav mi přijde nekonzistentní. Proč Mail si zaslouží vlastní jmenný prostor a Image ne? Pořád je to ucelená knihovnička náhodou patřící do Nette Frameworku.

Jemnost jmenných frameworků je pak pojistkou proti nepředvídatelné evoluci, kdy třeba z Latte filteru či Nette debugu vyrostly klíčové a velké součásti frameworku, ale oproti Nette\Caching nebo Nette\Reflection nemají vlastní jmenný prostor. Jen proto, že ze začátku byly zamýšlené jako jednoduché třídy, kde vlastní jmenný prostor „nebyl potřeba“.

Dalším (trochu subjektivním) důvodem je marketingové hledisko. Třeba v Symfony je malá knihovnička na procházení DOMu HTML stránky v jQuery stylu. Díky zajímavému nápadu, ale z části i vlastnímu namespacu, z toho může Fabien Potencier udělat v prezentacích a na webu velké téma. V Nette takový reklamní potenciál mají dle mého názoru třeba Image, Neon, Finder, o Debugu nemluvě, v budoucnu se snad přidá DI. Ostatně… svůj názor, co si zaslouží vlastní namespace, jsem vyjádřil tom návrhu výše.

Souhlas až na to sjednocení množné/jednotné. Trochu logického myšlení do toho a nespletete si to. I jiné jazyky to tak mají a žádné zmatení v tom nevidím.

Doufam, ze IDE mi bude vzdycky napovidat, ale stejne jako je uzus ohledne lomitek v cestach, nevidim duvod proc mit neporadek v jednotnych a mnoznych cislech (pokud k tomu neni jasny duvod). Cili ja bych sjednotil:-)