Как помочь Java определиться с типами?

Question

[Сергей Семенко]

Имеется код:

listenForWhisper(socketManager.<MessagingUser>presenceChannel("dialog-" + dialog.getId()),
    WhisperEvent.EVENT_TYPING, TypingEvent.class);

Метод listenForWhisper:

public <T> SocketChannel listenForWhisper(SocketPresenceChannel<T> channel, String name,
                                          Class<? extends WhisperEvent<T>> event) {
        if (!channels.containsKey(channel.getName())) {
            channels.put(channel.getName(), channel);
        }
        return channel.listenForWhisper(name, event);
    }

Как можно заметить, мне приходится явно указывать тип данных, что выглядит довольно сомнительно. Писать я могу и так, мне не сложно, но что на счёт читабельности такого кода?

Или лучше до такого не доводить?

Comments

[Алексей]

А зачем вам дублирование информации о типе ивента? WhisperEvent.EVENT_TYPING, TypingEvent.class
Имя же можно хранить в TypingEvent. Или это внешняя библиотека?
Зачем вы заинлайнили создание SocketPresenceChannel? Так же менее понятно и можно запутаться.

Кроме того, какой именно тип данных вам приходится указывать явно? Event? MessagingUser? Если Event - можно упростить выкинув имя ивента которое вы выцепляете из отдельного класса, перенеся его внутрь каждого ивента.

[Толстый Лорри]

мне приходится явно указывать тип данных

Если речь про дженерики (MessagingUser), то как компилятор должен узнать необходимый вам тип? =) Автоматически типы выводятся только из аргументов, а здесь, очевидно, параметры заданы конкретным типом - String.

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

[Сергей Семенко]

Алексей:
1. Имя события может генерироваться динамически.
2.

Зачем вы заинлайнили создание SocketPresenceChannel,

Немного не понял. Предлагаете сделать вот так?

SocketPresenceChannel<MessagingUser> channel 
                = socketManager.presenceChannel("dialog-" + dialog.getId());
        listenForWhisper(channel, WhisperEvent.EVENT_TYPING, TypingEvent.class);

Но суть в том, чтобы сделать это в одну строку.
3. Нужно указать тип для WhisperEvent, в данном случае MessagingUser, структура которого выглядит примерно так:

public class WhisperEvent<T>
{
    private T data;

    ... Другие данные, общие для всех событий
    
    public T getData() {
        return data;
    }
    
    public WhisperEvent setData(T data) {
        this.data = data;
        return this;
    }
}

[Сергей Семенко]

Толстый Лорри:

то как компилятор должен узнать необходимый вам тип?

Надеялся, что он обратится за помощью к классу TypingEvent:

public class TypingEvent extends WhisperEvent<MessagingUser>
{
    ...
}