OOP i projektowanie Opcje

[eVolution]

Witam,
Jako, że dopiero zaczynam naukę OOP, chciałbym się Was zapytań, czy jest to dobrze napisana klasa.

Klasa do obsługi bazy danych

[PHP] pobierz, plaintext 
<?php
 
class DB
{
	public function __construct()
	{
		try
	   	{
	       $this->baza = new PDO('mysql: host=localhost; dbname=db', 'login','pass');
	   	}
	   	catch(PDOException $e)
	   	{
	      throw new Exception($e->getMessage());
	   	}
	}
 
	public function prepare($query)
	{
		return $result=$this->baza->prepare($query);
	}
}
 
?>
[PHP] pobierz, plaintext 

Klasa do obsługi użytkownika czyli dodanie, kasowanie

[PHP] pobierz, plaintext 
<?php
class User
{
	protected $_db;
 
	public function __construct(DB $db)
	{
		$this->_db = $db;
	}
 
	public function add($login, $pass)
	{
		$result = $this->_db->prepare("INSERT INTO user VALUES(:login, :pass)");
		$result->bindParam(':login', $login, PDO::PARAM_STR);
		$result->bindParam('pass', $pass, PDO::PARAM_STR);
 
		if(!$result->execute()){
			throw new Exception("Wystąpił błąd podczas dodawania usera.");
		}
	}
 
	public function delete($id)
	{
		$result = $this->_db->prepare("DELETE FROM user WHERE id=:id");
		$result->bindParam(':id', $id, PDO::PARAM_INT);
 
		if(!$result->execute()){
			throw new Exception("Wystąpił błąd podczas usuwania usera.");
		}
	}
}
[PHP] pobierz, plaintext 

I oczywiście osobno UserManger czyli odpowiedzialna za pobranie danych typu login, logowanie etc...

Czy w tym kodzie jest wszystko ok? Kolejna sprawa, blok try - catch dawać w metodach w klasie, czy w miejscu gdzie dana metoda jest wywołana?

Ten post edytował eVolution 20.03.2012, 18:36:29

[wookieb]

Po co ci klasa DB, która nie wprowadza żadnej funkcjonalności poza bezsensownym wstawieniem na sztywno danych dla połączenia?

[eVolution]

Dlatego założyłem ten temat, żeby mi ktoś wyjaśnił co i jak, i czy kod jest dobry. Ale już widzę, że nie. Czyli rozumiem, że najlepiej zawartość konstruktora z klasy DB dać do klasy User w konstruktor, i jest to lepszym rozwiązaniem?

Ten post edytował eVolution 20.03.2012, 18:49:45

[wookieb]

Nikt Cię tutaj nie nauczy OOP bo to bardzo szeroka dziedzina
Od tego są odpowiednie książki helion -> wzorce projektowe

[by_ikar]

Czyli rozumiem, że najlepiej zawartość konstruktora z klasy DB dać do klasy User w konstruktor, i jest to lepszym rozwiązaniem?

Nie. @wookieb'owi chodziło o to że twoja klasa DB nie robi nic innego prócz połączenia na sztywno, bo zarówno nie można przekazać żadnych ustawień, jak i ta klasa poza łączeniem się z bazą nic nie robi. I coś takiego większego sensu nie ma, bo równie dobrze gdzieś wcześniej możesz utworzyć połączenie w PDO i instancje PDO przekazać do klasy użytkownika. Pomijam już kwestię że praktycznie żadnych danych nie sprawdzasz..

Czy w tym kodzie jest wszystko ok? Kolejna sprawa, blok try - catch dawać w metodach w klasie, czy w miejscu gdzie dana metoda jest wywołana?

A to już zależy od tego w jaki sposób chcesz obsłużyć wyjątek. Może masz tam jakiegoś handlera, wówczas nie musisz stawiać wszędzie try/catch, a jeżeli musisz wykonać inne działanie niż zatrzymanie aplikacji i komunikat o błędzie, to używaj try/catch..

[eVolution]

Skrobnąłem na szybko coś takiego:

[PHP] pobierz, plaintext 
<?php
class Kalkulator
{
	function dodaj($liczba, $liczba2)
	{
		$this->wynik = $liczba + $liczba2;
		return $this;
	}
 
	function odejmij($liczba, $liczba2)
	{
		$this->wynik = $liczba - $liczba2;
		return $this;
	}
 
	function mnozenie($liczba, $liczba2)
	{
		$this->wynik = $liczba * $liczba2;
		return $this;
	}
 
	function wynik()
	{
		return $this->wynik;
	}
}
 
$kalkulator = new Kalkulator;
echo $kalkulator->odejmij(2,5)->wynik();
[PHP] pobierz, plaintext 

Czy tu jest wszystko ok? I mi nie chodzi o to, żeby ktoś mnie uczył OOP, tylko o to, że mam problem z tym, co np w danej klasie powinno się znaleźć. Np co powinno być w klasie odpowiedzialnej z 1 ucznia itp tak jak np w tej podanej powyżej to wiem co powinno być, ale jak już np chodzi o zrobienie bardziej sensownej klasy to już jest problem.

Poprawcie mnie jeśli się mylę:
Mam np obiekt samochód, czyli jego metody to skręcanie, hamowanie, przyśpieszenie, zwalnianie?
Kolejne mam obiekt użytkownik, jego metody to dodaj, usuń, aktualizuj, i UserManager czyli pokaz wybranego, pokaż wszystkich etc...? Wiem głupie pytania, ale żadna książka akurat tego nie opisuje (IMG:style_emoticons/default/wink.gif)

Ten post edytował eVolution 21.03.2012, 02:12:59

[viking]

Może ściągnij sobie http://framework.zend.com/manual/en/learni...tart.intro.html i zobacz jak to może być zrobione.

[eVolution]

No właśnie ZF czy Kohana3 ? Oczywiście, bardziej z myślą o pracy jako progamista. Na ZF z tego co wiem jest większe zapotrzebowanie.

[phpion]

Na ZF jest większe zapotrzebowanie, a co za tym idzie także konkurencja większa. Weź pod uwagę, że poznając dany framework od podstaw musisz poświęcić sporo czasu by nabrać doświadczenia i posługiwać się nim biegle.

[eVolution]

Kohana3 wydaje się być prostym i lekkim frameworkiem, co przemawia na jego korzyść (IMG:style_emoticons/default/smile.gif) W KO2 miałem okazję pracować, jakieś pół roku.

Ten post edytował eVolution 21.03.2012, 13:24:38

[viking]

Ale to wcale nie znaczy że musisz tylko do tych się ograniczać. Oprócz większych Jak ZF, Symfony czy przytoczona Kohana jest też sporo całkiem ciekawych projektów. Jest Yii, jest pisany trochę na wzór Sinatry (Ruby) Laravel czy w końcu http://lithify.me/. A przytaczam tylko skupiające większą społeczność i aktywnie rozwijane. Wszystkie też są sporo szybsze niż ZF i Symfony.

[Orzeszekk]

Naucz sie programowania obiektowego w Javie i pozniej przenies sie do php jak bedziesz jeszcze chcial. obiektowosc php jest troche toporna i mozesz sobie wyrobic zle nawyki.