Jmenné prostory – Last Call Announcement

Nové jmenné prostory

Přehled všech tříd se změněným názvem:

• bez prostoru
  • InvalidStateException → Nette\InvalidStateException
  • NotImplementedException → Nette\NotImplementedException
  • NotSupportedException → Nette\NotSupportedException
  • DeprecatedException → Nette\DeprecatedException
  • MemberAccessException → Nette\MemberAccessException
  • FatalErrorException → Nette\FatalErrorException
  • ArgumentOutOfRangeException (zrušeno)

• Nette
  • IComponent → Nette\ComponentModel\IComponent
  • Component → Nette\ComponentModel\Component
  • IComponentContainer → Nette\ComponentModel\IContainer
  • ComponentContainer → Nette\ComponentModel\Container
  • AmbiguousServiceException → Nette\DI\AmbiguousServiceException
  • IContext → Nette\DI\IContext
  • Context → Nette\DI\Context
  • Configurator → Nette\DI\Configurator
  • Debug → Nette\Diagnostics\Debugger
  • DebugPanel → Nette\Diagnostics\Panel
  • IDebugPanel → Nette\Diagnostics\IPanel
  • DebugHelpers → Nette\Diagnostics\Helpers
  • ITranslator → Nette\Localization\ITranslator
  • SafeStream → Nette\Utils\SafeStream
  • Finder → Nette\Utils\Finder
  • ArrayTools → Nette\Utils\Arrays
  • Json → Nette\Utils\Json
  • Neon → Nette\Utils\Neon
  • Paginator → Nette\Utils\Paginator
  • String → Nette\Utils\Strings
  • Tools → rozděleno do tříd Nette\DateTime Nette\Utils\CriticalSection Nette\Utils\MimeTypeDetector
  • CallbackFilterIterator → Nette\Iterators\Filter
  • GenericRecursiveIterator → Nette\Iterators\Recursor
  • InstanceFilterIterator → Nette\Iterators\InstanceFilter
  • MapIterator → Nette\Iterators\Mapper
  • RecursiveCallbackFilterIterator → Nette\Iterators\RecursiveFilter
  • SmartCachingIterator → Nette\Iterators\CachingIterator

• Nette\Application
  • PresenterRequest → Nette\Application\Request
  • IPresenterResponse → Nette\Application\IResponse
  • DownloadResponse → Nette\Application\Responses\FileResponse
  • ForwardingResponse → Nette\Application\Responses\ForwardResponse
  • JsonResponse → Nette\Application\Responses\JsonResponse
  • RedirectingResponse → Nette\Application\Responses\RedirectResponse
  • RenderResponse → Nette\Application\Responses\TextResponse
  • CliRouter → Nette\Application\Routers\CliRouter
  • MultiRouter → Nette\Application\Routers\RouteList
  • Route → Nette\Application\Routers\Route
  • SimpleRouter → Nette\Application\Routers\SimpleRouter
  • RoutingDebugger → Nette\Application\Diagnostics\RoutingPanel
  • Link → Nette\Application\UI\Link
  • IRenderable → Nette\Application\UI\IRenderable
  • ISignalReceiver → Nette\Application\UI\ISignalReceiver
  • IStatePersistent → Nette\Application\UI\IStatePersistent
  • IPartiallyRenderable → Nette\Application\UI\IPartiallyRenderable
  • Presenter → Nette\Application\UI\Presenter
  • PresenterComponent → Nette\Application\UI\PresenterComponent
  • PresenterComponentReflection → Nette\Application\UI\PresenterComponentReflection
  • Control → Nette\Application\UI\Control
  • AppForm → Nette\Application\UI\Form
  • BadSignalException → Nette\Application\UI\BadSignalException
  • InvalidLinkException → Nette\Application\UI\InvalidLinkException

• Nette\Forms
  • Button → Nette\Forms\Controls\Button
  • Checkbox → Nette\Forms\Controls\Checkbox
  • FileUpload → Nette\Forms\Controls\UploadControl
  • FormControl → Nette\Forms\Controls\BaseControl
  • HiddenField → Nette\Forms\Controls\HiddenField
  • ImageButton → Nette\Forms\Controls\ImageButton
  • MultiSelectBox → Nette\Forms\Controls\MultiSelectBox
  • RadioList → Nette\Forms\Controls\RadioList
  • SelectBox → Nette\Forms\Controls\SelectBox
  • SubmitButton → Nette\Forms\Controls\SubmitButton
  • TextArea → Nette\Forms\Controls\TextArea
  • TextBase → Nette\Forms\Controls\TextBase
  • TextInput → Nette\Forms\Controls\TextInput
  • FormGroup → Nette\Forms\ControlGroup
  • FormContainer → Nette\Forms\Container
  • IFormControl → Nette\Forms\IControl
  • DefaultFormRenderer → Nette\Forms\Rendering\DefaultFormRenderer

