Encapsulate your classes and avoid exposing references to mutable private fields
This posting discusses the (too unknown) concept of encapsulating data. When a private field is mutable, you will not achieve much encapsulation if you provide a public accessor returning a direct reference to the data that you declared as private.