warning: suggest parentheses around assignment used as truth value 警告:真偽値として使用するなら代入式にカッコつける[-Wparentheses]
■これがヨーダ記法 if(ret == ERROR) //音読できるif(ERROR == ret) //音読できない⇦これがヨーダ記法■代入【=】と比較【==】の書き間違い int f(int x)
{
if(x = 0) { //最近のコンパイラは警告してくれる
return 0 ;
}
return 1;
}C言語の代表的な間違いに == と = の書き間違いがあります。
if(NOT_ERROR != ret) //音読できますか？やはり品質のためなら読めないコードを量産しても良いという考えは時代遅れではないでしょうか？
■音読できない【ヨーダ記法】 int f(int x)
{
//読みづらい「ヨーダ記法」は時代遅れとなった
if(0 == x){
return 0;
}
return 1;
}現代では不自然なヨーダ記法を使うのではなく -Werror のようにコンパイラの警告オプションを使ってケアレスミスを刈り取る手法が主流となりつつあります。
とはいえ下記に示すようにコンパイラが全ての間違いを見つけてくれるわけではないので注意が必要です。
■意図的か間違いか判断できない例 static int func()
{
return 1;
}
int f(void)
{
int ret ;
if(ret = func()){ //warning: suggest parentheses around assignment used as truth value [-Wparentheses]
return 1;
} else {
return 0;
}
}
■意図的な場合の警告抑止方法 int f(void)
{
int ret ;
if((ret = func())){ //代入式を丸かっこで囲むと警告されなくなる
return 1;
} else {
return 0;
}
}代入式を丸カッコで囲むと
■gccでも間違いを指摘してくれない例 int f(int x,int y)
{
//でも、丸かっこで囲まれた(y = 1)は
//意図的みなしgccは警告しないので注意
if((x == 0) && (y = 1)){
return 0 ;
}
return 1;
}
第二条件の代入は丸カッコで囲まれているので意図的と見なされ
参考:
■アンケート
