let's make these private with getters/setters. Things like name or id I think should be immutable