You shouldn't need this and the constructor should be private