Tak jak w tytule tematu, skrypt zwraca mi błąd:

[PHP] pobierz, plaintext 
class db
{
	protected static $host;
	protected static $port;
	protected static $user;
	protected static $pass;
	private static $pdo;
 
	public function __construct()
	{
		self::$host = HOST;
		self::$port = PORT;
		self::$user = USER;
		self::$pass = PASS;
		$options  = array(
            PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES UTF8'
        );
        try 
		{
            self::$pdo = new PDO("mysql:host={self::$host};port={self::$port}", self::$user, self::$pass, $options);
			self::$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		} 	
		catch(PDOException $error) 
		{
            die("ERROR CONNECT: " .  $error->getMessage());
        }
	}
 
	public static function query($query='',$value=array(),$type="true") 
	{
		try 
		{
			$query = self::$pdo->prepare($query);
			switch($type)
			{
				case 'true':
				case 'false':
					$tab = array('true' => array(0 =>'true',1=>'false'),'false' => array(0 =>'false',1=>'true'));
					$result = $query->execute($value) ? $tab[$type][0] : $tab[$type][1];					
					break;
				default:
					$result = $query->execute($value) ? $query->$type() : false;
			}
		} 
		catch(PDOException $error) 
		{ 
			die("ERROR QUERY: " . $error->getMessage()); 
		}
 
		return $result;
	}	
}
 
[PHP] pobierz, plaintext 

dokładnie chodzi o tą oto linijkę

Sie zdecyduj. Albo uzywasz STATIC albo normalnie obiektow. Bo self::$pdo inicjalizujesz w konstruktorze, ale konstruktor odpala sie tylko dla obiektow a ty masz funkcje statyczna:
public static function query
ktora pewnie tez odpalasz jaka statyczna.

Racja mój błąd, przerobiłem to w taki oto sposób

[PHP] pobierz, plaintext 
class db
{
	private $pdo;
 
	public function __construct()
	{
		$options  = array(
            PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES UTF8'
        );
        try 
		{
            $this->pdo = new PDO("mysql:host={HOST};port={PORT}", USER, PASS, $options);
			$this->pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		} 	
		catch(PDOException $error) 
		{
            die("ERROR CONNECT: " .  $error->getMessage());
        }
	}
 
	public function query($query='',$value=array(),$type="true") 
	{
		try 
		{
			$db = new db();
			$query = $db->pdo->prepare($query);
			switch($type)
			{
				case 'true':
				case 'false':
					$tab = array('true' => array(0 =>'true',1=>'false'),'false' => array(0 =>'false',1=>'true'));
					$result = $query->execute($value) ? $tab[$type][0] : $tab[$type][1];					
					break;
				default:
					$result = $query->execute($value) ? $query->$type() : false;
			}
		} 
		catch(PDOException $error) 
		{ 
			die("ERROR QUERY: " . $error->getMessage()); 
		}
 
		return $result;
	}	
}
 
[PHP] pobierz, plaintext 

Teraz chyba jest dobrze napisane.

Nie, nie jest dobrze napisane....

Po grzyba w query robisz
$db = new db();
przeciez tworzysz ten sam obiekt w tej samej klasie...

Obiekt pdo stworzyles raz w konstruktorze i to z niego masz korzystac a nie tworzyc nowe dziekie kombinacje

to w jaki sposób wywołać obiekt pdo ? $this->pdo nie działa

Pokaz jak odpalasz funckje query(). I ogolnie pokaz caly kod

No więc na szybko


metoda query jest statyczna a w niej

self::$pdo po prostu nie istnieje bo KONSTRUKTOR nie jest wykonany !

Jeśli chcesz robić statyczną klasę do obsługi bazy to nie w konstruktorze jest łączenie z bazą a w statycznej metodzie

np statyczna klasa implementująca PDO

[PHP] pobierz, plaintext 
use PDO;
 
class Database extends PDO
{
    public static function get($confi = false)
    { 
        $confi = !$confi ? array (
            'type' => DB_TYPE,
            'host' => DB_HOST,
            'name' => DB_NAME,
            'user' => DB_USER,
            'pass' => DB_PASS
        ) : $group;
 
        $type = $confi['type'];
        $host = $confi['host'];
        $name = $confi['name'];
        $user = $confi['user'];
        $pass = $confi['pass'];
 
        $id = "$type.$host.$name.$user.$pass";
        if (isset(self::$instances[$id])) {
            return self::$instances[$id];
        }
 
        try {
            $instance = new Database("$type:host=$host;dbname=$name;charset=utf8", $user, $pass);
            $instance->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            self::$instances[$id] = $instance;
            return $instance;
        } catch (PDOException $e) {
            Logger::newMessage($e);
            Logger::customErrorMsg();
        }
    }
[PHP] pobierz, plaintext 

i wywołujesz to sobie w dowolnym modelu

[PHP] pobierz, plaintext 
class Model {
    protected $_db;
 
    public function __construct(){
       $this->_db = Database::get(); // to jest Twoje połączenie z bazą za pomocą PDO
    }
 
