Database class

Luc · 4 October 2011

Beste leden,

Omdat ik graag meer wil leren van classes wil ik graag jullie mening over mijn begin van de database class die ik aan het maken ben.

PHP

class Database
{
	private $host;
	private $user;
	private $password;
	private $database;
	private $connection;
	private $connectionTrue;
	
	
	function dbData
	{
		$this->host = 'localhost';
		$this->user = '';
		$this->pw = '';
		$this->db = '';
		$this->connectionTrue = FALSE;
	}
	
	function connect($host, $user, $password, $database)
	{
		if($connectionTrue === FALSE)
		{
			$connection = @new MYSQLi($host,$user,$password,$database);
			
			if(! $this->connection)
			{
				$error = 'Er kon geen verbinding met de database aangegaan worden.';
				die($error);
			}
			else
			{
				$connectionTrue === TRUE;
			}
		}
	}
	
	
	
}

Toon Meer

Hopelijk hebben jullie veel kritiek waar ik verder mee kan. Houdt het wel netjes, en beargumenteer je mening. Het is mijn eerste opzet, en mijn eerste grote class die ik wil gaan schrijven.

Reza · 4 October 2011

Ik heb eigenlijk geen kritiek.
Ik vind het wel goed geschreven.

Tim · 4 October 2011

Misschien een Javadoc eraan toevoegen?

Overigens vind ik het zelf altijd erg vervelend als de waardes true en false met hoofdletters geschreven worden, maar dat is iets persoonlijks.

Verder mis ik bij dbData haakjes. En waarom initialiseer je ze via een functie? Je maakt de variabelen al in het begin aan maar doet en geeft geen waarde eraan.

Ook snap ik niet dat je parameters aan de connectie functie geeft. Je kan alles al ophalen via this neem ik aan.

Edit:
Misschien de namen public en final toevoegen aan een functie.

Stefan.J · 4 October 2011

@BrokenTrack: JavaDoc? PHPDoc in PHP. :p

Ook ik snap de functie van dbData niet. Wat doet deze functie? En waar is deze goed voor?

Je connect methode stopt de uitvoer van het gehele script als het verbinden met de database mislukt. Dit is een slecht principe: Als de database verbinding mislukt, moet de class dit terug geven, en moet de gebruiker van de class moet het probleem oplossen. Zo kun je beslissen of je de script uitvoer wilt stoppen, of een deel over wil slaan.

Je zou een exception kunnen gooien als de verbinding mislukt, of de methode connect true of false laten returnen.

http://php.net/manual/en/language.exceptions.php

Luc · 4 October 2011

PHP

