IT rekvalifikácia. Seniorní programátori zarábajú až 6 000 €/mesiac a rekvalifikácia je prvým krokom. Zisti, ako na to!

Diskusia – VirtualCompany v Nette

Späť

Upozorňujeme, že diskusie pod našimi online kurzami sú nemoderované a primárne slúžia na získavanie spätnej väzby pre budúce vylepšenie kurzov. Pre študentov našich rekvalifikačných kurzov ponúkame možnosť priameho kontaktu s lektormi a študijným referentom pre osobné konzultácie a podporu v rámci ich štúdia. Toto je exkluzívna služba, ktorá zaisťuje kvalitnú a cielenú pomoc v prípade akýchkoľvek otázok alebo projektov.

Komentáre
Avatar
thejoeejoee
Člen
Avatar
thejoeejoee:17.1.2015 13:52

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
Editované 17.1.2015 13:53
 
Odpovedať
17.1.2015 13:52
Avatar
Jan Demel
Tvůrce
Avatar
Odpovedá na thejoeejoee
Jan Demel:17.1.2015 14:04

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...

Odpovedať
17.1.2015 14:04
To co se zdá být nemožné, je vždy možné.
Avatar
thejoeejoee
Člen
Avatar
Odpovedá na Jan Demel
thejoeejoee:17.1.2015 18:41

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...

 
Odpovedať
17.1.2015 18:41
Avatar
Josef Kuchař - Pepa489
Tvůrce
Avatar
Josef Kuchař - Pepa489:19.1.2015 17:07

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?… :` :D

Odpovedať
19.1.2015 17:07
2x piš, jednou debuguj
Avatar
Tomáš Martin Holub
Tvůrce
Avatar
Odpovedá na Jan Demel
Tomáš Martin Holub:30.7.2018 23:03

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

 
Odpovedať
30.7.2018 23:03
Robíme čo je v našich silách, aby bola tunajšia diskusia čo najkvalitnejšia. Preto do nej tiež môžu prispievať len registrovaní členovia. Pre zapojenie sa do diskusie sa zaloguj. Ak ešte nemáš účet, zaregistruj sa, je to zadarmo.

Zatiaľ nikto nevložil komentár - buď prvý!