在函数中使用向量指针时出错

Question

我有这段代码，但它无法编译，我不明白哪里出了问题——我猜向量的指针不正确。 我的想法是在 main() 中收集一些数字并将它们存储在向量和数组中，然后将它们的内存地址传递给函数，并使用指针打印存储的数据。

当我读到一些关于指针的东西说我应该使用它们来节省内存时，我想到了这个，所以 IMO 下面的代码不会复制向量和数组的内容，而是使用指针来访问它们内存中的位置 - 这就是我想要做的。

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

void function(vector<int>* a, int *s)
{
    cout << "function starts.." << endl;
    for(int i=0;i<a->size();i++)
    {
        cout << a[i] << endl;
        cout << s[a[i]] << endl;
    }
    cout << "function ends..." << endl;
}


int main(void)
{
    vector<int> m;
    int s[102];
    for(int i=0;i<10;i++)
    {
        m.push_back(i*i);
        s[i*i] = i-2;
    }
    function(&m, &s);
    return 0;
}

我在编译时收到几个错误，有问题。

请告诉我我的代码有什么问题以及如何修复它。谢谢...

Answer 1

你应该通过引用而不是指针来传递向量：

void function(vector<int>& a, int *s)

然后

function(m, ...);

在指向向量的指针上使用 [] 肯定会导致奇怪的问题，因为它的行为就像 a 指向一个 std::vectors 数组（虽然它实际上只指向一个）。向量本身永远不会被索引。您也可以使用(*a)[...] 通过指针来索引向量。

Answer 2

如果你坚持按指针解析，那么正确的语法应该是：

void function(vector<int>* a, int *s[]) 
{ 
    cout << "function starts.." << endl; 
    for(int i=0;i<a->size();i++) 
    { 
        cout << (*a)[i] << endl; 
        cout << (*s)[(*a)[i]] << endl; 
    } 
    cout << "function ends..." << endl; 
} 

Answer 3

首先在主程序中 s 是一个指向 int 的指针，而 m 是一个向量。因此函数调用应该如下：

函数(&m, s);

其次在函数a中是一个指向向量的指针，所以应该按如下方式索引：(*a)[i]。

但是，您确实应该使用 const 引用来传递您的向量：

void 函数(const vector& a, int *s) { .. cout

然后这样称呼它：

函数(m, s);

Answer 4

（已修正）

&s 实际上是int(*)[102]：指向一个包含 102 个项目的数组的指针。

你应该说

function(&m, s);

这是因为根据旧的 C 遗留规则，数组本质上是一个指向其索引为 0 的项的 const 指针。所以s 已经是int*

Answer 5

此版本有效：

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

void function(const vector<int>& a, int s [102])
{
    cout << "function starts.." << endl;
    for(int i=0;i<(int)a.size();i++)
    {
        cout << a [i] << endl;
        cout << s[a [i]] << endl;
    }
    cout << "function ends..." << endl;
}


int main(void)
{
    vector<int> m;
    int s[102];
    for(int i=0;i<10;i++)
    {
        m.push_back(i*i);
        s[i*i] = i-2;
    }
    function(m, s);
    return 0;
}