File logs

[illitain]

Заготовка под логи

[nypi]

Посмотрел код, раз был запрос на ревью. Мне кажется, что нет явного повода форсить AutoCloseable у листнеров с точки зрения контракта. И закрытие листнеров внутри матча нарушает подход «открыли/закрыли ресурс на одном уровне». То есть, матч получает ссылку на уже открытый ресурс, но при этом его закрывает. Можно Closeable оставить только у MatchFileLogger (не форсить для всех реализаций интерфейса), а закрытие логгера перенести на уровень, где он создается

try (MatchFileLogger logger = new MatchFileLogger(folder)) {
listeners.add(logger);
Match match = new Match(…);
new MatchRunner(match, botSessions).run();
}

[illitain]

Точно, спасибо, классное замечание! Поправил.