1. Almost no comments. This code may be a particularly written. Of course, more may be more bad.
2. Documents are in some places and actual code. That is the issue of documentation.
3. The code for operating the database section is not separated, almost mixed throughout the program, if you want to use other databases ... can only be rewritten. What is the so-called "no separation" refers to? If you want to change the database, you will be rewritten, indicating that it is indeed bad. If the data table structure is a bit change, it should be modified, and it is normal. In terms of experience, the data table structure is the highest in the entire system, and it is not found to find a suitable method, and I hope some people doll.
4. A large number of macros used in "#define example string (" example "), resulting in low efficiency at runtime. It is not very well understood by people who write a program.
5. As the class of the base class, there is no false arming function. Ibid.
6. Use the "==" and "Compare" method of the String class. It is not very well understood by people who write the program.
7. Many new statements are followed by no checking. This is relatively confused. "Effective C " does not notice that NEW is unsuccessful. But the prawn is really checked for every new statement? I really want to ask how the code is written. Also, this is just the case in C . Do you check it again after writing a Java or C # program?
8. Class with pointers, there is no replication constructor, and there is also no explicit prohibited copy object. The same 4.
9. Many places enforce the base class's pointers to derived pointers. This situation is not seen wrong. If the derived class does have the properties and methods that are not available in the base class. It's not when you need to use the adapter mode.
10. There is no place to use Const. Or, the entire program does not have a const identifier. This is not a good habit, but it is not a fatal bad habit.
11. Debug the statement used, no "#ifdef _debug" is included in. This is! Speechless.