In general it's fine to have this kind of code in the constructor since it'll only ever be called once (from the constructor) and doesn't do any crazy logic, it can be easier to read. If you were doing a ton of stuff in here then it's good to have the method, otherwise it's just more lines of code.