一段STL模版的代码（C++），属于书写错误。现结贴！！

toxyboy

1 #include <iostream>;
      2 #include <string>;
      3 #include <vector>;
      4 #include <algorithm>;
      5 using namespace std;
      6
      7 template <typename Type>;
      8 inline Type max(Type t1,Type t2){return t1 >; t2 ? t1:t2 ;}
      9
     10 template <typename elemType>;
     11 inline elemType max(const vector<elemType>; &vec)
     12         {return *max_element( vec.begin(),vec.den() );}
     13
     14 template <typename arrayType>;
     15 inline arrayType max(const arrayType* parray,int size)
     16         { return *max_element(parray,parray+size);}
     17
     18 int main()
     19 {
     20         string sarray[]={"we","were","her","pride","of","course"};
     21         vector<string>; svec(sarray,sarray+7);
     22
     23         int iarray[]={2,3,55,6,7,88,45};
     24         vector<int>; ivec(iarray,iarray+7);
     25
     26         float farray[]={2.5,3.5,33.2,5.6,45,7.9};
     27         vector<float>; fvec(farray,farray+6);
     28
     29         int imax = max(max(ivec),max(iarray,7));
     30         float fmax = max(max(fvec),max(farray,7));
     31         string smax = max(max(svec),max(svec,6));
     32
     33         cout << "imax should be -- found: " << imax << endl;
     34         cout << "fmax should be -- found: " << fmax << endl;
     35         cout << "smax should be -- found: " << smax << endl;
     36         return 0;
     37 }
     # g++ test.cpp -o test
出错举例：
temp.cpp:31: no matching function for call to `max (vector<string,
allocator<string>; >; &, int)'
and so on!!!

THEBEST

max(svec,6)你准备调用什么函数?

idisc

12 {return *max_element( vec.begin(),vec.den() );}
应该为 {return *max_element( vec.begin(),vec.end() );}
31 string smax = max(max(svec),max(svec,6));
应该为string smax = max(max(svec),max(sarray,6));

whyglinux

C++标准库中已经提供了 max()函数的实现，不需要你自己再定义。因此，你应该把第7、8行的代码去掉。

至于10-12行的代码只是对 max_element() 函数的一个简单的包装。这种情况下，你不应该使用和标准库中的函数同名的函数，用另外的函数名对这个函数进行命名。另外，我认为与其使用你这个函数，还不如直接使用max_element()函数直观和简单些。

对于14-16行的代码同上：不要使用max命名自定义函数，用另外一个函数名；或者把这些代码从程序中去除，直接使用max_element()函数。

你代码中的算法很奇怪。比如，ivec和iarray存储的是相同的整数序列，因此max(ivec)和max(iarray,7)即ivec中的最大值和iarray中的最大值是相同的。既然如此，还有必要再比较这两个最大值哪个更大吗？不过，如果你只是为了测试你写的max()函数的话我倒无话可说。

程序中还存在着一处错误：sarray中明明只有6个数组元素，你在下面却这样用sarray初始化svec：vector<string>; svec(sarray,sarray+7);。这一句中的“7”应该改为6才对吧。要细心一点。

按照我的理解，你的程序可以作如下改写（不使用自定义的max()之类的函数。在你这个题目中完全没有必要自定义max，使用C++标准库提供的函数即可）。你可以跟你的程序比较一下，看哪一个更简洁好用些。

1. 
   
2. #include <iostream>;
   
3. #include <string>;
   
4. #include <vector>;
   
5. #include <algorithm>;
   
6. using namespace std;
   
7. 
   
8. int main()
   
9. {
   
10.   string sarray[]={"we", "were", "her", "pride", "of", "course"};
    
11.   //  vector<string>; svec(sarray,sarray+7);
    
12.   vector<string>; svec(sarray,sarray+6);
    
13. 
    
14.   int iarray[]={2, 3, 55, 6, 7, 88, 45};
    
15.   vector<int>; ivec(iarray,iarray+7);
    
16. 
    
17.   float farray[]={2.5, 3.5, 33.2, 5.6, 45, 7.9};
    
18.   vector<float>; fvec(farray,farray+6);
    
19. 
    
20.   //  int imax = max(max(ivec),max(iarray,7));
    
21.   // 按照上一句的算法，可实现如下：
    
22.   int imax = max( *max_element(ivec.begin(), ivec.end()),
    
23.                   *max_element(iarray, iarray+7) );
    
24.   // 其实，上面的算法是不必要的，简单地如下实现即可：
    
25.   // int imax = *max_element(ivec.begin(), ivec.end());
    
26. 
    
27.   // 下面的求最大值的方法采取上面的后一种方法来实现
    
28. 
    
29.   //  float fmax = max(max(fvec),max(farray,7));
    
30.   float fmax = *max_element(fvec.begin(), fvec.end());
    
31. 
    
32.   //  string smax = max(max(svec),max(svec,6));
    
33.   string smax = *max_element(svec.begin(), svec.end());
    
34. 
    
35.   cout << "imax should be -- found: " << imax << endl;
    
36.   cout << "fmax should be -- found: " << fmax << endl;
    
37.   cout << "smax should be -- found: " << smax << endl;
    
38. }
    

复制代码

toxyboy

呵呵，sorry，误导大家了。纯粹书写错误。。
whyglinux      
风云使者
你说的很多，是不用那么复杂，我只是为了测试使用一下模版而已。

whyglinux

原帖由 "toxyboy" 发表：
呵呵，sorry，误导大家了。纯粹书写错误。。
whyglinux      
风云使者
你说的很多，是不用那么复杂，我只是为了测试使用一下模版而已。

真的是纯粹书写错误吗？那样的话我也就没有必要说那么多了。主要的错误还在于你自己定义的max()函数和标准库中的同名函数发生冲突的问题吧。你说纯粹是书写错误，不知道你是怎样在改正了几个书写错误后通过编译的。