2.C言語のよくある間違い


C言語のよくある間違い : Review-Cで検出するエラーの解説です。
Review-C に付属のサンプルソースをコンパイルして実際に実行して確認することができます。



エラー番号順サンプルソース参照回数順  を選択変更可能です。

参照回数[エラー番号]エラータイトル  となっています。
66989 [e1]C言語仕様で未定義!
45910 [e2]ゴミの値を使ってはダメ
97583 [e3]配列の添字が大きすぎる!
79770 [e4]シフトのしすぎ!
70983 [e5]caseの外には制御が来ません
79389 [e6]if 文の無駄!
17968 [e7]絶対に成り立たない?
25358 [e8]常に成り立つ?
67999 [e9]ビットフィールドの設定!
67539 [e10]配列の添字が負値!
15079 [e11]-1の比較!
92739 [e12]default の書き間違い!
31851 [e13]break 文書き忘れ!
11514 [e14]条件文中に定数値有り!
24773 [e15]= と == の間違い?
87173 [e16]== で無く = の間違い?
16278 [e17]無意味な記述!
10938 [e18]負値との比較!
71031 [e19]無駄な比較!
10772 [e20]&register不可
89446 [e21]初期値が不適切!
16868 [e22]static を付けて関数宣言!
40462 [e23]char 型は遅いよ
13228 [e25]引数を未使用!
13549 [e26]未使用自動変数
58134 [e27]スタック浪費
16425 [e28]標準ヘッダを使おう
100067 [e29]ちゅうぶらりんelse
43098 [e30]せっかく作っても未使用!
61066 [e31]then節が空行です
23968 [e32]ヘッダファイルで static 宣言?
93303 [e33]制御が来ません!
17624 [e34]if 文の中がカラッポ!
62787 [e35]返却値がありません
68107 [e36]int か void か不明
18047 [e37]if(val & 2== 2)はBUG
87967 [e38]アドレスを半分削る?
26776 [e39]0bit は立ちません
30211 [e40]異常時処理は大丈夫?
62380 [e41]符号付き拡張(1)!
61837 [e42]符号付き拡張(2)!
10084 [e43]結合力は(3)?
9808 [e44]結合力は(4)?
10258 [e45]bit field問題
10553 [e46]システムマクロの値は?
11364 [e47]プロトタイプ宣言と異なる
13880 [e48]プロトタイプ宣言必要
72311 [e49]ポインタなの?整数なの?
62567 [e50]代入時、左辺と右辺の型が違う
12424 [e51]0x80000000以上正負不明式
9018 [e52]旧形式関数宣言は無駄
62024 [e53]引数の定義が異常
62266 [e54]未使用ラベル
12379 [e55]case文が不適切
63451 [e56]case -N:には絶対に来ない
63065 [e57]return(&自動変数)はBUG
9468 [e58]x|y==zの優先順位?
63779 [e59]enumの型が異なる
10475 [e60]struct/unionにメンバ名なし
69698 [e61]このファイル空ですよ!?
9762 [e63]関数定義/宣言の統一
17921 [e64]"??=%d"に注意
11463 [e65]構造体、typedef名組合せ異常
61483 [e66]引数と自動変数が同名です
66201 [e67]配列のサイズがゼロです
62016 [e68]ユニオン中の重複メンバ
8891 [e69]ブロック外のタグ定義が隠蔽
9306 [e70]明示キャストかU接尾子要演算子
61931 [e71]メンバ宣言の終わりに『;』なし
61692 [e72]アラーム文字は互換性なし
61911 [e73]『16進数値その物』互換性なし
9132 [e74]マクロ引数条件コンパイル要
9399 [e76]16進表記を検討
62775 [e77]括弧付ける『~=0』→『~={0}』
62868 [e78]未定義です
9235 [e79]引数の数が合いません
9292 [e80]関数呼出時、引数不足
65612 [e81]疑わしいエスケープ文字
66693 [e82]BUGです。if(常時真)
64009 [e83]BUGです。if(常時偽)
61456 [e84]このbitfield幅では比較は常に真です
61010 [e85]ゼロ除算
24778 [e86]初期化文字列が長すぎ
18859 [e87]#undefに不正な引数
18662 [e88]マクロの再定義
6210 [e89]SJISコメント問題
63097 [e91]#行の後にある物は無視
66279 [e93]使い道のない宣言
9944 [e94]初期値の型があわない
62375 [e95]外部とstaticの2重宣言
9241 [e96]関数の引数部で構造体を定義
9300 [e97]関数の引数部でunion を定義
13308 [e98]疑わしいタグですstruct
78290 [e100]mainはint型
11876 [e101]不適合ポインタ型の代入
62383 [e102]変更値に対するキャスト
62110 [e103]プロトタイプとの不一致
8663 [e104]タグ定義スコープ外
29609 [e105]配列の初期化要素過多
61654 [e106]列挙型タグの2重定義
65802 [e107]return?return 式?
62972 [e109]構造体/共用体メンバ未定義
9347 [e111]スカラの初期化要素過多
61576 [e113]関数外の空宣言は不要
9338 [e117]voidの演算子使用不可
18776 [e120]"/*"の中に更に"/*"
7369 [e143]記憶クラス指定子の記述場所
16004 [e148]case 森の石松:を忘れていませんか?
18713 [e149]enum を使った switch 文
64683 [e150]同名外部変数有り
11870 [e151]関数の引数の型が不適合
62499 [e152]引数で互換性なし
9073 [e153]同名内部変数有り
62552 [e154]関数の型が不明記述
12463 [e155]初期化に「{」必要
9893 [e156]初期化に「{」必要
62027 [e157]ビット演算と算術演算優先順
10194 [e158]算術演算^の周辺括弧必要
9850 [e159]算術演算|の周辺括弧必要
15330 [e160]ビットと関係演算優先順位
37279 [e161]char は遅い
62890 [e162]caseの値が過大
64768 [e163]if(a < b < c)はダメ
9467 [e164]16進表記と接尾子Uの使用
62263 [e165]プリント文途中切れ
61328 [e166]16進の文字エスケープ過大
10873 [e167]構造体初期化要素過多
62220 [e168]共用体初期化要素過多
65739 [e169]フォーマット対応引数はポインタ
63698 [e170]構造体転送の発生
62448 [e171]条件文は常に真です
9935 [e172]計算式"+"の結果が過大
10080 [e173]unsigned typeでも過大な整数
74543 [e175]意味がないコーディング
86569 [e177]シングルコート中の文字列
63037 [e178]プリント文の意味不明?
73873 [e179]charは127円まで
9457 [e180]関数定義/宣言の統一要
63697 [e182]%書き込み時注意
12646 [e183]出力なしプリント文
65021 [e187]2重定義です
65636 [e188]enum値がサイズオーバー
62143 [e189]整数返却値のポインタ化
63887 [e190]組込み関数型が競合
22358 [e192]返却値有り無しの混合
9416 [e195]期待する引数がこない
62063 [e198]組込み関数の非関数宣言
61325 [e199]組込み関数の未宣言使用
9059 [e200]型別の初期値
65137 [e201]関数呼出時に引数過多
9465 [e202]整定数をunsignedで処理
62238 [e203]構造体/共用体記述誤り
9287 [e204]最適化時、変数異常
64902 [e231]printf呼出時、引数過多
9261 [e232]組込み関数暗黙の型宣言
9091 [e233]以前の外部宣言の型と違う
61471 [e234]引数の型宣言を省略
60925 [e241]非存在構造体タグのサイズ
31583 [e242]コーヒーを丼で飲む?
29741 [e251]return後の更新は手遅れ
29201 [e252]配列の添字が大きすぎる!
29724 [e253]配列の添字が負値!
29928 [e255]何度も呼ぶな
29627 [e256]0と&演算すると0になる
9920 [e257]定数をキャストする前に型を見直すこと
29998 [e258]if(変数|定数)は常に真
29541 [e259]NULLにキャストをしない事
13946 [e260]変数宣言と同時に初期化すると楽
33068 [e261]ポインタ引数を更新しても、親関数には反映されない
32602 [e262]memset( )の第1引数が NULLポインタ
30142 [e263]memset( )の第3引数が 0
29234 [e264]エルは壱と紛らわしい
29645 [e265]roopは誤字です
29124 [e266]!演算子は否定形のときに使うと良い
29521 [e267]ビットの落としかた
29359 [e268]変数のアドレスは NULL にならない
29773 [e269]memcmpとmemcpyを間違えた
29761 [e270] !strcpy( )
29841 [e271]while(3)
6606 [e272]関数メトリクス
29916 [e273]ファイルメトリクス
5825 [e274]深すぎるネストは良くない
29852 [e275]セミコロンが多い
5431 [e276]回らないwhileループ
30204 [e277]回らないforループ
18862 [e279]変数 | 0に意味は無い
61313 [e303]関数呼出時の引数の数異常
61492 [e304]%表記異常
63226 [e305]printf呼出時引数不足
61716 [e306]printf呼出時引数過多
10632 [e307]printfの%の使い方が間違?
9219 [e308]引数の型がまちまちです
24264 [e309]短すぎる外部変数
9013 [e310]関数返却値は常時無視
8550 [e312]返却値の無い関数の返却値使用
59956 [e313]型はint
61939 [e399]マクロ関数引数の数が合いません
5371 [e400]正常時のみ考慮された変数
58527 [e402]本文が空行です
8946 [e403]構造体のメンバ変数、初期化ミス
9433 [e404]初期化されていない配列(1)
96520 [e405]初期化されていない配列(2)
59048 [e409]型違い初期化
58890 [e411]型違い演算
5402 [e412]論理結合演算子には boolean 値(1)
6119 [e413]論理結合演算子には boolean 値(2)
59276 [e415]カウンタはint型が適切
60373 [e418]型違い返却値 BYTE関数宣言
58666 [e419]読み条件
6500 [e420]書き条件
58973 [e421]論理演算子 || の右辺が(int)型です
58902 [e422]左辺が負値の可能性あり
5080 [e434]文字列に\0が入りません
59560 [e437]型違い引数
63513 [e442]memcpy 引数型違い
60780 [e445]関数 memcpy を引数 arg = NULLで呼び出し
7673 [e452]符号無しの値と定数の比較(1)
7382 [e453]符号無しの値と定数の比較(2)
58933 [e454]0の時だけ成り立ちます
7269 [e456]符号無しの値と定数の比較(3)
59506 [e458]double 型の危険な比較
60955 [e459]ループ停止の Break?
59228 [e460]解放不要?
14779 [e467]構造体メンバ変数の初期化漏れ
59001 [e471]引数にNULLポインタが渡る可能性有り
5215 [e472]領域が確保されていない引数使用
7461 [e473]型違いループ内比較
59654 [e476]8進数定数の誤記
59018 [e485]返却値にゴミを返します
58570 [e488]条件式が int 型です
58827 [e493]添字はint型が適切
7614 [e494]NULLポインタへの代入
13346 [e497]size_t の宣言違い
8760 [e501]三項演算子と enum の型
59370 [e504]無限ループ疑惑あり
6556 [e505]型違い代入
16372 [e511]構造体の二重定義
60777 [e512]共用体の二重定義
7037 [e699]コンパイルエラー相当です
11607 [e999]リンク相当チェック対象外
18771 [e4027]文字エスケープ過大
19210 [e4064]何のための宣言?
10529 [e4068]unsignedで負値を扱おうとしては駄目
18563 [e4069]入りきらない値
20342 [e4108]1bitのintビットフィールド
21243 [e4117]戻り値書き忘れ
18971 [e4128]到達しないループ
19016 [e4174]==では代入できません
5254 [e4175]配列の添字が大きすぎる!
5691 [e4185]実行されない初期化
9620 [e4186]符号無しの値と0を比較しています
18828 [e4187]if(x = 0)
18726 [e4236]if(定数)
18926 [e4237]switch(定数)
5163 [e4284]参照型でNULLを扱おうとするな
5050 [e4368]コンストラクタで初期化しないと意味が無いメンバ
8102 [e4430]一時オブジェクトの参照を返すな
17851 [e4460]引数が隠される
5131 [e4514]符号無しの値と負値を比較しています
17989 [e4533]例外ハンドラは子クラスから書く
5129 [e4611]オーバーライドのし忘れ?
17663 [e4767]ポインタが入りきらない型に入れています
21709 [e4836]ローカル変数の参照を返してもゴミが返るだけ
20016 [e4938]mainの戻り値型省略(C++)
5046 [e4940]戻り値書き忘れ(2)
19379 [e4951]mainはint宣言し、return(プログラム終了値)を記述するのが適切です
18022 [e4959]型に対して大きすぎるビットフィールド
19909 [e5000]無意味なextern
18037 [e5053]ポインタ型に大きな整数を入れています
18426 [e5057]一時オブジェクトのポインタを返すな
17660 [e5210]例外ハンドラの引数を隠している
17886 [e5221]宣言を隠す宣言
17914 [e5348]宣言を隠す宣言(2)
17941 [e5672]同じ定数値を比較している
5115 [e5673]if(val & 2 == 2)はBUG
5159 [e5674]結合力は(4)?
5036 [e5691]caseの値は入りきりません(ビットフィールド)
5160 [e5762]ポインタ型の引数に代入しても無意味
17774 [e5763]sizeof(x[N])は1要素分のサイズ
5552 [e5764]境界チェックは先に
20220 [e5765]NULLチェックは先に
18738 [e5766]自分自身に代入しても無意味
18712 [e5772]未使用ハンドラ引数
22193 [e5773]1バイトmemcpy
19006 [e5774]符号拡張
102667 [e5776]char*文字列を==で比較しては駄目
19182 [e5777]おかしいNULLチェック
5568 [e5779]毎回初期化される大きな配列
21424 [e5780]sizeof(定数)
5982 [e5782]memcpyの第三引数にsizeof(配列でも構造体でも無いただの整数型)
18990 [e5783]sizeof('A')はいくつ?
5417 [e5784]ビットフィールド範囲外の値との==,!=比較
5104 [e5785]セキュリティ上の問題があるファイル名生成関数の使用
29879 [e5786]sizeof(構造体ポインタ型)は変
18909 [e5787]上に跳ぶgoto
6340 [e5788]ポインタ引数のポインタを進めるか中身を増やすか
5177 [e5789]x == 1 && x == 2はあり得ない
5166 [e5790]代入先より大きいビット数左ビットシフトしている
19102 [e5791]sizeof(プリミティブポインタ型)は変



Review-Cのホームページへ