come dire se il record inserito non è corretto allora non copiarlo nel database

[ShAdOw WaLkEr]

ho elaborato questo codice che in pratica mi prende i dati inseriti da una form e li inserisce in un database dopo aver verificato se l'indirizzo e-mail è valido richiamando un altro script che controlla la validità della mail, vorrei ora fare un controllo sugli altri campi dicendo che se uno qualsiasi dei campi è vuoto, o non valido allora il record non deve essere inserito nel db, mentre col codice che ho gia mi dice semplicemente record inserito con successo, o indirizzo e-mail non valido ma nonostante questo inserisce comunque il record nel db grazie in anticipo

<?php
// CONNESSIONE AL DATABASE
include('connect.php');
// RECUPERA DATI DAL FORM
$nome = $_POST['nome'];
$cognome = $_POST['cognome'];
$indirizzo = $_POST['indirizzo'];
$email = $_POST['email'];
include('chkmail.php');
$telefono = $_POST['telefono'];
// INSERISCI NUOVO ARTICOLI IN DATABASE
$query = "INSERT INTO `datiform` ( `nome` , `cognome` , `indirizzo` , `email` , `telefono`)
VALUES ( '$nome', '$cognome', '$indirizzo', '$email', '$telefono')";
if (mysql_query($query)) {
echo "<div align=center class=testo> <b>Record inserito con successo.</b></div>";
} else {
echo "<div align=center class=testo> <b>Errore di inserimento record: ". mysql_error()
."</b></div>";
}
?>

[ShAdOw WaLkEr]

up

[satifal]

Magari con un IF ???

[L_C]

Uno modo molto semplice è quello di validare ogni campo con una if prima di eseguire la query d'inserimento esempio:
if (nome != "" AND cognome != "" AND indirizzo != "" AND email != "" AND telefono !="")
{
eseguire query
}
else
{
non esegui query
}

[WottaFacca]

Per evitare i campi vuoti ti basta inserire in ogni <input> la voce "required"

 <input type="text" name="nome" required>

In questo modo, se l'utente preme invio senza aver inserito dei dati, non viene inviato niente e compare una "finestra" con scritto di riempire il campo.

[ShAdOw WaLkEr]

ho ottenuto questo codice che in pratica prende i dati da un form li pubblica in un db se sono corretti e poi prendendo un codice da un'altra tabella codici lo visuallizza nel messaggio quando vengono inviati i dati, ora vorrei fare in modo che il codice preso da quella tabella possa essere preso solo una volta, ho aggiunto alla tabella codici il campo stato e ora vorrei fare che se il codice viene stampato stato diventa=1 e a quel punto dire che se stato=1 il codice non lo prendo, mi dite come potrei fare?

// RECUPERA DATI DAL FORM
$nome = $_POST['nome'];
$cognome = $_POST['cognome'];
$indirizzo = $_POST['indirizzo'];
$email = $_POST['email'];
$telefono = $_POST['telefono'];

if (($nome=="") || ($cognome=="") || ($indirizzo=="") || ($email=="") || ($telefono==""))  {echo "non c'è scritto nulla";}
else if(!preg_match( '/^[\w\.\-]+@\w+[\w\.\-]*?\.\w{1,4}$/', $email)) {
    echo "email non valida";
}
else {
    // CONNESSIONE AL DATABASE
    include('connect.php');
    // INSERISCI NUOVO ARTICOLI IN DATABASE
    $query = "INSERT INTO `datiform` ( `nome` , `cognome` , `indirizzo` , `email` , `telefono`)
    VALUES ( '$nome', '$cognome', '$indirizzo', '$email', '$telefono')";
    if (mysql_query($query)) {
        echo "Record inserito con successo";
        $query = "SELECT codice FROM codici ORDER BY RAND() limit 1";
        $result = mysql_query($query);
        while ($riga = mysql_fetch_object($result)){
            echo $riga-> codice;
        }
    } else {
        echo "Errore di inserimento record: ". mysql_error();
    }
}

[MySQL]

Direi che potrebbe essere sensibilmente migliorato.
Iniziamo col dire che isset è buono e giusto; poi suggerisco di usare real_escape_string o anche strip_tags.
Per pulire ancora meglio puoi predisporti una tua funzione di "sistemazione" delle stringhe, per limitare ad esempio la lunghezza massima.

[ShAdOw WaLkEr]

Direi che potrebbe essere sensibilmente migliorato.
Iniziamo col dire che isset è buono e giusto; poi suggerisco di usare real_escape_string o anche strip_tags.
Per pulire ancora meglio puoi predisporti una tua funzione di "sistemazione" delle stringhe, per limitare ad esempio la lunghezza massima.

grazie della risposta, ma non hai risposto alla mia domanda, come li dico che quando un codice viene preso alla riga corrispondente il campo stato diventa in sostanza stato= stato+1 io avevo pensato a una cosa simile

mysql_query("UPDATE codici SET stato = (stato + 1) WHERE codice = '$result' ");

ma non va, e allo stesso modo come faccio l'if dicendo che se stato=0 allora lo prendo altrimento no?

[ShAdOw WaLkEr]

up

[satifal]

up