0

動作していない Cookie コードがありますが、何が問題なのかわかりません。このコードは別のスクリプトで動作していますが、スクリプトでは動作していません。

コード：

function get_cookie(Name) {
    var search = Name + "=";
    if (document.cookie.length > 0) {
        offset = document.cookie.indexOf(search)

        // cookie exist, update expiry date of the same cookie
        if (offset != -1) {
            cookie_info = document.cookie.split('=');
            cookie_info = cookie_info[0]+"=" + cookie_info[1];

            // add 30 days expiry
            var date = new Date();
            date.setTime(date.getTime()+(30*24*60*60*1000));
            var expires = "; expires="+date.toGMTString();

            cookie_info += expires + ";path=/";

            document.cookie = cookie_info;
       }
   }}

function loadCokie() {
    if (get_cookie('popup')=='') {
        alert("hello");
        document.cookie="popup=yes"
    }
}

事前にTHnx

4

1 に答える 1

1

単なる提案ですが、次のことができます。

if ( ! localStorage.getItem('visited')) {
    localStorage.setItem('visited', true);
    alert('Hello, new visitor !');
}

古いブラウザーをサポートする必要がある場合は、自動的に Cookie にフォールバックするMDNのポリフィルを追加します。

編集：

元のコードに関しては、関数を実行することはなく、Cookie が存在しない場合は空の文字列が返されないため、ロジックに欠陥があるため、少し変更しました:

function get_cookie(c_name) {
  var c_value = document.cookie;
  var c_start = c_value.indexOf(" " + c_name + "=");
  if (c_start == -1) {
    c_start = c_value.indexOf(c_name + "=");
  }
  if (c_start == -1) {
    c_value = null;
  }else{
    c_start = c_value.indexOf("=", c_start) + 1;
    var c_end = c_value.indexOf(";", c_start);
    if (c_end == -1) {
      c_end = c_value.length;
    }
    c_value = unescape(c_value.substring(c_start,c_end));
  }
  return c_value;
}

function set_cookie(c_name,value,exdays) {
  var exdate=new Date();
  exdate.setDate(exdate.getDate() + exdays);
  var c_value=escape(value) + ((exdays==null) ? "" : "; expires="+exdate.toUTCString());
  document.cookie=c_name + "=" + c_value;
}

function loadCokie() {
  if ( ! get_cookie('popup')) {
    alert("hello");
    set_cookie('popup', 'yes', 30);
  }
}

loadCokie();
于 2013-07-05T14:48:27.380 に答える