Ik heb er al eerder problemen mee gehad, destijds in een functie (Topic), en nu dus weer, maar dan in een class.

Ik wil dus het aantal queries tellen, en de totale tijd die benodigd was om alle queries uit te voeren. Nu dacht ik dat zo op te lossen:

Class database:


<?php
class Database {
	var $link_database;
	var $queries	= 0;
	var $tijd		= 0;
	
	var $server		= 'localhost'; 			//DB server
	var $username	= '';					//DB username
	var $password	= '';					//DB password
	var $database	= '';					//DB database
	
	function verbind(){
		if($this->link_database = mysql_connect($this->server,$this->username,$this->password)){
			if(mysql_select_db($this->database,$this->link_database)){
				return true;
			}else{
				return false;
			}
		}else{
			return false;
		}
	}
	
	function query($input_query){
		$tijd_start = microtime();
		if(!isset($this->link_database)){
			if(!$this->verbind()){
				return false;
			}
		}
		
		if($output_query = mysql_query($input_query,$this->link_database)){
			$this->tijd += round(microtime() - $tijd_start,4);
			$this->queries++;
			return $output_query;
		}else{
			return false;
		}
	}
	
	function statistiek(){
		return array($this->queries,$this->tijd);
	}
}
?>


Class gebruikers:


<?php
class Gegevens {
	function gebruiker($input_gebruiker,$input_id){
		$database = new Database;
		
		$controle_gebruiker_array	=	array('id','loginnaam','email','voornaam','achternaam');
		$input_gebruiker_array	= 	explode(',',$input_gebruiker);
		
		if(empty($input_id) OR !ctype_digit($input_id)){
			return false;
		}
		
		foreach($input_gebruiker_array as $waarde){
			if(!in_array($waarde,$controle_gebruiker_array)){
				return false;	
			}
		}
		$query_gebruiker 		= "SELECT `";
		$query_gebruiker 		.= implode("`, `",$input_gebruiker_array);
		$query_gebruiker 		.= "` FROM `gebruikers` WHERE `id` = ";
		$query_gebruiker 		.= $input_id;
		$query_gebruiker		.= " LIMIT 1";
		
		$result_gebruiker_array = mysql_fetch_assoc($database->query($query_gebruiker));
		
		return $result_gebruiker_array;
	}
	
}
?>


Ik roep de class zo aan:


<?php
error_reporting(E_ALL);
require('database.php');
require('gegevens.php');
$gegevens = new Gegevens;
$database = new Database;
print_r($gegevens->gebruiker('id,loginnaam','2'));
print_r($database->statistiek());
?>


Nu krijg ik keurig de resultaten terug van de query, maar geen statistieken.
Ik heb het idee dat het aan de $database = new Database; (r 4 in de class Gebruikers), maar ik heb ook nog geen andere oplossing kunnen vinden voor het aanroepen van een class, in een class (dus class Database oproepen in class Gebruikers).
Moet ik nu alles in 1 grote class zetten? Moet ik dingen anders aanroepen?

Graag zou ik zo veel mogelijk tips, aan en opmerkingen willen. Aan de error-afhandeling ga ik nog werken, en ik ben me er van bewust dat nog niet alles klopt.

Ook is dit het eerste wat ik script in (wat ik hoop dat het is) OO stijl, hiervoor heb ik altijd alles in 'gewone' functies gedaan.

---

Oh ja, en kan iemand mij vertellen hoe ik in een class variabelen kan opvragen die gezet zijn voor de class? Voorbeeld:


<?php
$test_variabele = 'test';

class Experiment {
    function test2(){
        return $test_variabele;
    }
}

$test = new Experiment;
echo($test->test2());
?>


[edit]
Dode link..

p.s. Ik gebruik PHP4, dus geen PHP5.
[/edit]
Je kunt zorgen dat de ene class een membervariabele wordt van een andere class:
<?php
class test1 {
var $naam;

function test1 ($naam) {
$this->naam = $naam;
}
}

