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

Co to za dziwna nazwa klasy? Nie przekazujesz parametrow do konstruktora ktore sobie zdefiniowales

[Kerth]

No może rzeczywiście dziwna nazwa. Poprawiłem. Ale nie bardzo rozumiem jak mam przekazać parametry do konstruktora.

[PHP] pobierz, plaintext 
class Database{
	private $host = "localhost";
	private $username = "root";
	private $password = "";
	private $database = "portal";
 
	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.';
	        }
    	}
 
    public function getSomeData(){
	    $db = $this->pdo->query('INSERT INTO `users` (`nick`) VALUES (`dawidasd`)');
	}
}
[PHP] pobierz, plaintext 

W ten sposób wyświetlam w pliku index.php:

[PHP] pobierz, plaintext 
<?php 
 
/**
 * TITLE: Home page in OOP and MVC in PHP.
 */
 
    require_once("controller/ConfigDatabase.class.php");
    $connectDatabase = new Database;
    $connectDatabase->SQLConnection();
?>
[PHP] pobierz, plaintext 

Ten post edytował Kerth 10.03.2015, 00:11:25

[com]

Siedze na mobile to napisze Ci slownie. Tam gdzie wywulujesz w 9 lini w nawiasie () piszesz te wszystkie wartosci co masz w deklaracje. Oczywiscie to co tam ma byc Poczytaj o obiektowosci bo troche sie w tym gubisz jeszcze

[Kerth]

Ok. Dzięki Ci za pomoc. Masz rację, jeszcze się gubię w OOP. Jednakże odnośnie kodu to nie wiem dlaczego się tak dzieje: gdy wpiszę błędne dane połączenia to mimo wszystko pojawia się komunikat o poprawnym połączeniu z bazą danych:

[PHP] pobierz, plaintext 
<?php 
 
/**
 * TITLE: Home page in OOP and MVC in PHP.
 */
 
    require_once("controller/ConfigDatabase.class.php");
    $connectDatabase = new Database;
    $connectDatabase->SQLConnection($host, $username, $password, $database);
?>
[PHP] pobierz, plaintext 

Wiesz może dlaczego tak się dzieje?

Ten post edytował Kerth 10.03.2015, 00:21:35

[com]

Widze wpisales tam zmienne a je wgl gdzies masz zadeklarowane? A co do pytania to dlatego ze kod sie wykonuje pokolei jak podasz bledne dane to pdo sie nie wysypie tylko zwroci null i php pojdzie dalej. Blok try jest ok ale zabraklo warunku ktory sprawdza czy sie polaczyles i dopiero w nim ten komunikat

[Kerth]

W pliku: ConfigDatabase.class.php

[PHP] pobierz, plaintext 
class Database{
	public $host = "localhost";
	public $username = "root";
	public $password = "";
	public $database = "portal";
 
	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 

deklaruję zmienne publiczne zaraz pod klasą. Ten plik później załączam do index.php:

[PHP] pobierz, plaintext 
require_once("controller/ConfigDatabase.class.php");
$connectDatabase = new Database;
$connectDatabase->SQLConnection($host, $username, $password, $database);
[PHP] pobierz, plaintext 

Ostrzeżeń się pozbyłem. Mówisz o utworzeniu warunku sprawdzającego połączenie co dla mnie jest zaskoczeniem gdyż tego samego kodu połączenia używałem podczas programowania strukturalnego i nie było z nim problemów. O jakim warunku mówisz? Jakiej funkcji w nim użyć? Wiem, że się dopytuję zbyt dużo pewnie ale to dla mnie nowa sprawa.

[com]

To nie tak warunek to if(!empty($db))
To co tam deklarujesz narazie ma sie nijak do tego kodu zeby odwolac sie do zmiennej z wnetrza klasy musisz zrobic w twoim wypadku $this->host itd ale to wtedy w zasadzie nie potrzeba Ci w konstruktorze tych prametrow skoro robisz na sztywno. Musisz poczytac bo narazie nie widzisz tego zupelnie jak to wszystko dziala. A ja na telefonie tez Ci nie pomoge za bardzo. W index php zamiast zmienych wpisz normalnie dane. Bo tych zmiennych co zadeklarowales w tej postaci itak narazie nie uzywasz :-)

