Ｃ言語のよくある間違い

２．Ｃ言語のよくある間違い

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

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

参照回数

［エラー番号］エラータイトル　　となっています。
102667

[e5776]char*文字列を==で比較しては駄目
100089

[e29]ちゅうぶらりんｅｌｓｅ
97607

[e3]配列の添字が大きすぎる！
96729

[e405]初期化されていない配列(2)
93336

[e33]制御が来ません！
92762

[e12]default の書き間違い！
89467

[e21]初期値が不適切！
87988

[e38]アドレスを半分削る？
87196

[e16]＝＝で無く＝の間違い？
86584

[e177]シングルコート中の文字列
79789

[e4]シフトのしすぎ！
79412

[e6]if 文の無駄！
78316

[e100]mainはint型
74564

[e175]意味がないコーディング
73892

[e179]charは127円まで
72329

[e49]ポインタなの？整数なの？
71049

[e19]無駄な比較！
71005

[e5]caseの外には制御が来ません
69726

[e61]このファイル空ですよ！？
68127

[e36]int か void か不明
68019

[e9]ビットフィールドの設定！
67565

[e10]配列の添字が負値！
67022

[e1]Ｃ言語仕様で未定義！
66711

[e82]BUGです。if(常時真)
66297

[e93]使い道のない宣言
66220

[e67]配列のサイズがゼロです
65823

[e107]return？return 式？
65753

[e169]フォーマット対応引数はポインタ
65651

[e188]enum値がサイズオーバー
65629

[e81]疑わしいエスケープ文字
65153

[e201]関数呼出時に引数過多
65045

[e187]２重定義です
64922

[e231]printf呼出時、引数過多
64788

[e163]if(a < b < c)はダメ
64700

[e150]同名外部変数有り
64024

[e83]BUGです。if(常時偽)
63907

[e190]組込み関数型が競合
63794

[e59]enumの型が異なる
63716

[e182]％書き込み時注意
63714

[e170]構造体転送の発生
63527

[e442]memcpy 引数型違い
63466

[e56]case -N:には絶対に来ない
63242

[e305]printf呼出時引数不足
63111

[e91]＃行の後にある物は無視
63078

[e57]return(＆自動変数)はBUG
63052

[e178]プリント文の意味不明？
62987

[e109]構造体/共用体メンバ未定義
62903

[e162]caseの値が過大
62883

[e78]未定義です
62816

[e77]括弧付ける『～=0』→『～={0}』
62803

[e35]返却値がありません
62585

[e50]代入時、左辺と右辺の型が違う
62575

[e154]関数の型が不明記述
62515

[e152]引数で互換性なし
62461

[e171]条件文は常に真です
62413

[e41]符号付き拡張(1)！
62397

[e102]変更値に対するキャスト
62390

[e95]外部とstaticの２重宣言
62281

[e165]プリント文途中切れ
62279

[e54]未使用ラベル
62256

[e203]構造体／共用体記述誤り
62240

[e168]共用体初期化要素過多
62155

[e189]整数返却値のポインタ化
62121

[e103]プロトタイプとの不一致
62077

[e198]組込み関数の非関数宣言
62040

[e53]引数の定義が異常
62040

[e157]ビット演算と算術演算優先順
62033

[e68]ユニオン中の重複メンバ
61953

[e399]マクロ関数引数の数が合いません
61943

[e71]メンバ宣言の終わりに『;』なし
61928

[e73]『16進数値その物』互換性なし
61851

[e42]符号付き拡張(2)！
61732

[e306]printf呼出時引数過多
61706

[e72]アラーム文字は互換性なし
61667

[e106]列挙型タグの２重定義
61588

[e113]関数外の空宣言は不要
61510

[e304]％表記異常
61497

[e66]引数と自動変数が同名です
61484

[e234]引数の型宣言を省略
61470

[e84]このbitfield幅では比較は常に真です
61357

[e199]組込み関数の未宣言使用
61345

[e166]16進の文字エスケープ過大
61326

[e303]関数呼出時の引数の数異常
61077

[e31]then節が空行です
61026

[e85]ゼロ除算
60969

[e459]ループ停止の Break?
60938

[e241]非存在構造体タグのサイズ
60804

[e445]関数 memcpy を引数 arg = NULLで呼び出し
60792

[e512]共用体の二重定義
60391

[e418]型違い返却値 BYTE関数宣言
59968

[e313]型はint
59669

[e476]8進数定数の誤記
59574

[e437]型違い引数
59521

[e458]double 型の危険な比較
59383

[e504]無限ループ疑惑あり
59298

[e415]カウンタはint型が適切
59243

[e460]解放不要？
59061

[e409]型違い初期化
59041

[e485]返却値にゴミを返します
59018

[e471]引数にNULLポインタが渡る可能性有り
58987

[e421]論理演算子 || の右辺が(int)型です
58948

[e454]0の時だけ成り立ちます
58914

