Почему не правильно работает проверка времени жизни?

Question

[v-grabko]

window.Cache = {
    Get: function (name, call, time) {
        var st = this.Storage();
        var data = st.getItem(name);
        if (data != undefined) {
            data = JSON.parse(data);
            var times = parseInt(new Date().getTime() / 1000) - data.time;
            console.info(times);
            if (times >= time) { // не срабатывает
                return data.data;
            } else {
                st.removeItem(name);
                var gg = call();
                st.setItem(name, JSON.stringify({
                    data: JSON.stringify(gg),
                    time: parseInt(new Date().getTime() / 1000)
                }));
                return gg;
            }
        } else {
            var gg = call();
            st.setItem(name, JSON.stringify({
                data: JSON.stringify(gg),
                time: parseInt(new Date().getTime() / 1000)
            }));
            return gg;
        }
    },
    Set: function (name, value) {
        var st = this.Storage();
        st.setItem(name, JSON.stringify({
            data: JSON.stringify(value),
            time: parseInt(new Date().getTime() / 1000)
        }));
    },
    Del: function (name) {
        var st = this.Storage();
        st.removeItem(name);
    },
    Clear: function () {
        var st = this.Storage();
        st.clear();
    },
    /**
     * Создаёт обьект localStorage. Если произошла ошибка емулирует localStorage. 
     * (все данные хранит в озу, пропадают при перегрузке страницы). 
     */
    Storage: function () {
        try {
            return window['localStorage'];

        } catch (e) {
            return {
                storage: {},
                clear: function () {
                    delete this.storage;
                    this.storage = {};
                },
                getItem: function (key) {
                    return this.storage[key];
                },
                removeItem: function (key) {
                    return delete this.storage[key];
                },
                setItem: function (key, value) {
                    this.storage[key] = value;
                }
            };
        }
    }

};

Почему то всегда срабатывает замыкание.
А console.info(times);
возвращает 3, 6 , 100 и т.д.

Answer 1

[v-grabko]

window.Cache = {
    Get: function (name, call, time) {
        var st = this.Storage();
        var data = st.getItem(name);
        if (data != undefined) {
            data = JSON.parse(data);
            var times = parseInt(new Date().getTime() / 1000) - data.time;
            console.info(times);
            console.info(time);
            if (times <= time) { 
                console.info('cache');
                return data.data;
            } else {
                console.info('STcache');
                st.removeItem(name);
                var gg = call();
                st.setItem(name, JSON.stringify({
                    data: JSON.stringify(gg),
                    time: parseInt(new Date().getTime() / 1000)
                }));
                return gg;
            }
        } else {
            console.info('NO cache');
            var gg = call();
            st.setItem(name, JSON.stringify({
                data: JSON.stringify(gg),
                time: parseInt(new Date().getTime() / 1000)
            }));
            return gg;
        }
    },
    Set: function (name, value) {
        var st = this.Storage();
        st.setItem(name, JSON.stringify({
            data: JSON.stringify(value),
            time: parseInt(new Date().getTime() / 1000)
        }));
    },
    Del: function (name) {
        var st = this.Storage();
        st.removeItem(name);
    },
    Clear: function () {
        var st = this.Storage();
        st.clear();
    },
    /**
     * Создаёт обьект localStorage. Если произошла ошибка емулирует localStorage. 
     * (все данные хранит в озу, пропадают при перегрузке страницы). 
     */
    Storage: function () {
        try {
            return window['localStorage'];

        } catch (e) {
            return {
                storage: {},
                clear: function () {
                    delete this.storage;
                    this.storage = {};
                },
                getItem: function (key) {
                    return this.storage[key];
                },
                removeItem: function (key) {
                    return delete this.storage[key];
                },
                setItem: function (key, value) {
                    this.storage[key] = value;
                }
            };
        }
    }

};

Answer 2

[Александр Аксентьев]

Данные-то какие передаете?

Логика вообще не ясна, как вы используете аргумент time в вызове Get.
Кажется что это должно быть время жизни, но у вас получается наоборот. В коде если время жизни больше чем переданное время жизни, то возвращаем из кеша значение - бесконечное время жизни.

ps.
в целом идея какая-то ужасная.
Почему GET УСТАНАВЛИВАЕТ значения которые получены из колбека какого-то.

Comments

[v-grabko]

https://habrahabr.ru/post/145317/

[v-grabko]

Во время установки кеша я пишу время его установки. Потом я узнаю сколько времени прошло после установки. Если время после установки меньше чем переданное time то возвращаю из кеша.

[Александр Аксентьев]

v-grabko: пример в студию на jsfiddle. В цифрах какие передаете данные, текстом и так из кода понятно.

[v-grabko]

Александр Аксентьев: https://jsfiddle.net/o2dq07sv/

[Александр Аксентьев]

v-grabko: условие наоборот должно быть. Сейчас если время после установки больше чем переданное то из кеша берется...