Forum PHP Romania - Discutii despre PHP, MySQL, Javascript, AJAX, etc

[dragosm]

Am un formular de adaugare a unor date.Dupa ce utilizatorul apasa butonul "Adauga", informatiile sunt adaugate in baza de date.Totul ok pana aici, insa daca cineva da "Refresh" din click dreapta informatiile sunt dublate in baza de date !

[ottonel]

asa este, ai dreptate :D . aceeasi problema o am shi eu.
intrebare:
se poate ca atunci cind fac refresh pe o pagina de genul "index.php?action=insert" sa nu se ia in considerare actiunea de inserare? :?:

[dragosm]

am primit o solutie de la un prieten, dar nu imi pare viabila....el spune sa verific tot timpul baza de date sa vad daca nu se repete inregistrarile...dar eu am tabela cu zeci de mii de inregistrari si cu 10 campuri...nu pot sa fac verificarea asta la fiecare adaugare....

[DrStupid]

Pai are dreptate prietenul tau , sa faci o inteogare care sa verifice daca se dubleaza datele din baza de date ... daca nu sa bage informatiile in baza de date si daca da sa apare un mesaj de eroare ...

[ottonel]

eu am observat o chestie:
plec de la pagina add_new unde am o forma pe care o submitez si apelez aceeasi pagina cu request in coada add_new?action=insert . inregistrarea se face in tabela shi apoi afisez ce am in tabela. pagina curenta este add_new?action=insert - daca fac refresh (oricum buton,F5, s.a.) primesc mesajul ca fereastra nu poate fi refreshata decit daca retrimite informatziile, adica face iarasi submit la formul din add_new: rezulta o noua inregistrare in tabela cu aceleasi informatii.
poate te ajuta chestia asta. succes

[lupuleine]

solutia care o ofer este valabila numai in cazul in care metoda folosita este GET adica daca se pun la capatul fisierului stuff-uri de genu add.php?idx=2&name=orice o solutie usor de aplicat si care nu incruca pe nimeni ar fi


if ($REQUEST_URI!=$PATH_INFO)
{
Header("Location:".basename($PATH_INFO));
}


pe de alta parte daca oricare din campurile din baza de date este unic de felul lui nu prea cred ca se mai introduce inregistarea

mai bat capu sa gasesc o solutie cu post-urile

[biban]

uite cum va propun eu sa faceti:

presupunem ca aveti un formular cu un buton submit cu numele adauga
pagina pe care se afla butonul se numeste pagina.php
pagina.php?adauga=submit&alte_informatii_trimise_spre_adaugare va avea ca rezultat adaugarea de informatii in baza de date.

deci undeva in pagina.php vei avea un cod de genul
...
if($adauga)
{
adauga inregistrarile
}
.....
restul scriptului
.....

Va propun ca in if-ul de mai sus , inainte de inchiderea acoladei, sa faceti o redirectionare a browserului catre pagina.php, fara nici un alt parametru.
Astfel, cand se va da refresh se va da direct pagina.php, nu pagina.php?.....

[iugin]

E OK solutia lui Biban dar ideea simpla in chestia asta e sa faci redirect.

In unele cazuri ca sa mearga if($adauga) trebuie sa apesi efectiv pe buton, daca esti intrun camp text nu o ia in considerare(am patit).


Cel mai simplu elegant si corect dupa parerea mea e sa faci pagini care adaauga si pagini care afiseaza. La cele care adauga nu tiparesti nimic ssi la final dai un redirect undeva, altfel sunt baiuri

[lupuleine]

asha doar din pura curiozitate a incerc cineva sa puna chestia asta in pagina dupa ce se fac adugatile in baza de date ?!? numa asha sa vedeti ce efect are ?

//adaugi info in baza de date dupa care bagi linia de mai jos

if ($REQUEST_URI!=$PATH_INFO)
Header("Location:".basename($PATH_INFO));

[Zamolxe]

