2

Есть интерфейсы (много) такого типа

public interface OnProfileRequestListener {
     public void onResponse(Profile profile);
     public void onError(int errorType);
}

И такого типа

public interface OnDialogsRequestListener {
     public void onResponse(List<Dialog> dialogs);
     public void onError(int errorType);
}

Как мне лучше изменить их на wildcard'ы? Создать один общий и передавать в его тип как коллекции элементов так и сами элементы (OnResponseListener<Profile>, OnResponseListener<List<Dialog>>)

public interface OnResponseListener<T> {
     public void onResponse(T response);
     public void onError(int errorType);
}

Или оставить этот для элементов, а для коллекции другой создать? И, кстати, как такие интерфейсы называть? OnResponseListener?

2
  • 1
    1. По поводу выбора имен почитайте "Чистый код". Если кратко, то посторонний разработчик, смотря на имя класса, должен представлять его функционал. 2. Да, одного интерфейса на все более чем достаточно. Они же описывают конкретный функционал - классы их реализующие должны уметь onResponse и onError. И самому интерфейсу параллельно, какими именно response эти классы будут оперировать
    – rjhdby
    17 окт 2017 в 6:58
  • @rjhdby, спасибо. Ваш коммент имеет право стать ответом
    – user194625
    17 окт 2017 в 8:35

1 ответ 1

2

Про выбор имен процитирую Роберта Мартина (Чистый Код, глава 2)

Имя переменной, функции или класса должно отвечать на все главные вопросы. Оно должно сообщить, почему эта переменная (и т. д.) существует, что она делает и как используется. Если имя требует дополнительных комментариев, значит, оно не передает намерений программиста.

...

Имена классов и объектов должны представлять собой существительные и их комбинации: Customer, WikiPage, Account и AddressParser. Старайтесь не использовать в именах классов такие слова, как Manager, Processor, Data или Info. Имя класса не должно быть глаголом.

Резюмируя - имя должно быть таким, чтобы посторонний человек(разработчик) при взгляде на него мог в общих чертах понять, для чего конкретно (за что конкретно отвечает класс с именем DataManager?) предназначен этот класс.

По поводу интерфейсов.

В вашем случае все эти интерфейсы используются в реализации функций обратного вызова при обращении к одному конкретному REST API(если я правильно понял). В таком случае самым грамотным будет использовать единый, наиболее общий интерфейс, без разделения на списки и единичные объекты. Я бы назвал его, например, NeboMobiResponseListenerInterface. Не надо бояться длинных имен. Длинное понятное имя гораздо лучше, чем короткое и непонятное.

1
  • Благодарю! Чистый код скачал, погружаюсь в чтение :)
    – user194625
    17 окт 2017 в 9:18

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.