problema inserimento dati da form(REFRESH)

[E.d.i.73]

per l'idea che ti è venuta potresti controllare con una select se esiste nel database un record con $nome e $cognome.se non esiste fai la queri con L'INSERT, altrimenti esci.
esempio:

<?php
$query="SELECT nome,cognome FROM preventivi WHERE nome='$nome'AND cognome='$cognome'";
$res= mysql_query($query);
$num_rows=mysql_num_rows($res);
if($num_rows=="0")
 {

  $query="INSERT INTO preventivi(nome,cognome,email,password,scelta,commento,data,orario) VALUES('$nome','$cognome','$email','$password','$scelta','$commento','$data','$orario') " ;
  mysql_query($query);
 }
 else{

 }
?>

[ispuk]

ciao grazie per la risposta ,si comunque pensavo proprio a qualche cosa di questo tipo

adesso ho modificato il mio codice il problema è che non so dove sbaglio,volevo usare il tuo metodo in questa maniera ,inserendo anche uno switch

if($_POST['nome']&&$_POST['cognome']&&$_POST['email']&&$_POST['password']&&$_POST['scelta']&&$_POST['commento'] !='' )
{
$query="SELECT nome,cognome FROM preventivi WHERE nome='$nome'AND cognome='$cognome'";
$res= mysql_query($query);
$num_rows=mysql_num_rows($res);
switch($num_rows)
{
case "0":
 $query=mysql_query("INSERT INTO preventivi(nome,cognome,email,password,scelta,commento,data,orario) VALUES('$nome','$cognome','$email','$password','$scelta','$commento','$data','$orario') ") or die(mysql_error());
  echo '<a style="color:#ff0000;">Dati inseriti</a>';

break;

case "1":
echo' Dati già esistenti';
break;

}
else{echo'Devi riempire tutti  campi';} 

mi puoi aiutare? purtroppo sto sbagliando qualche cosa e causa grande stanchezza non trovo il filo della matassa....

[E.d.i.73]

Forse l'errore sta di non aver chiuso l'IF.Ho messo la graffa di chiusura prima dell'ELSE,vedi se ti va adesso:

<?php
if($_POST['nome']&&$_POST['cognome']&&$_POST['email']&&$_POST['password']&&$_POST['scelta']&&$_POST['commento'] !='' ) 
{ 
$query="SELECT nome,cognome FROM preventivi WHERE nome='$nome'AND cognome='$cognome'"; 
$res= mysql_query($query); 
$num_rows=mysql_num_rows($res); 
switch($num_rows) 
            { 
case "0": 
$query=mysql_query("INSERT INTO preventivi(nome,cognome,email,password,scelta,commento,data,orario) VALUES('$nome','$cognome','$email','$password','$scelta','$commento','$data','$orario') ") or die(mysql_error()); 
  echo '<a style="color:#ff0000;">Dati inseriti</a>'; 

break; 

case "1": 
echo' Dati già esistenti'; 
break; 

          } 
    }
else{echo'Devi riempire tutti  campi';}
?>

[Virus_101]

Premetto che non ho letto il codice.

Ma qui secondo me c'e' un errore di base.

Quando si sviluppano applicativi su web un'organizzazione strutturale MVC e' d'obbligo.

Eviti il problema dei recod duplicati a causa di refresh.

TI faccio un esempio di come dovresi disporri le pagine e gli script :

1) form html (view)
<invia i dati a>
2) script esterno (controller)
<controlla i dati, risponde (model)>
3) torno alla form html visualizzando la risposta dello script(view)


lo script non essendo incluso nella pagina viene invocato SE E SOLO SE viene cliccato il bottone o interrogato da altra risorsa.

Tale script legge i dati e li controlla.
Se i dati sono corretti li salva nel database e EFFETTUA REDIRECT alla form con codice di uscita che conferma il successo dell'operazione
Se i dati sono errati EFFETTUA REDIRECT alla form rispondendo il codice dell'errore

Come vedi se poi refreshi la pagina non interroghi lo script e non inserisci n volte gli stessi dati a causa di F5.

[ispuk]

grazie a tutti e due

il codice adesso funziona il problema era proprio quello che mi hai evidenziato

sono d'accordo anche che sarebbe meglio utilizzare pagine esterne con il redirect e non pagine incluse e infatti ora che il codice funziona tramuterò tutto come hai detto tu

grazie di cuore ragazzi a buon rendere

[Virus_101]

Np .

Buon codice