[OOP] Migliorare funzione per registrazione utente o modifica

[hutton]

Salve a tutti, vorrei sottoporvi due metodi di classe in php5 che servono per visualizzare un form a seconda che sia un nuovo utente che si deve registrare o un utente già esistente che deve modificare i suoi dati.. I metodi funzionano, ma vorrei sapere se fosse possibile migliorarli e semplificarli a livello di codice, sopratutto per quanto riguarda le variabili dei dati in entrata:

public function start_registration()
    {
        // se il form e i suoi input sono stati inviati
        if(isset($_POST['register']))
        {
            // controllo l'email
            $this->check_insert_registration($_POST['email'], $_SESSION['id']);
            
            if($this->verify == FALSE)
            {
                if(!isset($_SESSION['id']))
                {
                    // inseriemo all'interno del database i dati
                    $this->query_insert_registration($_POST);
                }
                else
                {
                    // aggiornamento dei dati del database
                    $this->query_change_registration($_POST);
                }
                return TRUE;
            }
            else
            {
                //return FALSE;
                echo '<div class="error">La mail inserita &egrave; gi&agrave; esistente!</div>';
            }
        }
        return FALSE;
    }
    
    // controlla esistenza della email
    // se esistono nel database ritorna un messaggio altrimenti ritorna una stringa nulla
    public function check_insert_registration($email, $id=null)
    {
        if(is_null($id))
        {
            $query = "SELECT id FROM users WHERE email = '$email'";
        }
        else
        {
            $query = "SELECT id FROM users WHERE email = '$email' AND id != $id";
        }
        
        $result = mysql_query($query);
        
        if(mysql_num_rows($result)>0)    
        {
            $this->verify .= TRUE;
        }
        else
        {
            $this->verify .= FALSE;
        }
        
        return $this->verify;
    } 

vorrei evitare ad esempio di inserire "$_POST" e "$_SESSION" in questa riga:

$this->check_insert_registration($_POST['email'], $_SESSION['id']); 

Un grazie a chi mi può aiutare

[badaze]

Per me sarebbe meglio se non ci fossero superglobali all'interno della classe. Senza, la classe potrebbe servire dappertutto o per altri siti che non sarebbero obbligati ad utilizzare le stesse variabili POST o potrebbero adirittura utilizzare variabili $GET.

[chumkiu]

Ciao,
ti do' un po' di consigli.

- Non usare le deprecate funzioni mysql_* ma usa l'interfaccia PDO

- Hai un problema di SQL Injection quando fai le query tipo

$query = "SELECT id FROM users WHERE email = '$email'"; 

Problema che con le PDO puoi mitigare facilmente.


- Nel codice fai un concatenamento di un valore booleano

$this->verify .= TRUE; 

che non ha molto senso.

- Per il tuo problema puoi semplicemente passare i valori che ti servono direttamente nel metodo start_registration.
Tipo:

if(isset($_POST['register'])) {
     $engine->start_registration($_POST['user'],$_SESSION['id']);
} 

oppure

if(isset($_POST['register'])) {
     $engine->start_registration($_POST);
} 

oppure ancora

if(isset($_POST['register'])) {
     $engine->start_registration(
           Array(
                   'email'  =>$_POST['email'],
                   'name' => $_POST['name']
// ecc. ecc. ecc.

            )
     );
} 

[hutton]

Grazie per i consigli... mi metto al lavoro!!

[hutton]

e come posso fare per eliminare le superglobali ma avere lo stesso il valore di esse?

[Al_katraz984]

Originariamente inviato da hutton
e come posso fare per eliminare le superglobali ma avere lo stesso il valore di esse?

singleton -> sconsiglio

composition -> meglio

[hutton]

??