[e422]左辺が負値の可能性あり
58903

[e411]型違い演算
58847

[e493]添字はint型が適切
58680

[e419]読み条件
58586

[e488]条件式が int 型です
58539

[e402]本文が空行です
58150

[e27]スタック浪費
45969

[e2]ゴミの値を使ってはダメ
43342

[e30]せっかく作っても未使用！
40501

[e23]char 型は遅いよ
37307

[e161]char は遅い
33092

[e261]ポインタ引数を更新しても、親関数には反映されない
32634

[e262]memset( )の第1引数が NULLポインタ
31869

[e13]break 文書き忘れ！
31603

[e242]コーヒーを丼で飲む？
30225

[e40]異常時処理は大丈夫？
30221

[e277]回らないforループ
30155

[e263]memset( )の第3引数が 0
30048

[e5786]sizeof(構造体ポインタ型)は変
30014

[e258]if(変数｜定数)は常に真
29946

[e255]何度も呼ぶな
29930

[e273]ファイルメトリクス
29867

[e275]セミコロンが多い
29855

[e271]while(3)
29785

[e269]memcmpとmemcpyを間違えた
29774

[e270] !strcpy( )
29761

[e251]return後の更新は手遅れ
29741

[e253]配列の添字が負値！
29656

[e265]roopは誤字です
29642

[e256]0と&演算すると0になる
29623

[e105]配列の初期化要素過多
29556

[e259]NULLにキャストをしない事
29534

[e267]ビットの落としかた
29371

[e268]変数のアドレスは NULL にならない
29247

[e264]エルは壱と紛らわしい
29212

[e252]配列の添字が大きすぎる！
29140

[e266]！演算子は否定形のときに使うと良い
26801

[e39]０bit は立ちません
25373

[e8]常に成り立つ？
24811

[e86]初期化文字列が長すぎ
24791

[e15]＝と＝＝の間違い？
24277

[e309]短すぎる外部変数
24026

[e32]ヘッダファイルで static 宣言？
22405

[e192]返却値有り無しの混合
22223

[e5773]1バイトmemcpy
21760

[e4836]ローカル変数の参照を返してもゴミが返るだけ
21444

[e5780]sizeof(定数)
21266

[e4117]戻り値書き忘れ
20358

[e4108]1bitのintビットフィールド
20256

[e5765]NULLチェックは先に
20052

[e4938]mainの戻り値型省略(C++)
19922

[e5000]無意味なextern
19393

[e4951]mainはint宣言し、return(プログラム終了値)を記述するのが適切です
19223

[e4064]何のための宣言？
19220

[e5777]おかしいNULLチェック
19114

[e5791]sizeof(プリミティブポインタ型)は変
19032

[e4174]==では代入できません
19028

[e5783]sizeof('A')はいくつ？
19022

[e5774]符号拡張
19011

[e4128]到達しないループ
18946

[e4237]switch(定数)
18922

[e5787]上に跳ぶgoto
18876

[e149]enum を使った switch 文
18871

[e87]#undefに不正な引数
18871

[e279]変数 | 0に意味は無い
18844

[e4187]if(x = 0)
18818

[e88]マクロの再定義
18788

[e120]"/*"の中に更に"/*"
18786

[e4027]文字エスケープ過大
18762

[e4236]if(定数)
18756

[e5766]自分自身に代入しても無意味
18747

[e5772]未使用ハンドラ引数
18577

[e4069]入りきらない値
18441

[e5057]一時オブジェクトのポインタを返すな
18079

[e37]if(val & 2== 2)はBUG
18050

[e5053]ポインタ型に大きな整数を入れています
18037

[e4959]型に対して大きすぎるビットフィールド
18019

[e4533]例外ハンドラは子クラスから書く
17984

[e7]絶対に成り立たない？
17961

[e64]"？？＝％ｄ"に注意
17960

[e5672]同じ定数値を比較している
17929

[e5348]宣言を隠す宣言(2)
17899

[e5221]宣言を隠す宣言
17863

[e4460]引数が隠される
17786

[e5763]sizeof(x[N])は1要素分のサイズ
17698

[e4767]ポインタが入りきらない型に入れています
17671

[e5210]例外ハンドラの引数を隠している
17669

[e34]if 文の中がカラッポ！
16895

[e22]static を付けて関数宣言！
16439

[e28]標準ヘッダを使おう
16418

[e511]構造体の二重定義
16294

[e17]無意味な記述！
16020

[e148]case 森の石松：を忘れていませんか？
15353

[e160]ビットと関係演算優先順位
15091

[e11]－１の比較！
14841

[e467]構造体メンバ変数の初期化漏れ
14089

[e260]変数宣言と同時に初期化すると楽
13897

[e48]プロトタイプ宣言必要
13581

[e26]未使用自動変数
13391

[e497]size_t の宣言違い
13332

[e98]疑わしいタグですstruct
13242

[e25]引数を未使用！
12675

