發送動態指針數組以使用C ++

Question

我試圖將數組發送給函數，但是我的程序被卡住了

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];
}

有什么辦法可以發送動態數組來起作用嗎？

Answer 1

如果在for循環后不使用方括號，則僅將第一條語句用作循環：

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

此循環將嘗試打印n次“ arr [＃]”，然后要求輸入（它將嘗試放置在數組（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;
    });
}

Answer 2

您的代碼要求最后輸入一個數字，因為最后一個cin>>n

將init_array else部分修復為：

 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];