 #внезапно #разное #мимими
Забавный глюк клиента: https://www.pichome.ru/images/2017/12/09/guWS96vq.gif NaN — Not a Number, специальное значение для результата математической операции, в ходе которой получилось "не число" или "не определено". Например, попытка разделить строку на число даст результат NaN. 9 декабря 2017
3 |
 |
Ластро
|
|
Styx, а как ты будешь следить за этими ответами? А тут ты автоматом блокируешь новые запросы :)
|
|
|
Ластро
|
|
Вот лямбда отвечающая на фанфиксе за ми-мишки...
https://pastebin.com/T0ZE5nRL |
|
|
Ластро
|
|
По моему ясно видно, что она никаких ответов не ждёт...
|
|
|
|
|
Ал Ластор
а как ты будешь следить за этими ответами? Замыкания же. она никаких ответов не ждёт... Строка 14 и сразу же 15. |
|
|
|
|
Вообще, проблема в строке 31. count_likes-1 может дать NaN, и там нет parseInt()
|
|
|
Ластро
|
|
>> Замыкания же.
И правда :( >> Строка 14 и сразу же 15. И верно :( проморгал. >> Вообще, проблема в строке 31. count_likes-1 может дать NaN, и там нет parseInt() А вот сюда можно звать ReFeRy! |
|
|
Ластро
|
|
Но это, в отличии от секундной заморозки, не решит проблему с двойным мимиком, и проблему с отрицательным мимиком.
|
|
|
|
|
Ал Ластор, не решит, да. Чтобы решить двойной/отрицательный мимими — надо блокировать последующие нажатия мимими, пока не пришёл ответ. Что-то типа такого: https://pastebin.com/btW9zyCg
|
|
|
|
|
А ещё лучший вариант — в ответе выдавать не только like/dislike, но и суммарное кол-во мимимишек, чтобы не клиент вычислял, а сразу знал, какое число туда подставлять. А like/dislike будет использоваться только для направления слайда. Хотя, при такой реализации, эти слайды вообще не нужны будут, потому что пока я мимимикаю — кто-то может отмимимикать, и получится, что слайдится вниз с 5 не на 6, а на 3, например :)
1 |
|
 |
|
|
Styx, давно хотел спросить: с помощью чего ты делаешь эти зачётные гифки?
1 |
|
|
|
|
|
|
Романтик, кстати, если понравится — маякни, я тебе инвайт пришлю, мне за это ништяки будут :)
|
|
|
|
|
Вот ссылка для регистрации (по моему инвайту) :)
https://my.cl.ly/r/2m2G200z0I0C0H2P |
|
|
|
|
Styx, попробую, спасибо :)
1 |
|
|
|
|
Great news... ***@***.** accepted your invitation! :)As soon as they create their 5th drop you’ll both get one month of CloudApp Pro for free. |
|
|
|
|
Styx, а эта тема платная?
|
|
|
|
|
Романтик, там есть бесплатный план: https://www.getcloudapp.com/pricing
1 |
|
 |
|
|
Добавил parseint, где надо, и поставил блокировку повторного вызова добавлением/снятием класса-флага. На локалке добиться NaN больше не получилось. На сайт выкачу вместе с заявками.
3 |
|
|
|
|
ReFeRy, супер, спасибо! :)
1 |
|
|
|
|
ReFeRy, только не забудьте, что у ajax-запроса может быть не только success, но и error :)
1 |
|
Включить тёмную тему
Популярное
VKontakte
WhatsApp
Telegram
Отключить рекламу
