Использование Checkstyle для проверки правильности получения / установки доменного объекта - PullRequest
1 голос
/ 13 октября 2011

У меня проблема или я искал правило для контрольного стиля, которое проверит следующее.Это тривиальная проверка, но было бы полезно правило, когда кто-то вручную меняет имя get / set.

Я хочу иметь правило, которое будет проверять get / sets и выводить сообщения об ошибках при обнаружении чего-либо подобногов коде.

Пример: исходный атрибут был описанием.Но разработчик должен изменить его на shortDescription, но испортит рефактор.

private String description;

public String getDescription() {
    return description;
}

public void setShortDescription(String description) {
   this.description = description;
}

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

Ответы [ 2 ]

2 голосов
/ 14 октября 2011

Хотя это не охватывается Checkstyle, если PMD может быть опцией, существует тест BeanMembersShouldSerialize , который выдает жалобу, если есть какие-либо нестатические и непереходные поля, которые не имеют методов получения и установки послеСоглашения об именах Java.

0 голосов
/ 13 октября 2011

* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * 1003

В вашем примере, какую ошибку вы ожидаете сообщить? Отсутствует сеттер для свойства description? Отсутствует свойство для setShortDescription сеттера?

Как checkstyle должен знать свойства, для которых вы хотите иметь геттеры / сеттеры? Я предполагаю, что вы не хотите иметь методы получения и для всех ваших личных переменных.

...