|
|
C言語のよくある間違い : Review-Cで検出するエラーの解説です。 Review-C に付属のサンプルソースをコンパイルして実際に実行して確認することができます。
エラー番号順/サンプルソース参照回数順 を選択変更可能です。参照回数 [エラー番号]エラータイトル となっています。91110 [e29]ちゅうぶらりんelse88149 [e3]配列の添字が大きすぎる!85062 [e33]制御が来ません!84508 [e12]default の書き間違い!81353 [e21]初期値が不適切!80951 [e38]アドレスを半分削る?79807 [e177]シングルコート中の文字列79321 [e16]== で無く = の間違い?73522 [e4]シフトのしすぎ!72970 [e6]if 文の無駄!72300 [e100]mainはint型69361 [e175]意味がないコーディング68432 [e179]charは127円まで66722 [e49]ポインタなの?整数なの?66119 [e5]caseの外には制御が来ません66096 [e19]無駄な比較!64281 [e61]このファイル空ですよ!?63001 [e36]int か void か不明62818 [e9]ビットフィールドの設定!62041 [e10]配列の添字が負値!61881 [e82]BUGです。if(常時真)61682 [e169]フォーマット対応引数はポインタ61414 [e93]使い道のない宣言61296 [e81]疑わしいエスケープ文字61142 [e67]配列のサイズがゼロです61104 [e107]return?return 式?60757 [e201]関数呼出時に引数過多60638 [e150]同名外部変数有り60541 [e188]enum値がサイズオーバー60487 [e231]printf呼出時、引数過多60164 [e187]2重定義です59968 [e163]if(a < b < c)はダメ59539 [e56]case -N:には絶対に来ない59474 [e83]BUGです。if(常時偽)59380 [e59]enumの型が異なる59226 [e170]構造体転送の発生58935 [e35]返却値がありません58876 [e91]#行の後にある物は無視58833 [e77]括弧付ける『~=0』→『~={0}』58803 [e78]未定義です58671 [e305]printf呼出時引数不足58631 [e162]caseの値が過大58608 [e178]プリント文の意味不明?58570 [e57]return(&自動変数)はBUG58402 [e165]プリント文途中切れ58367 [e182]%書き込み時注意58353 [e171]条件文は常に真です58321 [e95]外部とstaticの2重宣言58317 [e41]符号付き拡張(1)!58239 [e54]未使用ラベル58210 [e399]マクロ関数引数の数が合いません58175 [e68]ユニオン中の重複メンバ58146 [e203]構造体/共用体記述誤り58122 [e50]代入時、左辺と右辺の型が違う58102 [e103]プロトタイプとの不一致58088 [e71]メンバ宣言の終わりに『;』なし58025 [e102]変更値に対するキャスト57899 [e84]このbitfield幅では比較は常に真です57899 [e154]関数の型が不明記述57870 [e42]符号付き拡張(2)!57867 [e157]ビット演算と算術演算優先順57865 [e72]アラーム文字は互換性なし57859 [e53]引数の定義が異常57846 [e109]構造体/共用体メンバ未定義57820 [e73]『16進数値その物』互換性なし57797 [e304]%表記異常57784 [e152]引数で互換性なし57774 [e190]組込み関数型が競合57750 [e168]共用体初期化要素過多57745 [e198]組込み関数の非関数宣言57726 [e189]整数返却値のポインタ化57715 [e66]引数と自動変数が同名です57701 [e1]C言語仕様で未定義!57692 [e306]printf呼出時引数過多57682 [e106]列挙型タグの2重定義57675 [e113]関数外の空宣言は不要57630 [e199]組込み関数の未宣言使用57625 [e166]16進の文字エスケープ過大57573 [e234]引数の型宣言を省略57554 [e303]関数呼出時の引数の数異常57471 [e241]非存在構造体タグのサイズ57303 [e31]then節が空行です57250 [e85]ゼロ除算57191 [e442]memcpy 引数型違い57118 [e459]ループ停止の Break?56755 [e418]型違い返却値 BYTE関数宣言56431 [e512]共用体の二重定義56208 [e476]8進数定数の誤記56202 [e313]型はint55621 [e504]無限ループ疑惑あり55212 [e458]double 型の危険な比較55209 [e415]カウンタはint型が適切55185 [e460]解放不要?55055 [e471]引数にNULLポインタが渡る可能性有り55043 [e493]添字はint型が適切55030 [e485]返却値にゴミを返します55020 [e437]型違い引数55012 [e445]関数 memcpy を引数 arg = NULLで呼び出し55009 [e454]0の時だけ成り立ちます54991 [e422]左辺が負値の可能性あり54978 [e488]条件式が int 型です54957 [e421]論理演算子 || の右辺が(int)型です54955 [e409]型違い初期化54954 [e411]型違い演算54931 [e419]読み条件54922 [e402]本文が空行です51729 [e27]スタック浪費38000 [e2]ゴミの値を使ってはダメ31321 [e23]char 型は遅いよ30814 [e161]char は遅い28841 [e30]せっかく作っても未使用!26361 [e242]コーヒーを丼で飲む?25772 [e262]memset( )の第1引数が NULLポインタ25651 [e255]何度も呼ぶな25624 [e273]ファイルメトリクス25623 [e253]配列の添字が負値!25609 [e277]回らないforループ25601 [e259]NULLにキャストをしない事25599 [e264]エルは壱と紛らわしい25593 [e275]セミコロンが多い25585 [e271]while(3)25582 [e251]return後の更新は手遅れ25572 [e263]memset( )の第3引数が 025571 [e261]ポインタ引数を更新しても、親関数には反映されない25563 [e265]roopは誤字です25561 [e270] !strcpy( )25559 [e267]ビットの落としかた25552 [e258]if(変数|定数)は常に真25543 [e252]配列の添字が大きすぎる!25542 [e256]0と&演算すると0になる25530 [e269]memcmpとmemcpyを間違えた25526 [e266]!演算子は否定形のときに使うと良い25510 [e268]変数のアドレスは NULL にならない24598 [e40]異常時処理は大丈夫?20939 [e39]0bit は立ちません20428 [e13]break 文書き忘れ!19955 [e8]常に成り立つ?19210 [e15]= と == の間違い?18986 [e309]短すぎる外部変数16698 [e192]返却値有り無しの混合15066 [e5773]1バイトmemcpy15054 [e4174]==では代入できません15045 [e5777]おかしいNULLチェック15024 [e5765]NULLチェックは先に14998 [e5791]sizeof(プリミティブポインタ型)は変14997 [e5783]sizeof('A')はいくつ?14996 [e4108]1bitのintビットフィールド14993 [e5776]char*文字列を==で比較しては駄目14991 [e86]初期化文字列が長すぎ14987 [e5766]自分自身に代入しても無意味14984 [e5787]上に跳ぶgoto14982 [e5774]符号拡張14981 [e4069]入りきらない値14978 [e4237]switch(定数)14976 [e4128]到達しないループ14969 [e4187]if(x = 0)14962 [e5772]未使用ハンドラ引数14957 [e5780]sizeof(定数)14955 [e87]#undefに不正な引数14951 [e4236]if(定数)14948 [e4117]戻り値書き忘れ14945 [e4064]何のための宣言?14942 [e120]"/*"の中に更に"/*"14933 [e4027]文字エスケープ過大14920 [e279]変数 | 0に意味は無い14361 [e37]if(val & 2== 2)はBUG14319 [e4938]mainの戻り値型省略(C++)14297 [e4951]mainはint宣言し、return(プログラム終了値)を記述するのが適切です14277 [e5000]無意味なextern14260 [e5053]ポインタ型に大きな整数を入れています14253 [e5221]宣言を隠す宣言14251 [e5057]一時オブジェクトのポインタを返すな14249 [e4959]型に対して大きすぎるビットフィールド14240 [e5672]同じ定数値を比較している14238 [e5763]sizeof(x[N])は1要素分のサイズ14238 [e5210]例外ハンドラの引数を隠している14226 [e5348]宣言を隠す宣言(2)14218 [e4836]ローカル変数の参照を返してもゴミが返るだけ14211 [e4767]ポインタが入りきらない型に入れています14198 [e4460]引数が隠される13652 [e7]絶対に成り立たない?13503 [e4533]例外ハンドラは子クラスから書く13356 [e405]初期化されていない配列(2)12722 [e22]static を付けて関数宣言!12679 [e28]標準ヘッダを使おう12630 [e34]if 文の中がカラッポ!12206 [e148]case 森の石松:を忘れていませんか?12154 [e17]無意味な記述!12084 [e64]"??=%d"に注意11206 [e11]-1の比較!10233 [e32]ヘッダファイルで static 宣言?9558 [e26]未使用自動変数9215 [e48]プロトタイプ宣言必要9104 [e25]引数を未使用!8403 [e51]0x80000000以上正負不明式8112 [e151]関数の引数の型が不適合8035 [e155]初期化に「{」必要7984 [e999]リンク相当チェック対象外7643 [e14]条件文中に定数値有り!7594 [e98]疑わしいタグですstruct7560 [e65]構造体、typedef名組合せ異常7352 [e183]出力なしプリント文7291 [e101]不適合ポインタ型の代入7028 [e20]&register不可7021 [e18]負値との比較!6890 [e47]プロトタイプ宣言と異なる6821 [e307]printfの%の使い方が間違?6788 [e55]case文が不適切6418 [e46]システムマクロの値は?6266 [e105]配列の初期化要素過多6202 [e44]結合力は(4)?6190 [e60]struct/unionにメンバ名なし6120 [e156]初期化に「{」必要6060 [e43]結合力は(3)?6037 [e63]関数定義/宣言の統一5967 [e45]bit field問題5935 [e76]16進表記を検討5916 [e58]x|y==zの優先順位?5909 [e167]構造体初期化要素過多5848 [e173]unsigned typeでも過大な整数5843 [e159]算術演算|の周辺括弧必要5795 [e111]スカラの初期化要素過多5791 [e172]計算式"+"の結果が過大5764 [e195]期待する引数がこない5759 [e94]初期値の型があわない5682 [e117]voidの演算子使用不可5630 [e497]size_t の宣言違い5615 [e74]マクロ引数条件コンパイル要5591 [e308]引数の型がまちまちです5585 [e79]引数の数が合いません5561 [e96]関数の引数部で構造体を定義5528 [e233]以前の外部宣言の型と違う5482 [e232]組込み関数暗黙の型宣言5475 [e70]明示キャストかU接尾子要演算子5432 [e202]整定数をunsignedで処理5416 [e69]ブロック外のタグ定義が隠蔽5409 [e204]最適化時、変数異常5405 [e200]型別の初期値5379 [e180]関数定義/宣言の統一要5367 [e52]旧形式関数宣言は無駄5362 [e80]関数呼出時、引数不足5346 [e160]ビットと関係演算優先順位5338 [e158]算術演算^の周辺括弧必要5282 [e97]関数の引数部でunion を定義5259 [e310]関数返却値は常時無視5247 [e164]16進表記と接尾子Uの使用5197 [e153]同名内部変数有り5181 [e104]タグ定義スコープ外5103 [e312]返却値の無い関数の返却値使用4522 [e511]構造体の二重定義4511 [e149]enum を使った switch 文4365 [e467]構造体メンバ変数の初期化漏れ4107 [e452]符号無しの値と定数の比較(1)4080 [e501]三項演算子と enum の型3924 [e453]符号無しの値と定数の比較(2)3897 [e494]NULLポインタへの代入3894 [e473]型違いループ内比較3864 [e456]符号無しの値と定数の比較(3)3736 [e143]記憶クラス指定子の記述場所3345 [e699]コンパイルエラー相当です3088 [e404]初期化されていない配列(1)2860 [e420]書き条件2817 [e403]構造体のメンバ変数、初期化ミス2796 [e505]型違い代入1716 [e400]正常時のみ考慮された変数1696 [e412]論理結合演算子には boolean 値(1)1687 [e413]論理結合演算子には boolean 値(2)1585 [e472]領域が確保されていない引数使用1517 [e274]深すぎるネストは良くない1458 [e260]変数宣言と同時に初期化すると楽1457 [e5786]sizeof(構造体ポインタ型)は変1411 [e272]関数メトリクス1401 [e276]回らないwhileループ1391 [e257]定数をキャストする前に型を見直すこと1339 [e4186]符号無しの値と0を比較しています1328 [e4430]一時オブジェクトの参照を返すな1318 [e89]SJISコメント問題1313 [e5782]memcpyの第三引数にsizeof(配列でも構造体でも無いただの整数型)1311 [e5673]if(val & 2 == 2)はBUG1306 [e5674]結合力は(4)?1286 [e434]文字列に\0が入りません1285 [e5779]毎回初期化される大きな配列1285 [e4284]参照型でNULLを扱おうとするな1284 [e5784]ビットフィールド範囲外の値との==,!=比較1284 [e5764]境界チェックは先に1283 [e4185]実行されない初期化1282 [e5789]x == 1 && x == 2はあり得ない1281 [e5762]ポインタ型の引数に代入しても無意味1281 [e4611]オーバーライドのし忘れ?1279 [e88]マクロの再定義1279 [e4368]コンストラクタで初期化しないと意味が無いメンバ1278 [e5788]ポインタ引数のポインタを進めるか中身を増やすか1276 [e4940]戻り値書き忘れ(2)1275 [e5785]セキュリティ上の問題があるファイル名生成関数の使用1275 [e4175]配列の添字が大きすぎる!1275 [e4068]unsignedで負値を扱おうとしては駄目1274 [e4514]符号無しの値と負値を比較しています1273 [e5691]caseの値は入りきりません(ビットフィールド)1266 [e5790]代入先より大きいビット数左ビットシフトしている |