リファクタリング プログラミング

メソッドを切り出すメリット。リーダブルなコードで行こう!

投稿日:

概要

大きなクラス、長いメソッドを書いたり読んだりしていて、ロジックが追えなくなってきています。
私の周りではすごく理解が良く、長い処理でも追っていける人がいますが私はそうではありません。

私はごくごく短いロジック、意図が明確な名称でないとちゃんと頭に入りません。
読み返してももう一回理解し直さなくてはならなかったりとかで、結構つらいものがあります。

そんな私が自分自身に対して、もっと短くて、もっとわかり易くて、もっと美しいコードを書きたいという思いと、周りの人にもそのようにしてほしい(自己都合)という願いで執筆しました。
 

実装例

リーダブルでないコード

Q. さて、このコードは何を行っているでしょうか?

A. エラーがある場合は全てを表示する。

簡単なコードなので見てすぐわかる人にはわかると思いますが、
見にくく(醜く)ないですか?

私は見にくいです。
それに、視覚的に頭に入ってきません。
これをリーダブルに変えます。
 

リーダブルなコード

 
<改善ポイント>

  • 各変数で正確な意味が表現されている。
  • ロジックがメソッド名で明確に表現されている。
  • ifのネストが無くなり、判定ロジックが独立するようになった。(メンテナンスしやすい!)
  • エラーを追加する列の位置を合わせたので、見やすくなっている。

 
実際、現場で自分またはレビュー対象のコードをリーダブルにしようと思ったら、これくらいの単位から行うものなんじゃないかと思う。

リーダブルであることは「文化」なので、読みにくいコードを書いているプロジェクトや会社ではそもそもリーダブルにしようとすることを多くの人が諦めているだろうと思っています。

でも諦めないでほしい!
まずは自分から、ちょっとでも読みやすいコードを書こう!
ソースコードだけじゃなく、設計書もリーダブルにしていこう!
理解しやすいとコミュニケーションロスがなくなって、仕様が明確になって、手戻りが減る!
良いこと尽くめ!!

私も今のプロジェクトで悪戦苦闘しながら、少しでもリーダブルになる努力を続けています。
 

まとめ

  • 変数やメソッド名で意図を明確に表現するとわかり易い。
  • 列を揃えると、思いのほか読みやすい!
  • ifをネストしないのは読みやすさの定番。
  • 自分の状況がどうであれ、リーダブルに行こう!

 
最後までお読み頂き、ありがとうございました!
ご意見・ご要望がありましたら、遠慮なくコメント下さい!
もし内容が良かったらランキング評価を頂けると励みになります(^^)

ランキング評価する

-リファクタリング, プログラミング
-,

執筆者:


comment

メールアドレスが公開されることはありません。

CAPTCHA


関連記事

elseを使わないことでパフォーマンスが下がる場合の対処法

  状況 前回記事(なぜelse、else ifを書かないようにすべきなのか?)のような ifのみで判定されており、それがfor文等の繰り返し処理の中にある場合は パフォーマンスが下がる。 …

【Java入門】プリミティブ型と参照型とは?

概要 Javaのデータ型にはプリミティブ型と参照型があります。 ここではそれぞれの型はどのようにデータを保持しているのか? また、どのような種類があるのか? について学んでいきます。   プリミティブ …

【Java】ArrayListの使い方まとめ

概要 ArrayListはOracleの公式ページでも明記しているクラスです。 Java SE 11 Programmer I (1Z0-815-JPN) 試験 インタフェースによる抽象化 インタフェ …

junit, テスト, junit5

【JUnit5】基本的なAssertionまとめ

概要 JUnit5における基本的なAssertionをまとめました。 検証方法 等価性 値が等価であるかを検証するメソッドです。 以下はorg.junit.jupiter.api.Assertions …

【Node.js】Excel⇔JSON変換する方法

やること 1.ExcelファイルをJSONで取得。 2.JSONの編集後、新しいExcelファイルに出力する。 環境 Node.js ・インストール  https://nodejs.org/ja/   …

SpringFramworkの良書

■おすすめ教材
Javaの基本を学んだ人が、次のステップとして読む本(中級者向け)

※SpringはJavaのFrameworkの1つです。
変更のしやすさ、保守性の高さが特徴です。