Hoe groot zijn jullie ..
OOP User classe
Ben sinds kort bezig met OOP. Hieronder tref je het begin van mijn User.class.php aan. Wat ik mij nu afvraag: Is het nu de bedoeling dat ik voor elke eigenschap (lees: elke var) van de class een aparte functie schrijf om het betreffende veld uit de database te plukken (dus: function email(), function username(), function registerDate(), etc.)? Dat lijkt me nogal inefficient.
En is het dan handig dat ik de functies gewoon email, username, etc. noem? Of kan dat dan beter iets zijn in de trant van getEmail, getUsername,etc. Want in dat laatste geval kan je net zo makkelijk een algemene getField() functie maken.
Het kan zijn dat het zo hoort hoor? Graag hoor ik jullie aanwijzingen.
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
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
<?php
/****
*
* User class
*
****/
class User{
private $_db;
private $_id;
private $_username;
private $_email;
private $_registerDate;
private $_permission;
public function __construct($id=0, $db)
{
$this->_id = $id;
$this->_db = $db;
}
public function email()
{
$sql = "SELECT email FROM user WHERE id = '".$this->_id."'";
$res = $this->_db->query($sql);
while($row = $this->_db(fetchAssoc($res)){
$this->_email = $row['email'];
}
}//-> email
}//--> User
?>
/****
*
* User class
*
****/
class User{
private $_db;
private $_id;
private $_username;
private $_email;
private $_registerDate;
private $_permission;
public function __construct($id=0, $db)
{
$this->_id = $id;
$this->_db = $db;
}
public function email()
{
$sql = "SELECT email FROM user WHERE id = '".$this->_id."'";
$res = $this->_db->query($sql);
while($row = $this->_db(fetchAssoc($res)){
$this->_email = $row['email'];
}
}//--> User
?>
Code (php)
1
$this->_Username = $row['Username'];
gebruiken
en dan een aparte functie
Code (php)
1
2
3
4
2
3
4
public function getUsername () {
return $this->_Username;
}
return $this->_Username;
}
maarja, ik ben ook maar 1 weeekje bezig met oop.
Dus weet niet of ik het helemaal goed heb.
doe wel wat met je foutafhandeling
zoiets:
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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
<?php
/****
*
* User class
*
****/
class User{
private $_db;
private $_id;
private $_username;
private $_email;
private $_registerDate;
private $_permission;
public function __construct($id=0, $db)
{
$this->_id = $id;
$this->_db = $db;
$sql = "SELECT id,username,email FROM user WHERE id = '".$this->_id."'";
$res = $this->_db->query($sql);
while($row = $this->_db(fetchAssoc($res)){
$this->_email = $row['id'];
$this->_username = $row['username'];
$this->_id = $row['email'];
}
}
public function getUsername()
{
return $this->_username;
}
public function getEmail()
{
return $this->_email;
}
}
?>
/****
*
* User class
*
****/
class User{
private $_db;
private $_id;
private $_username;
private $_email;
private $_registerDate;
private $_permission;
public function __construct($id=0, $db)
{
$this->_id = $id;
$this->_db = $db;
$sql = "SELECT id,username,email FROM user WHERE id = '".$this->_id."'";
$res = $this->_db->query($sql);
while($row = $this->_db(fetchAssoc($res)){
$this->_email = $row['id'];
$this->_username = $row['username'];
$this->_id = $row['email'];
}
}
public function getUsername()
{
return $this->_username;
}
public function getEmail()
{
return $this->_email;
}
}
?>
Gewijzigd op 28/12/2010 12:14:38 door Dindong Veter
Als je PDO gebruikt, en je database goed in mekaar steekt, dan kan je in één keer je data ophalen en naar een User casten, het is inderdaad niet de bedoeling om voor iedere variable opnieuw data op te gaan halen.
En kijk ook even naar het overerven van een standaard Controller class, die al connectie maakt met een database, dan hoef je dat ook niet meer steeds te doen.
Ik snap het idee, maar ook dit lijkt me inefficient. Wel tof dat je meedenkt.
@Nicoow:
Ja, je hebt ook helemaal gelijk. Maar als ik het MVC probeer op te zetten loop ik compleet vast. Is het aan te raden om PDO te gebruiken? Ik heb dat een tijd terug uitgeprobeerd, maar toen raadde iedereen het mij af...
Volgens mij maak ik overigens niet steeds opnieuw verbinding met de database. Ik moet alleen wel het object meenemen in de class toch? Anders kan ik er niet bij. Dit is mijn database.class.php:
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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
<?php
/*****
*
* Class - Database
*
* Deze class handelt de communicatie met de MySQL Database af.
* De methods kunnen verder worden uitgebreid.
*
* ------------------------------------------------------------
*
* Gebruik:
*
* $result = $db->query("SELECT username FROM users");
*
* while($row = $db->fetchAssoc($result)) {
* echo($row['username']);
* }
*
*****/
class Database {
private $_link;
public function connect($server='', $username='', $password='', $new_link=true, $client_flags=0) {
$this->_link = mysql_connect($server, $username, $password, $new_link, $client_flags);
}
public function selectDatabase($db){
return mysql_select_db($db, $this->_link);
}
public function error() {
return mysql_error($this->_link);
}
public function errno() {
return mysql_errno($this->_link);
}
public function escape($string) {
return mysql_real_escape_string($string, $this->_link);
}
public function query($query) {
return mysql_query($query, $this->_link);
}
public function fetchArray($result, $array_type = MYSQL_BOTH) {
return mysql_fetch_array($result, $array_type);
}
public function fetchRow($result) {
return mysql_fetch_row($result);
}
public function fetchAssoc($result) {
return mysql_fetch_assoc($result);
}
public function fetchObject($result) {
return mysql_fetch_object($result);
}
public function numRows($result) {
return mysql_num_rows($result);
}
public function close() {
return mysql_close($this->_link);
}
}
?>
/*****
*
* Class - Database
*
* Deze class handelt de communicatie met de MySQL Database af.
* De methods kunnen verder worden uitgebreid.
*
* ------------------------------------------------------------
*
* Gebruik:
*
* $result = $db->query("SELECT username FROM users");
*
* while($row = $db->fetchAssoc($result)) {
* echo($row['username']);
* }
*
*****/
class Database {
private $_link;
public function connect($server='', $username='', $password='', $new_link=true, $client_flags=0) {
$this->_link = mysql_connect($server, $username, $password, $new_link, $client_flags);
}
public function selectDatabase($db){
return mysql_select_db($db, $this->_link);
}
public function error() {
return mysql_error($this->_link);
}
public function errno() {
return mysql_errno($this->_link);
}
public function escape($string) {
return mysql_real_escape_string($string, $this->_link);
}
public function query($query) {
return mysql_query($query, $this->_link);
}
public function fetchArray($result, $array_type = MYSQL_BOTH) {
return mysql_fetch_array($result, $array_type);
}
public function fetchRow($result) {
return mysql_fetch_row($result);
}
public function fetchAssoc($result) {
return mysql_fetch_assoc($result);
}
public function fetchObject($result) {
return mysql_fetch_object($result);
}
public function numRows($result) {
return mysql_num_rows($result);
}
public function close() {
return mysql_close($this->_link);
}
}
?>
Deze komt overigens van Blanche af als ik me niet vergis.
Bij MVC ga je query's uitvoeren in je model en met het resultaat van deze querys objecten opzetten. Later kunnen deze dan terug gestuurd worden naar je controller.
Moe BE op 28/12/2010 12:46:26:
Mij is altijd verteld om query's buiten je objecten te houden.
Bij MVC ga je query's uitvoeren in je model en met het resultaat van deze querys objecten opzetten. Later kunnen deze dan terug gestuurd worden naar je controller.
Bij MVC ga je query's uitvoeren in je model en met het resultaat van deze querys objecten opzetten. Later kunnen deze dan terug gestuurd worden naar je controller.
The Ultimate op 28/12/2010 12:28:48:
Het was dus ook mijn bedoeling om het niet MVC te doen maar wel OOP. ;-)... als ik het MVC probeer op te zetten loop ik compleet vast.
Moe BE op 28/12/2010 12:58:53:
Maar dat wil toch niet zeggen dat ik de werking van MVC helemaal begrijp? Geloof me, als ik MVC goed begreep zou ik het MVC doen.Je kan OOP toch perfect gebruiken in een MVC omgeving.
Maar hou je vooral niet in en toon mij de weg...
Heb jij een voorbeeld van een goede MVC UserController.class.php + User.class.php?
Gewijzigd op 28/12/2010 13:05:16 door The Ultimate
Maak eerst een duidelijke mappen structuur die verschil brengt in de verschillende lagen van het MVC.
bijvoorbeeld:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
2
3
4
5
6
7
8
9
10
11
12
controllers
- BaseController.php // Alle standaard functionaliteit welke elke controller erft die hem extend
models
- user
- enz
- enz
views
- scripts
- index ( controller naam )
- index.phtml ( action naam, methode naam o.i.d )
- partials
- helpers
- BaseController.php // Alle standaard functionaliteit welke elke controller erft die hem extend
models
- user
- enz
- enz
views
- scripts
- index ( controller naam )
- index.phtml ( action naam, methode naam o.i.d )
- partials
- helpers
Ik leg het MVC model altijd zo uit.
Wat is MVC en wat doet het:
MVC Pattern staat voor het Model View Controller patroon. Het model zorgt voor alle informatie uit een data bron bijvoorbeeld een database. Het model geeft deze informatie door aan de controller die deze verwerkt of bewerkt en deze doorgeeft aan de view ( de output). De controller is dus eigenlijk een knooppunt van alle ‘bronnen’.
- Het Model representeert de applicatie data
- De View maakt een presentatie van de model gegevens
- De Controller is verantwoordelijk voor het verwerken van de data
Hieronder een standaard MVC Request.
De bovenstaande figuur geeft een voorbeeld van een simpel MVC request weer. Hieronder is puntsgewijs weergegeven hoe daadwerkelijk het MVC een request correct afhandeld.
1. Client klikt op een link. Zijn browser maakt een request aan naar de webserver
2. De dispatcher ( uitvoerder ) controleert de request URL en geeft de request door aan de juiste controller
3. De controller voert de applicatie specifieke logica uit. Bijvoorbeeld of de client is ingelogd.
4. Daarnaast gebruikt de controller een of meerdere modellen om toegang tot de applicatie data te verkrijgen. Meestal representeert een model een database tabel, maar kan ook een tekst bestand wezen.
5. Zodra het model de data heeft verkregen en deze heeft doorgegeven aan de controller geeft de controller deze data door aan de view. De view pakt deze data op en maakt deze klaar voor presentatie aan de gebruiker. Views zijn normaliter in HTML format, maar dit zou evengoed een PDF, XML document of een JSON object kunnen zijn, afhankelijk van de wensen van de applicatie ontwikkelaar.
6. Als de view de data van de controller heeft gebruikt om een volledige view op te bouwen, wordt de inhoud van die view teruggestuurd naar de browser van de client.
Bij elke request die naar de applicatie wordt verstuurd volgt dit basispatroon.
Waarom MVC:
Waarom gebruik maken van MVC? Omdat het een bewezen software design patroon blijkt te zijn dat van een applicatie, een goed onderhoudbaar, modulair en snel te ontwikkelen pakket maakt. De taken van de toepassingen onderverdelen in aparte modellen, views en controllers maakt de applicatie geheel lichtvoetig.
Gewijzigd op 29/08/2011 20:13:11 door Niels K
Mooie uitleg. Stel nu dat je een user uit de database wil halen hoe zou jij dit dan doen?
Stel je request komt binnen op www.jouwsite.nl/inloggen
Dan heb je bijvoorbeeld een loginAction in de indexcontroller en in die loginaction wil je dan een user ophalen. Hoe zou je dat dan doen?
Zo?
Code (php)
Of zo?
Code (php)
Of heel anders?
Eerste optie bouw je een extra laag in het MVC, welke eigenlijk nergens toe dient. Niet doen dus ;)
Verder, maak je gebruik van iets als een BaseController ?
Gewijzigd op 28/12/2010 14:08:24 door Niels K
Is bovengenoemde niet goed (of niet gebruikelijk)?
Toevoeging op 28/12/2010 14:09:50:
Niels Kieviet op 28/12/2010 14:02:43:
Verder, maak je gebruik van iets als een BaseController ?
Wie ik?
Quote:
Volgens mij heb ik namelijk zo'n optie als optie 1 ook wel eens gezien. Dus $user = new User() en user spreekt dan met UserModel. En als je dan bijvoorbeeld de naam van de user wil dan zeg je echo $user->getName();
Waar precies? Want dat is geen goeie vertaling van het MVC pattern
Quote:
Wie ik?
Ja :-)
Volgens de Java richtlijnen moet je voor elke eigenschap die opvraag moet zijn een get functie maken bijvoorbeeld getUsername(). En ook een set functie voor elke eigenschap die muteerbaar moet zijn.
Je hebt in PHP5 ook magic functions __Set en __Get als iemand van buiten de scope dan $object->eigenschap aanroept dan word __Get aangeroepen in je klasse en word de gevraagde eigenschap als argument meegegeven.
@Topicstarter: Ik zou sowieso proberen niet meer de MySQL extensie te gebruiken, naast dat deze erg oud is zal deze verdwijnen in PHP6. MySQLi is daarentegen wel beschikbaar in PHP6. Ook PDO is een goed alternatief, en kan ik je aanraden om deze te gebruiken!
Waar staat dat de mysql extensie verdwijnt? Hij wordt alleen niet meer standaard ondersteund dacht ik.
En wat noem jij een vraag van de gebruiker? Een request?
@TS
Je kan ook eens kijken naar een bestaand framework?
Gewijzigd op 28/12/2010 14:15:10 door Niels K
Misschien mis ik een stukje uit de context, omdat ik niet zoveel actief ben hier. Maar de gebruiker (en daarmee bedoel ik in dit geval "The Ultimate" mee) vraagt iets over zijn user klasse en hij lijkt mij nog niet zo bekend met frameworks. Voor jou, mij en andere is MVC al vanzelfsprekend geworden wellicht maar ik zelf had er best wat moeite mee om het onder de knie te krijgen in het begin.
Daarom vind ik het onnodig om hem meteen advies te geven over hoe zijn applicatie zou moeten bouwen, maar zou het advies op de user klasse gericht moeten zijn in mijn ogen. Maargoed, zoals ik al zei ik ken de context niet en misschien was hij al eerder bezig met MVC o.i.d.
Niels Kieviet op 28/12/2010 14:12:52:
Library is voor je framework, en voor bijvoorbeeld een datum of een money converter, of een classe met allemaal constanten met een regex.
Waar precies? Want dat is geen goeie vertaling van het MVC pattern
Ja :-)
Quote:
Volgens mij heb ik namelijk zo'n optie als optie 1 ook wel eens gezien. Dus $user = new User() en user spreekt dan met UserModel. En als je dan bijvoorbeeld de naam van de user wil dan zeg je echo $user->getName();
Waar precies? Want dat is geen goeie vertaling van het MVC pattern
Quote:
Wie ik?
Ja :-)
Yep :)
"Waar precies? Want dat is geen goeie vertaling van het MVC pattern"
Volgens mij in mn vorige baan :)
Maar de library gebruik je dus voor algemene / generieke classes? Bijvoorbeeld een File class waarmee je een bestandsnaam kunt ophalen ofzo? $file_name = File::getName($file);
Zoiets bedoel je? Dat het dus eigenlijk een soort tools zijn?
En als je een user ophaalt zeg je dan echt $user = userModel()?
Gewijzigd op 28/12/2010 14:29:23 door Ozzie PHP
Quote:
Volgens mij in mn vorige baan :)
Ok dan, tja het is natuurlijk een eigen keuze en ze hebben er misschien wel een goede reden voor gehad, maar voor zover ik weet is het geen goede vertaling van het MVC.
Quote:
Maar de library gebruik je dus voor algemene / generieke classes? Bijvoorbeeld een File class waarmee je een bestandsnaam kunt ophalen ofzo? $file_name = File::getName($file);
Precies, of als je bijvoorbeeld mijn dependency injection container gebruikt, of iets van zend kan je die er gewoon ingooien en daar gebruik van maken. Je gooit gewoon je library map in je include path en gaan met die banaan :)
Quote:
En als je een user ophaalt zeg je dan echt $user = userModel()?
Welnee dat hoeft niet. Naamgeving is je eigen keuze. Bij zend framework maken ze gebruik van het table / row princiepe en daar zet je voor je database models Table_NaamVanModel. Dat staat voor 'in de map table' is een bestand wat NaamVanModel.php heet en in dat bestand heet de klasse Table_NaamvanModel
En dan heb je ook nog de Table_Row objecten ;) Google er eens op, leuke database implementatie is dat. Goed over nagedacht door zend :)
@TS
Ik zal vanavond wat extra commentaar plaatsen op jouw klasse heb ik nog wat commentaar op.
Gewijzigd op 28/12/2010 14:47:34 door Niels K
Niels Kieviet op 28/12/2010 14:44:58:
Precies, of als je bijvoorbeeld mijn dependency injection container gebruikt
je WAaaaAT????
Toevoeging op 28/12/2010 14:50:54:
Niels Kieviet op 28/12/2010 14:44:58:
Welnee dat hoeft niet. Naamgeving is je eigen keuze.
Herkent Zend Framework een model niet aan de naamgeving? Dus dat ie automatisch in de model map zoekt als je $user = userModel() aanroept?
Leuk topic dit overigens.. Gaat wel een beetje offtopic maar goed.
Quote:
Herkent Zend Framework een model niet aan de naamgeving? Dus dat ie automatisch in de model map zoekt als je $user = userModel() aanroept?
Dat zei ik toch? Als je Table_ ervoor zet zoekt hij in de map table/, maar je moet gewoon je model map in je include path zetten, dan zoekt php daar automatisch in, dat heeft niets met Zend te maken.
Gewijzigd op 28/12/2010 14:53:56 door Niels K