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时，第二个可能会溢出长度*宽度。

这取决于编程语言和编译器。如果在C中使用malloc，则无需键入cast，因为它会自动键入cast。然而，如果您使用的是C++，那么应该键入cast，因为malloc将返回void*类型。

void指针背后的概念是它可以被强制转换为任何数据类型，这就是malloc返回void的原因。此外，您必须了解自动打字。因此，虽然必须强制强制强制转换指针，但它有助于保持代码整洁，并有助于调试

不，您不强制转换malloc（）的结果。

一般来说，你不会向虚空施法或从虚空施法。

不这样做的一个典型原因是#include<stdlib.h>失败可能会被忽略。这在很长一段时间内不再是问题，因为C99使隐式函数声明非法，所以如果您的编译器至少符合C99，您将得到一条诊断消息。

但有一个更强烈的理由不引入不必要的指针强制转换：

在C语言中，指针转换几乎总是一个错误。这是因为以下规则（N1570中的§6.5 p7，C11的最新草案）：

对象的存储值只能由左值表达式访问，左值表达式具有以下类型：-与对象的有效类型兼容的类型，-与对象的有效类型兼容的类型的限定版本，-一种类型，它是与对象-类型，该类型是与对象的有效类型，-一种聚合或联合类型，在其成员（递归地包括子集合或包含联合的成员），或-字符类型。

这也称为严格别名规则。因此，以下代码是未定义的行为：

long x = 5;
double *p = (double *)&x;
double y = *p;

有时令人惊讶的是，以下内容也是：

struct foo { int x; };
struct bar { int x; int y; };
struct bar b = { 1, 2};
struct foo *p = (struct foo *)&b;
int z = p->x;

有时，您确实需要强制转换指针，但考虑到严格的别名规则，您必须非常小心。因此，代码中出现的任何指针强制转换都需要重新检查其有效性。因此，您永远不会编写不必要的指针强制转换。

tl；博士

简而言之：因为在C语言中，指针强制转换的任何出现都会给需要特别注意的代码带来危险，所以您不应该编写不必要的指针强制转换。

附带说明：

在某些情况下，您实际上需要转换为void*，例如，如果您想要打印指针：int x=5；printf（“%p\n”，（void*）&x）；这里强制转换是必要的，因为printf（）是一个可变函数，所以隐式转换不起作用。在C++中，情况不同。在处理派生类的对象时，强制转换指针类型有些常见（也是正确的）。因此，在C++中，与void*的转换不是隐式的，这是有意义的。C++有一整套不同风格的铸造。

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

您不需要强制转换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年历史的编译器呢？