[OOP] Ben ik op de goede weg ?
Het probleem is dat me eigenlijk nog niet geleerd is hoe ik klassen moet ontwerpen, maar aangezien dit best een simpel voorbeeld is (denk ik..) durf ik me er wel aan te wagen.
Ik hou van het maken van foto's en programmeren. Nu wou ik die twee hobby's combineren dus ik had bedacht om een remote shooting script te schrijven. Nu heb ik onder Linux het programma "gphoto2" geïnstalleerd, dit werkt uitstekend en het stelt me in staat om m'n spiegelreflex op een afstand te besturen.
Nu wil ik zorgen dat ik vanaf een PHP script deze software kan besturen. Nu weet ik wel dat dat met shell_exec() uitstekend gaat en is het me ook al gelukt om een OOP PHP script te schrijven dat prima werkt (zie http://mcdronkz.ath.cx/remotecapture/ password is "fahrftw") maar de klassen steken volgens mij onlogisch in elkaar en daar hoop ik door dit topic wat verandering in te brengen.
Ik wil het vanaf nu wat meer to the point gaan houden want ik begrijp dat bovenstaand stukje misschien een beetje te uitgebreid is beschreven, dus hier komt 'ie.
Eerst even kijken welke objecten ik nodig heb:
Ik heb een programma genaamd gphoto, deze is aan één of meerdere camera's gekoppeld, de geselecteerde camera moet een foto maken nadat iemand het goede wachtwoord heeft ingegeven.
Hier haal ik sowieso de volgende objecten uit: camera, foto, authenticatie.
Laten we beginnen met de makkelijkste, authenticatie:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
/*
Het is niet nodig om hier een instance van te maken, daarom gebruik ik een static class.
*/
static class Authentication () {
$password = "sdghsd235235uwdhghsdgs32523"; //MD5 string
public static function checkPassword($password) {
/*
Deze controleert of md5($password) gelijk is aan $this->password en returned uiteindelijk true of false.
*/
}
}
?>
/*
Het is niet nodig om hier een instance van te maken, daarom gebruik ik een static class.
*/
static class Authentication () {
$password = "sdghsd235235uwdhghsdgs32523"; //MD5 string
public static function checkPassword($password) {
/*
Deze controleert of md5($password) gelijk is aan $this->password en returned uiteindelijk true of false.
*/
}
}
?>
Ok, dan hebben we die gehad.
Dan krijgen we de camera. We willen dus één of meerdere camera's kunnen bedienen. Ik denk dat ik daarvoor een handler nodig heb, maar dat is van latere zorg.
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
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
<?php
class Camera () {
private $model; //Model is het merk en type van de camera, hiermee kun je meerdere camera's onderscheiden.
function __construct($model) {
setModel($model);
}
public function setModel($model) {
$this->model = $model;
}
public function getModel() {
return $this->model;
}
public function captureAndDownloadPicture($filename) {
/*
Maak een foto en sla deze op de schijf op met de ingegeven bestandsnaam. Maak vervolgens een instance van een picture en return deze.
*/
$picture = new Picture($filename);
return $picture; //$picture is van het type picture
}
public function getAperture() {
}
public function getShuttertime() {
}
public function getISOSpeed() {
}
}
?>
class Camera () {
private $model; //Model is het merk en type van de camera, hiermee kun je meerdere camera's onderscheiden.
function __construct($model) {
setModel($model);
}
public function setModel($model) {
$this->model = $model;
}
public function getModel() {
return $this->model;
}
public function captureAndDownloadPicture($filename) {
/*
Maak een foto en sla deze op de schijf op met de ingegeven bestandsnaam. Maak vervolgens een instance van een picture en return deze.
*/
$picture = new Picture($filename);
return $picture; //$picture is van het type picture
}
public function getAperture() {
}
public function getShuttertime() {
}
public function getISOSpeed() {
}
}
?>
Ik denk dat ik wat betreft de camera zo aardig compleet ben. Nu moet ik nog meerdere camera's kunnen bedienen, en dat wou ik met een cameraHandler object gaan doen. Het is overigens de bedoeling dat het procedurele gedeelte deze klasse rechtstreeks aanroept, en hier een instance van maakt. Dit is wel iets waarvan ik niet helemaal zeker ben, dus input is zeker gewenst hier.
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
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
<?php
class cameraHandler () {
private $cameras = array(); //Array waar camera objecten in komen te staan.
function __construct() {
/*
Maak instances van de camera objecten en plaats deze in $this->cameras.
*/
}
public function getCameras() {
return $this->cameras;
}
public function getPicture($model) {
/*
Vooral over deze functie ben ik onzeker. Het is de bedoeling dat ik straks de foto kan resizen en dergelijke, en
het lijkt me niet dat cameraHandler deze verantwoordelijkheid krijgt. Daarom ga ik straks een picture klasse en een
pictureHandler klasse schrijven. De camera maakt een instance van een picture, die ik hier vervolgens met een statische
pictureHandler object kan bewerken.
*/
if(in_array($model, $this->cameras)) {
$filename = "pictures/".time().".jpg";
if(($picture = $this->cameras[$model]->captureAndDownloadPicture($filename)) != false) {
pictureHandler::setPermissions($picture);
pictureHandler::resizePicture($picture);
pictureHandler::putLabelOnPicture($picture);
return $picture;
}
}
/*
Return false als het niet gelukt is om een foto te maken.
*/
return false;
}
}
?>
class cameraHandler () {
private $cameras = array(); //Array waar camera objecten in komen te staan.
function __construct() {
/*
Maak instances van de camera objecten en plaats deze in $this->cameras.
*/
}
public function getCameras() {
return $this->cameras;
}
public function getPicture($model) {
/*
Vooral over deze functie ben ik onzeker. Het is de bedoeling dat ik straks de foto kan resizen en dergelijke, en
het lijkt me niet dat cameraHandler deze verantwoordelijkheid krijgt. Daarom ga ik straks een picture klasse en een
pictureHandler klasse schrijven. De camera maakt een instance van een picture, die ik hier vervolgens met een statische
pictureHandler object kan bewerken.
*/
if(in_array($model, $this->cameras)) {
$filename = "pictures/".time().".jpg";
if(($picture = $this->cameras[$model]->captureAndDownloadPicture($filename)) != false) {
pictureHandler::setPermissions($picture);
pictureHandler::resizePicture($picture);
pictureHandler::putLabelOnPicture($picture);
return $picture;
}
}
/*
Return false als het niet gelukt is om een foto te maken.
*/
return false;
}
}
?>
Dan moeten we dus een picture klasse maken, deze bevat de bestandsnaam van de afbeelding maar kan in de toekomst natuurlijk uitgebreid worden met EXIF informatie return functies en dat soort zaken.
Code (php)
En dan de pictureHandler:
Code (php)
Dan zou je het volgende stukje procuderele code kunnen gebruiken:
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
<?php
$handler = new cameraHandler();
if($_SERVER['REQUEST_METHOD'] == "POST") {
if(Authentication::checkPassword($_POST['password'])) {
if(($picture = $handler->getPicture($_POST['model']) != false)) {
echo "Er is een foto gemaakt met de volgende filename: ".$picture->getFilename();
} else {
echo "Er ging iets fout tijdens het maken van de foto.";
}
} else {
echo "Voer het correcte wachtwoord in.";
}
} else {
/*
Toon een formulier met de camera's die gekozen kunnen worden en waar het wachtwoord ingevoerd kan worden.
*/
}
?>
$handler = new cameraHandler();
if($_SERVER['REQUEST_METHOD'] == "POST") {
if(Authentication::checkPassword($_POST['password'])) {
if(($picture = $handler->getPicture($_POST['model']) != false)) {
echo "Er is een foto gemaakt met de volgende filename: ".$picture->getFilename();
} else {
echo "Er ging iets fout tijdens het maken van de foto.";
}
} else {
echo "Voer het correcte wachtwoord in.";
}
} else {
/*
Toon een formulier met de camera's die gekozen kunnen worden en waar het wachtwoord ingevoerd kan worden.
*/
}
?>
Is dit nou de juiste approach ? Ik moet je eerlijk zeggen dat het behoorlijk lang geduurd heeft om deze vraag op te stellen omdat het voor mij nog moeilijk is om de relaties tussen verschillende objecten te zien en te bepalen. De code die ik hier geschreven heb kan ik voor mezelf wel rijmen denk ik, als ik kijk wat het doet is het voor mij vrij logisch.
Toch wou ik graag jullie kijk erop.
Gewijzigd op 01/01/1970 01:00:00 door Willem-Aart
Ik krijg net van mijn vader de opdracht om een 'bewakings camera' te bouwen. Ik ga een simpel linux bakkie draaien waar een cam aan hangt, een programmatje gaat alles netjes opnemen en ik stream het vervolgens dmv PHP naar het control panel. Ik wil ook wat shell cmds gaan gebruiken om het programma te bedienen vanuit de interface die ik ga creëren. Dit wil mijn vader voor op vakantie.
Tommy schreef op 29.11.2008 15:51:
Leuk idee.
Ik krijg net van mijn vader de opdracht om een 'bewakings camera' te bouwen. Ik ga een simpel linux bakkie draaien waar een cam aan hangt, een programmatje gaat alles netjes opnemen en ik stream het vervolgens dmv PHP naar het control panel. Ik wil ook wat shell cmds gaan gebruiken om het programma te bedienen vanuit de interface die ik ga creëren. Dit wil mijn vader voor op vakantie.
Ik krijg net van mijn vader de opdracht om een 'bewakings camera' te bouwen. Ik ga een simpel linux bakkie draaien waar een cam aan hangt, een programmatje gaat alles netjes opnemen en ik stream het vervolgens dmv PHP naar het control panel. Ik wil ook wat shell cmds gaan gebruiken om het programma te bedienen vanuit de interface die ik ga creëren. Dit wil mijn vader voor op vakantie.
Dat is wel even een ander verhaal natuurlijk, waarschijnlijk maak je dan gewoon gebruik van een webcam (voeding is dan wat makkelijker, gewoon via USB). Hierbij werk ik echt met losse bestanden :).
Ik heb m'n camera trouwens weer ingeschakeld voor de geïnteresseerden.
:-) Als je zo'n diagram kunt geven kunnen andere devvers wat makkelijker zien of je ontwerp klopt.
En als je voordat je gaat proggen zo'n diagram tekent kun je soms al wat ontwerp foutjes opmerken :)
In de meeste PHP OOP tutorials ontbreekt het stukje UML, terwijl dat bij OOP toch eigenlijk ook best belangrijk is bij het ontwerpen. Beetje zonde is dat..