Forum PHP.pl

napisalem prosty skrypt dodawania komentarzy na stronke i mam pytanie jak zrobic zeby skrypt nie dodawal ciagle tygo samego po nacisnieciu odswieżania??

----
Posty będące duplikacją postów już zawartych w temacie, będą bez ostrzeżenia usuwane. Ma to zapobiedz tworzeniu się zbędnego śmietnika
moderator

Możesz albo zablokować możliwość dwóch postów tego samego usera na dany temat pod sobą, albo sprawdzać np. czy pierwsze (powiedzmy) 50 znaków poprzedniego posta, różnią się od tego właśnie dodawanego.

Ponowny zapis do bazy po odświeżeniu strony

w prosty sposób można to osiągnąć, mianowicie:
np.:

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['AddReply']) && !isset($_POST['Re'])) $class->AddReply();
?>
[PHP] pobierz, plaintext 

a w forumularzu:

[PHP] pobierz, plaintext 
<form action="" method="post">
<?php if(!empty($_POST)) echo '<input name="Re" type="hidden" value="1">'; ?>
<input name="AddReply" type="hidden" value="1">
...
</form>
[PHP] pobierz, plaintext 

teraz po odświeżeniu strony, 2 raz nie doda się wpis do bazy

Ten post edytował huntercs 27.03.2006, 11:25:08

a jezeli chcem zrobic to przy zapisie do pliku .txt??
Moze jest to banalne pytanie ale naprawde niewiem

to nie ma znaczenia, co robisz po submicie formularza, w skrypcie

no nie do konca tak jest. tu chodzi o taki refresh przez wcisniecie F5 lub ctrl+r, a nie ponowne wcisniecie ręcznie submita.
Taki refresh o którym mowa, wysyla jeszcze raz ten sam formularz, z tymi samymi polami i wartościami. Tu nic nie da, ze ty dodasz dodatkowe pole, gdyż ono nie zostanie przy refreshu uwzględnione

to chyba najlepszym sposobem jest po nacisnieci submita zrobic zapis i przekierowac na stronke na ktorej sa wyswielane komentarze

Ten post edytował skowron-line 28.03.2006, 11:13:59

Ja to robię jeszcze inaczej.
PRzy tworzeniu formularza, generują unikalny identyfikator, na podsawie chociazby timestampa i czegos tam jeszcze. Identyfikator ten dokladam do formularza, jako wartość pola ukrytego. Po wyslaniu formularza sprawdzam, czy dany identyfikator jest zapisany w sesji. jesli nie, zapisuję go do sesji a formularz akceptuję jako prawidlowy. jesli jednak ten identyfikator jest juz zapisany w sesji, znaczy ze jest to refresh i dane z formularza odrzucam.
Dzieki takiemu podejsciu, mogę pozostac dalej na stronie z formularzem, bez przekierowania. Jest to mi czasami potrzebne.

a do bazy danych to wymyslilem ze moge wyciagnac ostatni rekord z bazy i porownac go z wpisywanym i jesli bedzie identczny to odrzucic. to chyba nienajgorsze rozwiazanie

Ten post edytował skowron-line 28.03.2006, 20:53:48

tylko po co mieszać do tego bazę? dodaktowe zapytanie...
lepiej zrobić z COOKIE/SESSION
coś podobnego do tego co zaproponował nospor

A nie prościej zrobić np. dodanie commenta po czym header("Location: adres.php");

I wszystkie F5 z głowy ?

Ależ jakie dodatkowe zapytanie?
Wystarczy nałożyć kluch UNIQUE.

Kliknij w linka, którego dałem na górze wątku.
Ja uważam że to jest najlepsze rozwiązanie.

Nie angażujesz żadnej sesji, ciasteczek ani niczego.

Po prostu działa

Ale nalozysz klucz na wszystkie pola? a co jesli rekord moze sie powtorzyc poprzez jawne wpisanie? A co jesli nie mozna zalozyc unique na zadne pole, gdyz kazde z osobna moze sie powtarzac. Osobiscie nie podoba mi ten sposob zabezpieczenia przed refreshem. Nie bedziemy przeciez zmieniac struktury bazy tylko pop to by uniknać refresha. Do tego są inne rzeczy juz tu wymienione

Więc należy wybarć kombinacją pól, które powtarzać się nie powinny.
W 99% przypadków kombinacja taka istnieje.


Prezcież to nie jest zmiana struktury bazy. Przy jej projektowaniu powinno się założyć unikalność na pola, które tego wymagają.
A poza tym gdyby to do aplikacji klienckiej miało należeć kontrolowanie unikalności wprowadzanych danych, to po co w bazach byłby w ogóle ten mechanizm.