• Nette\Caching
  • ICacheStorage → Nette\Caching\IStorage
  • DummyStorage → Nette\Caching\Storages\DevNullStorage
  • FileJournal → Nette\Caching\Storages\FileJournal
  • FileStorage → Nette\Caching\Storages\FileStorage
  • ICacheJournal → Nette\Caching\Storages\IJournal
  • MemcachedStorage → Nette\Caching\Storages\MemcachedStorage
  • MemoryStorage → Nette\Caching\Storages\MemoryStorage

• Nette\Config
  • ConfigAdapterIni → Nette\Config\IniAdapter
  • ConfigAdapterNeon → Nette\Config\NeonAdapter
  • IConfigAdapter → Nette\Config\IAdapter

• Nette\Database\Selector
  • GroupedTableSelection → Nette\Database\Table\GroupedSelection
  • TableRow → Nette\Database\Table\ActiveRow
  • TableSelection → Nette\Database\Table\Selection

• Nette\Loaders
  • LimitedScope → Nette\Utils\LimitedScope

• Nette\Mail
  • Mail → Nette\Mail\Message
  • MailMimePart → Nette\Mail\MimePart

• Nette\Reflection
  • ClassReflection → Nette\Reflection\ClassType
  • ExtensionReflection → Nette\Reflection\Extension
  • FunctionReflection → Nette\Reflection\GlobalFunction
  • MethodReflection → Nette\Reflection\Method
  • ParameterReflection → Nette\Reflection\Parameter
  • PropertyReflection → Nette\Reflection\Property

• Nette\Templates
  • LatteFilter → Nette\Latte\Engine
  • LatteMacros → Nette\Latte\DefaultMacros
  • LatteException → Nette\Latte\ParseException
  • CachingHelper → Nette\Caching\OutputHelper
  • TemplateException → Nette\Templating\FilterException
  • TemplateCachestorage → Nette\Templating\PhpFileStorage
  • TemplateHelpers → Nette\Templating\DefaultHelpers
  • TemplateFilters → přesunuto do doplňků
• Nette\Web
  • Html → Nette\Utils\Html
  • HttpContext → Nette\Http\Context
  • IHttpRequest → Nette\Http\IRequest
  • HttpRequest → Nette\Http\Request
  • IHttpResponse → Nette\Http\IResponse
  • HttpResponse → Nette\Http\Response
  • HttpRequestFactory → Nette\Http\RequestFactory
  • HttpUploadedFile → Nette\Http\FileUpload
  • ISessionStorage → Nette\Http\ISessionStorage
  • Session → Nette\Http\Session
  • SessionNamespace → Nette\Http\SessionNamespace
  • Uri → Nette\Http\Url
  • UriScript → Nette\Http\UrlScript
  • IUser → Nette\Http\IUser
  • User → Nette\Http\User

Špatně vidím, nebo skutečně chybí IPresenter a BadRequestException? Navrhuji přejmenovat AppForm na Form (v souladu s 2. pravidlem). Dále mi pořád nesedí RoutingDebugger v Application, ale v praxi mi to nevadí, protože ho nikdy neinstancuji.

Teď koukám, že chybí také celé Nette\Forms. Mám to chápat tak, že tam dojde ke změnám zároveň s uvedením nových formulářů?

paranoiq napsal(a):

Jan Tvrdík

AppForm → Form není v souladu s pravidlem c)

Je… AppForm s třídou Form jsem vedle sebe ještě neviděl použít.

pravidlo f

Zdá se mi divné… Vyčleňovat třeba Routery kvůli tomu, že je jich hodně? Dává mi větší smysl prohlásit celé routování za samostatný celek a dát do něj všechno, co s ním souvisí. Např. Nette\Application\Routing\IRouter, ono toho víc nebude.

