编码风格aginst警告检查

Question

在全新的编译警告，我遇到了下面的代码：

char *strcpy(char *dest, char *src) 
{ 
    unsigned int i; 

    while(dest[i] = src[i])//assignment in condition 
     i++; 

    return dest; 
} 

代码的基础功能应该是美好的，但编译器警告说，assignment in condition，请问这部分代码有任何潜在的风险？如果这种警告需要清理？

Answer 1

各种警告，需要清理。引入

此警告的原因=和==往往是来自程序员（你好，帕斯卡！）糊涂了，你的意图可以通过添加围绕赋值表达式的括号中明确指出：

if ((x = y)) // no warning 

Answer 2

不，你不需要解决这个问题。

警告的原因是，有时人们错误地键入=时，他们的意思是==，例如，

while (response = 'n') 

在有条件的转让是更可能是这样的错误不是分配要测试其值，所以编译器警告一下。您可以通过在测试包分配沉默警告：

while ((dest[i] = src[i]) != 0) 

在另一方面，我建议你总是把while或if的体内{}，哪怕它只是一个声明。见Why is it considered a bad practice to omit curly braces?

您还需要初始化i：

unsigned int i = 0;