Forum PHP.pl

Witam, posiadam dwie klasy, DB i Player, w player w konstrukcie zrobilem laczenie z DB. W funkcji getPlayer w Player mam dane o uzytkowniku np. $this->id = $array['id']; teraz gdy zrobilem nowa funkcje pod funkcja getPlayer w tej samej klasie gdy odwoluje sie do id czyli $this->id nie pobiera mi juz tego id. Czy musze robic jakas specjalna publiczna funkcje lub cos innego abym mogl z tego korzystac?

Nie mam pojęcia czemu, ale nadal nie działa. Nie dostarcza id do zapytania.

[PHP] pobierz, plaintext 
class DB
{
   public $mysqli = null;
   public function __construct() {
    /* polaczeni */
   }
}
class Player
{
	public $player;
 
	public function __construct()
	{
		mb_internal_encoding( 'UTF-8' );
        mb_regex_encoding( 'UTF-8' );
        mysqli_report( MYSQLI_REPORT_STRICT );
 
		$this->db = new DB();
	}
 
	public function getPlayer($array)
	{
		$object = array();
 
			$this->id = $array['id'];
 
			$user = $this->db->query('SELECT * FROM `user` WHERE `id` = '.$this->player->id);
         }
}
[PHP] pobierz, plaintext 

Z góry dziękuję.

Ten post edytował IdeaProgram 6.03.2016, 12:27:21

Nie wygląda żebyś ustawiał $this->player->id, jest tylko $this->id. Rób sobie zawsze var_dump($zmienna) jeżeli nie masz pewności co się dzieje. Możesz też wymusić typ tablicowy getPlayer(array $array) chociaż w getterze raczej się nie ustawia danych.

Uzupełniłem ręcznie zapytanie i mam problem z bazą.

[PHP] pobierz, plaintext 
class DB {
 
public $db = null;
 
    public function __construct() {
 
        include_once "system/config.php";        
 
        $this->db = new mysqli(DB_SERVER, DB_USER, DB_PASS, DBNAME);
 
        if ($this->db->connect_errno) {
 
            echo "Error MySQLi: (". $this->db->connect_errno . ") " . $this->db->connect_error;
 
            exit();
        }
    }
 
    public function connect()
    {
    	$this->db = new mysqli(DB_SERVER, DB_USER, DB_PASS, DBNAME);
    	return $this->db;
    }
 
    public function query($query)
    {
 
 
    	//
    }
}
 
class Player
{
	public $player;
 
	public function __construct()
	{
		mb_internal_encoding( 'UTF-8' );
        mb_regex_encoding( 'UTF-8' );
        mysqli_report( MYSQLI_REPORT_STRICT );
 
		$this->db = new DB();
	}
 
	public function getPlayer($array)
	{
		$object = array();
 
			$this->id = $array['id'];
 
			$user = $this->db->query('SELECT * FROM `user` WHERE `id` = 1');
 
				foreach($user as $key => $value)
				{
					array_push($object, $value);
				}
 
			debug($object);
 
	}
 
	public function getGuild()
	{
		/* */ 
	}
 
}
[PHP] pobierz, plaintext 

Bład:

[PHP] pobierz, plaintext 
Warning: Invalid argument supplied for foreach() in C:\xampp\htdocs\_class.php on line 59
Array
(
)
[PHP] pobierz, plaintext 

Coś skopałem chyba z bazą.

A co zwraca query()?

Właśnie tutaj sam nie wiem, w mysqli jest juz funkcja query, lecz chyba ja musze zdefiniować ja w obiektówce? Gdybym robił struturalnie to przeciez funkcji query nie musiał bym definiować, sam nie wiem. Gdy z klasy DB usunę funkcję query to wywala że nie może odnaleźć funkcji query(), lecz przeciez ona jest w mysqli.

Ten post edytował IdeaProgram 6.03.2016, 13:10:08

Pod $this->db->db->query jest natywna metoda http://php.net/manual/pl/mysqli.query.php ale ty ją definiujesz w klasie DB. connect() większego sensu nie ma bo robisz to samo w konstruktorze.
var_dump($user);

Ten post edytował viking 6.03.2016, 13:14:48

Dobra już zrozumiałem, końcowo funkcja query wygląda tak:

[PHP] pobierz, plaintext 
public function query($q)
    {
 
    	if($this->query = $this->db->query($q)) {
 
       		$results = $this->query;
 
      	  return $results;
 
    	}
 
    }
[PHP] pobierz, plaintext 

Ten post edytował IdeaProgram 6.03.2016, 13:33:42

A po co to robisz? To wszystko z konstruktora powinieneś do jakiegoś configa wyrzucić a z takim zastosowaniem jak pokazałeś, klasa DB nie ma sensu.

Dobra, dzieki.

Ostatni już problem, założmy mam 2 klasy Player i Guild, chce do konstruktora klasy Guild podać id gracza, czemu nie moge uzyć $this->player->id ?


Ten post edytował IdeaProgram 6.03.2016, 15:48:12