Well, the LPCSTR is just a windows defined typedef, like BOOL or DWORD. I don''t think that would be considered Hungarian notation. Technically, Hungarian notation would be char *szStr, which is readable, however that variable name basically says its a string twice because of how you named the variable (Str). But to me, having a variable called byLevel tells you that 1, it''s definately a variable, and 2, that it is a byte. Better than having to go looking through header files to find out the type whether it is someone elses code, or your own if you forgot (which can happen on extremely large projects).

User defined classes are a problem. Basically I just forgo Hungarian notation at that point, since you don''t want to have a 5 or 6 character prefix just to name a variable (defeats the point of making it easier to read).