Diskuze – VirtualCompany v Nette

Jen tak zběžně:

• proč existuje BasePresenter::re­turnRandomStrin­g, když používáš Nette? použij http://api.nette.org/…Strings.html#…
• BasePresenter::i­sUserLoggedIn má nelogický název, metoda nic nevrací o tom, jestli je uživatel přihlášen; lepší by bylo nějaké checkUser.
• V admin presenteru zbytečně opakuješ velké kusy kódu inejktování dat do šablony, přesuň to do beforeRender metody.
• Kód v Repository::get­PlayerTradesA­mount má dělat co? použij count()
• /** @return Nette\Applica­tion\UI\Form */ private $database - tak to asi ne
• $values->password == $values->verpassword jde jako podmínka zadat rovnou při vytváření formuláře
• setBody("Dobrý den, vaše nové heslo je: " . $values->newPassword); takhle se opravdu neresetuje heslo. bezpečnost nula nula nic. Při žádosti o reset se musí vygenerovat token a ten se poslat v podobě odkazu na akci. V té akci se ověří časová platnost tokenu a nabídne se formulář pro změnu hesla.
• $mailer = new Nette\Mail\SmtpMa­iler opravdu ne, použij DI container
• použij komponenty, třeba z MessagePresenter by šla udělat pěkná komponenta

Já už nic používat nebudu Jak jsem psal, tak projekt jsem dělal asi před půl rokem... Celé to bylo absolutně nerozvržené prostě psané úplně naslepo. Teď už bych to napsal celé jinak a hlavně bych si všechno rozmyslel. Sem jsem to dal jenom kdyby někdo měl ještě zájem se v tom trochu podrbat a něco třeba dotvořit...

Nepřijde mi vhodné zveřejňovat takový ne úplně čistý Nette kód na síti, kde se lidé učí programovat a měli by zde pochytit best practices. Ale proti gustu...

Jak už jsem vám psal před půl rokem, neopravily jste závažnou chybu, lze získávat libovolně předměty které nejdou vyrobit například železná ruda: http://virtualcompany.8u.cz/company/create?…

Možná bych ještě zkusil tu hru vylepšit, ale nevím