php - Error handling - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php

Bruger #10325 @ 30.01.10 22:14

Halløj,

Jeg sidder og er ved at prøve at få styr på OOP (sigh, fatter det ikke >.<

, og jeg er ved at prøve på at lave en simpel backbone. Jeg har lavet en hel simpel class:

Kode

class database_link
{
	private $host = "localhost";
	private $user = "root";
	private $pass = "something";
	
	function initiate_link()
	{
		if(!mysql_connect($this->host, $this->user, $this->password))
		{
			return false;
		}
		else
		{
			return true;
		}
		
	}
	
	function db_link()
	{
		if(!mysql_select_db($this->db))
		{
			return false;
		}
		else
		{
			return true;
		}
	}
}

(ved ikke engang om det er rigtig gjort >.>

, men anywho; Jeg sidder så og vil have lavet noget error handling. Så hvis jeg på min view-page laver database connection:

Kode

<?php 
$database_connection = new database_link();

if($database_connection->initiate_link == true)
{
	echo "Database connection successfully initiated";
	echo "<br />Initiating Database Selection...";
	
	if($database_connection->db_link == true)
	{
		echo "<br />Database selection successfull.";
	}
	elseif($database_connection->db_link == false)
	{
		echo "<br />Database selection failed!";
	}
}
elseif($database_connection->initiate_link == false)
{
	echo "Database connection failed!";
	echo "<br />Aborting database selection...";
	
}

?>

Hvordan laver jeg så en class der kan handle errors?

Eventuelt hvis det ser helt skidt ud med OOP så ville jeg sætte pris på et link til en guide, men det skal helst være en guide der viser noget praktisk som jeg kan bruge til noget (ie. en database del, og så et script af en art f.eks login), bare de der standard oop guides hvor man laver nogle classes som så holder en person, som du så udbygger med hvilken farve øjne de har etc. fatter jeg ikke en skid af. Jeg tror bare jeg skal have fat i konceptet af hvordan man bruge generel database handling inden i classes, altså input, select, delete etc.

Håber der er nogen der kan vise noget super godt da jeg ikke lige kan få fat på det

Indlæg senest redigeret d. 31.01.2010 00:24 af Bruger #10325

9 svar postet i denne tråd vises herunder
2 indlæg har modtaget i alt 4 karma

Sorter efter stemmer Sorter efter dato

Bruger #10216 @ 31.01.10 11:22

4.283

Hvis du skal bruge ValidateInput:

ecureVar() på den måde som du gør der, så ja.

Dog vil jeg ikke anbefale det, men i stedet holde de to ting adskilt.

Her har du et eksempel for hvordan din controller "UserSystem" = "UserController" kan arbejde - så skal du få resten af din backbone til at håndtere det trivielle arbejde:

Kode

<?php
class UserController {
	public function createAction() {
		$input = new Input();
		
		// validates
		$input->validatePost('username',new Validate('not-empty'));
		$input->validatePost('password',new Validate('not-empty'));
		
		// create user
		if($input->isValidPost(array('username','password'))) {
			$user = new User();
			$user->setUsername($input->getPost('username'));
			$user->setPassword($input->getPost('password'));
			if(!$user->save()) {
				die('Could not create user.');
			}
		} else {
			die('Input is not valid.');
		}
	}
	
	public function editAction() {
		// ....
	}

	public function deleteAction() {
		// ....
	}
}
?>

Indlæg senest redigeret d. 31.01.2010 11:23 af Bruger #10216

Bruger #15047 @ 31.01.10 12:42

Jeg har selv lavet denne class, den giver lidt fejl osv.

PHP kode

 <?php
     class mysql {
         public $host = 'localhost';
         public $username = 'root';
         public $password = '';
         var $database;
         
         public function error_msg($size, $text, $color, $extra="") {
             if($extra == "lower") {
                 $text = strtolower($text);
             }
             if($extra == "upper") {
                 $text = strtoupper($text);
             }
             $er = "<font color='$color' font-size='$size'> $text </font>";
             return $er; 
         
         }
         
         public function mysql_connection() {
             mysql_connect($this->host, $this->username, $this->password) or die($this->error_msg('2', 'Error(1): Could not connect to MySQL database', 'red', ''));
         }
         public function mysql_disconnection() {
             mysql_close() or die($this->error_msg('2', 'Error(2): Could not disconnect from MySQL database', 'red', ''));
         }
         public function mysql_database() {
             mysql_select_db($this->database) or die($this->error_msg('2', 'Error(3): Could not select database <b>' . $this->database . '</b>', 'red', ''));
         }
     }
     $std = new mysql;
     $std->mysql_connection();
     $std->database="applecake";
     $std->mysql_database();
     //$std->mysql_disconnection();
 ?>

Bruger #10216 @ 30.01.10 22:47

4.283

Du laver en væsentlig fejl ved at anvende dine funktioner som properties:

Kode

if($database_connection->initiate_link == true)

Du skal sætte et sæt parenteser på:

Kode

if($database_connection->initiate_link() == true)

Her er din database klasse gjort lidt mere enkel, prøv at lure den:

Kode

<?php
class Database
{
	private $host = "localhost";
	private $user = "root";
	private $pass = "dappe1591";
	
	function connect()
	{
		if(!mysql_connect($this->host, $this->user, $this->password))
		{
			return false;
		}

		if(!mysql_select_db($this->db))
		{
			return false;
		}
		
		return true;
	}
}

// Her begynder "the database actions"...
$db = new Database();
if(!$db->connect()) {
	die('Could not connect to database');
}

Bruger #10325 @ 31.01.10 02:11

Nå nu er jeg ved at være godt træt, men har leget lidt med det. Det jeg vil opnå med alt det her OOP er at komme ind i en programmeringsform som er lidt mere modul-baseret (og derved gør det nemmere for mig), men er kommet til et problem.

Min idé var at lave en backbone i form af essentielle classes (database link, query håndtering, validering af inputs etc) og så have andre classes som moduler. Det første modul jeg ville prøve at lave er et user modul (altid et godt et at starte med), og har lavet en function som bruges til at registrere en user med.

Den er i en anden fil end min backbone, og der løber jeg ind i lidt problemer. Jeg er kommet frem til noget i stil med:

Kode

class UserSystem 
{
	
	function CreateUser($username, $password, $acceslvl)
	{
		if(!$this->SecureVar($username, $password, $acceslvl))
		{
			return false;
		}
		else
		{
			return true;
		}
	}
}

Men da funktionen SecureVar ligger i klassen ValidateInput i backbonen får jeg en fejl:

Kode

class ValidateInput
{
	function SecureVar()
	{
		foreach($var as $Key => $Value)
		{
			if(!isset($var))
			{
				return false;
			}
			else
			{
				return true;
			}
			if(empty($var))
			{
				return false;
			}
			else
			{
				return true;
			}
		}
		
		return true;
	}
}

Fatal error: Call to undefined method UserSystem:

ecureVar()

Betyder det at jeg skal extende mit user-modul fra backbone classes? Eller kan jeg lave noget halløj med public functions?

Bruger #10325 @ 31.01.10 11:29

Hmm skal lige være sikker på jeg følger

Du vil lave en ny klasse ( Input(); ), som er i backbonen. Her henter du så funktionen validatePost fra. !!Åbenbaring mens jeg skriver!! Altså du laver bare en ny instance af objektet NÅR du skal bruge det (i den her sammenhæng i userControl systemet), og så ville jeg gøre det i alle henseender hvor jeg skal validere noget?

Indlæg senest redigeret d. 31.01.2010 11:29 af Bruger #10325

Bruger #10216 @ 31.01.10 12:01

4.283

Tanken er at man laver en ny instance af Input, når man skal bruge inputdata fra brugeren. Brugerdata skal altid valideres, og det kan man så gennemtvinge vha. denne klasse.

Validering kan ske på mange måder. Her opstilles nogle validerings kriterier og senere checkes så om det hele er valid data.

Hvis alt valideres korrekt kan man oprette brugeren.

Bruger #10325 @ 31.01.10 12:29

Hmm du har altså

class Input (her henter du post/get data?) og,

class Validate (her henter du funktionerne til at validere med?)

$input->validatePost('password',new Validate('not-empty'));

Det må jo være klassen input der holder funktionen validatePost, som tager 2 properties, data (password). Men så laver du en ny instance af en anden class, og hvorfor/hvordan kan den class tage properties (troede det var funktioner kun?)

Undskylder hvis jeg er lidt langsom, men det hele er stadig lidt kryptisk for mig.

Bruger #10216 @ 31.01.10 12:54

4.283

For hvad der er tekniske muligt, vil jeg anbefale dig at læse:
http://dk2.php.net/manual/en/language.oop5.php

Enhver metode/funktion i en klasse kan tage x-antal parametre.

klasser har en række magic-methods som f.eks __construct()
http://dk2.php.net/manual/en/language.oop5.decon.php

__construct() kaldes automatisk, når man opretter en ny instans af en klasse. __construct kan eksempelvis skrives:

Kode

class Database {
__construct($host,$user,$password,$db) {}
}

Dvs. at du skal definere alle fire parametre før du kan oprette en instans af Database.

Bruger #10325 @ 02.02.10 12:27

Okay, jeg leger stadigvæk med det her. Har et nyt problem:

Backbone (database, message funktion)

Kode

class Backbone
{
	private $host	= "localhost";
	private $user	= "root";
	private $pass	= "pw";
	private $db	= "db";
	
	function connect()
	{
		
		if(!mysql_connect($this->host, $this->user, $this->pass))
		{
			return false;
		}
		
		if(!mysql_select_db($this->db))
		{
			return false;
		}
		
		return true;
	}
	
	function disconnect()
	{
		if(!mysql_close())
		{
			return false;
		}
		
		return true;
	}
	
	function query($q)
	{
		if(!mysql_query($q))
		{
			return false;
		}
	
		return true;
	}

	function newMsg($size, $text, $type)
	{
		if($size == 0)
		{
			$text = strtolower($text);
		}
		elseif($size == 1)
		{
			$text = strtoupper($text);
		}
		elseif($size == 2)
		{
			$text = $text;
		}
		
		if($type == 0)
		{
			$text_class = "error";
		}
		elseif($type == 1)
		{
			$text_class = "succes";
		}
		$msg = "<p class='". $text_class ."'>". $text ."<p>";
		
		return $msg;
		
	}


}

Validation (alt fra om ting er empty, isset til at kontrollere om et username er taget etc)

Kode

class Validate extends Backbone
{
	function validation()
	{
		//Har ikke helt fundet ud af hvordan jeg får det her til at virke, derfor returner den kun true lige nu.
		return true;
	}
	
	function usernameTaken($username)
	{
		if(!$q = $this->query("SELECT * FROM users WHERE username='$username'"))
		{
			return false;	
		}
		else
		{
			$row = mysql_fetch_array($q); //bare noget halløj pt.
		}
		
		if(mysql_num_rows($q) != 0)
		{
			
		}
		
		return true;
	}
}

UserControl (save user, delete user, login etc vil være at finde her)

Kode

class UserControl extends Backbone
{
	function userSave($username, $password)
	{
		$q = new Backbone();
		$validation = new Validate();
		
		if(!$validation->validation($username, $password))
		{
			return false;
		}
		
		if(!$validation->usernameTaken($username))
		{
			echo $q->newMsg(2, "Error: Username already taken", 0);
			return false;			
		}
		else
		{
			if(!$q->query("INSERT INTO users (username, password, acceslvl) VALUES('$username', '$password', '0')"))
			{
				return false;
			}
		}
	
		return true;
	}
}

Mit problem ligger i at når jeg vil kontrollere hvorvidt der allerede er en user med et givent navn, så siger den:

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource

Det er fordi jeg bruger

Kode

$q = $this->query("SELECT * FROM users WHERE username=$username");

og det kan den ikke finde ud af, hvis jeg derimod bare skriver:

Kode

$q = mysql_query("SELECT * FROM users WHERE username=$username");

if(mysql_num_rows($q) != 0)
{
     return false;
}

return true;

så vil den godt.

Desuden er jeg lidt lost med hensyn til hvordan det er mest hensigtsmæssigt at hente data ud fra DB'en. Kommentare til hele skidtet er selvfølgelig meget velkommen.

Error handling

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler