Правила     Закладки     Карма    Календарь    Журналы    Помощь    Поиск    PDA    Чат   
        СМС-ки
   
Пейджер выключен!
 
Фильтр авторов:    показать 
  скрыть
  Ответ в темуСоздание новой темыСоздание опроса

> Не записывает cookie, Не записывает cookie
webkirpich  
 ۩  [x]    Дата
Цитировать сообщение

Пользователя сейчас нет на форуме



Новичок
*

Профиль
Группа: Пользователь
Сообщений: 3
Пользователь №: 43300
На форуме: 3 месяца, 12 дней
Карма:




Привет друзья, есть проблема не могу понять почему не работает cookie?

<script type="text/javascript">
$(document).ready(function () {
if (document.cookie.indexOf("ModalShown=true")<0) {
setTimeout(function() {
function posMouse(e){
var mouX = 0, mouY = 0;
if (!e) e = window.event;
if (e.pageX || e.pageY) {
mouX = e.pageX;
mouY = e.pageY;
} else if (e.clientX || e.clientY) {
mouX = e.clientX + (document.documentElement.scrollLeft || document.body.scrollLeft) - document.documentElement.clientLeft;
mouY = e.clientY + (document.documentElement.scrollTop || document.body.scrollTop) - document.documentElement.clientTop;
}
return {"x":mouX, "y":mouY}
}


$(document).ready(function(){
$('#exitposition').mouseover(function(){

$("#ExitModal").modal("show");
$("#myModalClose").click(function () {
$("#ExitModal").modal("hide");
});
});
});
}, 10000);
expireDate = new Date
expireDate.setDate(expireDate.getDate()+30)
document.cookie = "ModalShown=true; expires=" + expireDate.toGMTString();" path=/";
}
}
);
</
script>
PMПисьмо на e-mail пользователю
    0   Для быстрого поиска похожих сообщений выделите 1-2 слова в тексте и нажмите сюда Для быстрой цитаты из этого сообщения выделите текст и нажмите сюда
killer8080  
[x] Дата
Цитировать сообщение

Пользователя сейчас нет на форуме



Здесь живет
******

Профиль
Группа: Эксперт
Группа переписки
Сообщений: 8737
Пользователь №: 26630
На форуме: 5 лет, 9 месяцев, 5 дней
Карма: 591




было бы странно, если бы эта каша работала user posted image
если нормально отформатировать код, всё станет очевидно
http://jsbeautifier.org/
$(document).ready(function() {
if (document.cookie.indexOf("ModalShown=true") < 0) {
setTimeout(function() {
function posMouse(e) {
var mouX = 0,
mouY = 0;
if (!e) e = window.event;
if (e.pageX || e.pageY) {
mouX = e.pageX;
mouY = e.pageY;
} else if (e.clientX || e.clientY) {
mouX = e.clientX + (document.documentElement.scrollLeft || document.body.scrollLeft) - document.documentElement.clientLeft;
mouY = e.clientY + (document.documentElement.scrollTop || document.body.scrollTop) - document.documentElement.clientTop;
}
return {
"x": mouX,
"y": mouY
}
}


$(document).ready(function() {
$('#exitposition').mouseover(function() {

$("#ExitModal").modal("show");
$("#myModalClose").click(function() {
$("#ExitModal").modal("hide");
});
});
});
}, 10000);
expireDate = new Date
expireDate.setDate(expireDate.getDate() + 30)
document.cookie = "ModalShown=true; expires=" + expireDate.toGMTString();
" path=/";
}
}
);
PMПисьмо на e-mail пользователю
    0   Для быстрого поиска похожих сообщений выделите 1-2 слова в тексте и нажмите сюда Для быстрой цитаты из этого сообщения выделите текст и нажмите сюда
webkirpich  
 ۩  [x] Дата
Цитировать сообщение

Пользователя сейчас нет на форуме



Новичок
*

Профиль
Группа: Пользователь
Сообщений: 3
Пользователь №: 43300
На форуме: 3 месяца, 12 дней
Карма:




