[php] Pytanie o procentową jakość skryptu w OOP Opcje

[Danone]

Witam (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Jestem na jakimś etapie nauki OOP, strukturalnie już jest ok:)

Mam do Was pytanie, w jakim stopniu ten skrypt poniżej jest Obiektowy? Może być ocena procentowa, jeśli macie jakieś uwagi to też bardzo proszę pisać (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
class MyClass{
	public $host="localhost";
	public $login="root";
	public $pass="nokia";
	public $connection;
	public $baza;
	public $zapytanie;
 
	//Konstruktor do połączenia z bazą...
	function __construct(){
		$this->connection= mysql_connect("$this->host", "$this->login", "$this->pass");
			if(is_resource($this->connection)){
				echo "Polaczenie do " .$this->host. " udane";
			}
		$this->baza= mysql_select_db("dzienniczek");
 
		if($this->baza){
			echo "<br />ok";
		}
 
		}
	//wyswietlenie wszystkich rekordów w tabeli
	public function show(){
		$this->zapytanie="SELECT * FROM user";
		$this->wybor=mysql_query($this->zapytanie);
		echo "<table border=1>
			  <tr>
			  <td><a href='klasa_czwarta.php?sort=id'>ID</a></td>
			  <td><a href='klasa_czwarta.php?sort=imie'>Imię</a></td>
			  <td><a href='klasa_czwarta.php?sort=nazwisko'>Nazwisko</a></td>
			  </tr>";
		while($row=mysql_fetch_array($this->wybor)){
			echo"
			<tr>
			<td>".$row['id']."</td>
			<td>".$row['imie']."</td>
			<td>".$row['nazwisko']."</td>
			</tr>
			";
		}
		echo "</table>";
	}
	//funkcja odpowiedzialna za sortowanie danych
	public function sort(){
		if(isset($_GET['sort'])){
		$sortowanie="SELECT * FROM user ORDER BY ".$_GET['sort']."";
		$sort=mysql_query($sortowanie);
		echo "<table border=1>
		<tr>
		<td bgcolor=red>ID</td>
		<td bgcolor=red>Imie</td>
		<td bgcolor=red>Nazwisko</td>
		</tr>";
		while($row2=mysql_fetch_array($sort)){
			echo"
			<tr>
			<td>".$row2['id']."</td>
			<td>".$row2['imie']."</td>
			<td>".$row2['nazwisko']."</td>
			</tr>";
		}
		}
	}
}
 
 
$test=new MyClass;
$test->show();
$test->sort();
?>
[PHP] pobierz, plaintext 

[dr_bonzo]

0%

1. nie ocenia sie tego procentowo, co nazwyzej: jest dobra, ujdzie, i 'czemu-nie-dales-tego-szitu-na-przedszkolu' (edit: ja myslalem ze to jest w forum OOP (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) )
2. nazwa 'MyClass' nic nie mowi o przeznaczeniu klasy
3. jest niezykozystywalna (na stale zapisane parametry polaczenia z baza)
4. czemu funkcja 'sort' wyswietla dane? a nie sortuje
5. powielasz kod wyswietlania danych
6. polaczenie z baza powinno byc w osobnej klasie z ktorej bys korzystal

itd.

[Danone]

Jeśli chodzi o ten podpuntk?

3. jest niewykorzystywalna (na stale zapisane parametry polaczenia z baza)

Poczytałem trochę i czy teraz połączenie z bazą jest odpowiednie? Parametry można zmieniać.

[PHP] pobierz, plaintext 
<?php
Class MYSQL{
 
	public $polaczenie;
 
	function __construct($dbHost, $dbUser, $dbPass){
		$this->DBHost =$dbHost;
		$this->DBUser =$dbUser;
		$this->DBPass =$dbPass;
 
	}
 
	function connect(){
		$this->polaczenie = mysql_connect($this->DBHost, $this->DBUser, $this->DBPass);
 
		if($this->polaczenie){
			echo "Jestes polaczony jako: " .$this->DBUser. " :)";
		}
	}
}
 
$DB= new MYSQL(localhost, root, nokia);
$DB->connect();
?>
[PHP] pobierz, plaintext 

Ten post edytował Danone 29.05.2007, 22:34:27

[Amorph]

Główną zaletą OOP jest skalowalność, i podział w którym każda klasa odpowiada za obiekt lub grupę obiektów. W twoim wypadku napisałeś klasę która nic nie ma związanego z OOP oprócz tego że jest klasą (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Powinieneś napisać jedną klasę odpowiadającą za obsługę bazy danych, drugą za wyświetlanie danych jakie zostaną do niej przypisane....
Zobacz ile funkcji musiałbyś napisać, jeśli chciałbyś wyświetlać dane np. w takich kolejnościach:
1. id, imie, nazwisko
2. id, nazwisko, imie
3. imie, nazwisko, id
4. imie, id, nazwisko
itd.