Witam

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ć

[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 

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 )
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.

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 

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ą
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.