Можно ли использовать паттерн или как лучше решить задачу?

Question

[Senbonzakuraa]

Имеется socket.io server и класс Random

class Random {
    constructor(timer) {
        this.timer = timer
        this.status = 0
    }
    async setStatus() {
        const interval = setInterval(()=> {
            this.timer -= 1
            if(this.timer <= 0) {
                this.status = 1
            }
        }, 1000)
    }

}

Код Socket.io server

const Random = require('../Random').Random
const chanels = {}
const timers = [{timer: 20, name: "chanel1"},{timer: 400, "chanel2"}]
timers.map((timer,id) => chanels[timer.name] = new Random(timer.time))

socket.on('startTimer', async (name, callback) => {
    await chanels[name].setStatus()
})

Проблема:

Я хочу выполнять определенное событие на socket-сервере в том случае, если this.status = 1. Т.е мне нужно отслеживать смену this.status и оповещать об этом socket часть

Вопрос:

Каким образом это можно реализовать? Пытался сделать через setInterval на стороне сокета запрашивая каждую секундуthis.state, но на мой взгляд это крайне плохое решение. Наткнулся на паттерн observer, но возможна ли его реализация в моем случае?

Comments

[Евгений Самсонов]

Возможна

[Senbonzakuraa]

Евгений Самсонов, можно поподробнее? Не понимаю как это реализовывать на стороне сокета. Может подтолкнете в нужном направлении?

Answer 1

[Алексей]

Мб чем поможет

function Timer(t, ch) {
	this.time = t;
	this.channel = ch;
	
	this.interval = setInterval(() => {
		if (--this.time < 1) {
			clearInterval(this.interval);
			// здесь можно делать ченить с сокетом, он же полюбому глобальный
			// либо сделать свои ивенты через эммитер
			socket.to(this.channel).emit('TIMEOUT - STATUS ZERO');
		};
	}, 1000);
	
	this.getTimeleft = () => this.time;
	
	socket.to(this.channel).emit(`TIMER STARTS, YOU HAVE - ${this.time} SECONDS TO DO SOMETHING`);
};

var timer = new Timer(10, 'chanel2');
var timer2 = new Timer(3, 'chanel5');

Comments

[Senbonzakuraa]

ну вот как раз я и не хочу мешать класс с сокетами, все разделено

[Алексей]

Senbonzakuraa, ну убери тогда сокет оттуда и сделай дополнительный интервал

var timers = [ new Timer(10, 'chanel2'), new Timer(3, 'chanel5') ]
setInterval(() => {
	timers.forEach(timer => {
		if (timer.getTimeleft() < 1) {
			// ченить с сокетом
		};
	});
	// ну и здесь чистка от ненужных таймеров
}, 1000);

[Алексей]

Senbonzakuraa, я просто логику не понимаю, ты хочешь чтобы по истечению таймера сокет об этом знал, тоесть самое эффективное это добавить сокет в сам класс, но при этом не хочешь чтобы там был сокет.

[Senbonzakuraa]

Алексей, ну да, вот я и спрашиваю про Observer

[Алексей]

Senbonzakuraa, ну дак какая разница, хоть обсервер, хоть геттеры, хоть прокси - ты в любом случае будешь обращаться к сокету, если не задумано другого.

[Алексей]

Senbonzakuraa,
Оно бессмысленно спрашивать про паттерн, даже если тебе щас тут напишут как это сделать с использованием обсервера, что ты будешь делать с остальным кодом ? Так же здесь чтоле спрашивать чтобы и другое допиливали ? Просто почитай про любой паттерн и глядя на свой код подумай нужен ли он тут или нет, если на этот вопрос ты сможешь ответить, то и написать у тебя проблем не составит.