C++——编程常见错误

C++库函数

C++标准库比C标准库要复杂很多，需要大家认真学习。C++标准库建立时间较晚，解决了C标准库的一些问题。通过认真学习、熟练掌握会对代码质量的提高有一定帮助。

一些建议：
1. 尽量使用迭代器代替下标操作
2. 尽量避免使用指针和数组 ，尽可能使用vector和迭代器
3. 采用 string 类型取代 C 风格字符串（使用标准库类型 string，除了增强安全性外，效率也提高了，因此应该尽量避免使用 C 风格字符串）
4. 用指针操作数组时应首先选用类似于迭代器的那种用法
5. const使用时，如果等价，尽量将const放在类型后，以避免类似于typedef使用中的错误
6. 一般而言，应该优先使用标准库抽象类而少用语言内置的低级数组和指针。尤其是应该使用 string 类型取代 C 风格以空字符结束的字符数组。现代 C++ 程序不应使用 C 风格字符串。

下面是一些在我们程序中发现的错误：

1. 不会用 iterator（其实就是没用过STL）;
for (iter=v.begin; iter!=v.end(); iter++) {
...
}
if (iter == v.end()) {
}
（漏掉iter == v.end()的判断，直接使用*iter。导致死机。安徽与宁夏出现过，非标准写法的EPG引发。）

2. 多继承时使用reinterpret_cast<>转换。 (Hippo中出现过。死机。)

3. override函数时参数不一致导致被认作两个不同的函数。(Hippo中出现过导致mp对象建立失败，视频无法播放。)

 

不严谨的习惯，写程序不留心环境、上下文

1. getString只传buffer指针，不传buffer大小。
示例：int sysNtvuserGet(char *buf);
点评：在我们程序员的心中总隐藏着一些约定、为程序限定工作场景。要知道需求会变的，记忆会被淡忘的，隐藏在内心的约定是不方便和其他同事交流的。所以要牢记——不要给程序做隐式的限制，所有的限定必须直观，必须落到代码上、文字上。
修正：int sysNtvuserGet(char *buf, int bufLength);

2. 写程序随意性大，不与代码上下关联思考
示例1：if (!strncmp("PVR.Schedule.get", buff, 15)) {
do something ;
} else if () {
。。。。。。
A large number of code
。。。。。。
} else if (!strncmp("PVR.Schedule.getByID", buff, 19)) {
never do here.
}
点评：不整理，不归类，代码可读性差，问题越藏越深，不易发现

示例2：char LogUploadInterval_buf[URL_LEN] = {0};
memset(LogUploadInterval_buf, 0, URL_LEN);
sprintf(LogUploadInterval_buf,"%d",XXX);
点评：一个buffer竟然被清了三次0，不知是认真过头了，还是粗心过头了。

最后请注意：上面多次提到“显式”，“直观”，切记。