change variables to private