Calling toString()
or clone()
on an object should always return a string or an object. Returning null
instead contravenes the method's implicit contract.
public String toString () { if (this.collection.isEmpty()) { return null; // Noncompliant } else { // ...
public String toString () { if (this.collection.isEmpty()) { return ""; } else { // ...