I wonder if it's worth having this as a separate method as it's only ever called from __init__. Personally I do like it for the encapsulation it provides, but I'm not sure it's the way we usually code this type of thing.