HALLOWEEN JE TADY: Získej 66 % extra kreditů zdarma při nákupu od 1199 kreditů s promo kódem NEBOJSEIT66. Zjisti více:
NOVINKA: Začni v IT jako webmaster s komplexním akreditovaným online kurzem Tvůrce WWW stránek. Zjisti více:

Diskuze – VirtualCompany v Nette

Zpět

Upozorňujeme, že diskuze pod našimi online kurzy jsou nemoderované a primárně slouží k získávání zpětné vazby pro budoucí vylepšení kurzů. Pro studenty našich rekvalifikačních kurzů nabízíme možnost přímého kontaktu s lektory a studijním referentem pro osobní konzultace a podporu v rámci jejich studia. Toto je exkluzivní služba, která zajišťuje kvalitní a cílenou pomoc v případě jakýchkoli dotazů nebo projektů.

Komentáře
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
Editováno 17.1.2015 13:53
Avatar
Jan Demel
Tvůrce
Avatar
Odpovídá 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...

Odpovědět
To co se zdá být nemožné, je vždy možné.
Avatar
thejoeejoee
Člen
Avatar
Odpovídá 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...

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

Odpovědět
2x piš, jednou debuguj
Avatar
Tomáš Martin Holub
Tvůrce
Avatar
Odpovídá na Jan Demel
Tomáš Martin Holub:30.7.2018 23:03

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

Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 5 zpráv z 5.