最終更新日 2025年11月18日
warning: unused variable
警告:未使用変数 ‘yagni’
[-Wunused-variable]
■この記事の概要
この記事は、C言語で未使用変数に関する警告を最適化し、コードの可読性を向上させる方法を紹介します。
特に無駄なコメント行を避ける方法に焦点を当て、必要ない変数やコードを削除する重要性についても詳述しています。
■コメントで2行増やすのを止めよう
/*削除開始*/⇦ コメントを追加しないで
// 削除int unused_variable ;⇦この行を削ろう
/*削除終了*/⇦ コメントを追加しないで/* 削除開始 */と
/*削除終了*/のコメントで
2行増やさず未使用変数の1行を削りましょう。
■未使用変数は単に削除しよう
int main(void){
int used,it,a,Long,time,ago;//昔使った
int may,use,It,someday;//いつか又使うかも
int copipe = 0 ;//コピペしたけど使わない
}不要な変数はどんどんコードから削除しましょう。
不要な物がなくなるとそれだけコードの見晴らしが良くなります。
いつか使うかもと大事に残しておく人も多いですが、
経験的に未使用変数が後で役に立つ事はありません。
この警告は不要変数を削除するだけでアルゴリズムの修正は無いので比較的簡単に修正できます。
自分で作ったプログラムでは必ず対応しましょう。
■コメントで古いコードを残すのは止めよう
int main(void){
/*静的解析ツール対策 開始
int used,it,a,Long,time,ago;//昔使った
静的解析ツール対策 終了*/
/*コンパイラ警告対策 START */
//int may,use,It,someday;//いつか又使うかも
/*コンパイラ警告対策 ENDS */
#if 0 /* 削除開始 */
int copipe = 0 ;//コピペしたけど使わない
#endif /* 削除終了 */
} 本来コンパイラや静的解析ツールの警告に従い
不要コードを削除すると
コードは綺麗になりますが、
上記のような修正をするとコードはどんどん汚れてしまいます。
これは掃除をすればするほど部屋が汚れていくようなもので
何のために掃除をしているのか分からなくなります。
不要な変数を削除しないでこのようにコードを残す修正は絶対にしてはいけません。
またこのような修正履歴コメントには、
コンパイラ警告対策 とか
静的解析ツール対策 とか
まるでゴジラを迎え撃つ自衛隊のような
”～～対策”のコメントがよくありますが、
コンパイラを敵視したり、
コンパイラと喧嘩するのは止めて
コンパイラをうまく使いましょう。
■アンケート
参考:
MSC12-C. プログラムに対して作用しないコードを検出して削除する
