1

Задача простая - создать всплывающее окно которое будет показываться на сайте один раз за сессию, похимичив с кодом вот что получилось - смотрю через хром, там в куки попадает - но окно всплывает каждый раз,почему?

    var cookie_value = 0

function get_cookie(name){
var cookie_name = name + "="

if (document.cookie.length > 0){
start_index = document.cookie.indexOf(cookie_name)

if (start_index != -1) {
start_index += cookie_name.length

end_index = document.cookie.indexOf(";", start_index)

if (end_index == -1){
end_index = document.cookie.length
}

cookie_value=unescape(document.cookie.substring(start_index, end_index))

}

}

if(cookie_value == 0){
document.cookie="pop_once = 1"
PopupMe()
}

}

function PopupMe(){
setTimeout('__PopupMe()',1000)
}

function __PopupMe(){
myleft=(screen.width)?(screen.width+500)/2:1;mytop=(screen.height)?(screen.height-200)/2:50;
settings='top=' + mytop + ',left=' + myleft +',width=250,height=150,location=no,directories=no,menubar=no,toolbar=no,status=no,scrollbars=no,resizable=yes,fullscreen=no'
PopupWin=window.open('http://ampheek.com/popped.htm','PopupWin', settings);
PopupWin.focus();
setTimeout('PopupWin.close()',7000);}
3
  • Потому что у вас ошибка. По смыслу все правильно, а искать ошибку в таком желания не возникает
    – Snow
    28 июл 2014 в 23:04
  • @donkey совсем плохой код? 29 июл 2014 в 5:07
  • @ГошкаТарасов, в коде нигде не вызывается get_cookie, соответственно, значение cookie_value не меняется и всегда 0
    – Grundy
    5 янв 2016 в 9:07

1 ответ 1

2

Видимо значение сравнивается с нулем (типа (string)document.cookie == (boolean)0 - false), а не со строкой.

Я бы вот так сделал вообще:

function getCookie(name) {
    var value = "; " + document.cookie;
    var parts = value.split("; " + name + "=");
    if (parts.length == 2) return parts.pop().split(";").shift();
}

if(!getCookie('pop_once')){
    document.cookie = 'pop_once=true';
    PopupMe();
}

А еще таймауты:

setTimeout(__PopupMe,1000);
1
  • на самом деле просто нигде getCookie не вызывался
    – Grundy
    5 янв 2016 в 9:00

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.