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:
vorrei evitare ad esempio di inserire "$_POST" e "$_SESSION" in questa riga:Codice PHP: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 è già 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;
}
Un grazie a chi mi può aiutareCodice PHP:$this->check_insert_registration($_POST['email'], $_SESSION['id']);
![]()

Rispondi quotando