Prosta klasa rotatora linków do analizy, Chciałbym się dowiedzieć co robię źle i jak to powinno wyglądać. Opcje

[emtiej]

Już od dawna miałem zamiar zacząć pisać coś w OOP, do tej pory tylko teoria z której wiadomo dużo bez praktyki do głowy nie wejdzie i postanowiłem napisać prosty skrypt rotatora linków, stworzyłem klasę "lr" i klasę "admin" która jest rozszerzeniem klasy lr(chciałbym, aby w lr znajdowały się funkcje które wyciągają linki z bazy i takie tam, co chciałbym też wykorzystać w panelu administracyjnym. Dopiero zacząłem i już mam jakieś wątpliwości co do logiki i użyteczności kodu. To mój pierwszy zamach na OOP, proszę o wskazówki co źle robię, jak wy to byście zrobili itp. Rotator ma być prosty, jego funkcje to: dodawanie, usuwanie, edycja dodanych linków. W bazie zrobiłem sobie tabelę "config" w której przechowuję login i hasło administratora.

Tu plik lrclass.php

[PHP] pobierz, plaintext 
<?php
 
//-----------------------------------------------------------
//-                 Klasa rotatora linków                   -
//-----------------------------------------------------------
 
class lr
{
	private function db()
	{
		$this->dbhost = "localhost";
		$this->dbname = "lr";
		$this->dbuser = "lg";
		$this->dbpass = "lgpass";
 
		$conn = new mysqli($this->dbhost, $this->dbuser, $this->dbpass, $this->dbname);
 
		return $conn;
 
	}
 
	public function lr()
	{
		print_r($this->GetConfig());
	}
 
	public function GetConfig()
	{
	// funkcja pobiera i zwraca tabel? config
		$db = $this->db();
		$query = $db->query("SELECT * FROM config");
		$result = $query->fetch_assoc();
 
		return $result;
	}
	public function GetLinks()
	{
		$db = $this->db();
		$query = $db->query("SELECT * FROM links");
		$result = $query->fetch_assoc();
 
		return $result;
	}
 
 
 
 
}
 
?>
 
[PHP] pobierz, plaintext 

adminclass.php

[PHP] pobierz, plaintext 
<?php
 
//-----------------------------------------------------------
//-            Klasa panelu administracyjnego               -
//-----------------------------------------------------------
require_once("lrclass.php");
 
class admin extends lr
{
 
  var $admin;
 
	public function admin()
	{
		session_start();
		if(self::IsAdmin())
		{
			echo "Jesteś zalogowany jako administrator";
 
			if($_GET['action'] == 'logout')
			{
				self::StopAdminSession();
			}
		}else{
			if($_GET['login'] && $_GET['pass'])
			{
				if(self::login($_GET['login'], $_GET['pass']))
				{
					echo "Dobre dane do logowania";
				}else{
					echo "Złe dane do logowania!";
				}
			}else{
				echo "Tu formularz logowania";
				echo $_SESSION['login'];
			}
		}
 
	}
	public function login($login, $pass)
	{
		$this->adminlogin = self::GetAdminLogin();
		$this->adminpass = self::GetAdminPass();
 
		if($login == $this->adminlogin && $pass == $this->adminpass)
		{
			self::StartAdminSession();
			return TRUE;
		}else{
			return FALSE;
		}
 
 
	}
	public function StartAdminSession()
	{
		session_start();
		$_SESSION['login'] = TRUE;
 
	}
	public function StopAdminSession()
	{		
		$_SESSION = array();
		session_destroy();
	}
	public function GetAdminLogin()
	{
		$db = parent::db();
		$query = $db->query("SELECT value FROM config where name='adminlogin'");
		if(!$db) DIE("Zapytanie się nie powiodło");
		$result = $query->fetch_object();
		$result = $result->value;
 
		return $result;	
	}
	public function GetAdminPass()
	{
		$db = parent::db();
		$query = $db->query("SELECT value FROM config where name='adminpass'");
		if(!$db) DIE("Zapytanie się nie powiodło");
		$result = $query->fetch_object();
		$result = $result->value;;
 
		return $result;	
	}
	public function IsAdmin()
	{
		if($_SESSION['login']){
			return TRUE;
		}else{
			return FALSE;
		}	
	}
}
 
?>
[PHP] pobierz, plaintext 

W klasie lr jest funkcja db(), czy to jest dobry pomysł żeby ją tam umieszczać? Może lepszym wyjściem byłaby osobna klasa do łączenia z bazą? Naprawdę teoretycznie trochę wiem, ale w praktyce wymiękam hah (IMG:style_emoticons/default/smile.gif) Pomóżcie początkującemu ^^

Ten post edytował emtiej 22.03.2010, 13:33:34

[nospor]

var $admin;
public function admin()

Jesli juz uzywasz php5 to nie stosuj VAR - jest to wylacznie dla php4 a w php5 tolerowane.



self::StopAdminSession();
wiesz do czego sluzy self:: ?

Włącz sobie wyswietlanie wszystkich bledow a zobaczysz ile ich masz (IMG:style_emoticons/default/smile.gif)

[emtiej]

Jesli juz uzywasz php5 to nie stosuj VAR

To pozostało z testów

Włącz sobie wyswietlanie wszystkich bledow a zobaczysz ile ich masz

(IMG:style_emoticons/default/mellow.gif) ciężko mi je opanować hah (IMG:style_emoticons/default/smile.gif) Masakra

self::StopAdminSession();
wiesz do czego sluzy self:: ?

Teraz już wiem hah (IMG:style_emoticons/default/smile.gif)

Jednak teorii za mało jeszcze hah (IMG:style_emoticons/default/smile.gif) Ok na razie dam się na spokój, ale nie zamykaj proszę tego tematu, wrzucę tu poprawiony kod za kilka dni (IMG:style_emoticons/default/smile.gif)

[phpion]

hah i staraj się nie używać tablic $_GET/$_POST/$_SESSION "na żywca" wewnątrz klas hah lepiej przekazywać konkretne wartości w formie parametrów hah hah hah.

[zend]

Wywyłując metode getDb kilka razy nawiązujesz połączenie z bazą danych kilkakrotnie

[PHP] pobierz, plaintext 
 
myclass {
static $DB = null;
 
public function getDb()
{
	if(self::$DB === null)
	{
		self::$DB = new mysqli('xxx' , 'xxx');
	}
}
 
}
 
[PHP] pobierz, plaintext 

Takie rozwiązanie zapobiegnie kilkukrotnemu nawiązywaniu połączenia z bazą. Nie myśl też że z poziomu deklarując metodę jako prywatną zapobiegniesz wielokrotnym połączeniom, wystarczy wywołać pobliczną metodę getConfig() dwukrotnie i już mamy jedno niepotrzebne połączenie z bazą.

Kolejną sprawą jest kwestia ustawiania parametrów do połączenia z bazą, powinieneś mieć możliwość ustawienia tych danych z zewnątrz klasy, a nie ustawiać je od środka i w dodatku przy każdym wywołaniu db

Ten post edytował zend 26.03.2010, 20:15:28