    public function insert($table, $array){        
        return $this->_db->insert($table,$array);
    }    
 
....
[PHP] pobierz, plaintext 

Wykonałem to w taki sposób,

Czy jest to prawidłowe ?

[PHP] pobierz, plaintext 
class db
{	
	public function connect()
	{ 
		$host = HOST;
		$port = PORT;
		$user = USER;
		$pass = PASS;	
		$options  = array(
            PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES UTF8'
        );
        try 
		{
			$connect = new PDO("mysql:host={$host};port={$port}", $user, $pass, $options);
			$connect->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			return $connect; 
		}
		catch(PDOException $error) 
		{
            die("ERROR CONNECT: " .  $error->getMessage());
        }
	}	
 
	public function query($query='',$value=array(),$type="true",$cashe=0) 
	{
		try 
		{
			$query = db::connect()->prepare($query);
			switch($type)
			{
				case 'true':
				case 'false':
					$tab = array('true' => array(0 =>'true',1=>'false'),'false' => array(0 =>'false',1=>'true'));
					$result = $query->execute($value) ? $tab[$type][0] : $tab[$type][1];					
					break;
				default:
					$result = $query->execute($value) ? $query->$type() : false;
			}
		} 
		catch(PDOException $error) 
		{ 
			die("ERROR QUERY: " . $error->getMessage()); 
		}
 
		return $result;
	}
[PHP] pobierz, plaintext 

funkcję query wykonuje tak:

[PHP] pobierz, plaintext 
db::query('SELECT * FROM `'.CMS.'`.`web_config`',array(),'fetch');
[PHP] pobierz, plaintext 

Cos ty sie uparl na tworzenie tych obiektow?

W konstruktorze masz raz utworzyc obiekt $pdo a potem wszedzie indziej tylko
$this->pdo

Jesli ci nie dzialalo znaczy ze zrobiles to zle wczesniej. Dlatego prosilem o kod. No ale widze swoje wiesz lepiej...

edit:

db::query
To jest wywoalnie statycznie. query ma byc wywolywane tak: $db->query gdzie $db =new db(). Temu ci wczesniej nie dzialalo $this->pdo bo co ty odpalales ciagle metody statyczne. Zapomnij na tym etapie o metodach statycznych tylko uzywaj normalnie obiektow

no ale kiedy robię to w taki sposób:

[PHP] pobierz, plaintext 
class db
{	
 
	protected $pdo;
 
	public function __Construct()
	{ 
		$host = HOST;
		$port = PORT;
		$user = USER;
		$pass = PASS;	
		$options  = array(
            PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES UTF8'
        );
        try 
		{
			$this->pdo = new PDO("mysql:host={$host};port={$port}", $user, $pass, $options);
			$this->pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		}
		catch(PDOException $error) 
		{
            die("ERROR CONNECT: " .  $error->getMessage());
        }
	}	
 
	public function query($query='',$value=array(),$type="true") 
	{
		try 
		{
			$query = $this->pdo->prepare($query);
			switch($type)
			{
				case 'true':
				case 'false':
					$tab = array('true' => array(0 =>'true',1=>'false'),'false' => array(0 =>'false',1=>'true'));
					$result = $query->execute($value) ? $tab[$type][0] : $tab[$type][1];					
					break;
				default:
					$result = $query->execute($value) ? $query->$type() : false;
			}
		} 
		catch(PDOException $error) 
		{ 
			die("ERROR QUERY: " . $error->getMessage()); 
		}
 
		return $result;
	}
}
[PHP] pobierz, plaintext 

zwraca mi błąd


chodzi o tą linijkę:

[PHP] pobierz, plaintext 
$query = $this->pdo->prepare($query);
[PHP] pobierz, plaintext 

Nie wiem może jestem jakiś tępy albo coś ale nie rozumiem gdzie robię błąd

Wszystko dlatego ze query odpalassz statycznie: db::query

Mowie ci: zrob to normalny obiekt z tego - to sa naprawde podstawy obiektow, przez ktore w ogole nie przeszedles :/

[PHP] pobierz, plaintext 
 
$db = new db();
db->query(...);
[PHP] pobierz, plaintext 

No ale jeżeli 10-15 plików będzie się tak generować to będzie zwalniać stronę chyba i dlatego statycznie chciałem to napisać.

chyba że, się mylę w tej kwestii to popraw mnie

Obiekt db tworzy sie tylko i wylacznie RAZ.
Raz utworzony obiekt db przekazuje sie tylko wszedzie tam gdzie potrzeba. Poczytaj o wzorcu DI (Dependency Injection)

no ale przy każdym załadowaniu strony jest generowane połączenie do bazy chyba tak ?

To zależy. Jest możliwość wykonania "lazy loading" czyli zainicjowanie klasy tylko jeśli jest potrzebna.
Możesz sobie zerknąć np. w kod https://github.com/zendframework/zend-servicemanager oraz jego dokumentację.

To w jaki sposób działa to w joomli ? bo ja własnie starałem bazować się na jej kodzie.

Zmień obiekt zainteresowania (czy. Jomla)

Na czym mogę bazować?

Podałem ci już link. Zobacz inne komponenty typu zend db. Zainteresuj sie dobrze napisanymi frameworkami typu symfony, zend. Ucz się od najlepszych a nie z tego syfu jomli.
Zobacz ostatnie artykuły z mojej stopki. Ostatnio pisałem jak wykorzystywać komponenty zenda we własnych projektach.

Dzięki wielkie zaraz sobie pobiorę i zobaczę jak to działa :-)

LowiczakPL a Ty jak chcesz pomagać, to czytaj uważnie, bo kod już dawno nie był statyczny i autor by sobie poradził, gdybyś nie wprowadzał w błąd jego