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

[viking]

Napisane jest dobrze tylko nie ma większego uzasadnienia. Po pierwsze i najważniejsze jest PDO, ewentualnie mysqli.

[Raven1122]

a moge o jakas podpowiedz poprosic? przyklad na ktorym bede mogl se wzorowac?

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

[Raven1122]

a tak lopatologicznie jakis przyklad bym poprosil

[viking]

http://php.net/manual/en/book.pdo.php

Czy źle napisane to kwestia dyskusyjna. Jasne że według sztuki poprawne będzie coś w stylu:

[PHP] pobierz, plaintext 
class mysqlAdapter {
 
protected $_host, $_user, $_password, $_database;
 
public function construct($host, $user, $password, $database){
  mysql_connect($host, $user, $password) or die(mysql_error());
  mysql_select_db($database); 
  //zwrócić jeszcze connection
}
 
public function setHost($host) {
  $this->_host = $host;
}
 
public function getHost() {
  return $this->_host;
}
 
}
[PHP] pobierz, plaintext 

Ale nie znaczy to że sposób wyżej jest od razu zły.