Maybe I am off, but I thought other classes already cannot inherit from it, because it has a private constructor.
-----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of Stefan Seelmann Sent: Monday, November 22, 2010 7:58 AM To: Apache Directory Developers List Subject: Re: Code review - constants in interface On Mon, Nov 22, 2010 at 2:50 PM, Guillaume Nodet <[email protected]> wrote: > AFAIK, the best pattern for that is to use a final with a private > constructor and declare public static final fields in it. > > public class Constants { > private Constants() { > // non-instantiable class > } > > public static final String TYPE = "type"; > } Sounds good. One additional improvement would be to declare the class as final, just to avoid that other classes inherit from it. Kind Regards, Stefan
