Почему такое странное поведение у JS?

Question

[Михаил]

Здравствуйте. Постараюсь кратко описать проблему. Есть класс:

class RedisAPI {
    constructor(redis) {
        this.masterID = new Array(); //здесь хранятся уникальные ID всех воркеров
        this.master = redis.createClient();
    }

    addWorker(id) {
        this.masterID.push(id);
        console.log(`Worker ${id} is working!`);
    }

    //отправление сообщения на обработку одному из воркеру
    emit() {
        let needID = this.masterID.splice(0, 1);
        console.log(`arr: ${this.masterID.length}; first: ${needID[0]}`);
        this.masterID.push(needID[0]);
    }

    //отправление сообщения на обработку одному из воркеров каждые ms секунд
    emitTimeout(ms) {
        setInterval(() => {
            this.emit();
        }, ms);
    }
}

После двух асинхронных добавления данных в RedisAPI.masterID код начинает себя странно вести, а именно:

const queue = new RedisAPI(redis);
queue.emitTimeout(1000);

А в консоли вижу:
arr: 1; first: undefined
arr: 1; first: id1499261061279
arr: 1; first: undefined
arr: 1; first: id1499261061279

И так чередуется. То есть не получается грамотно удалить первый элемент масиива, хотя он всегда есть. В чём может быть проблема?

Answer 1

[Abdula Magomedov]

//отправление сообщения на обработку одному из воркеру
    emit() {
        let length = this.masterID.length;
        let needID = this.masterID.splice(0, 1);
        console.log(`arr: ${length}; first: ${needID[0]}`);
        this.masterID.push(needID[0]);
    }

Comments

[Михаил]

Этот код существенно не отличается от моего. Проблема осталась

[Abdula Magomedov]

Михаил:
проверь что там. и есть ли.
console.log(this.masterID);

Answer 2

[Kovalsky]

Это вот зачем всё?

let needID = this.masterID.splice(0, 1);
...
this.masterID.push(needID[0]);

Вы этим алгоритмом первый элемент ставите на последнее место. Если бы я не знал задачи, я бы вам сказал "зачем так сложно, делайте просто this.masterID.reverse()". К тому же вы нигде не проверяете пустая ли очередь, в любом случае делаете ваше тёмное дело.

С проверкой всё работает без фокусов

class RedisAPI {
    constructor() {
        this.masterID = new Array(); //здесь хранятся уникальные ID всех воркеров
    }

    addWorker(id) {
        this.masterID.push(id);
        console.log(`Worker ${id} is working!`);
    }

    //отправление сообщения на обработку одному из воркеру
    emit() {
        if (this.masterID.length) {
            let needID = this.masterID.splice(0, 1);
            console.log(`arr: ${this.masterID.length}; first: ${needID[0]}`);
            this.masterID.push(needID[0]);
        } else {
            console.log('The queue is empty!');
        }
    }

    //отправление сообщения на обработку одному из воркеров каждые ms секунд
    emitTimeout(ms) {
        setInterval(() => {
            this.emit();
        }, ms);
    }
}

var queue = new RedisAPI();

queue.emitTimeout(1000);

setTimeout(function() {
    queue.addWorker(42);
}, 1500);

Comments

[Михаил]

Это не всё, конечно. Я потом применяю этот id в других методах, а уже после в конец добавляю

[Михаил]

Что касается проверки, то аналогично. Я просто не включил её в пример

[Kovalsky]

Михаил: а зачем добавлять, так надо - циклическое выполнение воркеров?

[Михаил]

Никита Полевой: Вообще это не оговорено, да и сейчас не так важно. Я не понимаю, почему при удалении первого элемента из непустого массива я получаю undefined. Это первоочерёдная проблема

[Kovalsky]

Михаил: не-не-не, Девид Блейн, именно из-за этих опущенных проверок код ведёт себя так, потому что проверка на длину и сплайс пустого массива это разные вещи: пока у вас массив пустой, сплайс возвращает также пустой массив, а значит вы в конце алгоритма запихиваете в ваш пустой массив undefined. Надеюсь понятно объяснил: "После двух асинхронных добавления данных в RedisAPI.masterID код начинает себя странно вести" - соль именно в этом, т.к. пока массив с ID пуст, вы в коде пушите в него undefined.

[Stalker_RED]

И вообще, почему splice(0, 1), а не shift(), например?

[Kovalsky]

Stalker_RED: в рамках "вообще" можно также спросить зачем вообще все эти операции с массивами в каждой итерации, если есть возможность просто хранить индекс текущего воркера.