¿Deberían preferirse las anotaciones en jar305.jar sobre las anotaciones similares en anotación.jar para FindBugs?

Question

En la distribución FindBugs, annotations.jar es no un subconjunto de jsr305.jar. Sin embargo, varias anotaciones parecen estar duplicadas (ya sea exactamente o muy de cerca). ¿Debo preferir una anotación en jsr305.jar si tengo otra opción?

Tenga en cuenta que no estoy solo interesado en saber que sería "mejor" utilizar las anotaciones de jsr305.jar simplemente porque representan un estándar. Más bien, quiero saber si la herramienta FindBugs realizará el mismo (o mejor) análisis si prefiero la versión jsr305.jar de una anotación particular. Puede ser que algunas anotaciones de jsr305.jar sean las preferidas, pero otras no.

Estoy usando FindBugs 1.3.9 que es the most recent version al momento de escribir esto. Con esta versión, veo las siguientes opciones (por favor, actualice esta table si hay otros):

edu.umd.cs.findbugs.annotations.CheckForNull → javax.annotation.CheckForNull
edu.umd.cs.findbugs .annotations.CheckReturnValue → javax.annotation.CheckReturnValue
edu.umd.cs.findbugs.annotations.NonNull → javax.annotation.Nonnull (NB capitalización)
edu.umd.cs.findbugs.annotations.Nullable → javax.annotation .Nullable
edu.umd.cs.findbugs.annotations.Cuando → javax.annotation.meta.Cuando

Además, toda la JCIPannotations están duplicados:

net.jcip.annotations.GuardedBy → javax.annotation.concurrent.GuardedBy
net.jcip.annotations.Immutable → javax.annotation.concurrent.Immutable
net.jcip.annotations.NotThreadSafe → javax.annotation.concurrent.NotThreadSafe
net.jcip.annotations.ThreadSafe → javax.annotation.concurrent.ThreadSafe

Answer 1

Sí, debería preferir la anotación JSR305 si es posible. ¿Por qué? Porque el JSR305 es un estándar y tiene mucho sentido apegarse a los estándares siempre que sea posible. Mientras porté mis propias aplicaciones, no noté ningún problema o cambio en el comportamiento. Además, incluso puede definir su propia anotación @NotNull y findbugs la recogerá (siempre que la nombre NotNull) consulte this blog entry for more details.

Por lo que puedo ver mirando las fuentes, findbugs está utilizando los mismos métodos de análisis internamente. El envío se realiza solo en función del nombre de las anotaciones. Como se menciona en el blog post vinculado anteriormente, eche un vistazo a las clases edu.umd.cs.findbugs.ba.NullnessAnnotation y NullnessAnnotationDatabase para obtener una vista inicial de cómo se hace internamente. Eche un vistazo a este paquete y encontrará clases similares para las otras anotaciones, como las jcip.

Por lo tanto, desde el punto de vista de las implementaciones, realmente no importa. Para todo el mundo que todavía no esté seguro de qué anotaciones usar, consideraría utilizar las anotaciones estándar o las autodefinidas para evitar que su código dependa de la biblioteca de findbugs.