TL；博士

int *sieve = (int *) malloc(sizeof(int) * length);

有两个问题。类型转换，并且您使用类型而不是变量作为sizeof的参数。相反，请这样做：

int *sieve = malloc(sizeof *sieve * length);

长版本

不您不必计算结果，因为：

这是不必要的，因为在这种情况下，void*会自动安全地升级为任何其他指针类型。它给代码增加了混乱，强制转换不太容易阅读（特别是如果指针类型很长）。这会让你重复自己，这通常是不好的。如果您忘记包含＜stdlib.h＞，它可能会隐藏错误。这可能会导致崩溃（或者更糟糕的是，直到稍后在代码的某个完全不同的部分发生崩溃）。考虑如果指针和整数的大小不同会发生什么；然后你通过投射来隐藏警告，可能会丢失返回地址的一部分。注意：从C99开始，隐式函数从C中消失，这一点不再相关，因为没有自动假设未声明的函数返回int。

作为澄清，请注意我说的是“你不需要投”，而不是“你不必投”。在我看来，这是一个失败的演员阵容，即使你做得对。这样做根本没有好处，但有一系列潜在的风险，包括演员阵容表明你不知道这些风险。

另外，正如评论家指出的，上面提到的是直接C，而不是C++。我非常坚信C和C++是独立的语言。

此外，代码不必要地重复类型信息（int），这可能会导致错误。最好取消引用用于存储返回值的指针，将两者“锁定”在一起：

int *sieve = malloc(length * sizeof *sieve);

这还将长度移到前面以增加可见性，并删除sizeof的多余括号；只有当参数是类型名时，才需要它们。许多人似乎不知道（或忽略）这一点，这使得他们的代码更加冗长。记住：sizeof不是函数！：）

虽然在一些罕见的情况下，将长度移到前面可能会增加可见性，但也应注意，在一般情况下，最好将表达式写成：

int *sieve = malloc(sizeof *sieve * length);

在这种情况下，由于首先保持sizeof，确保乘法至少使用size_t数学运算完成。

比较：malloc（sizeof*sive*length*width）与malloc，当width和length的类型小于size_t时，第二个可能会溢出长度*宽度。

这就是GNU C库参考手册所说的：

您可以将malloc的结果存储到任何指针变量中强制转换，因为ISO C会自动将类型void*转换为另一个必要时的指针类型。但演员阵容在上下文中是必要的而不是赋值运算符，或者如果您可能希望代码运行在传统C。

事实上，ISO C11标准（第47页）是这么说的：

如果分配成功，则返回的指针将适当对齐它可以分配给指向任何类型对象的指针基本对齐要求，然后用于访问这样的对象或此类对象的数组（直到空间被显式释放）

在C语言中，可以隐式将void指针转换为任何其他类型的指针，因此不需要强制转换。使用一个可能会向不经意的观察者暗示需要一个的原因，这可能会误导人。

在C语言中，您得到了从void*到任何其他（数据）指针的隐式转换。

void指针是泛型指针，C支持从void指针类型到其他类型的隐式转换，因此不需要显式类型转换。

然而，如果您希望相同的代码在不支持隐式转换的C++平台上完美兼容，则需要进行类型转换，因此这一切都取决于可用性。

您不需要强制转换malloc的结果，因为这样做会给代码增加无意义的混乱。

人们之所以选择malloc的结果，最常见的原因是他们不确定C语言是如何工作的。这是一个警告信号：如果你不知道特定的语言机制是如何工作的，那么就不要猜测。查找或询问堆栈溢出。

一些意见：

空指针可以转换为任何其他指针类型，而无需显式转换（C11 6.3.2.3和6.5.16.1）。但是，C++不允许在void*和另一个指针类型之间进行隐式转换。因此，在C++中，演员阵容是正确的。但是如果你用C++编程，你应该使用new而不是malloc（）。而且您永远不应该使用C++编译器编译C代码。如果需要使用相同的源代码同时支持C和C++，请使用编译器开关来标记差异。不要试图用相同的代码来满足两种语言标准，因为它们不兼容。如果C编译器因为忘记包含标头而找不到函数，则会出现编译器/链接器错误。因此，如果您忘记包含<stdlib.h>，这没什么大不了的，您将无法构建您的程序。在遵循超过25年的标准版本的古代编译器上，忘记包含＜stdlib.h＞将导致危险行为。因为在那个古老的标准中，没有可见原型的函数隐式地将返回类型转换为int。显式地从malloc转换结果将隐藏这个错误。但这真的不是问题。你不是在使用一台25年前的计算机，那么你为什么要使用一台有25年历史的编译器呢？