Убедитесь, что пружинный компонент не имеет состояния

Question

Мы столкнулись с проблемой многопоточности, когда разработчик ввел изменчивость в компонент Spring. Что-то вроде этого:

@Component //singleton 
public class MyComponent { 
... 
private String intermediateResults; 
public String businessMethod() { 
... fills in intermediateResults; 
} 

public String thisGetterShouldNotBeHere() { 
    return intermediateResults; 
} 
} 

, который привел к ошибке с многопоточностью - полевые intermediateResults обращались из разных потоков.

Есть ли способ предотвратить добавление состояния к Spring Singleton, например. каким-то статическим анализатором? Плагины SonarQube? Плагины Eclipse? Спасибо за предложения.

Answer 1

Google, MutabilityDetector кажется способным сделать именно то, что вам нужно:

Изменчивость детектор предназначен для анализа классов Java и сообщать о том неизменны экземпляры данного класса. Его можно использовать:

• В модульном тесте с утверждением типа assertImmutable (MyClass.class). Является ли ваш класс действительно неизменным? А что после этого изменения вы только что сделали?
• Как плагин FindBugs. Те классы, которые вы комментировали с помощью @Immutable, на самом деле? Во время выполнения. Требуется ли для вашего API обязательные объекты? Из командной строки. Вы хотите быстро запустить Mutability Detector на всей кодовой базе?

я все равно советую добавить четкий договор о том, что класс должен быть неизменен либо через javadoc или через @Immutable аннотацию на самом классе, чтобы позволить (чувственные) разработчикам сохранить реквизиты класса. (Если детектор Google не смог обнаружить определенные типы неизменяемости, например: Are String, Date really immutable?)

Answer 2

Вы могли бы реализовать свои собственные правила с любым статическим анализатором (как FindBugs, PMD и Checkstyle), чтобы проверить, что ваш класс:

• позволяет только конечные свойства
• Расширяет данный класс
• является окончательным
• Использует инъекции зависимостей конструктора

Однако, насколько я знаю, нет никакого инструмента specif для этого настроена из коробки.

В качестве альтернативы вы можете создать аннотацию @Immutable и выполнить там проверки.