Is it a good idea to do "heavy" stuff like this in the constructor?