[PHP]Tworzenie klas Opcje

[Raven1122]

Witam napisalem takie cos:

[PHP] pobierz, plaintext 
<?php
 
class mysql_db_connect {
 
	public $host;
	public $user;
	public $password;
	public $database;
 
public function connect($host, $user, $password, $database){
	mysql_connect($host, $user, $password) or die(mysql_error());
	mysql_select_db($database);
 
}
}
 
?>
 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
require_once("mysql.Class.php");
 
$db = new mysql_db_connect();
$db->connect('localhost', 'user', 'password', 'database');
 
?>
[PHP] pobierz, plaintext 

Czy jest to dobrze napisane? Prosze tez o wyrozumialosc, zaczynam dopiero co kolwiek robic obiektowo.
Z gorzy dziekuje z odpowiedz (IMG:style_emoticons/default/smile.gif)

[redeemer]

@viking: Napisane jest to źle.

@Raven1122: Atrybuty w klasie w Twoim wypadku są niepotrzebne - bo nigdzie ich nie używasz. Zresztą i tak powinny mieć inny zakres widoczności. Ja osobiście zostawiłbym te atrybuty (ze zmienioną widocznością) i dane potrzebne do połączenia przekazał w konstruktorze, natomiast w funkcji connect() tylko bym się do nich odwołał.

A vikingowi chodziło o to, że jak masz zamiar pisać własną klasę do obsługi bazy danych, to istnieją już gotowe i sprawdzone rozwiązania.