vraagje over ophalen van informatie en in een tabel zetten D.M.V OOP
Ik ben een beetje OOP PHP aan het uitproberen, en daar heb ik een vraag over.
Op het moment wil ik een simpel ding voor elkaar krijgen en dat is het volgende:
Ik heb een tabel in de database met de velden id en merk, en deze informatie wil ik uitlezen DMV een functie in de class (en dit in een tabel stoppen).
Ik heb het voor elkaar gekregen dat de informatie uit de database wordt gehaald, er een object van wordt gemaakt, en in een tabel wordt gezet. Maar ik heb de opbouw van de tabel in de functie zelf gedaan, en Dit is waarschijnlijk niet zoals het hoort. Dit omdat de functie nu maar voor 1 ding kan worden gebruikt, en dat is het ophalen van de informatie, en deze informatie wordt dan in een tabel getoond. Dit is te functie waar ik het over heb:
Code (php)
Ik heb het al geprobeerd om de self::find_by_sql($query) in een variabele te stoppen en deze te returnen. Maar als ik bijv $merk->id wil gebruiken, krijg ik de error dat ik geen private velden buiten de class kan gebruiken (wat logisch is, omdat ik deze private heb gemaakt in de class).
Heeft iemand een idee hoe ik dit op een nette manier kan doen (ik weet namelijk niet of dit is toegestaan)? dit omdat ik zelf nog niet veel ervaring heb met OOP, en het super graag wil leren.
alvast enorm bedankt!!
Gewijzigd op 07/10/2012 01:55:43 door Henk de Vries
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
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
<?php
class Persoon {
private $naam;
public function Persoon($naam) {
$this->naam = $naam;
}
public function getNaam() {
return $this->naam;
}
public function setNaam($naam) {
$this->naam = $naam;
}
};
$p = new Persoon("Frank");
echo $p->getNaam()."<br>";
$p->setNaam("Menno");
echo $p->getNaam()."<br>";
?>
class Persoon {
private $naam;
public function Persoon($naam) {
$this->naam = $naam;
}
public function getNaam() {
return $this->naam;
}
public function setNaam($naam) {
$this->naam = $naam;
}
};
$p = new Persoon("Frank");
echo $p->getNaam()."<br>";
$p->setNaam("Menno");
echo $p->getNaam()."<br>";
?>
probeer echo's in de onderliggende classes te vermijden.
beter meer kleine functies in je class dan grote met veel if else etc
self:: zul je niet zo gauw nodig hebben, gebruik voorlopig maar $this->
Gewijzigd op 07/10/2012 02:47:51 door Frank Nietbelangrijk
Ik heb nu de volgende code (ik laat even de hele class zien)
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
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
<?php
include_once("MySQLDatabase_class.php");
class Automerk{
private $id;
private $merk;
public function __construct($id='',
$merk='')
{
$this->id = $id;
$this->merk = $merk;
}
public static function find_by_sql( $query )
{
global $database;
$result_set = $database->fire_query( $query );
$merk = array();
while ( $row = mysqli_fetch_array($result_set))
{
$object = new Automerk();
$object->id = $row["id"];
$object->merk = $row["merk"];
$merk_array[] = $object;
}
return $merk_array;
}
public static function get_all_merken(){
$query = "select * from `merk`";
$result = self::find_by_sql($query);
return $result;
}
public static function create_table_merken(){
$get_merken = self::get_all_merken();
echo "<Table width='200' border=1>";
foreach ( $get_merken as $merk){
echo "<tr>";
echo "<td>$merk->id</td>";
echo "<td>$merk->merk</td>";
echo "<tr>";
}
echo "</table>";
}
}
?>
include_once("MySQLDatabase_class.php");
class Automerk{
private $id;
private $merk;
public function __construct($id='',
$merk='')
{
$this->id = $id;
$this->merk = $merk;
}
public static function find_by_sql( $query )
{
global $database;
$result_set = $database->fire_query( $query );
$merk = array();
while ( $row = mysqli_fetch_array($result_set))
{
$object = new Automerk();
$object->id = $row["id"];
$object->merk = $row["merk"];
$merk_array[] = $object;
}
return $merk_array;
}
public static function get_all_merken(){
$query = "select * from `merk`";
$result = self::find_by_sql($query);
return $result;
}
public static function create_table_merken(){
$get_merken = self::get_all_merken();
echo "<Table width='200' border=1>";
foreach ( $get_merken as $merk){
echo "<tr>";
echo "<td>$merk->id</td>";
echo "<td>$merk->merk</td>";
echo "<tr>";
}
echo "</table>";
}
}
?>
Ik heb er nu voor gekozen om de tabel in een andere functie te zetten. Nu kan ik door simpelweg create_table_merken() op te roepen de tabel met merken tevoorschijn halen, en kan ik de get_all_merken() voor meerdere dingen gebruiken (als ik bijvoorbeeld list items van de merken wil maken).
Ik hoop dat iemand mij feedback kan geven of ik de juiste richting op ga, of dat ik een aantal zaken moet bijschaven.
alvast thx!
Gewijzigd op 07/10/2012 19:29:32 door Henk de Vries
In OO probeer je je applicatie zo flexibel mogelijk te maken, alles moet met een paar regels verschil of slechts met 1 klasse te veranderen aan te passen zijn. Om hiertoe te komen is er 1 hele belangrijke regel:
Elke klasse en elke method heeft maar 1 verantwoordelijkheid, zodra je bij de omschrijving wat een klasse/method doet het woordje en gebruikt splits je het op in meerdere klassen/methoden
In jou geval ga je hier flink tegenin. De automerk klasse houdt en de gegevens vast van het merk en haalt de gegevens op uit de database. Deze 2 splits je op in klassen: 1 die het automerk ophaalt uit de database/een bestand/ect. en 1 die de gegevens vast houdt. Die nieuwe klassen noemen we een DataMapper. Kijk ook eens naar deze uitleg en voorbeelden: http://www.phphulp.nl/php/forum/topic/oop-gedachtengang/85017/1/#606620 en http://www.phphulp.nl/php/forum/topic/oop-in-combinatie-met-database/81754/#580025
De verwerking van data in klassen is 1 van de moeilijkste dingen in OO en daarom zijn er heel wat oplossingen voor. Naast de DataMapper, welke ik aanraadt, wordt ook het Active Record pattern gebruikt: http://www.phphulp.nl/php/forum/topic/applicatie-naamgeving/81370/3/#578116 en ook een ORM (zoals Doctrine of Propel) en nog veel meer oplossingen.