Нормально ли расширять стандартный объект?

Question

[Dvorak]

Пишу телеграм-бота. У меня есть объект app, методы которого позволяют принимать информацию от пользователя и оправлять её. Код выглядит следующий образом:

app.on('message', msg => {
  const text = msg.text;
  switch(text) {
    case '/add':
      add(redis, mongo, message, app, users);
    break:

    case '/remove':
      add(redis, mongo, message, app, users);
    break:
  }
})

И так для каждой команды. Так как при вызове каждой функции у меня в большинстве случаев передаются одинаковые аргументы, то решил расширить объект app:

app.redis = redis;
app.mongo = mongo;

app.add = (){...};

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

Comments

[Lynn «Кофеман»]

По моему вполне нормально. Хотя в express для этого есть специальное место: app.locals

Answer 1

[Алексей Уколов]

Если нет, то как лучше поступить?

Лучше использовать какой-нибудь IoC-container, их для js довольно много. Это позволяет уменьшить связанность кода, а это всегда хорошо.