клиентском приложении сделать класс Logger.js синглтоном.
И появилась такая мысль:
В приложении есть также несколько классов App.js, EventsHandler.js (EventEmitter), ErrorHandler.js, ServerHandler.js (ну и Logger.js, о котором выше).
Все классы внутри App.js.
Экземпляры Logger.js, EventsHandler.js (EventEmitter) и ErrorHandler.js соответственно используются в разных классах.
Есть ли смысл вообще все классы приложения сейчас сделать синглтонами? Или это плохая практика, и надо делать только Logger.js, EventsHandler.js (EventEmitter) и ErrorHandler.js — потому что больше шансов впоследствии, что кто-то может попытаться создать ещё экземпляры (ну, вряд ли же кто-то App.js ещё раз попробует создать 🙂 )?
Все классы внутри App.js. - это плохо. Разбивай на файлы Синглтон - зачем синглтон? какой в этом для тебя профит, почему нельзя не синглтоном? какую он проблему решает что в нем возникла потребность?
Не, не, имеется в виду, что App.js — главный класс App, внутри него все остальные экземплятры в конструкторе инициализируются, а так классы в разных файлах. Нужна потребность в singleton'е, потому что надо, чтобы Logger.js писал из разных частей приложения (наверное, может чтобы race condition избежать, я толком сам плохо понимаю). А у остальных классов, наверное, нет такой необходимости, просто я подумал, что раз их экземпляры нужны только по 1 разу, может быть их тоже сделать синглтонами. Но вообще, мне говорят, что лучше так не надо делать
У нас в одном проекте все вот просто так, и не надо никаких танцев с бубнами class Singleton { } const singleton = new Singleton(); export default singleton;
Я понял, что ты имеешь в виду, да, спасибо, но вопрос был в том, надо ли что-то ещё кроме Logger.js делать в моём случае синглтонами. Я хотел сделать всё, мне говорят, что ничего кроме Logger.js не стоит делать синглтоном
Чтобы применять паттерн, сперва надо знать, какую проблему он решает
Ну, в отношении остальных классов, кроме Logger.js, он точно ничего не решает (во всяком случае на данном этапе разработки) Про Logger.js — я точно сам не знаю, но у меня есть такая задача. Я видел, как это реализовано в других приложениях — там стараются делать синглтонами логгеры
Обсуждают сегодня