c++ - 数组数据未对输出产生正确的计算

Question

所以我有一个体重计算器作业，我正在为 Uni 做，但我的老师无法解决我的问题。我花了大约三天时间试图解决这个问题。我也手动执行了代码并跟踪了步骤但无法弄清楚。

我没有收到任何错误，但计算没有产生合乎逻辑的结果。

奇怪的是，它是一个非常小的程序，并且在多个设备上产生相同的结果，所以它不仅仅是我的 PC 或平板电脑。我认为问题可能首先在于重量数据的存储，而不是计算本身。

我会尽量不放入太多代码，但同时我希望你们了解我是如何构建程序的。我相信没有任何东西被正确存储到 result.weight

我是 C++ 的新手，我已经过渡到 Pascal。如果我的代码逻辑有问题，请帮助我，因为它在我看来应该有效。

part_data read_part()
{
    int option;
    part_data result;
    result.name = read_string("Please Enter a name: ");
    result.kind = read_part_kind();
    result.weight = read_float("Please Enter weight: ");
}

float weight_of_kind(part_array &data, partkind kind)
{
    int i = 0;
    float result = 0.0;

    for (i = 0; i< data.num; i++);
    {
        if (data.array[i].kind == kind)
        {
            result += data.array[i].weight;
        }
    }
    return result;
}

float total_weight(part_array &data)
{
    float result = 0.0;
    int i;
    for (i = 0; i < data.num; i++)
    {
        result += data.array[i].weight;
    }
    return result;
}

void calculate_weight(part_array &data)
{
    int i;
    printf("Total weight: %4.2f\n", total_weight(data));
    printf("Total CRITICAL: %4.2f\n", weight_of_kind(data, CRITICAL));
    printf("Total DESIRED: %4.2f\n", weight_of_kind(data, DESIRED));
    printf("Total OPTIONAL: %4.2f\n", weight_of_kind(data, OPTIONAL));
}

另外，这里有可供引用的类型

enum partkind {
    CRITICAL, DESIRED, OPTIONAL
};

typedef struct
{
    my_string name;//array
    partkind kind;
    long weight;
} part_data;

typedef struct 
{
    part_data *array; // pointer to a part
    int num;          // number of parts
} part_array;

这是我尝试计算多人体重时得到的输出，按种类(关键、期望或可选)分类

---

更新: 以下是程序中可能帮助我们调试的其他相关函数。我很感激我得到的所有帮助，并且你们没有“否决”我知道这对某些人来说有多么诱人

partkind read_part_kind()
{
        printf("1 - Critical\n");
        printf("2 - Desired\n");
        printf("3 - Optional\n");
    switch( read_integer_range("Please select a part kind: ", 1, 3) )
    {
        case 1: return CRITICAL;
        case 2: return DESIRED;
        default: return OPTIONAL;
    }
}

part_data read_part()
{
    int option;
    part_data result;

    result.name = read_string("Please Enter a name: ");
    result.kind = read_part_kind();
    result.weight = read_float("Please Enter weight: ");

    return result;
}

void add_part(part_array &data)
{
    data.num++;
    data.array = (part_data*) realloc(data.array, sizeof(part_data)*data.num);
    data.array[data.num - 1] = read_part();
}

我希望这段代码有助于理解为什么输入没有被正确分配以及为什么 Total weight 产生(看似)随机内存。

Answer 1

您试图在函数 calculate_weight() 和 total_weight() 中使用指针 part_data* 数组 而不对其进行初始化，这会导致它包含随机垃圾。在使用该数组之前，您应该像这样为其分配内存:

array = new part_data[n]

其中 n 是数组中元素的数量。在您的情况下，您可能希望 n 等于 data.num。

此外，您声明 readPart() 函数将返回 part_data，但它不返回任何内容。