Каналы в чат сервере

[TidalPoo]

Заглянул тут недавно в чат сервер, и немного ох... Был ошарашен.
Меня тупо интересует вопрос, какой идиот придумал хардкодить статичные каналы в классе ChatChannels.java?
 

Спойлер

Автор конечно конченный идиот. Чат каналы (а точнее их названия) прилетают с клиента в пакете. Ну и зачем, скажите мне их харкодить?
Мама не учила создавать динамический справочник? Примерно вот так:
 

chatChannels.computeIfAbsent(channelIdentifier, k -> new ChatChannel())

Ещё и понаделали кучу бесполезных классов:

Цитата

TradeChannel
JobChannel
LangChannel

Это всё идентифицируется переменными от 3 до 6, достаточно одного класса с идентификатором
3 - Чат локации, 4 - Чат торговли, 5 - Чат поиска группы, 6 - Чат класса.

В итоге имеем лаконичный чат сервер.

Спойлер

 

Да и ещё, не используете lock в таких примерах:

 try {
     lock.lock();
     // do something
    } finally {
        lock.unlock();
    }

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

А говнокодеров, которые писали такой код, гнать нахер из программирования.