分享一次调试bug遇到的问题及解决方法

 一次调试遇上了这样的bug:

        error C2679: 二进制“=”: 没有找到接受...类型的右操作数的运算符(或没有可接受的转换)

例如：

#include<iostream> #include<iterator> #include<set> #include<string> using namespace std; typedef set<string,string>::const_iterator CIT; template<class T> void show(CIT&it,set<T>&s) { for(it=s.begin();it!=s.end();++it) { cou...

网上搜了下，一位大神这么解释的：

我用VS2008、VS2010、VS2013、GCC都试了一下，确实VS2008、VS2010编译出错，但是VS2013编译成功了（我也很纳闷为什么成功，从标准库给出的原型来看程序应该是不能编译通过的）！
你把typedef set<string,string>::const_iterator CIT;改成typedef set<string>::const_iterator CIT;就可以编译通过了！
你的问题是说完了，但我还有话说，如果你那本书上真是这样的代码，那你可以把那本书直接扔掉了，完全不值得看，误人子弟，害人害己，当然，我告诉你原因：
原罪一：
set模板类定义的原型是
template < class T,                        // set::key_type/value_type
          class Compare = less<T>,        // set::key_compare/value_compare
          class Alloc = allocator<T>      // set::allocator_type
          > class set;
这里第一个参数就是你要存储的数据的类型，第二个参数和第三个是可选的，第二个默认是less<T>，排序准则，是一种函数对象（重载了括号运算符的）其定义如下：
template <class T> struct less : binary_function <T,T,bool> {
bool operator() (const T& x, const T& y) const {return x<y;}
};
也就是说less<string>表示使用string的小于号作为排序准则（可以换成greater<string>就是按大于号排序）；第三个就不说了！
所以typedef set<string,string>::const_iterator CIT;这个东西严格意义来说是不正确的，你跟搞不清楚它要做什么，string作为排序准则，究竟是大于号？小于号？我刚查了下string没有重载operator ()，所以不算函数对象；但是VS2013和G++能编译通过，我很纳闷！

原罪二：
来看看这个模板函数的定义，这明显不是面向对象的思想，是面向过程（C语言）的思维方式
template<class T>
void show(CIT&it,set<T>&s)
{
        for(it=s.begin();it!=s.end();++it)
        {
                cout<<*it<<"\t";
        }
        cout<<endl;
}
C++的做法是什么呢
template <class T>
void show(const set<T>& s)
{
        for (typename set<T>::const_iterator citer = s.begin(); citer != s.end(); ++citer)
        {
                cout << *citer << "\t";
        }
        cout << endl;
}
我来解释一下，这个函数的作用就是输出set容器内的每一个元素。明明只需要传入一个参数的，为什么会有两个？太奇怪了。

原罪三：（这部分可以商议）
再看这一段奇葩代码
const int N=5;
string s1[N]={"mick","bill","gate","rose","jane"};
string s2[N]={"张峰","秦平","李力","陆放","胡涛"};
set<string>name1;
name1.insert(s1,s1+N);
set<string>name2(s2,s2+N);
首先这里定义了两个临时数组，s1，s2，从这里看是为了使用数组特有的初始化列表（C++11标准后set也支持了），但从这个例子里看不出来这样使用的好处！
而且name1和name2的构造方式还不一样（第一个使用默认构造函数+insert成员函数，第二个则使用了特定的构造函数），很明显现在这个程序里不好解释
更合适的做法是：
set<string> name1;
name1.insert("mick");
name1.insert("bill");
name1.insert("gate");
name1.insert("rose");
name1.insert("jane");

set<string> name2;
name2.insert("张峰");
name2.insert("秦平");
name2.insert("李力");
name2.insert("陆放");
name2.insert("胡涛");
很一目了然，那么有些人可能会说如果元素很多呢，这样的代码会很难看，那么你其实应该将这些字符串放入文件，然后由程序读入，并使用insert插入到set中

原罪四：
cout<<"查找name2中大于“李力”的第一个元素。\n";
cit=name2.upper_bound("李力");
cout<<*cit<<endl;
set在英文下是按照字典顺序排序的，汉字的话，则不很确定，比如我在linux下使用G++运行的结果就是：
查找name2中大于<李力>的第一个元素。
秦平
注释：name2中顺序是：张峰 李力 秦平 胡涛 陆放（非拼音顺序）
而在windows下使用VS运行的结果则是：
查找name2中大于<李力>的第一个元素。
陆放（按照的是拼音顺序）
所以，这里就根本不该使用汉字来举例子，使用英文单词就会没有任何异议！

原罪五：
整个程序的布局、缩进、规范化都很差，一看都是业余的，或者很早以前的C++代码，我特别做了整理，看下：
#include<iostream>
#include<set>
#include<string>

using namespace std;

template <class T>
void show(const set<T>& s)
{
        for (typename set<T>::const_iterator citer = s.begin(); citer != s.end(); ++citer)
        {
                cout << *citer << "\t";
        }
        cout << endl;
}

int main()
{
        set<string> names;
        names.insert("mick");
        names.insert("bill");
        names.insert("gate");
        names.insert("rose");
        names.insert("jane");

        cout << "输出names中各个元素：" << endl;
        show(names);

        cout << "查找names中大于或等于<x-men>的第一个元素." << endl;
        set<string>::const_iterator cit = names.lower_bound("x-men");
        if (cit != names.end()) {
                cout << *cit << endl;
        } else {
                cout << "很抱歉, 没有没有找到大于或等于<x-men>的元素" << endl;
        }

        return 0;
}
1、去掉了#include <iterator>
2、重写了show模板函数
3、去掉了name2，将name1改为names
4、使用多次insert代替数组加insert
5、尽量使用endl而非"\n"
6、使用const_iterator而非iterator，因为不修改其值
7、对cit的返回值做了判断，就想程序里写的，找不到的时候能够正确的处理；原来的程序在找不到的情况下，就会挂掉
8、合理的使用空格和换行，让程序美观而有调理

如果你看得进去，那我推荐你一本好的书《C++ Primer》看看吧