119 try {
120 var archive = new ArchiveStreamFactory().createArchiveInputStream(input);
121 var entries = new AbstractIterator<InputStream>() {
122 @Override protected InputStream computeNext() {123 try {124 return (archive.getNextEntry() == null)125 ? endOfData()126 : readInput(CloseShieldInputStream.wrap(archive));127 } catch (IOException e) {128 throw new UncheckedIOException(e);129 }130 }131 };
132 return new MultiInputStream(archive, entries);
133 } catch (ArchiveException e) {If a method of a superclass has one particular nullability annotation applied to it, avoid marking any overrides in subtypes with a different nullability annotation.
Make sure to use the same annotation present on the super method as much as possible.
This issue is raised when the parent method is annotated with a particular nullability annotation, and the child method is not, or is annotated with a different annotation than the parent method.
This issue will also be raised on parameters of such overloaded methods that may have differing, or no nullability annotations as well.
Bad Practice
Consider this code, where class B extends class A, and overrides A.a with @Nullable instead of @Nonnull as A.a has been marked.
class A {
@Nonnull
Integer a(int x) {
return x;
}
}
class B extends A {
@Nullable
@Override
Integer a(int x) {
if (x % 2 != 0) return x;
return null;
}
}
This would cause issues if the classes in question were used with polymorphism, like in the following code:
A someInstance = new A();
int someInt = someInstance.a(3); // works.
someInstance = new B(); // An instance of `B` is assigned to a variable of type `A`.
someInt = someInstance.a(4); // Throws an NPE!
Recommended
Use consistent nullability annotations in overriding methods, and avoid changing the method contract specified by the parent class as much as possible.
If you need to have entirely new behavior with different constraints, create an overload, or a new method entirely.
class B extends A {
@Nullable
Integer aNullable(int x) {
if (x % 2 != 0) return x;
return null;
}
}