<?php
class Database
{
	private $host = 'localhost';
	private $user = '';
	private $password = '';
	private $database = '';
	private $connection = '';
	private $connectionTrue = false;


	
	public function connect()
	{
		if($connectionTrue === false)
		{
			try
			{
				$this->connection = @new MYSQLi($this->host,$this->user,$this->password,$this->database);
				return $this->connectionTrue = true;
			}
			catch (Exception $e) {
			{
				return 'Caught exception: ',  $e->getMessage(), "\\n" $this->connectionTrue;
			}
		}
	}	
}
?>

Toon Meer

Is dit zo beter?

Verder ben ik c# classes gewend en daar gebruikte ik steeds getters en setters. Dar was dus eigenlijk die functie voor bedoeld:$

De === heb ik ergens ooit gelezen dat die ook echt waar moet zijn (hoofdletter gevoelig). Weet niet meer waar ik dat ooit heb gelezen, maar dat heb ik me een beetje aangeleerd om te gebruiken bij belangrijke functies.

Verder had ik static gedaan omdat ik dacht dat hij tog niet veranderd.. Heb het alweer weggehaald.

Tim · 4 October 2011

$connectionTrue moet natuurlijk zonder die quotes neem ik aan.
Verder ben je eigenlijk ??n false vergeten te veranderen naar kleine letters.

Overigens, waarom maak je hem static? Verder doe je normalitair "function" achteraan en public/private/etc als eerst.

Ook zeg je op het laatst, $connectionTrue === true. ??n isje is voldoende lijkt mij.

@Killing, ik heb nog nooit van PHPDoc gehoord. In wat voor opzicht verschilt dat met JavaDoc?

Stefan.J · 4 October 2011

Nee, eigenlijk niet. Waarom is je connect methode static? Dit is nergens goed voor. Daarbij maakt je statische methode connect gebruik van niet statische properties ($this->connect), wat in principe niet mogelijk is.

Tim · 4 October 2011

Dat van === klopt. Maar voor het geven van een waarde blijf je nog steeds een enkele = gebruiken. === is voor vergelijkingen.

Verder moet je niet vergeten om bij de rest ook $this te gebruiken, anders zeg je dat enkel $connectionTrue true is binnen de functie en niet binnen de klasse.

Luc · 4 October 2011

Haha ik zie het nu pas dat ik hem een waarde wil geven met === bedankt. En ik moet dus

$this->connection =
$this->connectionTrue = true; doen?

Tim · 4 October 2011

Ja.

Edit:
Mogelijk wil je ook false/true returnen ipv enkel een mogelijke exception zoals Killing eerder zei.

Stefan.J · 4 October 2011

De exception is prima. Wat je nu verkeerd doet, is het afvangen van de exception. Alleen het gooien van de exception doe je in de methode (throw), en het opvangen in de code die gebruikt maakt van je class.

Dit komt er dan zo uit te zien:

PHP

<?php


try {
  $database = new Database;
  $database->connect();
} catch(DatabaseException $ex) {
  echo 'Database verbinding mislukt: '. $ex->getMessage();
}


?>

Toon Meer

De class DatabaseException moet je dan nog wel even aanmaken, als uitbreiding op Exception (extends).

http://nl3.php.net/manual/en/keyword.extends.php

Luc · 4 October 2011

Moet die try catch in mijn code dan weg, of moet ik er throw new Exception van maken?

Stefan.J · 4 October 2011

De try catch moet weg, en je moet een throw toevoegen aan de methode inderdaad.

Luc · 4 October 2011

PHP

public function connect()
	{
		if($connectionTrue === false)
		{
			$this->connection = @new MYSQLi($this->host,$this->user,$this->password,$this->database);
			
			if(! $this->connection)
			{
				throw new Exception('Er kon geen verbinding worden gemaakt met de daabase.');
			}
			else
			{
				return $this->connectionTrue = true;
			}
		}
		
	}

Toon Meer

Is deze functie zo goed? Of zijn er nog meer verbeteringen. En hebben jullie suggesties welke andere functies er in een goede database class moeten zitten?

Stefan.J · 4 October 2011

Dit begint er inderdaad erg op te lijken! De variabele connectionTrue zou ik hasConnection noemen. De return waarde is niet nodig: De methode return tenslotte altijd true, of gooit een exception.

Luc · 4 October 2011

PHP

<?php
class Database
{
	private $host = 'localhost';
	private $user = '';
	private $password = '';
	private $database = '';
	private $connection = '';
	private $hasConnection = false;


	public function connect()
	{
		if($hasConnection === false)
		{
			$this->connection = @new MYSQLi($this->host,$this->user,$this->password,$this->database);
			
			if(! $this->connection)
			{
				throw new Exception('Er kon geen verbinding worden gemaakt met de daabase.');
			}
			else
			{
				$this->hasConnection = true;
			}
		}
		
	}
}
?>

Toon Meer

Dit heb ik tot nu toe dan, nog meer verbeter punten? En wat staat er nog meer in een goede database class (insert select update delete).

Tim · 4 October 2011

Documentatie!

Darsstar · 5 October 2011

1) Haal die @ weg aub, het is over het algemeen geen goed idee om fouten te onderdrukken.
2) Voeg meer methoden toe, nu heb je er niets aan.
3) Als je verbinding moet maken met twee verschillende databases, hoe ga je dat nu doen?
- de class copy-pasten en de gegevens aanpassen?
- setters maken?
- iets anders?
4) Je gebruikt in de if $hasConnection terwijl het $this->hasConnection hoort te zijn.
5) === controleert op waarde en op type, voorbeeldje:

