[SQL][PHP]Początki w PHP OOP - problem z połączeniem PDO Opcje

[Panicz74]

Witam,

Zaczynam się dopiero uczyć PHP OOP. Utworzyłem skrypt połączenia z bazą i logowania. Kiedy próbuję się zalogować otrzymuję komunikat: Fatal error: Call to a member function prepare() on null in C:\xampp\htdocs\try\CLASS\phpLoader.class.php on line 18

Bardzo proszę was o pomoc i wyrozumiałość bo naprawdę mnie ta obiektówka rozłożyła. Oto kod:

config.class.php:

[PHP] pobierz, plaintext 
<?php
 
class config
{
static public $pdo;
 
 static public function connect()
 {
	 ob_start();
	 session_start();
 
		try
		{
			$pdo = new PDO('mysql:host=localhost; dbname=baza', 'root', '');
			return true;
			echo "Połączono";
		}
		catch(PDOException $e)
		{
			echo 'Wystąpił problem z połączeniem z bazą danych '.$e->getMessage();
		}
 }
 
 static public function disconnect()
 {
	 ob_end_flush();
 }
 
 
 static public function clear($text)
 {
    // jeśli serwer automatycznie dodaje slashe to je usuwamy
    if(get_magic_quotes_gpc())
	{
        $text = stripslashes($text);
    }
    $text = trim($text); 
    $text = mysql_real_escape_string($text); 
    $text = htmlspecialchars($text);
    return $text;
 
  }
 
 static public function codepass($password)
 {
	return sha1(md5($password).'#!%Rgd64');
 }
 
}
 
?>
[PHP] pobierz, plaintext 

phpLoader.php:

[PHP] pobierz, plaintext 
<?php
 
class phpLoader extends config
{
	// funkcja: logowanie
	static public function login()
	{		
		require_once 'config.class.php';
		config::connect();
		if(empty($_SESSION['logged'])) 
		{
		if(isset($_POST['name'])) 
		{
        $_POST['name'] = config::clear($_POST['name']);
        $_POST['password'] = config::clear($_POST['password']);		
        $_POST['password'] = config::codepass($_POST['password']); 
 
        $query = config::$pdo->prepare("SELECT `user_id` FROM `users` WHERE `user_name` = '{$_POST['name']}' AND `user_password` = '{$_POST['password']}' LIMIT 1");
		try
		{
			$query->execute();
			if($query->rowCount() > 0)
			{
				$dane = $query->fetch(PDO::FETCH_ASSOC);
				if($dane['user_pasword'] === $_POST['password'])
				{
					$_SESSION['logged'] = true;
					$_SESSION['user_id'] = $row['user_id']; 
				}
				else
				{
					echo "Złe hasło";
				}
 
			}
				else
				{
					echo "Zły użytkownik";
				}
		}
	catch(PDOException $e)
	{
		$e->getMessage();
	}
		}
		}
		config::disconnect();
	}
 
}
 
?>
[PHP] pobierz, plaintext 

login.php:

[PHP] pobierz, plaintext 
<?php
 
require_once 'CLASS/config.class.php';
require_ONCE 'CLASS/ModuleLoader.class.php';
require_once 'CLASS/phpLoader.class.php';
 
ModuleLoader::setDoctype();
phpLoader::login();
ModuleLoader::setLogo();
ModuleLoader::setTopbar();
ModuleLoader::setSidebar();
ModuleLoader::setContent();
ModuleLoader::setFooter();
 
?>
[PHP] pobierz, plaintext 

Na kodowanie hasła, itd. nie patrzcie na razie. Na razie chciałbym się po prostu zalogować. Co robię źle? Dlaczego to nie działa? Co powinienem zmienić/dopisać?

[Damonsson]

Nie nie nie, wszystko źle.

Nie wolno używać static! Wolno w różnych przypadkach, ale na tym etapie nauki przyjmij, że nie wolno i koniec, będziesz zmuszony myśleć obiektowo.
Poczytaj o Dependency Injection, poszukaj czegoś w google w stylu: php login db dependency injection i oprzyj na tym swój przykład.
Staraj się używać autoloadera vide Composer.
Staraj się używać przestrzeni nazw, które weszły do PHP w 2009 roku, mamy 2015.

Nie zastanawiaj się nawet nad tym błędem, bo szkoda Twojego czasu, tylko napisz to porządnie od nowa.


Edit: akurat użycie static do metody public function clear jest zasadne (IMG:style_emoticons/default/smile.gif)

Ten post edytował Damonsson 8.12.2015, 23:24:13