[e183]出力なしプリント文
12477

[e155]初期化に「{」必要
12438

[e51]0x80000000以上正負不明式
12412

[e55]case文が不適切
11897

[e101]不適合ポインタ型の代入
11883

[e151]関数の引数の型が不適合
11623

[e999]リンク相当チェック対象外
11529

[e14]条件文中に定数値有り！
11477

[e65]構造体、typedef名組合せ異常
11380

[e47]プロトタイプ宣言と異なる
10957

[e18]負値との比較！
10887

[e167]構造体初期化要素過多
10788

[e20]＆register不可
10651

[e307]printfの％の使い方が間違？
10566

[e46]システムマクロの値は？
10547

[e4068]unsignedで負値を扱おうとしては駄目
10490

[e60]struct/unionにメンバ名なし
10273

[e45]bit field問題
10217

[e158]算術演算＾の周辺括弧必要
10116

[e173]unsigned typeでも過大な整数
10106

[e43]結合力は(3)？
9963

[e94]初期値の型があわない
9957

[e257]定数をキャストする前に型を見直すこと
9954

[e172]計算式"+"の結果が過大
9904

[e156]初期化に「{」必要
9875

[e159]算術演算｜の周辺括弧必要
9829

[e44]結合力は(4)？
9779

[e63]関数定義／宣言の統一
9655

[e4186]符号無しの値と0を比較しています
9490

[e58]x|y==zの優先順位？
9480

[e164]16進表記と接尾子Ｕの使用
9478

[e202]整定数をunsignedで処理
9474

[e180]関数定義／宣言の統一要
9457

[e404]初期化されていない配列(1)
9452

[e195]期待する引数がこない
9413

[e76]１６進表記を検討
9362

[e117]voidの演算子使用不可
9360

[e111]スカラの初期化要素過多
9318

[e70]明示キャストかＵ接尾子要演算子
9312

[e97]関数の引数部でunion を定義
9308

[e204]最適化時、変数異常
9305

[e80]関数呼出時、引数不足
9279

[e232]組込み関数暗黙の型宣言
9252

[e96]関数の引数部で構造体を定義
9250

[e79]引数の数が合いません
9230

[e308]引数の型がまちまちです
9143

[e74]マクロ引数条件コンパイル要
9107

[e233]以前の外部宣言の型と違う
9087

[e153]同名内部変数有り
9071

[e200]型別の初期値
9057

[e52]旧形式関数宣言は無駄
9046

[e310]関数返却値は常時無視
8979

[e403]構造体のメンバ変数、初期化ミス
8929

[e69]ブロック外のタグ定義が隠蔽
8778

[e501]三項演算子と enum の型
8677

[e104]タグ定義スコープ外
8565

[e312]返却値の無い関数の返却値使用
8124

[e4430]一時オブジェクトの参照を返すな
7687

[e452]符号無しの値と定数の比較(1)
7628

[e494]NULLポインタへの代入
7477

[e473]型違いループ内比較
7395

[e453]符号無しの値と定数の比較(2)
7385

[e143]記憶クラス指定子の記述場所
7283

[e456]符号無しの値と定数の比較(3)
7050

[e699]コンパイルエラー相当です
6635

[e272]関数メトリクス
6570

[e505]型違い代入
6511

[e420]書き条件
6385

[e5788]ポインタ引数のポインタを進めるか中身を増やすか
6224

[e89]SJISコメント問題
6138

[e413]論理結合演算子には boolean 値(2)
5995

[e5782]memcpyの第三引数にsizeof(配列でも構造体でも無いただの整数型)
5841

[e274]深すぎるネストは良くない
5704

[e4185]実行されない初期化
5583

[e5779]毎回初期化される大きな配列
5569

[e5764]境界チェックは先に
5451

[e276]回らないwhileループ
5450

[e5784]ビットフィールド範囲外の値との==,!=比較
5412

[e412]論理結合演算子には boolean 値(1)
5399

[e400]正常時のみ考慮された変数
5277

[e4175]配列の添字が大きすぎる！
5227

[e472]領域が確保されていない引数使用
5212

[e5789]x == 1 && x == 2はあり得ない
5199

[e4284]参照型でNULLを扱おうとするな
5175

[e5790]代入先より大きいビット数左ビットシフトしている
5174

[e5762]ポインタ型の引数に代入しても無意味
5173

[e5674]結合力は(4)？
5146

[e4514]符号無しの値と負値を比較しています
5143

[e4611]オーバーライドのし忘れ？
5130

[e5673]if(val & 2 == 2)はBUG
5118

[e5785]セキュリティ上の問題があるファイル名生成関数の使用
5096

[e434]文字列に\0が入りません
5088

[e4368]コンストラクタで初期化しないと意味が無いメンバ
5081

[e4940]戻り値書き忘れ(2)
5050

[e5691]caseの値は入りきりません(ビットフィールド)

Review-Cのホームページへ