tagir_valeev Jan 24 2018 at 13:07

Пылесосим код IDEA Ultimate с помощью анализа потоков данных

5 min

JetBrains corporate blogProgramming*Java*

+41

Comments 15

artspb Jan 24 2018 at 16:47

А вы сообщили разработчикам? А патч отправили? Какую версию этого проекта вы проверяли? Master? Так надо 173! А вы проверяли проект IDEA Community? А что так МАЛО? А есть версия IntelliJ IDEA под Linux? Вы проверяли IDEA с помощью IDEA? Где статья о результатах проверки?

+10

tagir_valeev Jan 24 2018 at 17:31

Не распознают люди сарказм, вот и минусуют, прости их :-)

artspb Jan 24 2018 at 17:38

Я знаю, Тагир, я знаю :) И в профиль пользователя тоже не любят заглядывать.

-1

dougrinch Jan 24 2018 at 17:57

Распознают, и в профиль заглядывают, и даже видят там уже поставленный плюс в карму. Но конкретно этот коммент, что называется, «слишком толстый». И парсится уже не как шутка, а как глупое отвлечение от темы.

p.s. Сам пост отличный, спасибо!

tagir_valeev Jan 25 2018 at 10:25

Честно говоря, я больше опасался комментариев вроде «У… да ваша IDEA — глюкавище, как можно было такие баги допустить».

TheKnight Jan 25 2018 at 16:55

У… да ваша IDEA — глюкавище, как можно было такие баги допустить
Все, страшное случилось, дальше боятся смысла нет :)

Для фанатов Дюны

a__v__k Jan 24 2018 at 17:31

Есть ли возможность пометить, что именно проверяют свои функции-проверки, как то

boolean isNotEmpty(Object[] arr) { return arr != null && arr.length > 0; }

tagir_valeev Jan 24 2018 at 17:34

Есть контракты, но они довольно ограничены. Здесь можно написать @Contract("null -> false"), и это будет учитываться. А вот информация о длине массива во внешний метод не попадёт.

a__v__k Jan 24 2018 at 17:49

Спасибо. Попробую.

tagir_valeev Jan 24 2018 at 18:11

Если метод статический, скорее всего контракт сам выведется. Посмотрите по ctrl+q.

Hixon10 Jan 24 2018 at 22:29

Спасибо за интересный рассказ!

Один комментарий — очень плохо видно скриншоты кода. Понятно, что тут хочется показать реакцию IDEA, но факт — есть факт.

tagir_valeev Jan 25 2018 at 06:16

Да, со скриншотами не очень получилось. Шрифт явно не подходящий. В следующий раз учту :-)

ggo Jan 25 2018 at 09:45

IDE — хорошо, а как запустить инспекции в безлюдном режиме (в рамках CI) и получить отчет о найденных ошибках/предупреждениях? a la standalone static code analyzer

TheKnight Jan 25 2018 at 09:51

+1 к этому комментарию :) Желательно еще плагин, позволяющий автоматически постить результаты проверки в ПР на Github/Bitbucket/etc :)

tagir_valeev Jan 25 2018 at 10:23

https://www.jetbrains.com/help/idea/running-inspections-offline.html

А если у вас правильный CI, то всё будет работать из коробки :-)