Problem z odwołaniem się do funkcji Opcje

[Dominator]

Cześć,
jestem w trakcie przeprojektowania strony, konfiguracji i innych rzeczy i zmagam się z problemem od ponad godziny. A mianowicie mam dwa pliki - DatabaseConnector.php w którym łączę się z bazami danych oraz PlayerPanel.php.
Tak wygląda struktura pierwszego:

[PHP] pobierz, plaintext 
class DatabaseConnector
{
    public $pdo;
    public $pdo2;
    public $host = "xxx";
    public $port = 1234;
    public $username = "xxx";
    public $password = "xxx";
    public $database = "xxx";
    public $database2 = "xxx";
 
    public function __construct()
    {
        $this->connect();
    }
 
    public function connect()
    {
        try
        {
            $this->pdo = new PDO('mysql:host='.$this->host.';dbname='.$this->database.';port='.$this->port, $this->username, $this->password );
	    $this->pdo2 = new PDO('mysql:host='.$this->host.';dbname='.$this->database2.';port='.$this->port, $this->username, $this->password );
        }
        catch(PDOException $e)
        {
            echo 'Połączenie nie mogło zostać utworzone.<br />'.$e->getMessage();
			exit;
        }
    }
 
	public function __destruct() {
		$this->closeConnection();
    }
 
	public function closeConnection()
	{
		$this->pdo = null;
                $this->pdo2 = null;
	}
}
 
[PHP] pobierz, plaintext 

PlayerPanel.php

[PHP] pobierz, plaintext 
include "DatabaseConnector.php";
class PlayerPanel extends DatabaseConnector
{
    public $nick;
    public $kills;
    public $deaths;
    public $kd;
    public $guild;
 
    public function __construct($id)
    {
        if($id == null)
        {
            die("");
            exit();
        }
 
        $this->nick = $id;
 
        echo $this->getPlayerInfo($this->nick);
    }
 
    public function getPlayerKills()
    {
        return $this->kills;
    }
 
    public function getPlayerDeaths()
    {
        return $this->deaths;
    }
 
    public function getKDRatio()
    {
        return $this->kd;
    }
 
    public function getGuild()
    {
        return $this->guild;
    }
 
 
	public function getPlayerInfo($tag)
    {
		$stmt = $this->pdo2 -> prepare("SELECT * FROM `XXXX` WHERE player=:tag");
		$stmt -> bindValue(':tag', $tag, PDO::PARAM_STR); // 2
		$stmt->execute();
 
		$result = $stmt->fetchAll(PDO::FETCH_COLUMN, 0);
		return $result;
    }
 
	public function getPlayerGuild($nick)
	{
		$stmt = $this->pdo2 -> prepare("SELECT * FROM `XXXX` WHERE player=:tag");
		$stmt -> bindValue(':tag', $nick, PDO::PARAM_STR); // 2
		$stmt->execute();
 
		$results=$stmt->fetchAll(PDO::FETCH_ASSOC);
		$json=json_encode($results);
		return $json;
	}
 
 
}
[PHP] pobierz, plaintext 

Chcę sobie wyświetlić statystyki gracza metodą echo $this->getPlayerInfo($this->nick);, ale wygląda, że to wcale nie działa.

Próbowałem różnych metod, cały czas mój php twierdzi, że $this->pdo2 nie istnieje... błąd wygląda następująco: Fatal error: Uncaught Error: Call to a member function prepare() on null in .... 54 (pogrubiłem to w kodzie)

Ten post edytował Dominator 10.06.2017, 18:34:24

[nospor]

Pomijajac fakt, ze twoje klasy sa do bani, to w klasie PlayerPanel nigdzie nie odpalasz connect() z rodzica to co sie dziwisz ze nia masz polaczenia?

[Dominator]

Pomijajac fakt, ze twoje klasy sa do bani, to w klasie PlayerPanel nigdzie nie odpalasz connect() z rodzica to co sie dziwisz ze nia masz polaczenia?

W czym są beznadziejne?

[nospor]

A chociazby w tym, obiekt klasy bazodanowej przekazuje sie do innych obiektow a nie dziedziczy sie po tym

Plus cala masa innych rzeczy.

A takich kwiatkow
die("");
exit();
to juz nawet odechciewa sie komentowac

[Dominator]

A chociazby w tym, obiekt klasy bazodanowej przekazuje sie do innych obiektow a nie dziedziczy sie po tym

Plus cala masa innych rzeczy.

A takich kwiatkow
die("");
exit();
to juz nawet odechciewa sie komentowac

Na siłę próbujesz czepiać się rzeczy, które nie dotyczą tematu. Wspomniałem w pierwszym poście, że jestem w trakcie przeprojektowania, dlatego też owe "kwiatki" zostaną zmodyfikowane, aczkolwiek dziękuję za krytykę.
Po wywołaniu metody connect() w dalszym ciągu nie jestem w stanie wyświetlić informacji z bazy danych, co robię źle?

[nospor]

Nie czepiam sie na sile. Udzielilem ci odpowiedzi co robisz zle. Przy okazji tylko wspomnialem ze twoj kod jest do bani. To ty to ciagnales dalej nie ja

Zrobiles connect()? Super, to moze pokazesz nam jeszcze jak to zrobiles, bo najwyrazniej nadal zle.

ps:

Wspomniałem w pierwszym poście, że jestem w trakcie przeprojektowania

Takie rzeczy jak die i exit nie powinny sie pojawic w zadnej wersji tego kodu. Skoro pytales sie mnie, co jest zle, najwyrazniej nie zdawales sobie z tego nadal sprawy.

[Dominator]

Nie czepiam sie na sile. Udzielilem ci odpowiedzi co robisz zle. Przy okazji tylko wspomnialem ze twoj kod jest do bani. To ty to ciagnales dalej nie ja

Zrobiles connect()? Super, to moze pokazesz nam jeszcze jak to zrobiles, bo najwyrazniej nadal zle.

ps:

Takie rzeczy jak die i exit nie powinny sie pojawic w zadnej wersji tego kodu.

Chwilę pomyślałem i wystarczyło dodać parent::__construct(); do klasy PlayerPanel, wygląda to tak:

[PHP] pobierz, plaintext 
class PlayerPanel extends DatabaseConnector
{
    public $nick;
    public $kills;
    public $deaths;
    public $kd;
    public $guild;
 
    public function __construct($id)
    {
        parent::__construct();
 
        $this->nick = $id;
        $data_info = json_decode($this->getPlayerInfo($this->nick));
 
        $this->kills = $data_info[0]->kills;
        //...
    }
}
[PHP] pobierz, plaintext 

Zaczęło działać i mogę kontynuować pracę, bardzo dziękuję za pomoc Nie skapnąłem się, że nie wywołuję nigdzie tego.
A tak btw - czytałem trochę o tym całym OOP (niedawno wróciłem do programowania) i trochę nie mogę zrozumieć co w moim kodzie jest nie tak... chodzi o metodę getPlayerInfo, która w jakiś nieprawidłowy sposób pobiera dane? Albo klasa bazodanowa sama w sobie jest błędnie napisana? Byłoby miło gdyby ktoś mnie pokierował co robię źle, nie jestem na tak świetnym poziomie jak zapewne połowa tego forum i cały czas się uczę
Pozdrawiam

[nospor]

Napisalem ci na samym poczatku co jest zle. No dobra, napisalem tylko dwa, jeden z nich byl

A chociazby w tym, obiekt klasy bazodanowej przekazuje sie do innych obiektow a nie dziedziczy sie po tym

Poczytaj o wzorcu Dependcy Injection

Bledow jest cala inna masa, ale popraw wpierw jeden.