泥棒のテクニックから学ぶコード修正
コード修正のベストプラクティス(と言うより TIPS)
TL; DR (今北産業)
- 泥棒がタンスを開ける時、時短のため下の段から上へ開けていく。(らしい)
- コード修正もエラー解析結果の下から順に修正していく。(と良い)
- 下から修正して行けば、それより上のコードの行番号はズレないから。
例えば、下記のシェル・スクリプトの静的解析の結果を修正していく場合、先に 324 行目から潰して(修正して)いけば 312 行目のエラー箇所がズレない。
$ shellcheck -x -s sh */*.sh
In ./tests/run-tests.sh line 312:
php --version | grep Xdebug 2>/dev/null 1>/dev/null && {
^-- SC2015: Note that A && B || C is not if-then-else. C may run when A is true.
In ./tests/run-tests.sh line 324:
. "$path_file_conf_coveralls"
^-------------------------^ SC1091: Not following: ../tests/conf/COVERALLS.env: openBinaryFile: does not exist (No such file or directory)
TS; DR
静的解析、ユニット・テストや Linter などの結果を見ながら修正する際に、いまさらながら「泥棒のテクニック」が使えることに気付きました。
泥棒ってタンスをチェックする時に下の段から開けて行くんですって。ゲーム実況でアイテム探しをしているシーンを観て知りました。
同じように、解析結果で指摘されたコード行を「下の段から修正していく」ことで効率よく修正できるんですね。(同じファイルに複数の修正箇所がある場合に限る)
今まで、アホみたいに上から修正していました。修正するたびに、それ以降の対象行がズレていくものだから、迷子になるたびに解析を走らせてたのです。そう、タンスを上から開けては閉めて、次の段をチェックするがごとく。いやはや。
コーディングが早い人って、こう言う細かいことがサラッとできてるんだろうなぁ。
Author And Source
この問題について(泥棒のテクニックから学ぶコード修正), 我々は、より多くの情報をここで見つけました https://qiita.com/KEINOS/items/2c7988bfaae8c38a74dc著者帰属:元の著者の情報は、元のURLに含まれています。著作権は原作者に属する。
Content is automatically searched and collected through network algorithms . If there is a violation . Please contact us . We will adjust (correct author information ,or delete content ) as soon as possible .