[OOP] usare metodo in classe extends

[gaten]

database.php

    // eseguire una query
    public function execute($sql) {
        if($this->mysqli->query($sql)) 
                    return $this->mysqli->query($sql);
                else 
                    return false;
    }

    // numero di righe restituire da una query
    public function num_rows($sql) {
        $result = $this->execute($sql);

        return $result->num_rows;
    } 

utente.php

class Utente extends Database {
...

    public function checkUser($sql) {
        $result = $this->execute($sql);
        if ($result){
            if ($result->num_rows == 1)
                return true;
            else
                return false;
        }
    }
...
} 

index.php

...
if($login->checkUser($query))
echo "ok";
    else
echo "errore";
... 

Premetto che la query è esatta e i metodi usati in database funzionano, però qualora tento di usare il metodo execute che ho in Database nella classe Utente, mi dà l'errore.

Ho il seguente errore:
Trying to get property of non-object

Come posso risolvere?

[Leilond]

Qui sembra che il problema è un altro
$login non è proprio un oggetto, non è stata correttamente istanziata in precedenza

[gaten]

$login = new Utente; 

eppure, gli altri metodi di Utente funzionano alla perfezione.

[Leilond]

Prova intanto a correggere la funzione in modo che restituisca SEMPRE un risultato
Come l'hai scritta tu, nel caso $result sia false, non restituisce nè True nè False

Anche se non credo sia questo il problema... visto che il messaggio è piuttosto chiaro, nel momento in cui fa la if ( $login->checkUser( $query) ), $login non è ancora (o non è più) un oggetto

A meno che non si riferisca alla variabile $result

Secondo me il problema è il modo in cui utilizzi/confondi la proprietà "num_rows" con il metodo "num_rows"

Probabilmente la tua funzione funzionerebbe così

public function checkUser($sql) {
        if ($this->num_rows( $sql ) ){
                return true;
            else
                return false;
         }
    } 

[gaten]

Con una query esatta(testata) , con o senza l'else che dicevi tu, mi stampa a video sempre e solo errore

[gaten]

Originariamente inviato da Leilond

A meno che non si riferisca alla variabile $result

Secondo me il problema è il modo in cui utilizzi/confondi la proprietà "num_rows" con il metodo "num_rows"

Io ho il seguente metodo:

    public function num_rows($sql) {
        $result = $this->execute($sql);

        return $result->num_rows;
    } 

e funziona benissimo, cioè mi restituisce correttamente il numero di righe ottenute da una select, il problema è quando utilizzo il tutto all'interno della classe Utente, ora non mi dà più l'errore che ho postato all'inizio, ma mi stampa solo e sempre errore, cioè e come se:

if($result) non fosse mai vero, eppure la query è corretta.

[Leilond]

Prova a modificare il tuo metodo così

  public function checkUser($sql) {
        if ($this->num_rows( $sql ) ){
                return true;
            else
                return false;
        }
    } 

[Leilond]

Questa funzione inoltre è sbagliata

   public function num_rows($sql) {
        $result = $this->execute($sql);

        return $result->num_rows;
    } 

Poichè se il query $sql fosse formalmente sbagliato, il risultato di execut sarebbe FALSE (per come hai scritto execute all'inizio del post) e su un valore FALSE non puoi usare l'istruzione $result->num_rows;
La funzione corretta sarebbe

   public function num_rows($sql) {
        $result = $this->execute($sql);
        if ( $result ) {
          return $result->num_rows;
        } else {
          return 0; // oppure false
        }
    } 

[gaten]

Adesso il problema me lo dà qui:

    public function num_rows($sql) {
        $result = $this->execute($sql);

        return $result->num_rows;
    } 

precisamente la riga del return

[Leilond]

Originariamente inviato da gaten
Adesso il problema me lo dà qui:

    public function num_rows($sql) {
        $result = $this->execute($sql);

        return $result->num_rows;
    } 

precisamente la riga del return

Come ti ho scritto al post precedente, la funzione EXECUTE può restituirti FALSE e quindi, se questo è il caso, l'istruzione "return $result->num_rows;" va ovviamente in errore, perchè stai tentando di reperire la proprietà num_rows da una variabile che non è un oggetto, ma il valore false
Riscrivi la funzione così

    public function num_rows($sql) {
        $result = $this->execute($sql);
        if ( $result ) {
          return $result->num_rows;
        } else {
          return 0; // oppure false
        }
    }