cancellare messaggi "vuoti"

[goda]

Mi scuso subito per la mia ignoranza :master:

Ho creato un semplice libro degli ospiti:
http://www.alspigul.net/guest.php
e adesso vorrei che eventuali messaggi "vuoti", cioè senza nessun dato inserito, non venissero mostrati ma bensì cancellati automaticamente.

È possibile e se sì qualcuno ha la pazienza di spiegarmi come?

Grazie mille

[Inoki]

[supersaibal]Originariamente inviato da goda
Mi scuso subito per la mia ignoranza :master:

Ho creato un semplice libro degli ospiti:
http://www.alspigul.net/guest.php
e adesso vorrei che eventuali messaggi "vuoti", cioè senza nessun dato inserito, non venissero mostrati ma bensì cancellati automaticamente.

È possibile e se sì qualcuno ha la pazienza di spiegarmi come?

Grazie mille [/supersaibal]

Bhe più che cancellare i messaggi vuoti direi che sarebbe più efficiente, non memorizzarli proprio...

In fase di INSERT del messaggio, basta controllare che la varibile che contiene il messaggio (es $messaggio) non sia vuota...

quindi:

if($messaggio!=""){
     mysql_query("INSERTO INTO ....");
}else{
     echo "Inserire testo nel messaggio";
} 

Inotrle ti consiglierei di controllare e validare anche il testo che passano gli utenti, cercando di eliminare i tag html, e quant'altro...

[goda]

Ci provo...

ti ringrazio tanto

[goda]

Ho di nuovo bisogno di aiuto...
Ho inserito lo script che mi hai consigliato:

<?php

if($messaggio!=""){
mysql_query("insert into messggi (nome, messaggio, data_ora, sito) values (\"$HTTP_POST_VARS[nome]\", \"$HTTP_POST_VARS[testo]\", now(), \"$HTTP_POST_VARS[sito]\")");
}else{
echo "Inserire testo nel messaggio";
}

?>

Adesso non vengono memorizzati neppure i messaggi "non vuoti", non riesco a capirne il motivo.

grazie

[Inoki]

[supersaibal]Originariamente inviato da goda
Ho di nuovo bisogno di aiuto...
Ho inserito lo script che mi hai consigliato:

<?php

if($messaggio!=""){
mysql_query("insert into messggi (nome, messaggio, data_ora, sito) values (\"$HTTP_POST_VARS[nome]\", \"$HTTP_POST_VARS[testo]\", now(), \"$HTTP_POST_VARS[sito]\")");
}else{
echo "Inserire testo nel messaggio";
}

?>

Adesso non vengono memorizzati neppure i messaggi "non vuoti", non riesco a capirne il motivo.

grazie [/supersaibal]

Ma questo è tutto il tuo script, o solo una parte?

Perchè se è tutto lo script, è logico che non funziona.. Alla variabile $messaggio devi associare il valore del campo passato dal form..

Inoltre con le ultime versioni di PHP gli array HTTP_GET_VARS, HTTP_POST_VARS ecc ecc, sono stati sostituiti dagli array $_GET, $_POST, $_SESSION...

Quindi se fai:

$messaggio=$_POST['messaggio']; (posto che il campo nel form si chiami messaggio, puoi fare il controllo di cui sopra..

In ogni caso, per evitare problemi, ti consiglio di non inserire nella query le varibili che ti passano direttamente..
Controllale e validale, devi dare la possibilità di non inviare tag html pericolosi, e via dicendo...

Un saluto

[goda]

No è solo una parte, lo script intero funziona.

Proverò a seguire i tuoi consigli

[eiyen]

<?php 

if($HTTP_POST_VARS[testo]!=""){ 
mysql_query("insert into messggi (nome, messaggio, data_ora, sito) values (\"$HTTP_POST_VARS[nome]\", \"$HTTP_POST_VARS[testo]\", now(), \"$HTTP_POST_VARS[sito]\")"); 
}else{ 
echo "Inserire testo nel messaggio"; 
} 

?>

[goda]

[supersaibal]Originariamente inviato da eiyen

<?php 

if($HTTP_POST_VARS[testo]!=""){ 
mysql_query("insert into messggi (nome, messaggio, data_ora, sito) values (\"$HTTP_POST_VARS[nome]\", \"$HTTP_POST_VARS[testo]\", now(), \"$HTTP_POST_VARS[sito]\")"); 
}else{ 
echo "Inserire testo nel messaggio"; 
} 

?>

[/supersaibal]

grazie mille!!!!!

[Inoki]

[supersaibal]Originariamente inviato da goda
grazie mille!!!!! [/supersaibal]

Che è poi quello che ti avevo detto anche io...

Aggiungo comunque, che HTTP_POST_VARS è cambiato, e se hai le utlime versioni di PHP, dovresti aggiornarti anche tu..

Inoltre scrivere HTTP_POST_VARS[var], è ulteriormente non corretto, essendo var il riferimento all' array, e quindi dovrebbe essere racchiuso tra apici così come indicato nel manuale:

Ad esempio, $arr[item] è meglio che sia scritto come $arr['item'] poichè il PHP tenta di trattare "item" come costante.

[eiyen]

noto un velo di irona...


comunque: sarebbe opportuno usare $_POST["var"].

La mia indicazione invece si basa sull'esempio fornito (copia-e-incolla della variabile già utilizzata nella query). A volerla dire tutta ci sarebbero anche altre cosucce che sarebbe meglio aggiungere (come la codifica dei caratteri speciali). Ma può essere sufficiente così...