PHP

if (1 == true)
{
// wordt uitgevoerd omdat ze beiden als niet-nul waarden worden gezien
}


if (1 === true)
{
// wordt NIET uitgevoerd omdat de ene een int is en de andere een boolean
}


// van php.net
var_dump((bool) "");        // bool(false)
var_dump((bool) 1);         // bool(true)
var_dump((bool) -2);        // bool(true)
var_dump((bool) "foo");     // bool(true)
var_dump((bool) 2.3e5);     // bool(true)
var_dump((bool) array(12)); // bool(true)
var_dump((bool) array());   // bool(false)
var_dump((bool) "false");   // bool(true)

Toon Meer

6) Je zou Database::$hasConnection weg kunnen laten en "$this->connection instanceof MySQLi" kunnen gebruiken

Tim · 5 October 2011

Over 1.
Indien je een fout wilt onderdrukken is dit heel handig!
Soms wil je geen error laten zien of een vriendelijkere error laten zien!

Luc · 6 October 2011

Die @staat er omdat mysqli anders zegt dat $host geen variabele mag zijn.

Verder heb ik de class uitgebreid met query uitvoeren en rows teruggeven. Fetchen etc. komt er nog in.

Ik zit erover na te denken om voor select, update, delete en insert aparte functies te schrijven.

Dit is hem tot nu toe:

PHP

<?php
class Database
{
    private $host = 'localhost';
    private $user = '';
    private $password = '';
    private $database = '';
    private $connection = '';
    private $hasConnection = false;
	private $queryResult = '';
	
	// functie connectie aanmaken met de database
    public function connect()
    {
		// nieuwe connectie aanmaken als hij nog niet aangemaakt is.
        if($this->hasConnection === false)
        {
            $this->connection = @new MYSQLi($this->host,$this->user,$this->password,$this->database);
   
			// nette error meegeven als er geen verbinding gemaakt kon worden.
            if(! $this->connection)
            {
                throw new Exception('Er kon geen verbinding worden gemaakt met de daabase.');
            }
            else
            {
				// de connectie is aangemaakt en kan nu succesvol gebruikt worden.
                $this->hasConnection = true;
            }
        } 
    }
	
	// functie rijen teruggeven
	function num_rows($sql = NULL)
    {
        // Kijken of er een query meegegeven is, zo ja uitvoeren
        if($sql != NULL)
		{
            $this->makeQuery($sql);
		}
        
        // teruggeven van het aantal rijen.
        return $this->connection->query( $this->queryResult );
    }
	
	function makeQuery( $sql )
    {
        // query aanmaken en stoppen in de queryResult
        $this->queryResult = $this->connection->query( $sql, $this->db );
    
        // Kijken of de query uitgevoerd kon worden.
        if(! $this->queryResult )
		{
			// nette error meegeven als de query niet uitgevoerd kon worden.
			throw new Exception('Er is iets misgegaan bij het aanroepen van de query.');
		}
    }
}
?>

Toon Meer

Ik vraag me alleen af of $this->connection->query werkt..

Verder hoor ik graag verbeter punten etc.

Partner Gezocht om meerdere NFT Collecties op Open Sea te Plaatsen

Plesk installer

cpanel / whm themes

PHP Point of Sale

Probleem omtrent Tinymce

Kunnen jullie mij helpen met deze enquete

een eigen cloud sever

Probleem C++ en CL

193 Nieuwe domeinnamen April 2025

228 Nieuwe domeinnamen Maart 2025

285 Nieuwe domeinnamen Februari 2025

Database 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

Delen

Gebruikers die dit topic bekijken