2015-09-16 5 views
5

Я обновил свой сонар до версии 5.1.2, и на этом коде появляется вопрос Methods should not be empty:Ультразвуковые методы не должны быть пустыми в конструкторе

public User() {} 

public User (String login) { 
    this.login = login; 
} 

Это кажется ошибка. Я прав? Потому что этого раньше не было.

+0

Что версия Java плагин? что является ключевым правилом поднятого вопроса? – benzonico

+0

'Добавить вложенный комментарий, объясняющий, почему этот метод пуст, выведите исключение UnsupportedOperationException или завершите реализацию. Но это конструктор. Поэтому не требуется реализация. Потому что, если я это сделаю, я буду комментировать 'do nothing', что ничего не значит –

+0

Можете ли вы уточнить две информации, которые я задал? Зачем вам нужно явно объявлять пустой публичный конструктор без аргумента (если вы удалите этот конструктор, ваш код будет вести себя так же, как он будет объявлен неявно)? – benzonico

ответ

5

Я нашел эту джиру, это ошибка сонара.

https://jira.sonarsource.com/browse/SONARJAVA-1138

+1

Ну, после проверки, похоже, это не так. Собственно, эта проблема уже исправлена. Это может быть это исправление, в результате которого текущая проблема (re) появляется/регрессируется. Но ссылка выше - это * не * проблема, которую вы должны ждать, чтобы ее решить (поскольку она старая). –

0

Это действительно кажется, что неожиданное поведение. Я думаю, что это был ответ на билет выше того, что вызвало его. Я пытаюсь поговорить с разработчиком, чтобы установить правило S1186. Если вы хотите следовать:

https://jira.sonarsource.com/browse/SONARJAVA-1138

+0

, пожалуйста, обратитесь к [email protected], вместо того, чтобы повторять ответ на SO или комментировать закрытый билет. Благодарю. – benzonico