Javascript-форум (https://javascript.ru/forum/)
-   jQuery (https://javascript.ru/forum/jquery/)
-   -   неоконченные события jquery (https://javascript.ru/forum/jquery/45579-neokonchennye-sobytiya-jquery.html)

ecSpl01t 06.03.2014 17:41

неоконченные события jquery
 
Доброго времени суток.
Пишу комментарии с ответами, наткнулся на ошибку которую не понимаю и не знаю как решить. Ошибка заключается в дублировании функций либо событий либо неоконченного события.

Пример:
подгружается комментарий на страницу с помощью события on('click') в html есть кнопка 'ответить' но, она не срабатывает, я думал что при on() должно срабатывает. Далее я добавил после загрузки комментария на страницу функцию для того чтобы срабатывало события для нового комментария. При этом появилась проблема дублировании комментарий и событий, в ajax я это исправил чуть, добавил async:false при этом добавляется один комментарий, а остальные пишутся как ошибки.

Чтобы было Вам понятно мой рассказ я сделал тестовую страницу http://test.altme.org.ua/test_comm.php
попробуйте добавлять ответы на новых комментариях, потом на старых... короч попробуйте поюзать и поймете в чем дело, сам код js в исходном страницы можно увидеть.
Если много раз нажать на 'ответить' - то сколько раз Вы нажали сколько и отправится

p.s. смотрите консоль

Zuenf 06.03.2014 17:53

Вроде везде комменты добавляются нормально, только немного с анимацией проблемы.

ecSpl01t 06.03.2014 18:01

комментарии добавляются, но чуть не правильно, в анимации и дело, если смотреть на анимацию и открыть консоль видно сколько раз она запускается. Я сейчас уберу async:false, перейдите еще раз на страницу по нажимайте несколько раз по иконке ответить, и после чего напишите ответ и отправьте..

Zuenf 06.03.2014 18:13

А ну понятно, ты после получения ответа от сервера вешаешь еще один обработчик на все эти элементы. Убери строку после коммента об альтернативе, задай диву, в котором все комменты, класс и делай примерно так:
$(".тот_самый_класс").on('click', ".reply", mat_reply);

ecSpl01t 06.03.2014 18:54

не совсем понял, в success: вообще не должно быть $('.reply').on('click',mat_reply); ?
и такой тот самы класс ?

я просто не понимаю как работает on() читал куча инфы и так и не понял..

Zuenf 06.03.2014 19:31

Добавь класс(например .wrap-comments) диву, который содержит все комментарии(у которого маргин-топ: 40пкс).
И поменяй js примерно так:
$(document).ready(function() {
	var template_path = "http://test.altme.org.ua/altme/template/altme_1";

	$('.send_new_comment').on('click', function() {
			var message = $('.new_comment').val();
			var id_mat = $(this).attr('id_mat');
			$.ajax({
	            type: "POST",
	            url: template_path+"/function_php/all_function.php?page=material&type=send_new_commet",
	            dataType: "json",
	            async:false,
	            data: {
	            	message:message,
					id_mat:id_mat
	            },
	            success: function (data) {
	            	if(data['status']=='true'){
	            		$('.new_comment').val('');
						$('.event_other').removeClass('red').addClass('green').text('Успешно оставлен комментарий!').fadeIn().delay(3000).fadeOut();
	            		$('.new_comment_add').prepend(data['text'])
	            		.children()
	            		.css({backgroundColor:"rgba(255,237,116,0.2)"})
	            		.animate({backgroundColor:"rgba(255,255,255,0)"},4000)
	            		.fadeIn();
	            		$('.reply').on('click',mat_reply);
	            	}else{
	            		$('.event_other').removeClass('green').addClass('red').text(data['text']).fadeIn().delay(3000).fadeOut();
	            	}
	            }
	    	});
			return false;
		});	



	function mat_reply(){

				var reply = $(this);

				var slide = $(reply).parent().parent().parent().children('.add_commnet');

							$(slide).stop().toggleClass('open');

							if($(slide).hasClass('open')){
								$(slide).slideDown('fast');

								console.log("open");
							}else{
								$(slide).slideUp('fast');
								//event.preventDefault();
								//event.stopPropagation();
								//event.stopImmediatePropagation();
								//$(reply).off('click',mat_reply);
								console.log("close");
							}
						$(reply).parent().parent().parent()
						.children('.add_commnet')
						.children('.send_reply_comment')
						.on('click', function() {

							var message = $(this).parent().children('.reply_comment').val();
							var id_mat = $(reply).attr('id_mat');
							var id_mat_id = $(reply).attr('id_mat_id');

							$.ajax({
					            type: "POST",
					            url: template_path+"/function_php/all_function.php?page=material&type=send_reply_commet",
					           	dataType: "json",
					           	async:false,
					            data: {
					            	message:message,
									id_mat:id_mat,
									id_mat_id:id_mat_id
					            },
					     
					            success: function (data) {
					            		console.log("ajax done");
						            	if(data['status']=='true'){
						            		$('.reply_comment').val('');
											console.log('Успешно оставлен комментарий!');
						            		$('.reply_comment_add[id_mat_id = '+id_mat_id+']').append(data['text'])
						            		.children()
						            		.css({backgroundColor:"rgba(255,237,116,0.1)"})
						            		.animate({backgroundColor:"rgba(255,255,255,0)"},4000)
						            		.fadeIn();
						            		
						            		console.log("add: "+id_mat_id);

						 					if($(slide).attr('class').match(/open/)){
												$(slide).slideUp('fast');
											}

						 					var qw = $('.reply_comment_add[id_mat_id = '+id_mat_id+']').offset();
						 						qw['top']=qw['top']-220;

						            		$('html, body').animate({
											    scrollTop: qw.top
											}, 1000);
						            		//какая альтернатива ?
			//------------тут убрал строку-------------		            		

						            	}else{
						            		console.log(data['text']);
						            		console.log("ошибка");
						            	}
					            }
					    	});
					return false;
				});
				
	}
//---------И тут поправил---------------
	$('.wrap-comments').on('click', '.reply', mat_reply);
});

ecSpl01t 06.03.2014 20:14

добавил Ваш клас .wrap-comments, но при добавлении нового коммента/ответа, не работает событие

Zuenf 06.03.2014 20:44

Не туда класс добавил, либо поставь куда надо(перечитай что я писал), либо просто убери его и сделай так:
$('.matrial_comments').on('click', '.reply', mat_reply);

ecSpl01t 06.03.2014 21:25

сработала так $('#matrial_comments').on('click', '.reply', mat_reply);
можете объяснить как оно работает и еще , как убрать дублирование комментов.
При многократном нажимании на ответить, создаются много ответов

Zuenf 06.03.2014 21:57

Первый вопрос - это делегирование событий. Подробно как это реализуется и работает в нативном js - http://learn.javascript.ru/event-delegation.
А в jQuery это можно реализовать функцией .on(), если ей вторым параметром задать селектор отслеживаемого в родителе элемента при событии.
На счет второго - я посмотрю.


Часовой пояс GMT +3, время: 21:43.