Neměl bych problém ani s namespacem Nette\Routing, které by odpovídalo i pravidlu d.

PhpFileStorage do Templates patří, protože je implementací cachování šablon, ne? Viz původní název té třídy.

class Form extends Nette\Forms\Form mi nepřijde nijak problematické.

Nilp napsal(a):

Katalogizace je skvělá, navrhuji ještě jmenné prostory Exception, zvláště Nette\ by to prospělo.

tiez som za pridanie namespace Nette\Exceptions\, s vynimkami pracujem celkom casto a je treba si pamatat ich nazvy ked ich chcem vylovit, takto by som len napisal Nette\Exceptions\ stlacil ctrl+medzera a dostal by som zoznam vsetkych vynimiek. Cize podporuje to pohodlie programatora :)

Nette\Finder → Nette\FileFinder ?

Jenže Finder bude ve FileSystem, tak je to jasný, ne?

Osobně mi přijde hloupé pravidlo a), protože:

1. Velmi často koliduje s doporučením b) a v předloženém seznamu změn je jednou upřednostněna výstižnost názvu a jindy neduplicita. Sice je pravidlo a) „jasně“ definované požadavkem na podmět, ale to do celého problému jen vnáší další pravidlo, které celkem nevhodně nastavuje hranici mezi a) a b). Typickým matoucím příkladem je Application\Responses\JsonResponse (vítězí a) vs. Http\Response (vítězí b)
2. Pokud požadujeme „třída musí mít výstižný název i bez uvedení jmenného prostoru“, tak to člověka logicky vede k otázce – „A proč teda ty jmenné prostory zavádíme?“. Jednoduše řečeno – pravidlo a) mi připadá jako zpátečnické beznamespacové myšlení.

--

Taky se přimlouvám za přejmenování AppForm. Je pravda, že Form extends Form vypadá divně, ale to je interní věc frameworku, která moc lidí trápit nebude. Důležité je, že v aplikaci se používá buď Application\Form, nebo Forms\Form. Případů, kdy je rozumné oba přístupy mixovat moc nebude, natožpak případů, kdy se oba přístupy mixují v jednom souboru.

Editoval xificurk (7. 4. 2011 17:23)

Rozhodně nechat pravidlo a). S použitím use nebudu vůbec tušit, co že za třídy to v kódu mám.

AppForm nechat.

FileFinder++

Myslím že pravidlo a) by mělo mít větší prioritu než duplicity v názvu.

Editoval HosipLan (5. 4. 2011 15:05)

Ad použití Form + AppForm dohromady – nejspíš jsem ještě naučený z PHP 5.2 verzí, nicméně validační pravidla připisuji přes Form::SOMETHING.

Když už teda hlasujeme o prioritách, tak hlasuju pro nízkou prioritu pravidla a :)

Vyhnu se zbytečnému flame okolo všecho a vypíchnu jenom jednu věc a to AppForm vs Form. Předně jsem pro přejmenovat AppForm na Form. Kdo z vás kdy naposledy použil samotný Nette\Forms\Form? Podle mě ho používají pouze lidé kteří používají samotné formuláře. Takže to tak matoucí není.

Nehledě na:
varianta 1 (ta kterou nemám rád):

namespace App;

use Nette\Application\AppForm;

class FooPresenter extends BasePresenter
{
	protected function createComponentMyForm($name)
	{
		$form = new AppForm($this, $name);
		$form->addText('foo', "Foo")->addRule(AppForm::PATTERN, "[a-z0-9]");
	}
}

varianta 2 (hezčí srozumitelnější ale používám 2 různé „Form“):

namespace App;

use Nette\Forms\Form,
	Nette\Application\AppForm;

class FooPresenter extends BasePresenter
{
	protected function createComponentMyForm($name)
	{
		$form = new AppForm($this, $name);
		$form->addText('foo', "Foo")->addRule(Form::PATTERN, "[a-z0-9]");
	}
}

varianta 3 (po přejmenování Nette\Application\AppForm na Nette\Application\Form):

namespace App;

use Nette\Application\Form;

class FooPresenter extends BasePresenter
{
	protected function createComponentMyForm($name)
	{
		$form = new Form($this, $name);
		$form->addText('foo', "Foo")->addRule(Form::PATTERN, "[a-z0-9]");
	}
}

