[MySQL][PHP] Błąd połączenia PDO (Programowanie obiektowe) Opcje

[Kerth]

Cześć,
próbuję zrozumieć co jest źle w poniższym kodzie, że pojawiaja się ostrzeżenia. Co robię źle?

[PHP] pobierz, plaintext 
class __construct{
	private $host = "localhost";
	private $username = "root";
	private $password = "pass";
	private $database = "database";
 
	public function SQLConnection($host, $username, $password, $database){
        try{
	        $db = new PDO('mysql:host='.$host.';dbname='.$database.'', ''.$username.'', ''.$password.'',array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
	            	echo 'Połączenie nawiązane!';    
	        }catch(PDOException $e){
	            	echo 'Połączenie nie mogło zostać utworzone.';
	        }
 
}
[PHP] pobierz, plaintext 

Kod

Warning: Missing argument 1 for __construct::SQLConnection(), called in C:\WebServ\httpd\oop_mvc\index.php on line 12 and defined in C:\WebServ\httpd\oop_mvc\controller\ConfigDatabase.class.php on line 18

Warning: Missing argument 2 for __construct::SQLConnection(), called in C:\WebServ\httpd\oop_mvc\index.php on line 12 and defined in C:\WebServ\httpd\oop_mvc\controller\ConfigDatabase.class.php on line 18

Warning: Missing argument 3 for __construct::SQLConnection(), called in C:\WebServ\httpd\oop_mvc\index.php on line 12 and defined in C:\WebServ\httpd\oop_mvc\controller\ConfigDatabase.class.php on line 18

Warning: Missing argument 4 for __construct::SQLConnection(), called in C:\WebServ\httpd\oop_mvc\index.php on line 12 and defined in C:\WebServ\httpd\oop_mvc\controller\ConfigDatabase.class.php on line 18
Połączenie nawiązane!
Fatal error: Call to a member function query() on a non-object in C:\WebServ\httpd\oop_mvc\controller\ConfigDatabase.class.php on line 28

[com]

Boshi niestety w tym wypadku nie tak, tylko:

[PHP] pobierz, plaintext 
class Database
{
	private $db;
 
	public function __construct($host, $database, $username, $pass)
	{
		try
		{
			$this->db = new PDO('mysql:host='.$host.';dbname='.$database, $username,$pass,array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
			echo 'Połączenie nawiązane!';    
		}
		catch(PDOException $e)
		{
			echo 'Połączenie nie mogło zostać utworzone.'.$e->getMessage();
		}
	}
 
	public function getData()
	{
		return !empty($this->db) ? $this->db->query('SELECT * FROM users') : array();
	}
}
 
$ob=new Database('localhost','user','root','');
 
foreach ($ob->getData() as $value)
{
	echo $value['login'].'<br>';
}
[PHP] pobierz, plaintext 

Przykład Boshi zamyka te klase i w zasadzie staję się wtedy bezużyteczna, bo co jeśli zamiast chcieć wyświetlić coś w nowych liniach bd chciał zrobić to w tablece czy czymkolwiek. Takie podejście jak powyżej pokazuje tylko, że ktoś na sile chce upchnąć kod który tworzył strukturalnie do klasy i to jest złe, bardzo złe (IMG:style_emoticons/default/smile.gif)

Ten post edytował com 11.03.2015, 14:57:17