Ложноположительная ошибка MS_PKGPROTECT в FindBugs

Question

Я уверен, что получаю ложный результат от FindBugs. Мой код выглядит следующим образом:

public class MyClass { 
    protected static String fileNamePrefix; 
}   

И это ошибка я получаю:

поля должно быть пакет защищен

Изменяемого статическое поле может быть изменена вредоносным кодом или авария. Поле может быть защищено пакетом, чтобы избежать этой уязвимости.

Bug вид и модель: MS - MS_PKGPROTECT

Является ли это на самом деле ложный положительный или я что-то не хватает?

Answer 1

Защищенный модификатор позволяет стороннему подклассу получить повышенный доступ к полю.

class MyEvilSubClass extends MyClass { 
    static { 
     fileNamePrefix = "../" + fileNamePrefix; 
    } 
} 

FindBugs указывает, что «защищенный» не защищает поле от открытого доступа в дикой природе. Если поле было защищено пакетом, мы знаем, что намерение состоит в том, чтобы разделить какой-то секрет среди доверенных друзей.

В целом, спрячьте поле и создайте методы доступа, поскольку он более гибкий с точки зрения инкапсуляции, безопасности потоков и ленивой загрузки.