CODEWITHSUNDEEP
javaannotationsdeprecated
user1772643
user1772643

Reputation: 645

How to annotate deprecation of a class in Java?

I am going to deprecate a class in Java.

@Deprecated
class deprecatedClass

and I have list of this deprecated class, 

List<deprecatedClass> listOfDeperecatedClass

So do I need to add the @Deprecated tag for this list too?

Edit: @Deprecated should have a capital 'D'.
See: http://docs.oracle.com/javase/7/docs/api/java/lang/Deprecated.html

Upvotes: 18

Views: 26500

Answers (5)

AlpharettaTechy
AlpharettaTechy

Reputation: 390

Something to note for people who stumble on this question like I did... I am on OpenJDK 14 and I see capital D does not work but lower case D works! My Intellij says wrong tag @Deprecated vs. no such warning with lower case D.

Secondly, when I compile my maven project also only lower case D shows the warning below...

[WARNING] <....>.java: <....>.java uses or overrides a deprecated API.
[WARNING] <....>.java: Recompile with -Xlint:deprecation for details.

Upvotes: 0

SQB
SQB

Reputation: 4078

No, you don't need to. Adding the annotation @Deprecated to DeprecatedClass will generate a warning every time it's used.

What you should do however, is marking methods in other classes that take your deprecated class as an argument or return it, as deprecated as well. That goes for any access that other code may have to instances of your deprecated class — public fields, constants and so on. Those of course can't be used without an instance of your deprecated class, so the warning is given anyway, but in a correct deprecation annotation and comment, you should provide an explanation and point to an alternative, which is valuable information you need to give.

A method signature is like a contract and so is a class signature. You're telling other programmers what methods they can call and how they can call them. You're telling them which fields are accessible. Other programmers base their code on that. If you really need to break that contract, you first need to provide a substitute for that contract (a new method with the same functionality), and tell them and give them time to switch to that new contract (deprecate the old methods and classes).

Of course, the above assumes that you're coding to an audience. If you're the only one using your code and you just want to deprecate to clean up your code without breaking the build, just deprecate the class, fix the warnings, and remove it.

Upvotes: 9

Seid.M
Seid.M

Reputation: 185

Just marking as:

@Deprecated
List<deprecatedClass> listOfDeperecatedClass

Should be okay.

Upvotes: 1

BlackBox
BlackBox

Reputation: 2233

You only need to add the deprecation message to the declaration of anything you are deprecating. It serves as a warning that people should avoid an implementation which uses the deprecated class, such as in List<DeprecatedClass>.

Upvotes: 2

Keppil
Keppil

Reputation: 46209

Do you have operations on the List as part of your public interface? In that case, mark all those methods as deprecated too. Otherwise you should be fine.

Upvotes: 4

Related Questions