Witam, irytuje mnie pewna rzecz na mojej stronce, zrobilem sobie ajaxowy system komentarzy i po kliknieciu przesyla ajaxem na taki kod:

[PHP] pobierz, plaintext 
//srawdz czy juz jest
$sql=mysql_query("select id from komentarze where komentarz='$komentarz' and data='$date'");
$numsql=mysql_num_rows($sql);
if($numsql==0)
{
//jesli nie ma to zapisuje
$sql=mysql_query("insert into komentarze (`id`,`komentarz`) values ('','$komentarz');
}
else exit;
 
[PHP] pobierz, plaintext 

mimo że sprawdza czy juz istnieje, czasami dzieje się tak, że za jednym razem dodaje 2 te same komentarze, nawet się nie zmienia czas o sekundę.
Czy można temu jakoś zapobiec?
Wydaje mi się że tu problem jest po stronie ajaxa a nie php, ale nawet jesli wysyla 2 razy to za pierwszym razem już istnieje

przedebuguj dobrze kod... zobacz czy wysylasz jednego ajaxa np firebugiem, zobacz czy nie masz jakies petli... itd o ciezko z gory okreslic co nie nie tak

Dlaczego więc nie dałeś formularza i kodu ajax'a?
Co do samego PHP pokaż więcej kodu.

kod php jest cały on ma tylko dodawać do bazy.
a ajax to jquery

[PHP] pobierz, plaintext 
$('#dodajkomentarz').live("click",function(){
$.ajax({
'url':'dodajkoment.php',
  'data': {"param": komentarz},
  'type':'POST',
 
'success': function(msg) {
$('#komentarze').html(msg);
},
'error' :function(){alert('erroe');}
})
});
[PHP] pobierz, plaintext 

co do php to tylko pobieram dane i ustawiam date na początku

[PHP] pobierz, plaintext 
$date=date('Y-m-d H:i:s');
$komentarz=$_POST['komentarz'];
[PHP] pobierz, plaintext 

Podepnij tam zamiast ajax'a alert(1) i zobacz ile razy się wykona.

Jak ktoś ma syndrom lewego przycisku myszy to po jakiejś serii wielokliku uda mu się w czas zanim twój skrypt się zdąży ogarnąć.( chociażby połączenie z bazą danych). Możesz zrobić w sumie naiwne, ale działające zabezpieczenie w postaci wyłączenie przycisku do momentu odpowiedzi z serwera. Czyli:

[JAVASCRIPT] pobierz, plaintext 
$('#dodajkomentarz').live("click",function(){
	if(!$(this).hasClass('clicked')){ //sprawdzasz czy przycisk ma klase clicked zabezpieczenie przed cisnieciem myszy
		$(this).addClass('clicked'); // jesli nie mial to mu ja dajesz
		var obj = $(this);
		$.ajax({
		url:'dodajkoment.php',
		data: {"param": komentarz},
		type:'POST',
 
		success: function(msg) {
		$('#komentarze').html(msg);
			obj.removeClass('clicked'); //po wykonaniu zapytania zdejmujesz klase i mozna ponownie klikac
		},
		error :function(){alert('erroe');}
		});
	}
});
 
[JAVASCRIPT] pobierz, plaintext 

Takie proste ostudzenie nadpobudliwych

no w sumie mogę to niby tak zabezpieczyć, tylko dziwi mnie to że dodają się 2 komenty a nawet sekundy róznicy nie ma, srawdzę jeszcze z tym alertem, ale mi się tak raczej nie zdarza, tylko czasami poprostu się dodają 2 nie cały czas.

nie no teraz to już jest przeginka, jedno kliknięcie w button:

[HTML] pobierz, plaintext 
 
$('button.target').live("click",function(){
$id=$(this).attr('id'),
$.ajax({
'url':'inc/pages.php',
'data': {"id": $id},
'type': 'POST',
'success': function(msg) {
$('.container').html(msg);
}
})
});
 
[HTML] pobierz, plaintext 

wyskakuje mi średnio od 20 do 30 razy w firebugu. ustawilem alert na success i 20 razy wyskoczyl mi alert, cala aplikacja dziala poprawnie, tylko ten jeden button nie, może być tego przyczyną że jest to już kolejne wywołanie ajaxowe? czyli

jest sobie strona główna
wywoluje następną ajaxem
i z tej wywołanej ajaxem chce jeszcze wywołać następną tym buttonem.

Edit: zauważyłem , że po odświeżeniu strony jak kliknę poraz pierwszy to idzie raz, jeśli kliknę w to ponownie to idzie już 3 razy
później coraz więcej i więcej

Słuchaj a może to się jakoś zapętliło? Może ładujesz tym ajaxem stronę, która w sobie też ma link do tego skryptu i on to zdarzenie przypisuje jeszcze raz itd...?
Przez co za każdym razem miałbyś coraz więcej wysyłań po jednym kliknięciu im dłużej byś klikał ;P

Zupełnie nie używam żadnych frameworków pod JS ale może spróbuj tak:

[JAVASCRIPT] pobierz, plaintext 
$('button.target').onclick=function(){
    $id=$(this).attr('id'),
    $.ajax({
       'url':'inc/pages.php',
       'data': {"id": $id},
       'type': 'POST',
       'success': function(msg) {
           $('.container').html(msg);
       }
    })
};
[JAVASCRIPT] pobierz, plaintext 

Kod powyżej zapewnić Ci powinien nadpisywanie eventu onclick, nie wiem czy metoda live czasem nie dodaje kolejnych nie usuwając starych (czyli działa na zasadzie addEventListener)

HTH

Poza tematem - na przyszłość zapoznaj się też z unikalnymi kluczami - jakbyś to tutaj zastosował na polu id i komentarz to zamiast tego całego kodu wystarczył by sam insert bo nie duplikowałby wartości, kwestie czy dodało do bazy czy nie wyciągasz z mysql_insert_id()

masz rację , metoda live dziala jak addEventListener, co kliknięcie miałem o 1 wywowałanie więcej, ale bylem pewny że bez live to mi w ogole nie bedzie działało, jednak wszystko śmiga jak należy, dzięki wielkie... pozdrawiam.