redo constructor now it is a solution