Как правильно в данной ситуации писать код?

Question

[Lynatik001]

Как можно обойтись без создания переменных, которые выполняют по сути одно только действие? Как по мне, так код выглядит стрёмно. если, например ,у меня ещё в этом файле будет ещё десяток подключений "сцен" из других файлов и переход на них мне опять придётся вот так каждый раз обявлять по 3-5 переменных.

const BS = new BotScene();
    const Botscene = new Stage([BotScene.mainMenu()]);
    controlPanelScene.use(Botscene.middleware());

Comments

[Ярослав Иванов]

Правильно не уменьшать кол-во переменных а улучшать читаемость кода.

Answer 1

[Stalker_RED]

Я вижу две переменные, причем BS вообще не используется, а Botscene используется один раз, и ее можно заинлайнить вот так вот

controlPanelScene.use(new Stage([BotScene.mainMenu()]).middleware());

^{Правда читабельность страдает, хотя она и так быда не очень :)}

Объект этот все равно находится в памяти все время, пока он используется. Переменная - только удобный способ достучаться до этого объекта. Если вам она не нужна - можете объявлять.

Comments

[Lynatik001]

const BS = new BotScene();
    const Botscene = new Stage([BS.mainMenu()]);
    controlPanelScene.use(Botscene.middleware());

использовался - вот так выглядел код. ет прост я начал думать тогда над оптимизацией + норм читаемом выгляде.

вот так сделал.

controlPanelScene.use(() => new Stage([BotScene.mainMenu()].middleware()));

Потом заглянул сюда и увидел аналог. кста твой пример не компилит, только с стрелочной функцией