不匹配对'(std::pair<unsigned int，unsigned int>) (unsigned int&，unsigned int)‘的调用

Question

我不知道下面的代码有什么问题，它应该读取数字，并将它们的值和位置放在一个成对的向量中，然后对它们进行排序并打印出位置。我用sort删除了这个部分--我以为问题出在那里，但我又收到了编译错误。

#include <iostream>                                                                                                           
#include <vector>                                                                                                             
#include <algorithm>                                                                                                          
#include <utility>                                                                                                            
using namespace std;                                                                                                          

int main(void)
{
        unsigned int n,d,a[65],b[65],s,i,j,t,us=0;
        pair<unsigned int,unsigned int> temp;
        vector< pair<unsigned int,unsigned int> > v;
        cin >> n;
        for(i=0;i<n;i++)
        {
                cin >> t;
                temp(t, i+1);
                v.push_back(temp);
        }
        cin >> d;
        for(i=0;i<d;i++) cin >> a[i] >> b[i];
        for(i=0;i<v.size();i++)
        {
                cout << v[i].first << " -- " << v[i].second << endl;
        }
        return 0;
}

请告诉我问题出在哪里。谢谢。

Answer 1

问题出在temp(t, i+1);

您需要手动设置第一个和第二个

temp.first = t;
temp.second = i + 1;

或者，您可以在循环中声明temp (我可能会这样做)。

for(i=0;i<n;i++) 
{ 
    cin >> t; 
    pair<unsigned int,unsigned int> temp(t, i+1); 
    v.push_back(temp); 
} 

或者使用第二个替代方法，使用make_pair辅助函数，并完全删除temp (感谢KennyTM的提醒)

for(i=0;i<n;i++) 
{ 
    cin >> t; 
    v.push_back(make_pair(t, i+1)); 
} 

希望这能有所帮助

Answer 2

在循环之外创建一个变量并在循环中多次重复使用它通常是不好的形式，如果您只在该循环中使用它的话。只有当构建成本很高，并且重新分配比重新创建更便宜时，才能这样做。通常，C++变量应该在它们使用的作用域中声明，以便更容易读取程序的每个部分，并能够在以后对其进行重构。

在您的示例中，我将完全删除对temp的引用，并将push_back调用更改为v.push_back(make_pair(t, i+1))。