I think it would be cleaner if this were imutable with a constructor and getters.  This is minor, it just makes the code a bit cleaner.