Почему не изменяется переменная?

Question

[Space]

Всем привет. Есть код, который блокирует выполнения действия, если не прошло 3сек.

Таймер так сказать.

var socketEmitLastCallTime = 0;
	var aqwsde = Date.now() - socketEmitLastCallTime ;
	if(aqwsde >= 3 * 1000) {
		socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )
	    socketEmitLastCallTime = Date.now();
   };
   alert(aqwsde);

Проблема в чем:
Вывожу переменную alert`ом aqwsde, а она выводит текущее время. Т.е. эта переменная socketEmitLastCallTime не сохраняется. Почему?

Comments

[Space]

ссылка на эту тема Как сделать таймер

Answer 1

[Юрий Лобанов]

В связи с уточнениями:
Вот у Вас функция, например при клике на кнопку:
jsfiddle.net/iiil/FnbJn
Вы каждый раз обнуляете переменную socketEmitLastCallTime
Соответственно функция по клику срабатывает каждый раз.
А вот тут пример jsfiddle.net/iiil/56n9s/1
Мы присваиваем ноль только при объявлении, а при клике не обнуляем. Соответственно, алерт вызывается только раз в 3 секунды.

Comments

[Space]

@iiil мне посути надо не алерт, а чтобы

//socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )

отрабатывала не чаще 3 сек. Т.е. получается ее надо писать так:

var socketEmitLastCallTime = 0;
$('a').click(function(){
    var aqwsde = Date.now() - socketEmitLastCallTime;
	    if(aqwsde >= 3 * 1000) {
	        socketEmitLastCallTime = Date.now();
              socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )
        };
});

?

[Space]

@iiil и мне надо убрать click функцию. Если я сделаю так:

var socketEmitLastCallTime = 0;
    var aqwsde = Date.now() - socketEmitLastCallTime;
	    if(aqwsde >= 3 * 1000) {
	        socketEmitLastCallTime = Date.now();
              socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )
        };

правильно будет?

[Юрий Лобанов]

да

[Юрий Лобанов]

@ruslite нет, второй вариант не правильный! Как Вы вызываете функцию socket.emit ?

[Space]

@iiil

$( "#messageForm" ).submit( function(e) { e.preventDefault();
	var nameVal = $( "#nameInput" ).val();
	var msg = $( "#messageInput" ).val();
	var canalVal = $( "#canalInput" ).val();
	var socketEmitLastCallTime = 0;
    var aqwsde = Date.now() - socketEmitLastCallTime;
	if(aqwsde >= 3 * 1000) {
		socketEmitLastCallTime = Date.now();
		socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )
	    $( "#messageInput" ).val('');
			// Ajax call for saving datas
		$.ajax({
			url: "./ajax/insertNewMessage.php",
			type: "POST",
			data: { name: nameVal, message: msg, canal: canalVal },
			success: function(data) {
				 
			}
			
		});
   };

});

Вписал в код свой, но никакой задержки нет. Что не так?

[Space]

@iiil вот выше код. Вызывается если приходят данные по сокету var socket = io.connect( 'localhost:8080' );

[Юрий Лобанов]

var socketEmitLastCallTime = 0;
$( "#messageForm" ).submit( function(e) { e.preventDefault();
	var nameVal = $( "#nameInput" ).val();
	var msg = $( "#messageInput" ).val();
	var canalVal = $( "#canalInput" ).val();
    var aqwsde = Date.now() - socketEmitLastCallTime;
	if(aqwsde >= 3 * 1000) {
		socketEmitLastCallTime = Date.now();
		socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )
	    $( "#messageInput" ).val('');
			// Ajax call for saving datas
		$.ajax({
			url: "./ajax/insertNewMessage.php",
			type: "POST",
			data: { name: nameVal, message: msg, canal: canalVal },
			success: function(data) {
			}
		});
   };
});

[Space]

А данные по сокету идут после отправки формы#messageForm

[Юрий Лобанов]

@ruslite я делаю вывод, что Вы не читаете мои ответы. Почему Вы по прежнему обнуляете переменную внутри функции. Ее объявление и обнуление надо вынести.

[Space]

@iiil все сделал. спасибо. Понял теперь, что надо выносить, в этом и была проблема. Спасибо еще раз.

[Юрий Лобанов]

@ruslite уверен, что Вы могли и сами разобраться. Просто по шагам надо идти и смотреть, что имеете на данном шагу. В частности, Вы бы заметили, что вот в этом месте перед if имеем переменную socketEmitLastCallTime=0, а значит условие всегда true.

[Space]

@iiil вот уже понял, конечно с вашей помощью. вы бы знали как я решал проблему целые сутки, хорошо человек помог. Суть была очистить поле. Я вроде чистил, я протупил, что есть скрытое поле. И вот 30 часов я не спал, а надо было добавить $( "#messageInput" ).val('');

[Space]

@iiil и можно я вам сразу здесь вопрос задам.

if(aqwsde >= 2 * 1000) {
		socketEmitLastCallTime = Date.now();
		socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )
	    $( "#messageInput" ).val('');
			// Ajax call for saving datas
		$.ajax({
			url: "./ajax/insertNewMessage.php",
			type: "POST",
			data: { name: nameVal, message: msg, canal: canalVal },
			success: function(data) {
			}
		});
   }else {
   alert('Нельзя отправлять сообщение чаще 2 секунд');
   }
});

Как видите я уведомляю алертом. Как здесь написать вместо алерта, чтобы потом прописать на странице самой формы вывод сообщения в html коде? Т.е. чтобы выводилось не алертом, а в самом сайте.

[Юрий Лобанов]

Используйте jquery html()
jquery-docs.ru/attributes/html
Примерно так:

$('span.alert').html('Нельзя отправлять сообщения чаще, чем раз в 2 секунды');

Соответственно, создайте пустой спан с классом alert

[Юрий Лобанов]

@ruslite пишите, если будут вопросы.

[Space]

@iiil спасибо, хорошо)

[Space]

@iiil спасибо за код, уже сделал, так просто) на php то я знаю как, теперь и в js)

[Юрий Лобанов]

@ruslite пожалуйста

[Space]

@iiil и уж здесь еще спрошу, какой функцией и как сделать, чтобы это сообщение исчезало через опр время?

[Юрий Лобанов]

@ruslite см. ответ
Если надо, чтобы исчезала медленно - пишите вопрос )

[Space]

@iiil хорошо спасибо)

[Space]

@iiil я как понимаю вот эту реализацию задержки на реализовывать на сервере? ведь в обычной консоле можно удалить задержку и все :( Или какие варианты?

[Юрий Лобанов]

@ruslite не совсем понял о чем Вы говорите.

[Space]

@iiil врт таймер который вы помогли мне сделать, реализован в обычном скрипте, войдя в консоль его можно изменить под себя. Ведь консоль не только для просмотра. И получается я этот таймер стираю и все. И все можно флудить. Таймер то сделан чтобы спам не расслылали. Как быть?

[Юрий Лобанов]

@ruslite ну в принципе можно и скрипты отключить )
Это к сожалению не ко мне вопрос, думаю тут надо смотреть в сторону того, как Вы реализуете чат свой. Вы попробуйте свой вопрос так и задать, что вот есть чат, сделан так с использованием вот этого. Как сделать, чтобы не отправляли сообщения часто. Думаю, помогут.

[Space]

Хорошо ) спрошу. а что значит отключить скрипт?

[Юрий Лобанов]

@ruslite ну браузеры позволяют js отключать на сайтах, я об этом.

[Space]

@iiil хорошо. Задам вопрос новый. Спасибо

Answer 2

[Юрий Лобанов]

$('span.alert').html('Нельзя отправлять сообщения чаще, чем раз в 2 секунды');
setTimeout(function(){
    $('span.alert').html('');
},1000);

Comments

[Space]

@iiil спасибо, простая функция, уже читал, но забыл) работает.

Answer 3

[Юрий Лобанов]

Почему не продолжаете в том вопросе, где Вам подсказали это решение?
Что это у Вас такое: alert(var socketEmitLastCallTime = 0;
Откройте консоль и исправьте сначала все ошибки. А лучше приложите как минимум ссылку на тот вопрос.

var socketEmitLastCallTime = 0;
    setInterval(function(){
        aqwsde = Date.now() - socketEmitLastCallTime;
        if (aqwsde >= 3000) {
            console.log(aqwsde);
            socketEmitLastCallTime = Date.now();
        }
    },100);

Вот, вставьте в консоль, убедитесь, все работает. Интервал настроен через 100 млс, однако в консоль пишется только интервалы в 3000 (чуть более), т.е. 3 секундные интервалы.

Comments

[Space]

извините, код с дублировал когда скидывал.

var socketEmitLastCallTime = 0;
	var aqwsde = Date.now() - socketEmitLastCallTime ;
	if(aqwsde >= 3 * 1000) {
		socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )
	    socketEmitLastCallTime = Date.now();
   };
alert(aqwsde );

[Space]

@iiil тот вопрос затих потому что. ссылка наверху прикрепил

[Space]

@iiil мне сетинтервал не интересен. Просто как переменную которая внутри условия if перенести так, чтобы var aqwsde = Date.now() - socketEmitLastCallTime ; отработало правильно - текущее время минуc, то время которое было в условие if

[Space]

var socketEmitLastCallTime = 0; чтобы эта стала равна изменению в блоке if. А почему то значение не изм. надеюсь я нормально выразился

[Юрий Лобанов]

@ruslite посмотрите мой следующий ответ.

[Юрий Лобанов]

@ruslite и не пишите ничего такого вроде "мне интервал не интересен", попробуйте для начала разобраться в том, что я написал, раз сами не сечете ) Интервал я использовал в качестве аналога повторяющегося вызова функции, как будто кто-то с частотой в 100 млс нажимает кнопку.

[Space]

@iiil хорошо)

Answer 4

[pomeo]

Ну в смысле, у вас же

var socketEmitLastCallTime = 0;
var aqwsde = Date.now() - socketEmitLastCallTime;
alert(aqwsde);

вы и получаете текущее время. Никакого таймера я здесь не вижу.

Comments

[Space]

if(aqwsde >= 3 * 1000) {
socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )
socketEmitLastCallTime = Date.now(); // переменная которая выше была равна 0
};

[Space]

@pomeo if(aqwsde >= 3 * 1000) {
socket.emit( 'message', { name: nameVal, message: msg, canal: canalVal } )
socketEmitLastCallTime = Date.now(); // переменная которая выше была равна 0
};

[pomeo]

@ruslite и что, вы вообще понимаете что у вас написано? =) Во первых это условие сработает всегда, можно даже if не писать. Во вторых socket.emit срабатывает явно медленнее чем всё остальное за условием, поэтому вы alert получаете раньше со значением текущего времени.
@iiil там написал как это нормально должно быть.

[Space]

@ruslite @pomeo да значение socketEmitLastCallTime в условие почему не переносится, в эту же переменную которая выше. Это var socketEmitLastCallTime...