Цитата (killer8080 @ 25.08.2016 - 20:31)
было бы странно, если бы эта каша работала user posted image
если нормально отформатировать код, всё станет очевидно
http://jsbeautifier.org/
$(document).ready(function() {
if (document.cookie.indexOf("ModalShown=true") < 0) {
setTimeout(function() {
function posMouse(e) {
var mouX = 0,
mouY = 0;
if (!e) e = window.event;
if (e.pageX || e.pageY) {
mouX = e.pageX;
mouY = e.pageY;
} else if (e.clientX || e.clientY) {
mouX = e.clientX + (document.documentElement.scrollLeft || document.body.scrollLeft) - document.documentElement.clientLeft;
mouY = e.clientY + (document.documentElement.scrollTop || document.body.scrollTop) - document.documentElement.clientTop;
}
return {
"x": mouX,
"y": mouY
}
}


$(document).ready(function() {
$('#exitposition').mouseover(function() {

$("#ExitModal").modal("show");
$("#myModalClose").click(function() {
$("#ExitModal").modal("hide");
});
});
});
}, 10000);
expireDate = new Date
expireDate.setDate(expireDate.getDate() + 30)
document.cookie = "ModalShown=true; expires=" + expireDate.toGMTString();
" path=/";
}
}
);

Спасибо, впредь учту. Но всё равно не понимаю где ошибка (((
PMПисьмо на e-mail пользователю
    0   Для быстрого поиска похожих сообщений выделите 1-2 слова в тексте и нажмите сюда Для быстрой цитаты из этого сообщения выделите текст и нажмите сюда
killer8080  
Дата
Цитировать сообщение

Пользователя сейчас нет на форуме



Здесь живет
******

Профиль
Группа: Эксперт
Группа переписки
Сообщений: 8737
Пользователь №: 26630
На форуме: 5 лет, 9 месяцев, 5 дней
Карма: 591




Цитата (webkirpich @ 25.08.2016 - 21:24)
Но всё равно не понимаю где ошибка (((

да их тут целая пачка, читай код в порядке выполнения
1. $(document).ready(function() { навешиваем обработчик на готовность DOM

2. if (document.cookie.indexOf("ModalShown=true") < 0) { если кука не установлена...

2.1 setTimeout(function() { запускается таймер на 10 секунд

2.1.1 function posMouse(e) {внутри замыкания зачем то объявлена функция, которая в коде не используется. Для чего она внутри замыкания?

2.1.2 $(document).ready(function() {
$('#exitposition').mouseover(function() {
снова в jQuery добавляется обработчик на готовность DOM. Для чего? DOM уже загружен, его код не выполнится никогда!

2.2 expireDate = new Date
expireDate.setDate(expireDate.getDate() + 30)
тут почти всё нормально, за исключением оного но, для чего переменная expireDate объявляется глобально? Судя по коду, она не имеет смысла за пределами этой функции. Не обдуманное объявление глобальных переменных - зло, чреватое коллизиями имён, и как следствие не предсказуемыми багами!

2.3 document.cookie = "ModalShown=true; expires=" + expireDate.toGMTString();
" path=/";
ну тут ошибку ты и сам должен видеть wink.gif
PMПисьмо на e-mail пользователю
    0   Для быстрого поиска похожих сообщений выделите 1-2 слова в тексте и нажмите сюда Для быстрой цитаты из этого сообщения выделите текст и нажмите сюда
  Быстрый ответ
Информация о Госте
Введите Ваше имя
Кнопки кодов
Для вставки цитаты, выделите нужный текст и
НАЖМИТЕ СЮДА
Введите сообщение
Смайлики
:huh:  :o  ;) 
:P  :D  :lol: 
B)  :rolleyes:  <_< 
:)  :angry:  :( 
:unsure:  :blink:  :ph34r: 
     
Показать всё

Опции сообщения  Включить смайлики?
 Включить подпись?
 
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:

Опции темы Ответ в темуСоздание новой темыСоздание опроса