why don't you do this in the constructor? it seems like we need that anyways?