hm,yep, it's better to do in constructor