在 c++ 代码中将数组传递给函数错误

Passing arrays to functions in c++ code error

本文关键字：函数 错误 数组 c++ 代码      更新时间：2023-10-16

我正在学习C++，我不知道为什么这段代码输出正确数量的输出(4(，而只输出数字54。有人可以帮助修复它并解释我做错了什么吗？

#include <iostream>
using namespace std;
void pi(int arr[],int x){
for(int c = 0;c < x;c++){
cout << arr[x] << endl;
}
}
int main()
{
int _arr[4] = {3543,146,961262,-242};
pi(_arr, 4);
}

您将数组限制与索引器混淆了。应该是：

cout << arr[c] << endl;

数组的索引为零。您正在尝试打印出越界的元素，从而调用未定义的行为。根据平台的不同，您的输出可能看起来像四个0或类似-858993460.按照惯例，计数器变量通常以字母i开头。

由于x是 4，所以arr[x]是arr[4]。但是_arr只有四个条目，它没有第五个条目。所以你在输出垃圾。你可能的意思是arr[c]在循环中。

更改

cout << arr[x] << endl;

自

cout << arr[c] << endl;

函数内部循环中有一个拼写错误

for(int c = 0;c < x;c++){
cout << arr[x] << endl;
^^^
}

应该有

for(int c = 0;c < x;c++){
cout << arr[c] << endl;
^^^
}

它是用作数组索引的变量c。

然而，该程序有几个缺点。

不应使用以下划线开头的标识符。编译器实现可以保留此类名称。

不要使用"幻数"。它们通常是错误的原因。

要存储对象或数组的大小，请使用类型size_t而不是类型int。

该函数不会更改数组。所以相应的参数应该用限定符const来声明。

该程序可以如下所示

#include <iostream>
void pi( const int arr[], size_t n )
{
for ( size_t i = 0; i < n; i++ )
{
std::cout << arr[i] << std::endl;
}
}
int main()
{
int arr[] = { 3543, 146, 961262,-242 };
pi( arr, sizeof( arr ) / sizeof( *arr ) );
}

考虑到您可以使用标准算法输出数组，例如std::for_each或std::copy。

下面是一个演示程序，它使用标准算法std::copy输出数组。

#include <iostream>
#include <algorithm>
#include <iterator>
int main()
{
int arr[] = { 3543, 146, 961262,-242 };
std::copy( std::begin( arr ), 
std::end( arr ), 
std::ostream_iterator<int>( std::cout, "n" ) );
}
相关文章：