プロジェクト内の悪いコードの匂いを探す (名前付け)_PHP チュートリアル

はじめに

この期間中、私はプロジェクトに取り組んできたので、比較的忙しかったのですが、今日はようやく自分が書いたコードをじっくり見て、どこに問題があるのか​​を確認する時間ができました。 「悪臭」があります。

時間をかけてゆっくりとコードを思い出してください。悪臭をキャッチし始めます。

よくすること、考えること

Obscure if 条件

1) ||

上記のコードは、次のコードよりも理解するのに時間がかかります

コードの行数を減らすことは良い目標ですが、コードを理解するのに必要な時間はさらに良い目標です。 戻り値

498)this. style="max-width:90%" onmousewheel="javascript: return big(this)" alt="" src="http://www.bkjia.com/uploadfile/%202013/0904/20130904095421511.png">

上記の「$ret」は「名前が思いつかない」という言い訳です。このような空の名前を使用するのではなく、エンティティの価値や目的を説明する名前を選択してください。

$alias は、この変数がエイリアスを運ぶために使用されることを宣言し、この変数の目的を示します。欠陥の発見に役立つかもしれません

適切な名前は、変数の目的または変数が持つ値を説明する必要があります。 一時変数

498)this. style="max-width:90%" onmousewheel="javascript: return big(this)" alt="" src="http://www.bkjia.com/uploadfile/%202013/0904/20130904095422838.png">

ここでの $i は「一時変数」で、統計データを自動的に増やし、統計ポイントの重複を避けるために特別に使用されます。しかし、$i について最も重要なことは一時変数ではありません。 charset_index を使用して「統計データの添字」を表すと、より「説明的」になります。

ループイテレーター

私も私のjsコードで悪臭を発見しました。

i 変数名は非常に一般的なので、使用しないでください。

i、j、iter、it などの名前は、インデックスやループ反復子としてよく使用されます。名前は曖昧ですが、「私は反復者です」という意味であることは誰もが知っています。 - 実際、これらの名前を別の意味で使用すると、混乱を招く可能性があります。だからやめてください。

i、j、it のようなあいまいな名前を使用することに固執する場合は、自分を納得させる十分な理由が必要です。

まとめ

コーディングの過程で、さらに数秒かけて適切な名前を考え出すと、「ネーミング能力」がすぐに向上することがわかります。

私は通常、中国語の名前を最初に考えます。対応する英語の名前がどうしても思いつかない場合は、翻訳ツールを使用して思いついた中国語の名前を貼り付けてから、名前付きの変数または関数の名前を切り取ります。

最近見た素敵な名前をプレゼントしてください

http://www.bkjia.com/PHPjc/445677.htmlwww.bkjia.comtru​​ehttp://www.bkjia.com/PHPjc/445677.html技術記事はじめに: この期間中、私はプロジェクトに取り組んでいたので、比較的忙しかったです。今日、ようやく時間ができて、自分が書いたコードをじっくり見て、どこに問題があるのか​​、どこに悪臭があるのか​​を確認します。 ゆっくりしてください...