Jak nie zapomne to naskrobie Ci przyklad popoludniu jak bd na kompie. Ale mimo wszystko bd musial zajrzec do kursu oop bo bez tego ani rusz tymbardziej ze widze to sa jakies zalazki modelu mvc

Ten post edytował com 10.03.2015, 00:55:40

[Kerth]

Spróbowałem wykorzystać to $this i utworzyłem taki kod:

[PHP] pobierz, plaintext 
<?php
 
/**
 * TITLE: Connecting class with MySQL Databas
 */
 
 
 
class Database{
	public $host = "localhost";
	public $username = "root";
	public $password = "";
	public $database = "portal";
 
	public function SQLConnection($host, $username, $password, $database){
 
	    	try{
		        $db = new PDO(
		        	'mysql:host='.$this->host.';
		        	 dbname='.$this->database.'', 
		        	 ''.$this->username.'', 
		        	 ''.$this->password.'',
		        array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
 
		    }catch(PDOException $e){
		        	echo 'Connection failed: ' . $e->getMessage();
		        }
	    	}  	
 
}		
 
 
?>
[PHP] pobierz, plaintext 

Teraz wszystko jest OK. Po wpisaniu złych danych otrzymuję komunikat o błędzie. Obyło się bez warunku sprawdzającego. Czy możesz mi powiedzieć czy dobrze to zrobiłem czy coś jest nie tak pod względem optymalności?

[com]

Jest ok ale sa dwa ale pierwsze teraz parametry w deklkaracji i wywolaniu metody sa nie potrzebne. Drugie taka klasa nic Ci nie daje bo itak musisz do niej zajrzec zeby cos zmienic w polaczeniu. Najlepsze to obecnie jest stosowanie DI czyli przekazujesz sam obiekt do konstruktora

No to kolega Boshi dal Ci przyklad jak to w zasadzie powinno wygladac dla twojego przypadku kiedy chcesz robic to w taki sposob

@down maly blad sie wkradl $this->pole zamiast $this->field :-D

Ten post edytował com 10.03.2015, 02:02:54

[Boshi]

Jeżeli już to konstruktor powinien łączyć się z bazą przy tworzeniu instancji klasy. Nie robi się etykiet publicznych-łamiesz jedną z podstawowych zasad OOP- hermetyzacje. Nie wiem po co też tworzysz funkcje przyjmująca parametry a przy tym deklarujesz pola publiczne z danymi.
Tak jak kolega com. napisał, do zmiennych wewnątrz klasy odwołujemy się po przez $this->zmienna. A po to jest to zrobione by wiedzieć która zmienna jest lokalną a która polem klasy. ufff

[PHP] pobierz, plaintext 
class first
{
 private $field;
   public function __construct($field)
{
$this->field=$field;
}
public function Get()
{
 echo $this->field;
}
}//end class
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
class Database{
 
 
public function __construct($host, $database, $username,$pass )
{
 
		try
		{
	        $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();
			}
		}
 
}
 
 
        $host = "localhost";
	 $username = "root";
	 $password = "";
	 $database = "portal";
$ob=new Database($host,$database,$username,$password);
[PHP] pobierz, plaintext 

Edit: @up poprawione

Ten post edytował Boshi 10.03.2015, 02:46:00

[Kerth]

Dzięki Wam za pomoc. Podepnę się jeszcze pod ten temat, bo mam jeszcze problem, który niejako z niego wynika. Jak utworzyć jakieś proste(najprostsze) zapytanie do bazy danych? W OOP jest to jakoś ciężko mi zrozumieć. Mógłby mi ktoś pokazać jakiś przykładowy kod względem wcześniejszego łączenia z bazą danych?

[Boshi]

Dane są z mojej bazy

[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()
{
  $getData=$this->db->query('Select * From uzytkownicy');
 
  foreach ($getData as $value)
  {
	  echo $value['login']."<br>";
  }
 
 
  }
 
}
 
$ob=new Database('localhost','user','root', '');
$ob->getData();
[PHP] pobierz, plaintext 

[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

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