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

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

投稿日:

概要

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

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

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

実装例

リーダブルでないコード

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

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

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

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

リーダブルなコード

 
<改善ポイント>

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

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

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

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

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

まとめ

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

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

ランキング評価する

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

執筆者:


comment

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

CAPTCHA


関連記事

オブジェクト指向とは何なのかを簡単に説明してみる

オブジェクト指向とは何なのか? これを私の理解を元に、ごくごく簡単に説明してみたいと思います。   オブジェクト指向とは? オブジェクト指向とは何かを説明するために、 「オブジェクト」と「指 …

【Bootstrap】共通ヘッダーをナビバーとしてデザインしよう!

概要 ヘッダーをナビバーとして実装します。 (SpringBootチュートリアルの共通ヘッダーを基に解説していきます) また、デザインはBootstrapのサンプルを基にしていますので、 そちらも一度 …

【Java入門】判定処理(if, if else, switch文)の使い方まとめ

概要 判定処理には、if / if-else / switch文があります。 これらはとても基本的な判定処理ですが、基本的な使い方に加えて、ちょっとした構文上のTipsや注意点についても解説していきま …

Facadeパターンの使い道をコードとクラス図で説明します。

Facadeパターンはどんな役に立つか? 簡単にいうと、複数の処理のまとめ役。 なので、依頼1つでいろんな仕事をやってくれる点で役に立つ。 Facadeパターンの要件 Facadeクラスはあくまでサブ …

【Java入門】クラスとインスタンスの違いを分かり易い言葉で説明します。

概要 クラスとインスタンスの違いについて、できるだけ分かり易い言葉や表現で説明します。   クラス 設計書だよ。 以上です。 ほんとマジで。 どんなデータと操作が可能かを決めているだけなので設計書と表 …

SpringFramworkの良書

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

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