POST dentro __construct

[torrone]

mi sto avvicinando alla programmazione ad oggetti.
mi son letto l'intero guida qui presente su html.it e ora sto provando a buttar giù un pò di codice

partiamo da una classe per il login

<?php

class Login {

    function __construct() {

        $nickname = mysql_real_escape_string($_POST['nickname']);
        $password = mysql_real_escape_string($_POST['password']);

        if (isset($nickname) AND isset($password)) {

            $this->effettuaLogin($nickname, $password);

        }

    }

    function __destruct() {

    }


    private function effettuaLogin($nickname, $password) {

    // codice login

    }

}

?>

e la pagina dove vengono inviati tramite post i dati del login

<?php

include "Login.class.php";

$obgLogin = new Login;

?>

come faccio a far leggere i dati inviati tramite post alla classe?

[eiyen]

hai riscontrato che non ti legge il valore di $_POST in quella posizione?

[torrone]

Originariamente inviato da eiyen
hai riscontrato che non ti legge il valore di $_POST in quella posizione?

si dentro __construct non mi legge $_POST

[_debo]

Come hai verificato la mancata lettura? È possibile vedere anche il codice della form che invia i dati in post?

[torrone]

allora facendo in questo modo funziona e mi effettua il login

pagina ricezione dati post

$objLogin = new Login;
$objLogin->effettuaLogin($_POST['nickname'], $_POST['password']); 

classe

class Login {

    public function __construct() {

    }

    public function __destruct() {

    }


    public function effettuaLogin($nickname, $password) {
    // funzione login
    }

} 

mentre come facevo prima no..

il form è semplicemente

<form method="post" action="controllo-login.php">
Nickname <input type="text" name="nickname" value="" /> 
Password <input type="password" name="password" value="" /> 
<input type="submit" name="login" value="Login" />
</form>

[torrone]

se metto

private effettuaLogin($nickname, $password) 

non mi legge i dati POST dentro __construct

mentre se metto

public effettuaLogin($nickname, $password) 

mi legge i dati POST dentor __construct

come mai?

[_debo]

Mi puoi far vedere i var_dump che hai usato per affermare ciò? Il codice inizialmente postato in teoria deve funzionare a meno che non ci sia un errore all'interno del metodo privato che non hai pubblicato.

[torrone]

ora funziona così da un momento all'altro...

ecco il codice

class Login {

    public function __construct() {

        if ($_POST['nickname'] == false) {

            echo "ERRORE";
            exit;

        } else {

            $this->effettuaLogin($_POST['nickname'], $_POST['password']);

        }

    }

    public function __destruct() {

    }


    private function effettuaLogin($nickname, $password) {
    }

}

?> 

<?php

include "Login.class.php";

$objLogin = new Login;

?>

[_debo]

Chissà cosa avevi sbagliato, comunque, giusto per completezza il metodo che stai usando è errato.
Innanzitutto è meglio usare una isset per verificare l'esistenza dell'input, secondo è bene verificate che tutti gli input che ti aspetti vengano passati prima di dare errore ed in ultimo il secondo metodo che avevi adottato era quello migliore. Il costruttore non dovrebbe mai intercettare l'input ma semplicemente inizializzare l'oggetto.

[torrone]

Originariamente inviato da _debo
Chissà cosa avevi sbagliato, comunque, giusto per completezza il metodo che stai usando è errato.

e allora vediamo subito di sistemare

Innanzitutto è meglio usare una isset per verificare l'esistenza dell'input,

questo l'ho già fatto, era solo un esempio veloce questo che ho scritto qui

secondo è bene verificate che tutti gli input che ti aspetti vengano passati prima di dare errore

infatti controllo se sia il nickname che la password siano stati inviati, altrimenti do un errore (questo lo controllo direttamente però in "effettaLogin".
L'esempio che ho mostrato qui con il controllo mi serviva solo per capire se arrivava il POST

ed in ultimo il secondo metodo che avevi adottato era quello migliore. Il costruttore non dovrebbe mai intercettare l'input ma semplicemente inizializzare l'oggetto.

per cui non è corretto così?

Login.class.php

<?php

class Login {

    public function __construct() {

        $_POST['nickname'] = isset($_POST['nickname']) ? $_POST['nickname'] : "";
        $_POST['password'] = isset($_POST['password']) ? $_POST['password'] : "";

            $this->effettuaLogin($_POST['nickname'], $_POST['password']);

    }

    public function __destruct() {

    }


    private function effettuaLogin($nickname, $password) {

    // funzione con i vari controlli

    }

}

?>

controllo-login.php

<?php

include "Login.class.php";

$objLogin = new Login;

?>