0

我试图将数组发送到函数，但我的程序卡住了

int main()
{
    int n, i;
    bool random;

    cout << "number of elements in array:"; cin >> n;
    cout << "use random values?"; cin >> random;

    int* arr = new int[n]; //create int array with n size
    init_array(random, n, arr); //fill array using function

    for (i = 0; i <= n; i++) //display array
        cout << " " << arr[i];

    return 0;
}

这个函数应该用随机数填充数组或从键盘输入

void init_array(bool random, int n, int* arr)
{
    int i;
    if (random)
    {
        srand(time(0)); //initialize random;
        for (i = 0; i < n; i++)
            arr[i] = rand() % 100;
    }
    else
        for (i = 0; i<n; i++)
            cout << "arr[" << i << "]: "; cin >> arr[i];
}

有没有办法发送动态数组来运行？

4

2 回答 2

2

当您在 for 循环后不使用括号时，只有第一条语句用作循环：

else
    for (i = 0; i<n; i++)
        cout << "arr[" << i << "]: "; cin >> arr[i];

此循环将尝试打印“arr[#]”n次，然后请求输入（将尝试将其放置在数组 (UB) 中最后一个元素之后的项目 1 中。

你想要的是这样的：

else
{
    for (i = 0; i<n; i++)
    {
        cout << "arr[" << i << "]: "; 
        cin >> arr[i];
    }
}

您的输出也有问题：

for (i = 0; i < n; i++) // <= would attempt to print 1 more item than exists in the array

为了完整起见，当您使用为您完成所有这些操作的容器时，大多数这些问题都会消失：

int main()
{
    int n = 0;
    bool useRandom = false;
    std::cout << "number of elements in array:"; 
    std::cin >> n;
    std::cout << "use random values?"; 
    std::cin >> useRandom;

    std::vector<int> arr(n);
    init_array(useRandom, arr);

    std::copy(arr.begin(), arr.end(), std::ostream_iterator<int>(std::cout, " "));

    return 0;
}

void init_array(bool useRandom, std::vector<int>& vec)
{
    ::srand(time(0)); //initialize random;
    int n = 0;
    std::transform(vec.begin(), vec.end(), vec.begin(), [&](int i)
    {
        if (useRandom)
        {
            i = rand() % 100;
        }
        else
        {
            std::cout << "arr[" << n++ << "]:  ";
            std::cin >> i;
        }
        return i;
    });
}
于 2013-11-10T19:03:47.890 回答
2

由于 last，您的代码最后要求输入一个数字cin>>n

else将部分固定init_array为：

 else
        for (i = 0; i<n; i++)
            { //Notice braces
             cout << "arr[" << i << "]: ";
             cin >> arr[i];
            }

同时修复：

for (i = 0; i < n; i++) //display array from index 0 to n-1
    cout << " " << arr[i];
于 2013-11-10T19:04:41.920 回答