Cod:
//begin cod HTML
//formular aici unde se baga date
//Nume: ___________
//Prenume: _________
//Sex: _________
//Varsta: _________
[ADAUGA] //buton
//end cod HTML
//urmeaza validarea PHP
if($_POST['adauga']="ADAUGA"){
$nume=$_POST['nume'];
$prenume=$_POST['prenume'];
$sex=$_POST['sex'];
$varsta=$_POST['varsta'];
//inseram userul in baza de date
mysql_query("INSERT INTO useri (nume, prenume, sex, varsta) VALUES ('$nume', '$prenume', '$sex', '$varsta')") or die(mysql_error());
//redirectam userul pe pagina index.php unde avem GET['msg']=1 si afisam mesajul 'Datele au fost introduse cu succes'
header("Location: ./index.php?msg=1");
exit();
}


codul de mai sus este teoretic, si l-am scris pentru a demonstra ca exista metode simple pentru a scapa de problema REFRESHULUI. nu am inclus masurile de securitate, si de validare a continului (nu era nevoie in exemplu).

-------------
pentru a evita inregistrarile duble, in caz ca sa zicem nu avem incotro, ne folosim de id-uri sau de o cautare in prealabil in baza de date. sa zicem la codul de mai sus inainte de mysql_query() deci de query-ul propriu-zis de inserare a userului, bagam urmatoarul cod:
$validare=mysql_query("SELECT COUNT(*) FROM useri WHERE nume='$nume' AND prenume='$prenume'");
$validare2=mysql_num_rows($validare);
if($validare2=>1){ //daca mai mare sau egal cu 1 atunci
echo("utilizatorul exista deja in baza de date");
exit(); //codul de sub acest if nu se executa.
}

[luciansabo]

As dori sa fac cateva consideratii generale. Sunt cateva greseli de conceptie ale paginilor PHP pe care le-am observat din raspunsurile date.

In primul rand daca discutam de introducerea unor date intr-o baza de date nu folosim niciodata metoda GET pentru trimitere. Are exact acest efect la refresh. Vom folosi POST si vom prelua cu $_POST["var"]
apoi redirectam catre o pagina cu mesaj de confirmare eventual.

Citat: asha doar din pura curiozitate a incerc cineva sa puna chestia asta in pagina dupa ce se fac adugatile in baza de date ?!? numa asha sa vedeti ce efect are ?

//adaugi info in baza de date dupa care bagi linia de mai jos

if ($REQUEST_URI!=$PATH_INFO)
Header("Location:".basename($PATH_INFO));

Variabilele se preiau corect cu $_SERVER["REQUEST_URI"]... majoritatea configuratiilor sunt cu REGISTER_GLOBALS=OFF, unde scriptul nu va functiona din acest motiv.

PATH_INFO se umple cu ceva util doar la apelarea scriptului in forma:
http://localhost/script.php/cale/fictiva
altfel e un string gol.

Deci conditia de la if va fi intotdeauna evaluata adevarat daca apelezi http://localhost/script.php?id=5. E ca si cum ai spune if("/script.php"!="")
sau if(1!=0)

[Zamolxe]

propun o alta metoda care poate fi luata in considerare:
Cod:
<?php
if($_POST['adauga']=="Adauga"){
$nume=$_POST['nume'];
$prenume=$_POST['prenume'];
$*=$_POST['*'];
$varsta=$_POST['varsta'];
if(empty($nume) || etc...){
afisare_formular_html(); //functia de afisare a formularului html
exit();
}
$inserare=mysql_query("etc..."); //inseram userul
}
?>

functia afisare_formular_html() va contine un formular html a carui campuri vor contine valorile <?php echo($nume); ?> etc.. pentru a reafisa utilizatorului valorile introduse, dar si campurile pe care le mai are de completat.
Apelam exit() pentru a nu executa mysql_query() si a crea duplicate.

va rog sa ma corectati daca gresesc.

Serban

[luciansabo]

Este foarte corecta abordarea.

[aniflaviu]

o alta posibilitate de a scapa de dublarea integistrarilor la refresh, in functie de modul de construire al paginii web, este folosirea unui iframe la afisarea datelor


daca gresesc corectati-ma

[PHPRomania Bot]