c-CS50实验室4:卷-关于malloc()

Question

我正在学习CS50 CS入门课程——实验室4：卷。我想我对molloc函数有一些误解。起初，我使用malloc函数为标题部分分配空间，如下所示：

// TODO: Copy header from input file to output file
uint8_t *header = malloc(HEADER_SIZE);
if (header == NULL)
{
return 2;
}
fread(header, sizeof(header), 1, input);
fwrite(header, sizeof(header), 1, output);
free(header);

这个程序成功地编译了，但我认为它没有正确地复制标题。然后我尝试了这个：

// TODO: Copy header from input file to output file
uint8_t header[HEADER_SIZE];
fread(header, sizeof(header), 1, input);
fwrite(header, sizeof(header), 1, output);

这是有效的，但我不知道我这样做做做了什么改变。我只是认为这两种代码具有相同的效果。有人能告诉我区别吗？我犯了什么错误？我的意思是，如果我真的想使用malloc函数，代码应该是什么样的？以下是我为volunt.c编写的完整成功代码：

// Modifies the volume of an audio file
#include <stdint.h>
#include <stdio.h>
#include <stdlib.h>
// Number of bytes in .wav header
const int HEADER_SIZE = 44;
int main(int argc, char *argv[])
{
// Check command-line arguments
if (argc != 4)
{
printf("Usage: ./volume input.wav output.wav factorn");
return 1;
}
// Open files and determine scaling factor
FILE *input = fopen(argv[1], "r");
if (input == NULL)
{
printf("Could not open file.n");
return 1;
}
FILE *output = fopen(argv[2], "w");
if (output == NULL)
{
printf("Could not open file.n");
return 1;
}
float factor = atof(argv[3]);
// TODO: Copy header from input file to output file
uint8_t header[HEADER_SIZE];
fread(header, sizeof(header), 1, input);
fwrite(header, sizeof(header), 1, output);
// TODO: Read samples from input file and write updated data to output file
int16_t buffer;
while (fread(&buffer, sizeof(buffer), 1, input))
{
buffer *= factor;
fwrite(&buffer, sizeof(buffer), 1, output);
}
// Close files
fclose(input);
fclose(output);
}

Answer 1

在这个实验室工作时，我也遇到了类似的问题。我能够在不使用malloc的情况下解决它，但我想写另一个包含它的解决方案

malloc的问题在于没有提供正确的参数。给出了头的每个字节都是一个uint8_t(8位无符号int(。因此大小应该是sizeof(uint8_t(*HEADER_size。看看这里的例子。

因此正确的方法是：

uint8_t *header = malloc(sizeof(uint8_t) * HEADER_SIZE);

而不是uint8_t *header = malloc(HEADER_SIZE);

此外，您应该而不是在下一行检查header == NULL，因为我们刚刚初始化了这个指针，那里有什么并不重要。

谢谢你的提问，希望我的回答能有所帮助！