Aneb ve variantě 1 definuju pravidlo PATTERN ale musím psát AppForm proto si to ve variantě 2 usnadním a přidám use Nette\Forms\Form abych mohl psát jenom Form (tím ale vznikne trochu WTF proč instancuju AppForm když pro pravidla používám Form). No a varianta 3 je prostě žůžo labůžo… No ni?

Jó a jednu tak trochu „OFF TOPIC“ si neodpustím a to co App namespace pro sandbox?

Patrik Votoček napsal(a):

varianta 1 (ta kterou nemám rád):

…

varianta 2 (hezčí srozumitelnější ale používám 2 různé „Form“):

…

variant 3

namespace NetteSandbox;

use Nette\Application\AppForm as Form;

class FooPresenter extends BasePresenter
{
	protected function createComponentMyForm($name)
	{
		$form = new Form($this, $name);
		$form->addText('foo', "Foo")->addRule(Form::PATTERN, "[a-z0-9]");
	}
}

David Grudl napsal(a):

a) Název každé třídy i bez jmenného prostoru musí vystihovat její podstatu, tj. název musí obsahovat podmět.

Pravidlo a) neznamená, že názov triedy musí byť jednoznačný v rámci frameworku.

Mne sa veľmi páči zápis

<?php

use Nette\Http;

$request = new Http\Request;
$response = new Http\Response;

pretože krásne využíva vlastnosť PHP „importovať“ menné priestory. Analógia z aktuálneho stavu je zbytočne ukecaná:

<?php

use Nette\Http\HttpRequest,
    Nette\Http\HttpResponse;

$request = new HttpRequest;
$response = new HttpResponse;

Škoda len, že nie všetky triedy sa dajú takto zapísať…

Nette\Http\UploadedFile ++ (Nette\Http\FileUpload mi skutečně přijde hodně WTF)

ad Finder, prohledává i adresáře, takže FileFinder či jiná varianta s File by byla opět WTF

kravčo napsal(a):

David Grudl napsal(a):

a) Název každé třídy i bez jmenného prostoru musí vystihovat její podstatu, tj. název musí obsahovat podmět.

Jinými slovy, nemělo by být nutné používat AS v klauzuli USE.

na1k napsal(a):

Asi bych byl taky za přejmenování AppForm na normální Form (a přesun někam do Application) a nahrazení obyč Formu něčím jiným. Form využijí jen ti co používají standalone formuláře; těch je ale minimum. Zatímco většina jede na plné verzi Nette a musejí psát o tři znaky delší AppForm. Já hlasuji pro pohodlí většiny :)

Souhlas

redhead napsal(a):

Zdá se mi to nebo tu nikdo nepracuje v Javě, ta by vás naučila.

Ale tady nejsme v Javě. :)

Já ho četl a to 2x… :-/
Nechme tedy AppForm stranou možná se to ztratilo v kontextu ale co App namespace pro „sandbox“ / „app“?

O pravidle a) prosím vůbec nediskutujme, to je naprosto základní pilíř celé koncepce.

A zase si to rozmyslel a vrátil je.

David Grudl napsal(a):

A zase si to rozmyslel a vrátil je.

ono je to v podstatě fuk. kdyby byl opravdu pádný argument pro nebo proti, asi by jsme tady o tom nediskutovali

S Nette\Exceptions souhlasím – už jen díky tomu, že si člověk pak napíše jen Exceptions\ a dá ctrl-shift a vyjedou mu všechny možné Nette výjimky (což se hodí, protože každá výjimka začíná jiným slovem než Exception; a hledat nějakou konkrétní by bylo poměrně otravné, když si člověk nepomatuje její název zpaměti).

Honzo, s těmi výjimkami máš samozřejmě pravdu, ale né všichni vědí, že by měli psát výjimku zvlášť pro každou třídu/balíček, nebo to chtějí dodržovat. Já to dodržuji jen částečně. Rozhodně jsem pro zabalení Nette výjimek do Exceptions\, je to neuvěřitelně pohodlné.

David Grudl napsal(a):

A zase si to rozmyslel a vrátil je.

Jsem pro Nette\Application\Responses\ForwardingResponse. Sám to používám, když mám skupinu tříd, které dělají podobnou činnost (responses, Iterators, …).

Editoval HosipLan (7. 4. 2011 7:59)

Jsi si jistý, že to dostatečně popisuje co třída dělá? :) Možná u forward ano, ale co takový Json?