Ja uważam że najlepiej baza zadba sama o siebie, mało tego jest to sapoób właśnie do tego stworzony, i jeszcze mniej tego najwygodniejszy.
Nie powiesz mi ~nospor, że pisanie mechanizmów ciasteczek, zapisywanie informacji w sesji, ciągłe pilnowanie tego jest wygodniejsze niż blok try ... catch i wyrzucanie wyjątku z odpowienim kodem (1062 w tym przypadku).

dalej sie z tobą nie zgadzam. Istnieją przecież tabele, w których nie może byc pol unikatowcyh. nie dlatego ze baza na to nie pozwala, ale dlatego, ze zalozenia dla danej tabeli na to nie pozwalają. a konkretnie pozwalają na takie same rekordy. moze tak byc.

Obys sie nie zdziwil Zalezy czego uzywasz. Ja uzywam wlasnej klaski do formualrzy. Wiesz jak sa sprawdzam czy byl refresh czy nie? O tak:

[PHP] pobierz, plaintext 
<?php
if ($form->WasThisOperation()) echo 'To jest refresh';
?>
[PHP] pobierz, plaintext 

No i niepowiesz mi chyba, ze jest to nie wygodne

edit:
źle sie wyslowilem. chodzilo mi, ze nie bede zakladal uniqow jesli nie potrzebuje, tylko poto by refresha uniknąc

Ładnie, bardzo ładnie.

Ja to robię tak:

[PHP] pobierz, plaintext 
<?php
 
try
{
	$objModel->addQuestion(); // chwilowe uproszczenie
}
catch( SQLException $objException )
{
	switch( $objException->getCode() )
	{
		case 1062 : 
			$objRequest->setError( 'question', 'Takie pytanie już istnieje.' );
			break;
 
		default : 
			$objRequest->setError( 'error', 'Nie można dodać pytania.' );
			break;
	}
 
	return View::ERROR;
}
 
?>
[PHP] pobierz, plaintext 

Połączenie Agavi i Creole'a.

Jak mi wyskoczy wyjątek to mogę z aplikacją zrobić wszystko, zależy to od tego jaki.

Każdy ma swój najlepszy sposób

A że ja upierdliwy jestem...
A co w przypadku, gdy mamy formularz, ale jego wynik nie jest zapisywany do bazy, tylko wysylany mailem? Jest to tzw. formularz kontaktowy. Wtedy uniqi na bazie nam nic nie dadzą
Oczywiście można wówczas użyc header, lub sesje.

A jeszcze pytanko bo nie sprawdzalem tego nigdy. Zalozmy ze uzywamy header, zeby przekierowac po wyslaniu formularza. A czy jest mozliwa sytuacja, ze koles zanim zostanie przekierowany zdązy wcisnąć F5, czyli tym samym ponownie wysle ten sam formularz?

ja napisalem cos takiego i dziala

[PHP] pobierz, plaintext 
<?php
mysql_connect('localhost','root','');
mysql_select_db('skowron');
 
$wpis_id = $HTTP_POST_VARS['wpis_id'];
$podpis = $HTTP_POST_VARS['podpis'];
$wpis = $HTTP_POST_VARS['wpis'];
 
echo"<form action=f5.php method=post>
	podpis: <input type=text name=podpis size=20><br>
	wpis: <br><textarea rows=5 cols=10 name=wpis></textarea>
	<input type=submit name=zapisz value=zapisz>";
 
if(isset($zapisz))
	{	
		$zapytanie = 'SELECT wpis FROM wpisy ORDER BY wpis_id DESC LIMIT 1';
			if($a = mysql_query($zapytanie))
				{
					while($wiersz = mysql_fetch_array($a))
						{
							echo"<input type=hidden name=z_bazy value='".$wiersz['wpis']."'>";
						}
				}		
	echo"$z_bazy";
		if($wpis == $z_bazy)
			{
				echo"takie same";
			}
		else
			{
				$zapytanie = 'INSERT INTO wpisy values("","'.$podpis.'","'.$wpis.'")';
				if(!mysql_query($zapytanie))
					{
						echo"blad ".mysql_error()."";
					}
			}
	}
?>
[PHP] pobierz, plaintext 

prosze o lekka krytyke jesli komus sie nie bedzie podobac

ale o ile dobrze wczytuje sie w ten kod, to ty zrobiles to, co zaproponowal wczesniej huntercs (i to nie dokladnie tak). Jak juz pisalem wczesniej to nie zadzial przy refreshu.

W jaki sposob ty to testowales?
Bo to wyglada tak:
jesli wcisniesz zapisz, to dogenerowujesz do formularza hiddena, z ostatnim wpisem z bazy, ale ten wpis nie jest tym, co aktualnie dodajesz. Potem sprawdzasz, czy wpis z bazy z hiddena rowny jest aktualnemu. To bedzie rowne dopiero przy ktoryms tam razie.
A wsytuacji, gdy wygenerujesz formularz pierwszy raz, wscisniesz zapisz a potem dasz refresh, to ci nie zadziala. No chyba ze mamy co innego na mysli mówiąc refresh