class test2 {
var $test;
var $plaats;

function test2 ($plaats) {
$this->plaats = $plaats;
}

function show_info () {
foreach ($this as $key => $value) {
echo '<p>' . $ker . ': ' . $value . '</p>';
}
}
}

$t1 = new test1 ('jantje');
$t2 = new test2 ('alkmaar');
$t2->test = $t1;
$t2->show_info ();
?>
@Jan:

Dus, als ik het goed begrijp:

<?php
$t1 = new Database;
$t2 = new Gegevens;
$t2->database = $t1;
?>

En in gegevens:

<?php
class Gegevens{
var $database;
function gebruiker($input_gebruiker,$input_id){
(...)
$result_gebruiker_array = mysql_fetch_assoc($database->query($query_gebruiker));
(...)
}
}
?>

Zo ongeveer?
@Jan, op die manier vergeet je iets belangrijks van OOP , information hiding.
Wijzig zo min mogelijk variabelen binnen de classe rechtstreeks van buiten, zorg dat je dit via een zo genaamde set methode (is een netter woord voor maar dat kan ik zo even niet bedenken) doet. In dit geval zou het misschien ook voldoen deze via de constructor mee tegeven.

[edit]The magic word: mutator method[/edit]
@ Andries: Ja precies!
@ Boaz: heb je gelijk in. Voor elke membervariabelen set- en get-methoden in verband met data hiding, plus dat je controle en foutafhandeling aan de class kunt overlaten.
@Jan:

Ok, ik heb nu dit:

Class database:

Onveranderd


Class gebruikers:

    <?php
class Gegevens {
	var $database;
	function gebruiker($input_gebruiker,$input_id){
	    (...) Line 24:
	    $result_gebruiker_array = mysql_fetch_assoc($database->query($query_gebruiker));
	    (...)
    }
}
?>


Ik roep nu aan met:
<?php
error_reporting(E_ALL);
require('database.php');
require('gegevens.php');
$database = new Database;
$gegevens = new Gegevens;
$gegevens->database = $database;
print_r($gegevens->gebruiker('id,loginnaam','2'));
print_r($database->statistiek());
?>

Maar ik krijg een 2 tal errors op lijn 24 in de class gebruikers:

Notice: Undefined variable: database (on line 24 class gebruikers)
En:
Fatal error: Call to a member function query() on a non-object (on line 24 class gebruikers)

Hoe moet ik dit oplossen?
Regel 9 zou nu zo moeten:

print_r($gegevens->database->statistiek());
Ok, ik heb ook regel 9 aangepast, maar de fout op regel 24 is er nog steeds. Op een of andere manier kan ik dus zo de class niet inladen, of wordt $database niet gezet in class gegevens.

Heeft iemand nog een idee?
Verander deze regel:

$result_gebruiker_array = mysql_fetch_assoc($database->query($query_gebruiker));

in dit:

$result_gebruiker_array = mysql_fetch_assoc($this->database->query($query_gebruiker));

Je moet dan wel zorgen dat de membervariabele database al is toegekend.
@Jan:
Hartelijk dank! Dat was de oplossing! Ik begin nu ook te snappen hoe ik variabelen toeken in een class, bedankt!
Ok, nog 1 klein vraagje:
Ik probeer nu variabelen van buiten af in de class te krijgen door:
<?php
require('database.php');
require('gegevens.php');
$database = new Database;
$database->server = 'localhost'; //DB server
$database->username = ''; //DB username
$database->password = ''; //DB password
$database->database = ''; //DB database
$gegevens = new Gegevens;
$gegevens->database = new Database;
?>

En in database class:

<?php
class Database {
var $link_database;
var $queries = 0;
var $tijd = 0;
var $server;
var $username;
var $password;
var $database;

function verbind(){
if($this->link_database = mysql_connect($this->server,$this->username,$this->password)){
if(mysql_select_db($this->database,$this->link_database)){
(...)
}
}
}
}
?>

Echter komen nu de variabelen niet aan, hoe moet dit anders?

Reageren