[php] 2 klasy i łatwe poruszanie się po nich Opcje

[Lejto]

Mam 2 klasy w 2 plikach i 1 plik i nazwie admin.php. 1 klasa służy do łączenia z mysql a 2 do zalogowania.
Fragment 1 klasy

[PHP] pobierz, plaintext 
<?php
class mysql
{
    private $lacz;
     
    public function __construct()
    {
        $this->lacz = new mysqli(DB_HOST, DB_USER, DB_PASS, DB_NAME);
 
    }
    public function __dectruct()
    {
        $this->lacz->close();
    }
}
....[...]....
?>
[PHP] pobierz, plaintext 

tu jest wszystko ok
dalej 2 klasa

[PHP] pobierz, plaintext 
<?php
class Auth extends mysql{
    public function authUser($nick, $pass)
    {
        $q = 'SELECT * FROM admin where nick = "'.$nick.'" and pass = "'.sha1($pass).'"';
        $r = mysql::$this->lacz->query($q);
        
        if(mysql::$this->lacz->num_rows($r) == 1)
        {
            return true;
        }
        else
        {
            return false;
        }
    }
}
?>
[PHP] pobierz, plaintext 

i plik admin.php

[PHP] pobierz, plaintext 
<?php
include("db.class.php");
$db = new mysql();
 
include("login.class.php");
$log = new Auth();
 
$log->authUser('aaa','1111');
?>
[PHP] pobierz, plaintext 

gdy to wszystko odpalam to mi woła że nie mam połączenia z mysql i że query nie ma obiektu
źle to zintegrowałem? nie wywołałem poprawnie klasy mysql żeby mi się połączyło z klasie Auth?

[nospor]

nie:private $lacz;
a: protected $lacz;

Doczytaj sobie jakie są roznice miedzy private, protected a public

[Lejto]

ok dzięki już wiem zmieniłem ale jeszcze wywala błąd w num_rows ;/

Kod

Fatal error: Call to undefined method mysqli::num_rows() in C:\Serv\www\spin\login.class.php on line 8

[nospor]

komunikat dosc wyraźny: $lacz nie ma metody num_rows.
Zajrzyj do manuala a dowiesz sie jaki obiekt ma te metode
http://pl.php.net/manual/pl/book.mysqli.php

[Lejto]

muszę jeszcze w to włączyć wywołane query nie?
zrobiłem tak:

[PHP] pobierz, plaintext 
<?php
if(mysql::$this->lacz->$r->num_rows($r) == 1)
        {
            return true;
        }
?>
[PHP] pobierz, plaintext 

i teraz jeszcze inny błąd

Kod

Catchable fatal error: Object of class mysqli_result could not be converted to string in C:\Serv\www\spin\login.class.php on line 8

[nospor]

przeciez query to samo $r:
$r = mysql::$this->lacz->query($q);

nie:mysql::$this->lacz->$r->num_rows($r)
a:mysql::$r->num_rows

link do manuala podalem ci nie bez powodu
http://pl.php.net/manual/pl/mysqli-result.num-rows.php

[Lejto]

ok zrobiłem
ale teraz daje błąd co do uprawnień

Kod

Fatal error: Access to undeclared static property: mysql::$r in C:\Serv\www\spin\login.class.php on line 8

[nexis]

[PHP] pobierz, plaintext 
<?php
class mysql
{
    protected $_lacz;
    
    public function __construct()
    {
        $this->_lacz = new mysqli(DB_HOST, DB_USER, DB_PASS, DB_NAME);
    }
    public function __destruct()
    {
        $this->_lacz->close();
    }
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
class Auth extends mysql
{
    public function authUser($nick, $pass)
    {
        $q = sprintf
            (
                "SELECT * FROM `admin` WHERE `nick` = '%s' AND `pass` = '%s'",
                $nick,
                sha1($pass)
            );
        $r = $this->_lacz->query($q);
        
        return $r->num_rows == 1;
    }
}
?>
[PHP] pobierz, plaintext 

[Lejto]

ok działa, ale ta mała zmiana nie mówię o num_rows czemu ma służyć? bezpieczeństwu?

[Mephistofeles]

Autoryzacja nie powinna dziedziczyć po bazie - bo czy telefonistka dziedziczy po telefonie ;]?
Przekazuj połączenie w konstruktorze, albo użyj np. singletona/rejestru.
Poza tym mylisz zmienne statyczne z normalnymi - ten komunikat

Kod

Fatal error: Access to undeclared static property: mysql::$r in C:\Serv\www\spin\login.class.php on line 8

o tym mówi.

Ten post edytował Mephistofeles 13.03.2009, 15:59:35