Problema UPDATE database

Ciao a tutti, io ho fatto una pagina che modifica un determinato campo di un determinato utente, che va alla pagina con il seguente script php:

<?php

include('include/database_user_connection.php');

$id = "$_POST[id]";
$username = "$_POST[username]";
$password = "$_POST[password]";
$nome = "$_POST[nome]";
$squadra = "$_POST[squadra]";
$campionato = "$_POST[campionato]";
$email = "$_POST[email]";
$privilegi = "$_POST[privilegi]";

if(!empty($username)
mysql_query("UPDATE utenti SET username = '.$username.' WHERE id = '.$id.'");
elseif(!empty($password)
mysql_query("UPDATE utenti SET password = '.$password.' WHERE id = '.$id.'");
elseif(!empty($nome)
mysql_query("UPDATE utenti SET nome = '.$nome.' WHERE id = '.$id.'");
elseif(!empty($squadra)
mysql_query("UPDATE utenti SET squadra = '.$squadra.' WHERE id = '.$id.'");
elseif(!empty($campionato)
mysql_query("UPDATE utenti SET campionato = '.$campionato.' WHERE id = '.$id.'");
elseif(!empty($email)
mysql_query("UPDATE utenti SET email = '.$email.' WHERE id = '.$id.'");
elseif(!empty($privilegi)
mysql_query("UPDATE utenti SET privilegi = '.$privilegi.' WHERE id = '.$id.'");
else
echo "Devi modificare almeno un campo!";
?>

Ma mi da errore alla seguente linea (quindi anche le altre uguali):

mysql_query("UPDATE utenti SET username = '.$username.' WHERE id = '.$id.'"); 

Non mi pare di vedere errori come posso fare.

Ciao.

[miki92]

Sostituisci:

mysql_query("UPDATE utenti SET username = '.$username.' WHERE id = '.$id.'"); 
elseif(!empty($password) 
mysql_query("UPDATE utenti SET password = '.$password.' WHERE id = '.$id.'"); 
elseif(!empty($nome) 
mysql_query("UPDATE utenti SET nome = '.$nome.' WHERE id = '.$id.'"); 
elseif(!empty($squadra) 
mysql_query("UPDATE utenti SET squadra = '.$squadra.' WHERE id = '.$id.'"); 
elseif(!empty($campionato) 
mysql_query("UPDATE utenti SET campionato = '.$campionato.' WHERE id = '.$id.'"); 
elseif(!empty($email) 
mysql_query("UPDATE utenti SET email = '.$email.' WHERE id = '.$id.'"); 
elseif(!empty($privilegi) 
mysql_query("UPDATE utenti SET privilegi = '.$privilegi.' WHERE id = '.$id.'"); 

Con:

mysql_query("UPDATE utenti SET username = '".$username."' WHERE id = '".$id."'"); 
elseif(!empty($password) 
mysql_query("UPDATE utenti SET password = '".$password."' WHERE id = '".$id."'"); 
elseif(!empty($nome) 
mysql_query("UPDATE utenti SET nome = '".$nome."' WHERE id = '".$id."'"); 
elseif(!empty($squadra) 
mysql_query("UPDATE utenti SET squadra = '".$squadra."' WHERE id = '".$id."'"); 
elseif(!empty($campionato) 
mysql_query("UPDATE utenti SET campionato = '".$campionato."' WHERE id = '".$id."'"); 
elseif(!empty($email) 
mysql_query("UPDATE utenti SET email = '".$email."' WHERE id = '".$id."'"); 
elseif(!empty($privilegi) 
mysql_query("UPDATE utenti SET privilegi = '".$privilegi."' WHERE id = '".$id."'"); 

[binarysun]

Ma perché non li fai in una query sola?

Grazie miki, ma mi da ancora errore:

Parse error: syntax error, unexpected T_STRING in /membri/elettronicaqui/administrator/aggiorna_utente.php on line 15

Comunque non è una sola query perchè deve modificare solo le cose interessate... se nel form prima avevo messo un nuovo username lasciando vuoto tutti gli altri campi, questo mi modificava solo l'username.

Grazie.

<?php

include('include/database_user_connection.php');

$id = trim($_POST['id']);
$username = trim($_POST['username']);
$password = trim($_POST['password']);
$nome = trim($_POST['nome']);
$squadra = trim($_POST['squadra']);
$campionato = trim($_POST['campionato']);
$email = trim($_POST['email']);
$privilegi = trim($_POST['privilegi']);

if(!empty($username))
mysql_query("UPDATE utenti SET username = '$username' WHERE id = $id");
if(!empty($password))
mysql_query("UPDATE utenti SET password = '$password' WHERE id = $id");
if(!empty($nome))
mysql_query("UPDATE utenti SET nome = '$nome' WHERE id = $id");
if(!empty($squadra))
mysql_query("UPDATE utenti SET squadra = '$squadra' WHERE id = $id");
if(!empty($campionato))
mysql_query("UPDATE utenti SET campionato = '$campionato' WHERE id = $id");
if(!empty($email))
mysql_query("UPDATE utenti SET email = '$email' WHERE id = $id");
if(!empty($privilegi))
mysql_query("UPDATE utenti SET privilegi = '$privilegi' WHERE id = $id");
?>

Ciao!!

Funzionaaa Grazie mille Debug!!!

[binarysun]

Comunque non è una sola query perchè deve modificare solo le cose interessate... se nel form prima avevo messo un nuovo username lasciando vuoto tutti gli altri campi, questo mi modificava solo l'username.

Allora fai così

<?php

include('include/database_user_connection.php');

$id = trim($_POST['id']);
$username = trim($_POST['username']);
$password = trim($_POST['password']);
$nome = trim($_POST['nome']);
$squadra = trim($_POST['squadra']);
$campionato = trim($_POST['campionato']);
$email = trim($_POST['email']);
$privilegi = trim($_POST['privilegi']);

$SetString = "";
if(!empty($username))
     $SetString .= " username = '".$username."'";
if(!empty($password))
     $SetString .= " password = '".$password."'";
if(!empty($nome))
     $SetString .= " nome= '".$nome."'";
if(!empty($squadra))
     $SetString .= " squadra= '".$squadra."'";
if(!empty($campionato))
     $SetString .= " campionato = '".$campionato."'";
if(!empty($email))
     $SetString .= " email= '".$email."'";
if(!empty($privilegi))
     $SetString .= " privilegi= '".$privilegi."'";

if (!empty($SetString))
     mysql_query("UPDATE utenti SET $SetString WHERE id = $id");

?>

Ok grz

[fctobia]

ragazzi scusatemi, sono giorni che non riesco a capire che errore c'è in questa porzione di codice:

$q2 = "UPDATE infoteam SET user=$nlogin WHERE nometeam=$nt";
mysql_query($q2,$conn)or die('query fallita');

le variabili sono inizializzate correttamente, i nomi dei campi nel database sono esatti e non ci sono errori a causa degli apici...
cosa potrebbe essere allora???


EDIT:
dimenticavo di dirvi che la momento del caricamento della pagina in questione mi viene stampato la stringa "query fallita"

al posto di "query fallita" stampa la funzione mysql_error()

[miki92]

Questa volta qualche apice serve:

$q2 = "UPDATE infoteam SET user = '$nlogin' WHERE nometeam = '$nt'";
mysql_query($q2,$conn)or die('query fallita');