You shouldn't be only worried about variable / method being static or non - static but about other things too.
I would categorize your actions as - READ & WRITE and here you are trying to READ a default scoped , final
& static
variable in an INSTANCE , private
method.
Concept of statics exits to logically group variables and methods so if your method has only that line and there isn't going to be anything else in that method, I would suggest to keep that grouping consistent and make either that variable an instance variable ( which doesn't make sense if variable is constant among all objects ) and change its scope to private
( if you don't wish variable to be available in same package classes ) OR mark that method as static.
Reading a final
& static
variable in an instance method is perfectly OK even though writing is questionable ( though final
can't be written to but in case variable is not final
) .
Making that variable an instance one is favored if that variable is not going to be accessed by class name somewhere else and then if its going to be class level constant , make it static and change method to be static ( Initializing same constant field in every object will unnecessary cost you memory ) .