login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Is dit vies?

Offline xSc - 29/07/2010 11:49 (laatste wijziging 29/07/2010 11:50)
Avatar van xScOnbekend Onderstaande code werkt goed, maar ik vind het een beetje 'vies'. Is dat gevoel of kan het echt beter?

Plaatscode: 139776

11 antwoorden

Gesponsorde links
Offline valles10 - 29/07/2010 11:51
Avatar van valles10 HTML interesse die is deftig hoor^^
Offline xSc - 29/07/2010 11:53 (laatste wijziging 29/07/2010 11:55)
Avatar van xSc Onbekend Let op, die variable namen komen niet overeen (screens en screen), maar dat is het punt niet.

@valles10 ... vind je mijn code deftig? 
Offline valles10 - 29/07/2010 11:55
Avatar van valles10 HTML interesse
xSc schreef:
Let op, die variable namen komen niet overeen (screens en screen), maar dat is het punt niet.

ja, kwou net post aanpassen om dat te melden, maar je code is voor de rest 'niet vies' hoor toch niet voor mij 
Offline Martijn - 29/07/2010 12:15
Avatar van Martijn Crew PHP naar mijn smaak wat veel lege regels, other than that, niets mis mee.
Offline valles10 - 29/07/2010 12:35
Avatar van valles10 HTML interesse
Martijn schreef:
naar mijn smaak wat veel lege regels, other than that, niets mis mee.

ja die lege regels dacht ik ook... maar langs andere kant zorgt het er wel voor dat code overzichtelijk blijft...
Offline gnotrgnotr - 29/07/2010 13:21
Avatar van gnotrgnotr Nieuw lid deftig
Offline lemoinet - 29/07/2010 13:46 (laatste wijziging 29/07/2010 13:46)
Avatar van lemoinet PHP gevorderde ik gerbuik dit
  1. $storage = $this->_auth->getStorage();
  2. $storage->write($adapter->getResultRowObject(null, 'password'));

tijdens mijn login. hij gaat dus alle gegevens behalve password opslaan in identity.

wanneer ik getIdentity opvraag heb ik reeds al de gegevens van mijn user en moet ik dit
  1. $user = $_users->fetchRow($_users->select()->where('username=?', $auth->getIdentity()->username));
dus niet meer aanroepen,maar gewoon
  1. $user = $auth->getIdentity();


ook ben ik overgestapt van het gebruik van de generieke queries terug naar eigen geschreven queries.
wanneer je dit
  1. $_users->select()->where('username=?', $auth->getIdentity()->username)
uitvoerd, zal ZF 2 queries nodig hebben.
  1. 1. DESCRIBE users;
  2. 2. SELECT * FROM users WHERE username = '?';

niet echt performant dus. om dit proces te versnellen kan je wel je metadata cachen
http://framewor...table.html
Offline Stijn - 29/07/2010 13:50
Avatar van Stijn PHP expert $_users -> $modelUsers , lijkt me iets overzichtelijker
Offline xSc - 29/07/2010 13:56
Avatar van xSc Onbekend Bedankt!

@Lemoinet, ik begrijp je commentaar. Met dat laatste ben ik het niet helemaal eens. De eerste punten heb je gelijk in.

Stijn, kun je uitleggen wat je bedoelt?
Offline Wave6 - 29/07/2010 15:20
Avatar van Wave6 PHP beginner Ik vind hem goed;)
Offline Stijn - 29/07/2010 16:36
Avatar van Stijn PHP expert Een betere naamgeving . Zo weet je welk object die variabele vasthoudt, namelijk een Model.
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.329s