[Check] OOP class

J.Rijdes · 21 December 2013

Hallo,

Momenteel ben ik bezig met het oefenen van OOP programmeren. Ik heb nu een class gemaakt wat werkt, maar is het ook op de juist manier en gebruikt ik niet teveel overbodige dingen in mijn class? Graag hoor ik van jullie.

Mijn script:

PHP

<?php


include_once('connection.php');


class Melding {


	public $db;


	public function __construct(){
		$this->db = new Connection();
		$this->db = $this->db->dbConnect();
	}


	public function showMeldingen() {
		$st = $this->db->prepare("SELECT * FROM meldingen");
		$st->execute();
		$show = $st->fetchAll();


		foreach($show as $row) {
			echo "<b> " . $row['melding_titel'] . " </b> ";
			echo "<br>";
			echo nl2br($row['melding_eenheden']);
			echo "<hr>";
		}
	}
	public function postMelding($melding_titel, $melding_eenheden) {
		if($melding_titel == "") {
			return "Geen titel ingevuld.";
		} elseif($melding_eenheden == "") {
			return "Geen eenheden ingevuld";
		} else {
			$st = $this->db->prepare("INSERT INTO meldingen (melding_titel, melding_eenheden) VALUES (?, ?)");
			$st->BindParam(1, $melding_titel);
			$st->BindParam(2, $melding_eenheden);
			$st->execute();
		}
	}
}


?>

Toon Meer

Alvast bedankt.

Luc · 22 December 2013

Waarom heb je $db public gemaakt?

Voorkom in je class dat je gaat echo'en dat is niet super netjes.

WHMCSAddons · 22 December 2013

Met Luc eens, een variable die (ik vermoed) als database connectie gaat moet je binnen de class houden, erbuiten is niet nodig.

private - Only the current class will have access to the field or method.
protected - Only the current class and subclasses (and sometimes also same-package classes) of this class will have access to the field or method.
public - Any class can refer to the field or call the method.

http://stackoverflow.com/quest…ject-oriented-programming

J.Rijdes · 22 December 2013

Bedankt, dit ga ik gelijk aanpassen. Hoe kan ik dat met de echo beter oplossen? Door een return of een hele andere manier?

M.Beers · 22 December 2013

Citaat van Wmdiensten

Met Luc eens, een variable die (ik vermoed) als database connectie gaat moet je binnen de class houden, erbuiten is niet nodig.

private - Only the current class will have access to the field or method.
protected - Only the current class and subclasses (and sometimes also same-package classes) of this class will have access to the field or method.
public - Any class can refer to the field or call the method.

http://stackoverflow.com/quest…ject-oriented-programming

Nog een aanvullig hierop waarvan ik niet helemaal zeker weet of dit al doorgevoerd is in PHP maar vast en zeker gaat komen...
Wanneer je geen public, protected of private gebruikt is het alleen toegestaan binnen de package waar je classe zich in bevindt. Vanaf PHP 5.3 kan je gebruik maken van packages in php namespaces genoemd.

http://php.net/namespaces

Citaat

Bedankt, dit ga ik gelijk aanpassen. Hoe kan ik dat met de echo beter oplossen? Door een return of een hele andere manier?

Het is gebruikelijk om je uitvoer/acties en je layout gescheiden te houden. Het meest gebruikte pattern is hiervan het MVC pattern. Dus eigenlijk echo je nooit wat maar return je de array die je vervolgens verwerkt in je layout/html bestanden.

Victor · 22 December 2013

Een object georienteerde class hoor je te kunnen hergebruiken zonder de code aan te passen. Dat kan in dit geval niet. Mocht ik een Engelse site hebben, zou ik de meldingen moeten aanpassen. Misschien wil ik wel dat er helemaal geen melding komt, maar dat hij alleen wordt gelogd.

Stefan.J · 22 December 2013

Alles wat hierboven wordt gezegd klopt, maar wat ik nog niet heb gehoord is dat je fundamenteel de fout in gaat.

Een melding, wat kan die? Een melding kan denk ik niet een melding opslaan of alle meldingen weergeven. Een melding is in jouw geval een domeinobject en heeft vooral dingen en kan niet zozeer iets zelf doen. Een melding heeft bijvoorbeeld een titel en een bericht.

De class hierboven is daarom ook de MeldingService, en niet de melding zelf.. En dat object hoor je ook te hebben!

J.Rijdes · 22 December 2013

Bedankt allen. Ik heb even wat aanpassing gedaan en ben aan het nadenken hoe ik het beter zonder echo's of return's met tekst kan doen, maar met variabelen. Ik ben volgens mij een eindje op weg, maar ook op de goede weg?

PHP

<?php


include_once('connection.php');


class MeldingService {


	private $db;


	public function __construct(){
		$this->db = new Connection();
		$this->db = $this->db->dbConnect();
	}


	public function checkMeldingTitel($input, $error) {
			if($input == "") {
			return $error;
			} else {
				$error_count = 0;
			}
		}


	public function checkMeldingEenheden($input, $error) {
			if($input == "") {
			return $error;
			} else {
				$error_count = 0;
			}
		}


	public function saveMelding($melding_titel, $melding_eenheden, $error) {
			if($error_count == 0) {
			$st = $this->db->prepare("INSERT INTO meldingen (melding_titel, melding_eenheden) VALUES (?, ?)");
			$st->BindParam(1, $melding_titel);
			$st->BindParam(2, $melding_eenheden);
			$st->execute();
			} else {
				return $error;
			}
		}
	}
}


?>

Toon Meer

WHMCSAddons · 23 December 2013

Waarom maak je geen Class van je database en roep je alles door middel van 1 bestand die een autoload heeft ?

http://php.net/autoload

DNS update Directadmin

Graag advies voor mijn website

Ik wil op afstand studeren, kennen jullie Tech?

PC probleem

Let op Vimexx klanten

wie kan mij helpen met Security aanscherpen voor nmijn website

Ik ben op zoek naar een goede hosting

App bouwer gezocht

Dating software

250 Nieuwe domeinnamen April 2024

Xenforo buckaroo plugin

[Check] OOP class

Participate now!

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Het Belang van Standaardisatie

De Revolutionaire Rol van IT in Alzheimeronderzoek

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Delen

Gebruikers die dit topic bekijken