prefer std::string over char* in CDB constructor

2nd commit:
cleanup class private and public areas in walletdb
Will update...