[Check] OOP class

J.Rijdes

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

Waarom heb je $db public gemaakt?

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

WHMCSAddons

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/questions/1020…ted-programming

J.Rijdes

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

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/questions/1020…ted-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

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

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

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

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

ICTscripters

Dé plek voor IT

Dé plek voor IT

Beta-testers gezocht voor Crypto-oefenplatform

Na 15 jaar terug van weggeweest: iCriminals.nl is terug (BETA)!

Developer Gezocht

Op zoek naar de legends

[FREE] WeFact Hosting module

Help testers nodig voor android app Urgent

Versio vervanger

Afspraken systeem met planbeperking

350 Nieuwe Domeinnamen Januari 2026

321 Nieuwe Domeinnamen December 2025

Meerdere mafia game template te koop

[Check] OOP class

Participate now!

Fijne feestdagen

Kritieke update voor Really Simple Security-plug-in

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Functioneel ontwerp

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

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

Gebruikers die dit topic bekijken