LivePersonInc/cacherjs

Function prototype has 188 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    Cacher.prototype = (function () {
        /**
         * Method for initialization
         * @param {Object} [options] - the configuration options for the instance
         * @param {Number} [options.max] - optional max items in cache

Function _evict has 70 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function _evict(kickoutClosestTTL) {
            var callback;
            var item;
            var cbRes;
            var timeoutRes;

Consider simplifying this complex logical expression.

Open

            if (!this.initialized) {
                options = options || {};

                this.cache = {};                                                                                                               // Objects cache
                this.length = 0;                                                                                                               // Amount of items in cache

Function initialize has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function initialize(options) {
            var that = this;
            var stop = false;
            var index = 0;

Consider simplifying this complex logical expression.

Open

                if (eviction && (this.cache[key].callback || "function" === typeof this.ontimeout || "function" === typeof this.onkickout) || this.max && this.length > this.max) {
                    _evict.call(this, this.max && this.length > this.max);
                }

Avoid deeply nested control flow statements.

Open

                            if (callback) {
                                cbRes = callback(key, item);
                            }

Avoid deeply nested control flow statements.

Open

                            if (this.ontimeout) {
                                timeoutRes = this.ontimeout(key, item);
                            }

Avoid deeply nested control flow statements.

Open

                        else if (!removed && kickoutClosestTTL) {
                            if (!kickOut) {
                                kickOut = {
                                    key: key,
                                    timeout: this.cache[key].timeout

Avoid deeply nested control flow statements.

Open

                            if (typeof cbRes === 'object') {
                                touch.call(this, key, cbRes.ttl || this.cache[key].ttl, cbRes.callback || callback);
                            } else if (typeof timeoutRes === 'object') {
                                touch.call(this, key, timeoutRes.ttl || this.cache[key].ttl, timeoutRes.callback || callback);
                            } else if (cbRes !== false && timeoutRes !== false) {

Identical blocks of code found in 2 locations. Consider refactoring.

Open

                            } else if (cbRes !== false && timeoutRes !== false) {
                                remove.call(this, key);
                                removed++;
                            } else if (!removed && kickoutClosestTTL) {
                                if (!kickOut) {

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

                        if (this.cache[key].timeout <= (new Date()).getTime()) {
                            item = this.cache[key].item;
                            callback = this.cache[key].callback;

                            if (callback) {

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                this.ttl = options.ttl && !isNaN(options.ttl) && 0 < options.ttl ? parseInt(options.ttl, 10) : 0;                              // Time to leave for items (this can be overidden for specific items using the set method - 0 for unlimited

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                this.max = options.max && !isNaN(options.max) && 0 < options.max ? parseInt(options.max, 10) : 0;                              // Maximum items in cache - 0 for unlimited

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                this.interval = options.interval && !isNaN(options.interval) && 0 < options.interval ? parseInt(options.interval, 10) : 1000;  // Interval for running the eviction loop

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    function Cacher(options) {
        // For forcing new keyword
        if (false === (this instanceof Cacher)) {
            return new Cacher(options);
        }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                            if (typeof cbRes === 'object') {
                                touch.call(this, key, cbRes.ttl || this.cache[key].ttl, cbRes.callback || callback);
                            } else if (typeof timeoutRes === 'object') {
                                touch.call(this, key, timeoutRes.ttl || this.cache[key].ttl, timeoutRes.callback || callback);
                            } else if (cbRes !== false && timeoutRes !== false) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                            } else if (typeof timeoutRes === 'object') {
                                touch.call(this, key, timeoutRes.ttl || this.cache[key].ttl, timeoutRes.callback || callback);
                            } else if (cbRes !== false && timeoutRes !== false) {
                                remove.call(this, key);
                                removed++;

• Read up
  Read up