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

Football Team Soccer ↗
Diventa una stella del calcio! Crea, allena ed equipaggia il tuo giocatore. Unisciti a un club e vinci in Football Team Game!

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?

Entropia Universe ↗
Lascia che il tuo avatar esplori nuovi mondi e viaggi tra i pianeti in questo stupendo MmoRpg Sci-Fi Free to Play!

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

World of Tanks ↗
Lanciati in epiche battaglie spalla a spalla con altri giocatori. Conquista la supremazia nel mondo dei Carri Armati!

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...

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

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.

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

tdl - staff

SCRIVI

28/05/2009 16:47:00

meno male che c'è dark! Possibile che non l'ha visto nessun altro?

» Whinare «
- Community Italiana su World of Warcraft -

www.tanadelladro.com
staff@tanadelladro.com

quota

zacharia

SCRIVI

28/05/2009 17:32:14

infatti il problema principale era proprio quello, nel frattempo ho risolto anche altri bug! grazie mille!