Question

因此，我创建了一个程序来模拟掷骰子，然后打印结果（作业:)）。现在我有一个bad_array_new长度，我想我知道它发生的位置，但是我真的不明白为什么。这可能是一个逻辑错误，如果有人可以找到它或给我一个Pointer，那就太好了。

int* CalculateThrows(int dice_one, int dice_two, int throw_count)
{
    // I think the error occurs here 
    int *digits = new int[(dice_one * dice_two) + 1]{ 0 };
    digits[0] = dice_one * dice_two+1;

    if (throw_count > 0)
    {
        for (int i = 1; i < throw_count; i++)
            digits[GetRandomValue(dice_one) + GetRandomValue(dice_two)]++;
    }
    return digits;
}

这是我调用方法并稍后遍历数组的方式：

int main(int argc, char* argv[])
{
    int * values = CalculateThrows((int)argv[1], (int)argv[2], (int)argv[3]);
    for (int i = 1; i < values[0]; i++)
    {
        cout << i << ". Has been thrown: " << values[i];
    }
}

Answer 1

argv[1]是指向包含第一个命令行参数的字符串的指针。因此，如果您调用程序./program 6 5 4，则argv[1]将是字符串"6"，而不是数字6。您应该使用atoi(argv[1])来获取号码。

在使用argc之前，还应该检查4是否至少为argv[3]。

std::vector<int> CalculateThrows(int dice_one, int dice_two, int throw_count)
{
    std::vector<int> digits (dice_one * dice_two, 0);

    if (throw_count > 0)
    {
        for (int i = 0; i < throw_count; i++)
            digits[GetRandomValue(dice_one) + GetRandomValue(dice_two)]++;
    }
    return digits;
}

int main(int argc, char* argv[])
{
    if (argc < 4) return -1;
    std::vector<int> values = CalculateThrows(atoi(argv[1]), atoi(argv[2]), atoi(argv[3]));
    for (size_t i = 0; i < values.size(); i++)
    {
        cout << i << ". Has been thrown: " << values[i];
    }
}

C ++我得到这个错误bad_array_new_length

1 个答案: