Mikuláš je tu! Získaj 90 % extra kreditov ZADARMO s promo kódom CERTIK90 pri nákupe od 1 199 kreditov. Len do nedele 7. 12. 2025! Zisti viac:
NOVINKA: Najžiadanejšie rekvalifikačné kurzy teraz s 50% zľavou + kurz AI ZADARMO. Nečakaj, táto ponuka dlho nevydrží! Zisti viac:

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

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

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.

Zobrazené 5 správy z 5.