refactoring private member viriables