SQL query che non va..

SQL query che non va.. postato il 28/05/2009 12:47:31 nel forum programmazione, gdrcd, open source, hosting

salve a tutti, ho questa porzione di codice che non va, qualche idea?

$Login=$_SESSION['Login'];
$Master=$_SESSION['Master'];
$pg=$_POST['pg'];
$commento=$_POST['nuovo'];
$modifica=$_POST['modifica'];
$IDCommento=$_POST['ID'];
$cancella="no";
$cancella=$_POST['cancella'];
$nuovo=0;
$nuovo=$nuovo+$_POST['nuovoI'];

?>
<title>Commenti su <?= $pg ?></title>
</head>
<body style="background-image: url(img/schedapgqr7.jpg); color: black;">

<?
if($modifica!=""){
$MySql="UPDATE Table Commenti SET Commento='".$modifica."', Data= NOW() WHERE IDCommento='".$IDCommento."'";
$result=mysql_query($MySql) or die("Non modifico");
echo("Ho modificato");
}

Mi restituisce sempre Non modifico.

Se un essere umano lo ha creato, perché non dovrei essere in grado di capirlo?

quota

Pagine → 1

dyrr

SCRIVI

28/05/2009 12:58:12

Se metti un:
echo $MySql;

qui:

<?
if($modifica!=""){
$MySql="UPDATE Table Commenti SET Commento='".$modifica."', Data= NOW() WHERE IDCommento='".$IDCommento."'";

echo $MySql;

$result=mysql_query($MySql) or die("Non modifico");
echo("Ho modificato");
}

e poi incolli la query nel phpmyadmin, dovrebbe darti una descrizione più estesa del tipo di errore che genera la query.

Guardando cosi il codice non vedo nulla che possa far riferimento alla connessione al database.
Lo hai omesso per non postare il codice o manca proprio in quella pagina?

Perchè se manca proprio allora è logico che non ti vada la query anche se fosse scritta correttamente

CRSED: F.O.A.D. ↗
Brutale sparatutto MMO in cui vince l'ultimo giocatore che resta in vita nell'arena! Ogni giocatore sarà accompagnato da armi realistiche e magie!

quota

zacharia

SCRIVI

28/05/2009 13:04:28

la connessione è inclusa in un header che non ho incollato.

seguendo le indicazioni di mr_faber l'aggiornamento va a buon fine (non mi restituisce errore), però mi cancella l'intera riga dal db.

guardate un po' se ci capite qualcosa, il sorgente della pagina è questo:http://www.pastie.org/492508 ↗

nel database la tabella si chiama Commenti ed ha i seguenti campi:

Master varchar(20) latin1_general_ci No master
Personaggio varchar(20) latin1_general_ci No utente
Data datetime No
IDCommento int(255) No auto_increment
Commento varchar(255) latin1_general_ci No Commento

Se un essere umano lo ha creato, perché non dovrei essere in grado di capirlo?

quota

mwnemesis

SCRIVI

28/05/2009 13:42:24

Facendo quanto proposto da dyrr puoi postare cosa ottieni?

Exclusive Villa GdR ↗
Las Vegas, azzardo, vizi, soldi, intrighi... Più le sue luci sono scintillanti, più le sue ombre sono oscure. La città che non dorme mai, vi aspetta.

quota

zacharia

SCRIVI

28/05/2009 14:29:34

mwnemesis ha scritto: Facendo quanto proposto da dyrr puoi postare cosa ottieni?

UPDATE Commenti SET Commento='nuovoooooooooooooo, o forse no?', Data= NOW() WHERE IDCommento='2'Ho modificato

però subito dopo non mi visualizza più il commento e scompare dal database...

non vorrei che facesse casini qui:

else{
echo("<td>".htmlspecialchars(strip_tags($rs['Commento'],"<b><i><br>"))."</td><form action='commenti.php' method='post'><input type='hidden' name='ID' value='".$rs['IDCommento']."'><input type='hidden' value='cancella' name='cancella'><input type='submit' value='Cancella'></td> </form></tr>" );
}

Se un essere umano lo ha creato, perché non dovrei essere in grado di capirlo?

quota

mwnemesis

SCRIVI

28/05/2009 14:57:00

zacharia ha scritto:

mwnemesis ha scritto: Facendo quanto proposto da dyrr puoi postare cosa ottieni?

direi che è proprio qui il problema, vai ad impostare di default la variabile a cancella con il valore "cancella" questo fa sì che ogni volta oltre all'aggiornamento ti fa anche la cancellazione

Lineage II ↗
Entra in uno sconfinato mondo fantasy dominato da razze in contrasto tra loro. Scatena i tuoi poteri in uno dei Mmo più famosi al mondo!

quota

zacharia

SCRIVI

28/05/2009 15:00:05

sarebbe così se facessi il submit, ma se non faccio il submit non dovrebbe cancellare... inoltre ho notato che non cancella sempre, ma solo se faccio la modifica (l'else invece ci entra quando il commento non è del proprietario e quindi non è modificabile).

Se un essere umano lo ha creato, perché non dovrei essere in grado di capirlo?

quota

mwnemesis

SCRIVI

28/05/2009 15:04:53

Dovresti seguire completamente il percorso che fà la modifica, da qualche parte posta anche una cancellazione...dal codice che hai postato prima si capisce solo questo...

New World Order ↗
Diventa un agente del Nuovo Ordine Mondiale e affronta le forze occulte che minacciano il pianeta Terra in questo GDR Urban Fantasy per Mobile!

quota

darkside of breakfast

SCRIVI

28/05/2009 15:41:37 e modificato da darkside of breakfast il 28/05/2009 15:48:53

if($cancella="cancella"){

Da sostituire con

if($cancella == "cancella") {

oppure

if(isset($cancella)) {

--Modifico per chiarire:

if($cancella="cancella"){

Darà sempre true come risultato, perchè stai semplicemente impostando una stringa come contenuto di una variabile.
Quindi ti si ripeterà ogni volta :)
== invece è l'operatore matematico per indicare "è uguale a"
isset() è una funzione che verifica se la variabile è esistente.
Se la crei a prescindere, ma per essere davvero un comando cancellante deve essere riempita con il testo "cancella", allora basta usare empty(), che restituisce true se la variabile è vuota, oppure false se c'è del testo.
Ora ho notato che prima la imposti a "no" e poi, al massimo, la riscrivi prendendola da $_POST.
Perchè? XD
E' solo uno spreco di risorse, meglio usare i metodi sopracitati.