Комментарии 3
В старых инспекциях, после фикса они просто терялись, в новых — за этим стараются специально следить.
Сейчас мы, кстати, вышли на новый уровень фанатизма с комментариями. У нас гоняются рандомные проперти-тесты, где в различные джава-файлы вставляются комментарии в самые сумасшедшие места, потом применяются любые интеншны и квик-фиксы и проверяется, что все комментарии остались в коде. Многие старые инспекции уже починили на этот счёт. Сегодня мне свалилось, например, такое:
Source code:
if(fileType instanceof //simple end comment LanguageFileType) {
Warning: "ConditionfileType instanceof LanguageFileType
is always false"
Quick-fix: "SimplifyfileType instanceof LanguageFileType
to false"
Error: "Lost comment//simple end comment
"
Может никому и не придёт в голову вставлять сюда комментарий, но мы это починим :-)
+3
Такой Chaos Monkey, но не для микросервисов? :)
А где лежит этот тест, чтобы найти в исходниках Community Edition и заценить?
+1
А где лежит этот тест, чтобы найти в исходниках Community Edition и заценить?
JavaCodeInsightSanityTest#testPreserveComments и далее, там довольно развесистая система.
Такой Chaos Monkey, но не для микросервисов? :)
В некотором смысле да, даже названия классов намекают.
+1
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Что посмотреть на выходных? Обзор лучших докладов в свободном доступе. Часть вторая, JBreak 2017