Also, they should go on the constructor and be made immutable IMHO (at least these two hints)