[PHP]Uzywanie database w innej klasie Opcje

[goartur]

Czesc mam problem z uzywaniem mysql w innej klasie, jestem poczatkujacy w oop, lecz chce wysylac zapytania z klasy user_Access lecz niedziala funkcja $this->db_query, dotaje meldunek "Query method not found in class", probowalem tez uzywc: mysqli_query, lecz tez niedziala,
mam taka klase do polaczenia a mysql

[PHP] pobierz, plaintext 
class db
{
    public function db()
    {
        $db = new mysqli(DB_SERVER, DB_USERNAME, DB_PASSWORD, DB_DATABASE);
 
 
        if ($db->connect_errno) {
 
            echo "Error: Could not connect to database.";
 
            exit;
 
        }
        else { return $db; }
 
    }
 
 
}
[PHP] pobierz, plaintext 

a oto user_Access.php

[PHP] pobierz, plaintext 
    $db = new db();
$userDB = new user_access($db->db());
$userDB->check_login('artur','ol');
class user_access{
 
    public $db;
 
    public function __construct($db){
        $this->db = $db;
 
    }
    public function check_login($login,$password){
        $password = sha1($password);
        $sql = "SELECT * FROM admins WHERE username = '$login' AND password = '$password'";
        $result = $result = $this->db->query($sql);
 
        if($result->num_rows > 1){
            echo 'ok';
        }else{
            echo "not";
        }
    }
 
}
[PHP] pobierz, plaintext 

Moze mi ktos pokazac gdzie jest blad?

Ten post edytował goartur 27.06.2015, 10:19:39

[prz3kus]

[PHP] pobierz, plaintext 
class user_access{
 
    public $db; 
 
    public function __construct($db){
        $this->db = $db;
 
    }
    public function check_login($login,$password){
        $password = sha1($password);
        $sql = "SELECT * FROM admins WHERE username = '$login' AND password = '$password'";
        $result = $result = $this->db->query($sql);
 
        if($result->num_rows > 1){
            echo 'ok';
        }else{
            echo "not";
        }
    }
 
}
 
 
$db = new db();
$userDB = new user_access($db->db());
$userDB->check_login('artur','ol');
[PHP] pobierz, plaintext 

Masz

public $dba;

zamiast

public $db;

[goartur]

Nie to nie to ...
Notice: Trying to get property of non-object in C:\xampp\htdocs\brothers.traning\profiadmin\models\login_Class.php on line 24
not

[markuz]

[PHP] pobierz, plaintext 
<?php
 
class Database {
 
	private static $db;
 
	public static function connect() {
		self::$db = new mysqli(DB_SERVER, DB_USERNAME, DB_PASSWORD, DB_DATABASE);
	}
 
	public static function get() {
		return self::$db;
	}
 
}
 
class Model {
 
	private $db;
 
	public function __construct() {
		if(!Database::$db)
			Database::connect();
 
		$this->db = Database::get();
	}
 
}
 
class UserAccessModel extends Model {
 
	public function login($username, $password) {
		$password = sha1($password);
		$results = $this->db->query(
			"SELECT * FROM admins WHERE username = ? AND password = ?",
			array($username, $password)
		);
 
		return $result->num_rows > 0;
	}
 
}
 
$userAccess = new UserAccessModel();
if($userAccess->login('artur', 'ol'))
	// logowanie
else
	// błąd logowania
 
?>
[PHP] pobierz, plaintext 

[goartur]

[PHP] pobierz, plaintext 
<?php
 
class Database {
 
	private static $db;
 
	public static function connect() {
		self::$db = new mysqli(DB_SERVER, DB_USERNAME, DB_PASSWORD, DB_DATABASE);
	}
 
	public static function get() {
		return self::$db;
	}
 
}
 
class Model {
 
	private $db;
 
	public function __construct() {
		if(!Database::$db)
			Database::connect();
 
		$this->db = Database::get();
	}
 
}
 
class UserAccessModel extends Model {
 
	public function login($username, $password) {
		$password = sha1($password);
		$results = $this->db->query(
			"SELECT * FROM admins WHERE username = ? AND password = ?",
			array($username, $password)
		);
 
		return $result->num_rows > 0;
	}
 
}
 
