SQL query che non va..

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

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?

Risposte alla discussione

Pagine: 1 [2]

mr_faber

28/05/2009 12:56:20

prova cosi':

$MySql="UPDATE Commenti SET Commento='".$modifica."', Data= NOW() WHERE IDCommento='".$IDCommento."'";

Se non funziona prova a postare lo schema della tabella e l'errore che da SQL.

Homo Faber Fortunae Suae

dyrr

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

Dragon Lord
Conquista potenti draghi nel vasto universo di Dragon Lord, dove orchi, elfi, umani e draghi si scontrano in terre sconfinate!

zacharia

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:[Link ]

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?

mwnemesis

28/05/2009 13:42:24

Facendo quanto proposto da dyrr puoi postare cosa ottieni?

Elvenar
Da un piccolo villaggio a una potente città: nel gioco fantasy online Elvenar dovrai raccogliere risorse per potenziare e far crescere gli edifici...

mr_faber

28/05/2009 14:18:37

davvero molto strano che elimini un record con una query di update, c'e' qualcosa che non va, forse a monte

Homo Faber Fortunae Suae

zacharia

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?

mwnemesis

28/05/2009 14:57:00

zacharia ha scritto:
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>" );
}

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

Cuisine Royale
Un Battle Royale unico, nato come pesce d'aprile, in cui combattere contro altri 30 giocatori armati di utensili da cucina!

zacharia

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?

mwnemesis

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

eXtramondo
Esplora la più fedele e vasta TdM d'Italia. Segui l'avventura attraverso il gioco RPG in chat o con le numerose funzionalità automatiche che offre Em!

darkside of breakfast

28/05/2009 15:41:37

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.

Se darkside non va alla montagna...
Darkside va al mare.

[Link ]

Modificato il 28/05/2009 15:48:53 da darkside of breakfast

Pagine: 1 [2]

Discussione seguita da: » zacharia

Non puoi scrivere in questo forum