Tijden veranderen
Ontwerpen usermanagement
Ik zit er aan te denken om nog eens wat in PHP te gaan doen. Als basis wil ik eerst en vooral een degelijk 'usermangement system' opzetten. Dit in een object gericht concept.
Omdat hierbij zowel het databaseontwerp als het implementeren van de code zelf belangrijk zijn, zou ik deze graag beiden bespreken.
De database zou ik als volgt willen inrichten:
Misschien kort even het schema toelichten:
In de database wordt over elke gebruiker wat basisgegevens bijgehouden. Er kunnen eventueel een aantal opmerkingen gemaakt worden over een gebruiker. (bv. zijn specifieke functie op de website). Verder is er verplicht een koppeling met de permissies tabel. Uiteraard, behoord een 'permissie-entry' altijd aan 1 specifieke gebruiker. (kleine opmerking hieronder) Verder worden alle belangrijke zaken die een gebruiker uitvoert gelogd. Tot slot is er nog een tabel met sessies in (Ik ben van plan een eigen session handler te programmeren die via de database werkt.) Elke sessie verwijst naar 1 gebruiker.
[opmerking permissies]
Ik overweeg dit nog wat aan te passen, namelijk een toevoeging: groups. Elke gebruiker krijgt dan een group en een group krijgt op zijn beurt een aantal permissies.
In PHP zou ik dan volgende klassen bekomen:
Session, User, Log, Note. Wat deze doen, lijkt mij vanzelfsprekend. Indien dat niet het geval zou zijn, wil ik dit gerust even toelichten.
Graag jullie mening over dit 'model'!
Probeer wat meer te programmeren/denken naar een interface*. De objecten die je nu hebt zijn allemaal objecten via programmeren naar een implementatie. Dat is niet verstandig.
Zo heb je nu bijv. de Session implementatie, maar wil je straks liever een File implementatie van het opslaan van de gebruiker of een cookie implementatie. Het is dus beter om deze terug te leiden naar 1 interface: Storage. Deze bevat de basis, vervolgens heb je de SessionStorage klasse die voor de implementatie zorgt.
Ditzelfde moet je doen voor bijv. de Log klasse. Nu heb je een database log, straks een logfile, vervolgens een email log. Daarom moet je 1 interface Log hebben die voor de basis zorgt en verschillende subklassen die voor de implementatie zorgen. Een leuk voorbeeld van log kun je hier vinden: Een logging class in OO - Jelmer.
Tevens zal ik eens kijken naar het STRATEGY pattern. Hiermee regel je wat de gebruiker mag en kan. Even snel wat gemaakt in plain text (hoort natuurlijk in een UML diagram)
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
+---------------------+ +-------------------------+
| <<interface>> | | User |
| RemoveUserStrategy | +-------------------------+
+---------------------+ | ... |
| removeUser() |--------------> | removeUserStrategy |
+---------------------+ +-------------------------+
^ ^ | ... |
/ \ | addRemoveUserStrategy() |
/ \ +-------------------------+
+------------------+ +----------------+
| CannotRemoveUser | | CanRemoveUser |
+------------------+ +----------------+
| removeUser() { | | removeUser() { |
| // exception | | // removeUser |
| } | | } |
+------------------+ +----------------+
| <<interface>> | | User |
| RemoveUserStrategy | +-------------------------+
+---------------------+ | ... |
| removeUser() |--------------> | removeUserStrategy |
+---------------------+ +-------------------------+
^ ^ | ... |
/ \ | addRemoveUserStrategy() |
/ \ +-------------------------+
+------------------+ +----------------+
| CannotRemoveUser | | CanRemoveUser |
+------------------+ +----------------+
| removeUser() { | | removeUser() { |
| // exception | | // removeUser |
| } | | } |
+------------------+ +----------------+
Kort gezegd heb je de klassen op object niveau door, alleen moet je nu nog gaan nadenken over de objecten op implementatie (of eigenlijk juist niet op implementatie) niveau. Je moet nu gaan nadenken over de structuur van je applicatie en hoe je die zo flexibel mogelijk maakt.
* Met interface doel ik hier niet per se op een interface zoals we die in PHP kennen. Maar ik bedoel een superklasse, het kan dus ook een abstracte klasse zijn.
Gewijzigd op 28/06/2012 07:47:57 door Wouter J
Bedankt, hiermee kan ik zeker wat. Misschien nog een toevoeging, is het interessant om de 'log' klasse te gaan verwerven met een Exception klasse? Dan kan een exception bv. een bepaalde log-methode worden meegegeven.
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
Hierbij vang je dus een exception op. Vervolgens laat je die afhandelen door de ExceptionHandler. De code van de Exception is vervolgens het niveau van de log, zoals je in dat voorbeeldje van Jelmer kan zien.
En je moet eigenlijk altijd naar een interface/abstracte klasse programmeren, dat is het meest flexibel. Als je ook nog methods hebt die in elke subklasse gelijk zijn (of zouden zijn, altijd verder denken dan wat je nu hebt) dan maak je geen interface maar een abstracte klasse.
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
<?php
$container = new Container();
$container->set('log.class', 'FileLog');
$container->set('log.config', array('logs/exceptions.log'));
$container->set('log', function($c) {
$reflection = new \ReflectionClass($c->get('log.class'));
if (!$reflection->isSubclassOf('Log')) {
throw new InvalidArgumentException('log.class does not implement the Log interface');
}
return $reflection->newInstanceArgs($c->get('log.config'));
});
$container->set('exceptionHandler.class', 'ExceptionHandler');
$container->set('exceptionHandler.config', array($container->get('log')));
$container->set('exceptionHandler', function($c) {
$reflection = new \ReflectionClass($c->get('log.class'));
if (!$reflection->isSubclassOf('Handler')) {
throw new InvalidArgumentException('exceptionHandler.class does not implement the Handler interface');
}
return $reflection->newInstanceArgs($c->get('exceptionHandler.config'));
});
$exceptionHandler = $container->get('exceptionHandler');
try {
$foo = new Foo();
$foo->setBar('12'); // gooit met exceptions
} catch (\Exception $e) {
$exceptionHandler->handle($e);
}
?>
$container = new Container();
$container->set('log.class', 'FileLog');
$container->set('log.config', array('logs/exceptions.log'));
$container->set('log', function($c) {
$reflection = new \ReflectionClass($c->get('log.class'));
if (!$reflection->isSubclassOf('Log')) {
throw new InvalidArgumentException('log.class does not implement the Log interface');
}
return $reflection->newInstanceArgs($c->get('log.config'));
});
$container->set('exceptionHandler.class', 'ExceptionHandler');
$container->set('exceptionHandler.config', array($container->get('log')));
$container->set('exceptionHandler', function($c) {
$reflection = new \ReflectionClass($c->get('log.class'));
if (!$reflection->isSubclassOf('Handler')) {
throw new InvalidArgumentException('exceptionHandler.class does not implement the Handler interface');
}
return $reflection->newInstanceArgs($c->get('exceptionHandler.config'));
});
$exceptionHandler = $container->get('exceptionHandler');
try {
$foo = new Foo();
$foo->setBar('12'); // gooit met exceptions
} catch (\Exception $e) {
$exceptionHandler->handle($e);
}
?>
Gewijzigd op 29/06/2012 16:56:15 door Wouter J
Alleen, wat is precies de bedoeling van 'reflection'? Overigens, wat is het voordeel van zo'n container?
Of als je de locatie van het bestand wilt veranderen hoef je alleen regel 5 te veranderen.
Zo maak je je code beter aanpasbaar.
De ReflectionClass is een ingebouwde PHP klasse. Deze zorgt ervoor dat je klassen kunt aanmaken. In dit geval gebruik ik hem omdat je argumenten wilt meegeven aan de constructor (welk bestand, welke logger). Daarom maak ik een ReflectionClass aan met als argument de naam van de klasse (al eerder gedefinieerd) en vervolgens gebruik ik newInstanceArgs, welke de klasse aanmaakt met de argumenten die je opgeeft (in dit geval de array die ook al eerder is aangemaakt).
Edit:
Ik heb de code even bewerkt, waardoor je ook meteen het voordeel ziet van werken met interfaces. De reflectionclass gebruik ik nu ook om te controleren of ze wel een subklasse zijn van de Log of Handler interfaces.
Gewijzigd op 29/06/2012 16:57:02 door Wouter J
Ik begrijp dat het handig is dat je simpelweg door op lijn 4 wat aan te passen dat dit erg eenvoudig werkt. Langs de andere kant, het is weer extra code, extra objecten, ... (extra werkruimte, opslag) Weegt deze flexibiliteit op den duur nog op tegen de performantie? Daar heb ik eerlijk gezegd mijn twijfels bij. Verder bedenk ik dan ook, als je dit toepast, dan dien je eigenlijk al een hele denkpiste af te leggen eer je werkelijk doet wat je wilt doen. Dan vraag ik me af, is het niet eenvoudiger om simpel DI te gebruiken (m.a.w. via contructoren/setters doorgeven wat nodig is)?
Ik denk eerlijk gezegd dat je dit niet -hoeft- te gebruiken, maar dat het wel kan. Komt het eigenlijk er niet op neer dat je voor jezelf een bepaald niveau van flexibiliteit vastlegt, en hier naar streeft?
Quote:
Weegt deze flexibiliteit op den duur nog op tegen de performantie?
Nee. Dat is echt micro-optimalisatie, als je kijkt naar websites die gemaakt zijn met een framework dan zijn er zo'n 100 objecten nodig voor een pagina aanroep. Dit is dus echt geen verschil.
Quote:
Verder bedenk ik dan ook, als je dit toepast, dan dien je eigenlijk al een hele denkpiste af te leggen eer je werkelijk doet wat je wilt doen. Dan vraag ik me af, is het niet eenvoudiger om simpel DI te gebruiken (m.a.w. via contructoren/setters doorgeven wat nodig is)?
En dat is nou net het verschil tussen Flat scripten en OO scripten. In OO moet je voordat je begint met scripten eerst alles al hebben uitgedacht. Pas als je alles hebt uitgedacht en hebt uitgetekend in UML diagrammen begin je met de code en je zult zien dat je dan maar heel kort of te programmeren, omdat je alles al hebt uitgedacht.
Bij Flat PHP begin je gewoon ergens en zie je wel waar het schip strand. En dat is dus niet de manier hoe programmeren werkt, volgens OO mensen.
Quote:
Ik denk eerlijk gezegd dat je dit niet -hoeft- te gebruiken, maar dat het wel kan. Komt het eigenlijk er niet op neer dat je voor jezelf een bepaald niveau van flexibiliteit vastlegt, en hier naar streeft?
OO is eigenlijk alleen maar kijken tot welk level je wilt. Wil je gaan tot een BlogPost klasse, of ga je ook een extra Author klasse erbij maken? Dat zijn vragen die je je zelf moet stellen.
Maar ik heb geleerd uit ervaring dat je beter teveel klassen kan hebben dan te weinig. Hoe beter je alles opsplitst hoe flexibeler het wordt en dat is nou net OO. OO is niet zo zeer het denken in objecten, OO is het flexibel maken van je code.
Eerst en vooral ben ik het geheel eens met je reactie Wouter. Waar ik met mijn vorige post op doelde heb jij in jouw post kort en kracht gezegd: "OO is eigenlijk alleen maar kijken tot welk level je wilt."
De 'strategy pattern' begrijp ik geheel. Zonder de term te kennen heb ik dit in het verleden al toegepast. Niet in PHP maar in Java, maar goed dat maakt niets uit. Alleen ik zie niet in waarvoor jij dit zou gebruiken i.v.m. de gebruikerspermissies.
Misschien heeft er iemand wat aan, ik bekeek het artikel 'Strategy Design Pattern' en een filmpje in een of andere tutorialreeks: 'Strategy Pattern'
Misschien moet ik voor de zekerheid even kort toelichten welke 'mogelijkheden' ik zeker en vast wil gaan inbouwen. In eerste instantie wil ik een onderscheid kunnen maken tussen: bezoeker, 'geregistreerde zonder rechten', 'geregistreerde met rechten' Hiermee wil ik simpelweg bereiken, sommige pagina's kunnen door iedereen bekeken/gebruikt worden, anderen door ingelogde gebruikers en dan is er nog de admin sectie.
Als je er even mijn ERD bijhaalt en daar Groups aan toevoegt is dit makkelijk te verkrijgen. In de 'permissions table' een veld met die refereert naar de 'group table'. group 1 is bv. 'geregistreerde zonder rechten' en groep twee is logischerwijs 'geregistreerde met rechten'. Later zou ik dan willen uitbreiden met meer specifieke rechten, bv. een gebruiker mag een (andere) gebruiker verwijderen, bewerken, toevoegen, ... Dit zowel op 'group' niveau als op één specifieke gebruiker.
Ik zie echter niet in hoe ik dit op jouw manier / een andere efficiënte manier dit perfect kan bekomen. Op database niveau is dit geen enkel probleem:
Een gebruiker moet een record hebben in 'permissions' en optioneel is er een link naar een 'group'.
Ik hoop dat mijn vraag min of meer duidelijk is. Ik wil alvast iedereen bedanken, en in het bijzonder natuurlijk Wouter :-).
Gewijzigd op 03/07/2012 11:58:26 door Write Down
http://phphulp.wouterj.nl/usermanagement.zip
Toch nog een paar vragen. Een gebruiker logt in, wat is de beste manier om dan te bepalen wat voor een gebruiker hij is? Want met jouw voorbeeld, maak je dus voor een admin een ander object aan dan voor een user. Ik denk ook als deze vraag beantwoord is, dat mijn 2de vraag beantwoord zal zijn: wat is het nut van Guest, aangezien die toch geen rechten heeft.
Je zou dan gewoon alleen een User klasse kunnen hebben. Doormiddel van de set...Strategy kun je dan alle rechten instellen aan de hand van de data uit de DB.
En een Guest. Tja, soms moet je verplicht zijn om ergens in gelogd te zijn maar wil je je niet registreren, dan is er soms een Guest optie. Dan zit je in het Guest object, deze Guest heeft vaak een naam (Guest124) en zal dus ook een eigen object krijgen die bijv. deze naam vast houdt.
Namespaces zijn niet heel moeilijk. Misschien dat mijn uitleg hier wel helpt: http://www.phphulp.nl/php/forum/topic/oop-gedachtengang/85017/5/#609868 (al vind ik het een moeilijk uit te leggen onderwerp)
Gewijzigd op 03/07/2012 14:20:59 door Wouter J
De enige vraag die mij dan rest...
Vroeger hield ik bv. de username, hash, ... bij (sessie) en keek ik op elke pagina na of de ingelogde recht heeft om de huidige pagina te bekijken. Indien ja: gebruiker kan alles doen wat er op die pagina mogelijk is. Indien nee: gebruiker krijgt melding dat hij onbevoegd is.
Aangezien de gegevens nu in een user object terecht komen, hoe geef je dit object dan door? Alles in een sessie gaan stoppen? (lijkt me niet dé oplossing) Of ga je op elke pagina na a.d.h.v. user_id is deze gebruiker gerechtigd (authenticatie)? Ja: maak user object; nee: niet gemachtigd (opnieuw inloggen / melding).
Toevoeging op 03/07/2012 14:32:21:
Jeroen vd op 03/07/2012 14:23:28:
euh, even inbreken.... wat wil je bereiken met de Permissions tabel?
Bijhouden wat een gebruiker / groep al dan niet mag?
Eerst heb je de Authentication, je kijkt wie deze gebruiker is, of hij is ingelogd ect. Vervolgens doe je de Authorization die kijkt welke rechten je nodig hebt en welke rechten de Authentication heeft opgeleverd.
Gewijzigd op 03/07/2012 14:33:44 door Wouter J
In mijn vroegere applicaties liet ik inderdaad inloggen en keek ik op de beveiligde pagina: is de gebruiker wel nog dezelfde (IP veranderd, geprutst met parameters, ...)
Ga je in dit geval bij de 'authorization' nagaan wat ik hierboven beschrijf en telkens opnieuw de permissies opnieuw uit de database halen, user-object maken (set...Strategy), ...? Om daarna dan te bepalen: aha, dit mag dit niet?
Quote:
Bijhouden wat een gebruiker / groep al dan niet mag?
waarom in een aparte tabel? het lijkt mij dat een gebruiker maar 1 soort van controle heeft, hij kan geen admin en gast tegelijk zijn. zet in je usertabel een paar kolommen bij met de permisions, dit is namelijk uniek voor elke gebruiker, en nutteloos om daar dan een aparte tabel voor aan te maken (zie het als een wachtwoord - die zet je toch ook niet in een aparte tabel?)
Daar ben ik het niet mee eens. Waarom zou je het niet splitsen? Dit verhoogt naar mijn mening de flexibiliteit. Stel morgen maak ik een applicatie die maar een paar permissie instellingen kent. Overmorgen maak ik een totaal andere applicatie met meer complexe permissies. De 'functie' gebruiker is in beide applicaties hetzelfde, de permissies niet.
Verder is er dan ook nog het indelen van groepen, dit is makkelijker met een aparte tabel voor permissies. Overigens heb je vaak de permissies niet nodig, gewoon zuiver gebruikersnaam en dergelijke. Dan is je query sneller afgehandeld, aangezien er minder werk is. Langs de andere kant, als je permissies nodig hebt, heb je een extra tabel nodig. Echter is het zo dat als je op de juiste wijze JOINS gebruikt dit erg efficiënt werkt.
@Wouter
Lijkt me een goed plan. Volgens mij is het dan het handigste om in de User class een serialize en deserialize functie te voorzien? Of eventueel eerst in een interface stoppen, zodanig dat andere objecten op dezelfde manier geserialiseerd kunnen worden.