UserIdentity inheritance

nicopc · February 9, 2011, 2:20pm

Quick question:

Why does the class CBaseUserIdentity define:




	const ERROR_USERNAME_INVALID=1;

	const ERROR_PASSWORD_INVALID=2;

As I understood based on the guide, the whole idea of having a useridentity is to allow multiple forms of authentication, not only username/password-based authentication.

Then these specific errors do not belong in this baseclass but should be defined in the child class CUserIdentity which also defines username and password properties.

Or does anybody know of a reason to put these definitions in this abstract base class?

mikl · February 10, 2011, 3:35pm

I think you have a point here. Only ERROR_NONE (and maybe ERROR_UNKOWN_IDENTITY) fit in this base class. Since CUserIdentity describes itself as "base class for representing identities that are authenticated based on a username and a password" it might be the better candidate for these two constants.