Следует ли объявлять методы в интерфейсе Java с модификатором общего доступа или без него?

Я всегда пишу, что бы использовал, если бы не было интерфейса, и я писал прямую реализацию, то есть использовал бы public.

Модификатор public не следует использовать в интерфейсах Java (на мой взгляд).

Поскольку он не добавляет никакой дополнительной информации, он просто отвлекает внимание от важных вещей.

Большинство руководств по стилю рекомендуют вам не указывать его, но, конечно же, самое важное - обеспечить единообразие всей кодовой базы, особенно для каждого интерфейса. Следующий пример может запутать человека, не владеющего Java на 100%:

public interface Foo{
  public void MakeFoo();
  void PerformBar();
}

Я бы не стал ставить модификаторы, которые применяются по умолчанию. Как уже отмечалось, это может привести к непоследовательности и путанице.

Худшее, что я видел, - это интерфейс с методами, объявленными abstract ...

Это абсолютно субъективно. Я опускаю избыточный модификатор public, поскольку он кажется беспорядком. Как отмечали другие, последовательность - ключ к этому решению.

Интересно отметить, что разработчики языка C# решили усилить это. Объявление метода интерфейса общедоступным в C# на самом деле является ошибкой компиляции. Согласованность, вероятно, не важна для разных языков, поэтому я предполагаю, что это не имеет прямого отношения к Java.

Я использовал методы объявления с модификатором public, потому что он делает код более читабельным, особенно с подсветкой синтаксиса. Однако в нашем последнем проекте мы использовали Checkstyle, который показывает предупреждение с конфигурацией по умолчанию для модификаторов public в методах интерфейса, поэтому я переключился на их исключение.

Так что я не совсем уверен, что лучше, но мне очень не нравится использование public abstract в методах интерфейса. Eclipse иногда делает это при рефакторинге с помощью «Extract Interface».

JLS проясняет это:

It is permitted, but discouraged as a matter of style, to redundantly specify the public and/or abstract modifier for a method declared in an interface.

Я предпочитаю пропускать, где-то читал, что интерфейсы по умолчанию public и abstract.

К моему удивлению, книга - Шаблоны проектирования Head First, использует public с объявлением интерфейса и методами интерфейса ... что заставило меня еще раз переосмыслить, и я попал на этот пост.

В любом случае, я считаю, что избыточную информацию следует игнорировать.

Люди будут изучать ваш интерфейс на основе автозавершения кода в своей среде IDE или в Javadoc, а не при чтении исходного кода. Так что нет смысла помещать слово «общедоступно» в источник - никто его не читает.

Причина того, что методы в интерфейсах по умолчанию являются общедоступными и абстрактными, мне кажется вполне логичной и очевидной.

Метод в интерфейсе по умолчанию является абстрактным, чтобы заставить реализующий класс предоставить реализацию, и по умолчанию является общедоступным, поэтому реализующий класс имеет доступ для этого.

Добавление этих модификаторов в ваш код является избыточным и бесполезным и может привести только к выводу, что вам не хватает знаний и / или понимания основ Java.

Несмотря на то, что этот вопрос был задан давно, но я чувствую, что подробное описание проясняет, почему нет необходимости использовать общедоступные абстрактные перед методами и общедоступные статические финальные перед константами интерфейса.

Прежде всего, интерфейсы используются для определения общих методов для набора несвязанных классов, для которых каждый класс будет иметь уникальную реализацию. Следовательно, невозможно указать модификатор доступа как частный, поскольку он не может быть доступен другим классам для переопределения.

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

В-третьих, интерфейс не имеет реализации данных, включая переменные и методы экземпляра. Если есть логическая причина для вставки реализованных методов или переменных экземпляра в интерфейс, тогда это должен быть суперкласс в иерархии наследования, а не интерфейс. Учитывая этот факт, поскольку ни один метод не может быть реализован в интерфейсе, все методы в интерфейсе должны быть абстрактными.

В-четвертых, интерфейс может включать только константу в качестве членов данных, что означает, что они должны быть окончательными, и, конечно же, конечные константы объявляются как статические, чтобы сохранить только один их экземпляр. Следовательно, static final также необходим для констант интерфейса.

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

С введением модификаторов private, static, default для методов интерфейса в Java 8/9 ситуация усложняется, и я склонен думать, что полные объявления более читабельны (требуется Java 9 для компиляции):

public interface MyInterface {

    //minimal
    int CONST00 = 0;
    void method00();
    static void method01() {}
    default void method02() {}
    private static void method03() {}
    private void method04() {}

    //full
    public static final int CONST10 = 0;
    public abstract void method10();
    public static void method11() {}
    public default void method12() {}
    private static void method13() {}
    private void method14() {}

}

Я не согласен с популярным ответом, что общедоступность подразумевает, что есть другие варианты, и поэтому их там не должно быть. Дело в том, что теперь с Java 9 и далее ЕСТЬ другие варианты.

Я думаю, что вместо этого Java должна обеспечивать / требовать указания «public». Почему? Потому что отсутствие модификатора означает «пакетный» доступ повсюду, и наличие этого как особого случая приводит к путанице. Если вы просто допустили ошибку компиляции с четким сообщением (например, «Доступ к пакету не разрешен в интерфейсе»), мы бы избавились от очевидной двусмысленности, которую вводит возможность не указывать «общедоступный».

Обратите внимание на текущую формулировку: https://docs.oracle.com/javase/specs/jls/se9/html/jls-9.html#jls-9.4

"A method in the body of an interface may be declared public or private (§6.6). If no access modifier is given, the method is implicitly public. It is permitted, but discouraged as a matter of style, to redundantly specify the public modifier for a method declaration in an interface."

Смотрите, что теперь разрешено "частное" IS. Я думаю, что последнее предложение следовало убрать из JLS. К сожалению, «неявно общедоступное» поведение когда-либо было разрешено, поскольку теперь оно, вероятно, останется для обратной совместимости и приведет к путанице, что отсутствие модификатора доступа означает «общедоступный» в интерфейсах и «пакет» в других местах.