$userAccess = new UserAccessModel();
if($userAccess->login('artur', 'ol'))
	// logowanie
else
	// błąd logowania
 
?>
[PHP] pobierz, plaintext 

Dzieki za odpowiedz dostaje taki blad:

Fatal error: Cannot access private property Database::$db in C:\xampp\htdocs\brothers.traning\profiadmin\login_Class.php on line 27

[markuz]

Ok. Błąd dotyczy tego fragmentu kodu:

[PHP] pobierz, plaintext 
	public function __construct() {
		if(!Database::$db)
			Database::connect();
 
		$this->db = Database::get();
	}
[PHP] pobierz, plaintext 

Database::$db jest prywatne. W jaki sposób się do niego dostać? Oto zagadka dla Ciebie Rozwiązanie znajduje się w tym samym fragmencie kodu.

[goartur]

Ok. Błąd dotyczy tego fragmentu kodu:

[PHP] pobierz, plaintext 
	public function __construct() {
		if(!Database::$db)
			Database::connect();
 
		$this->db = Database::get();
	}
[PHP] pobierz, plaintext 

Database::$db jest prywatne. W jaki sposób się do niego dostać? Oto zagadka dla Ciebie Rozwiązanie znajduje się w tym samym fragmencie kodu.

Musze zmienc private $db na public ?, jesli tak to dostaje taki blad:

[PHP] pobierz, plaintext 
Warning: mysqli::query() expects parameter 2 to be long, array given in C:\xampp\htdocs\brothers.traning\profiadmin\login_Class.php on line 44
 
Notice: Undefined variable: result in C:\xampp\htdocs\brothers.traning\profiadmin\login_Class.php on line 46
 
Notice: Trying to get property of non-object in C:\xampp\htdocs\brothers.traning\profiadmin\login_Class.php on line 46
[PHP] pobierz, plaintext 

[markuz]

Ogólnie powinieneś przenieść cały ten fragment w construct() do metody get w Database ale nieważne.. Możesz ustawić na public.

Co do drugiego błędu to pomyliłem mysqli z PDO. Zmień wykonywanie zapytania na :

[PHP] pobierz, plaintext 
$results = $this->db->query("SELECT * FROM admins WHERE username = '$username' AND password = '$password'");
[PHP] pobierz, plaintext 

Ten post edytował markuz 27.06.2015, 11:45:35

[goartur]

Ogólnie powinieneś przenieść cały ten fragment w construct() do metody get w Database ale nieważne.. Możesz ustawić na public.

Co do drugiego błędu to pomyliłem mysqli z PDO. Zmień wykonywanie zapytania na :

[PHP] pobierz, plaintext 
$results = $this->db->query("SELECT * FROM admins WHERE username = '$username' AND password = '$password'");
[PHP] pobierz, plaintext 

Nadal ten sam blad:

[PHP] pobierz, plaintext 
Notice: Undefined variable: result in C:\xampp\htdocs\brothers.traning\profiadmin\login_Class.php on line 43
 
Notice: Trying to get property of non-object in C:\xampp\htdocs\brothers.traning\profiadmin\login_Class.php on line 43
[PHP] pobierz, plaintext 

[markuz]

Tym razem wkradła się literówka (results, result). Usiądź nad kodem trochę samemu i spróbuj znaleźć błąd inaczej będzie Ci ciężko cokolwiek napisać..

[goartur]

Tym razem wkradła się literówka (results, result). Usiądź nad kodem trochę samemu i spróbuj znaleźć błąd inaczej będzie Ci ciężko cokolwiek napisać..

Wiem widze zauwazylem to,

[PHP] pobierz, plaintext 
Notice: Trying to get property of non-object in C:\xampp\htdocs\brothers.traning\profiadmin\login_Class.php on line 43
[PHP] pobierz, plaintext 

Zawsze mam ten sam problem i nikt mi nie moze powiedziec o co w tym chodz ...

[viking]

Dostałeś odpowiedź a i kod błędu wszystko mówi. Aby ->metoda() zadziałała zmienna musi być klasą. var_dump($zmienna) i zobaczysz gdzie jest błąd. Będzie tam pewnie null. Private ma zasięg tylko w Model, zmień na protected.