c++ - 在 C++ 中对数组进行洗牌和平方的函数的问题

Question

我正在尝试编写一个函数来随机化数组中元素的顺序，并对这些元素进行平方。我设法用两个参数来编写它，但是我想只用一个参数来编写它，但出于某种原因，我总是得到奇怪的值作为解决方案。有时它们是正确的，有时则不是。这是我的函数的代码:

int *randomizedField(int field[]){
    int temp = 0;
    int random = 0;
    int i = 0;
    int j = 0;
    while(field[i] != '\0'){
        field[i] = field[i] * field[i];
        i++;
    }
    while(field[j] != '\0'){
        random = rand() % i;
        temp = field[j];
        field[j] = field[random];
        field[random] = temp;
        j++;
    }
    return field;
}

Answer 1

你的循环中有未定义的行为:

while(field[i] != '\0')

你一直读到一个空终止字符。显然，您永远不知道什么时候到达那个位置，它也不会结束您的数组。因此，您通过了指针的边界并输入了undefined bahavior。土地。

另一方面，您无法找到数组的大小，因为当传递给函数时，数组会衰减为指针，您无法使用 sizeof(field)/sizeof(int) 找到它的大小.

要解决您的问题，请将数组的长度传递给函数，或者理想情况下使用 STL 容器之一，例如std::vector或 std::array .

第一个循环将是:

for(int i = 0; i < field.size(); ++i) { 
   field[i] = field[i] * field[i]; 
   i++;
}

函数签名可以是:

void randomizedField(std::vector<int>& field)