Задача решена -но возник вопрос по другому поводу :-)

Вопрос по задаче

Решен

Всем привет... Задачу решил-но попутно возник еще один вопрос, пока не забыл :-) Итак... В этой задаче мы указываем путь до метода в статическом классе - из метода main. С этим понятно:

Movie movie = MovieFactory.getMovie("cartoon")

Причем, если не указать путь- то IDE предложит импортировать этот метод в наш код вот в таком виде:

import static com.javarush.task.task14.task1414.Solution.MovieFactory.getMovie;

С этим тоже понятно. Непонятно другое: какой способ предпочтительнее в коде: прописывать подробный путь до метода в статическом классе или импортировать его? Что лучше и в каких случаях?

Комментарии (5)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Стас ПасинковSoftware Developer в Zipy

Master

1 февраля 2018, 23:48

решение

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

Айдар Загретдинов

Уровень 20

Expert

2 февраля 2018, 07:59

насчет одинаковой сигнатуры- доходчиво!

fanar6

Уровень 41

2 февраля 2018, 09:22

Не соглашусь. Нагромождение букв в коде делает его менее читаемым. Если Вы пользуетесь любой IDE (Eclips, IDEA, NetBeans), то определить откуда что дергается не составляет труда - достаточно зажать ctrl и кликнуть левой кнопкой мыши - вас автоматически перекинет на нужный метод... А кто такие "многие", которые считают? Можно ссыль - почитать для саморазвития - может изменю мнение?

Стас ПасинковSoftware Developer в Zipy

Master

2 февраля 2018, 15:18

многие - это те люди, с которыми у меня был разговор на тему статических импортов) про Ctrl+клик - вы правы) но допустим у меня в классе есть метод max(...) и я делаю статический импорт класса Math а потом где-то в коде вызываю метод max(...) из него. поскольку не указано Math.max(...) - я буду думать, что идет обращение к методу моего же класса. и если логика этих методов слегка отличается - то можно довольно надолго зависнуть не понимая почему возвращается одно, а должно возвращаться немного другое) конечно, пример притянут за уши, но реально может быть ситуация подобная просто не с такими тривиальными названиями методов. единственный вариант когда статический импорт реально оправдан (как мне кажется) - это импорт энумов. тут уж ошибиться или недопонять - тяжело)) во всех остальных случаях стоит подумать: а правильно ли поймет такой код другой человек? а заходить в каждый подозрительный метод чтобы проверить, что вызывается действительно именно тот метод, о котором ты думаешь - будет слишком накладно по времени :) короткий код - это, конечно, хорошо. но фанатично все сокращать не всегда оправдано :) я предпочитаю, пусть будет название переменной/метода/класса подлиннее (даже из 5-7 слов), но по названию будет четко понятно что это и зачем) так что я больше сторонник спрингового подхода)) с его названиями классов типа GenericInterfaceDrivenDependencyInjectionAspect или даже HasThisTypePatternTriedToSneakInSomeGenericOrParameterizedTypePatternMatchingStuffAnywhereVisitor :D

Lenar Khabibullin

Уровень 23

1 февраля 2018, 21:18

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