Vorrei chiedere un opinione su questo script

[rikkardo]

Questa funzione serve per cambiare la password di un utente... sono sicuro che non funziona... Non ho mai fatto una cosa che funziona al primo colpo XD

function cambiaPassword($vecchiapassword, $nuovapass, $mail) {
$conn_db = mysql_connect($db_host, $db_user, $db_pass);
if(!conn_db) { die(mysql_error();); 
$cambio = false;
}
else {
mysql_select_db($db_name, $conn_db);
$query = "UPDATE SET password = '$nuovapass'";
$query .= "FROM $tbl_name";
$query .= "WHERE username = '$username', password = '$vecchiapassword', email = '$mail';";
mysql_query("$query");
$cambio = true;
}
if(!$cambio) {
echo "Errore nel cambio della password... Assicurati che i dati siano giusti!"; }
else {
echo "Cambio avvenuto con successo"; }
} 

[miki92]

Nella query

$query = "UPDATE SET password = '$nuovapass'"; 

Hai mancato il nome della tabella:

UPDATE tabella SET campo = 'valore'

E

$query .= "FROM $tbl_name"; 

Può essere eliminata, e comunque molto più velocemente potevi fare:

$query = "UPDATE tabella SET password = '$nuovapass' WHERE username = '$username', password = '$vecchiapassword', email = '$mail'";
mysql_query("$query"); 

O anche direttamente:

mysql_query("UPDATE tabella SET password = '$nuovapass' WHERE username = '$username', password = '$vecchiapassword', email = '$mail'"); 

Comunque la prossima volta prova e se non funziona cerca di capire il problema e alla fine se proprio non va...posta

C'è ancora un errore, nella clausola where le condizioni sono concatenate da una virgola e non da un AND o un OR (in questo caso è l'AND che serve).

$query = "UPDATE tabella SET 
                 password = '$nuovapass' 
              WHERE 
                 username = '$username' AND 
                 password = '$vecchiapassword' AND 
                 email = '$mail'"; 
mysql_query($query); 

[rikkardo]

{per miki92} Ok, ma chiedevo solo un opinione

[miki92]

Ma tu non hai specificato, nel testo, di che tipo di opinione così mi sono limitato a correggerti quelli che per me erano errori o inutilità.

Comunque se vuoi un opinione posso dirti che quel codice può diventare più corto ma comunque è buono