skillup

技術ブログ

Java

Javaでのコーディングで気を付けること

投稿日:

会社でSoftWareDesignを定期購読しており、Javaのコーディングで気をつけることなどがあったので、ちょっとチェック。

以前も似たようなエントリーは書いたことがあったのだけど、再確認。メモ書きです。

良いコードとは

  • 良いコード=保守性が高いコード
  • 何を言っているのか理解しやすい
  • 不具合を発見しやすい

個人的には修正に強い、バグの影響を最小限にしやすいのような意味だとも思ってます。

具体的なテクニック

変数の初期化と変数のスコープの最小化

不要な初期化をさけること。String str =””などとしない。スコープはなるべく最小限にすることと、変数の宣言、代入はなるべく近い距離で行う。

メソッドは短く(目安として20~30行に)

長いメソッドは何を言っているかがわからず、処理を理解するのに非常に労力を使う。処理の塊ごとに区切ること。目安は20~30行。

finalを検討する

変数の変更は少なければ少ないほど良い。値が変わらなければその変数をおう労力を少なくできる。finalをつけることで変更がないことを宣言できる。そうでなくても変数の変更はなるべく行わないようにすべき。

ラムダ式のすすめ

Javaのみだが、複雑な処理を簡潔に書くことができる、配列関連で有効なメソッドがあり、非常に有用。

Optional

nullがあるものに関してはOptional。そうでないものは通常の型(Stringなど)で返せば、通常の型の時にいちいちnullチェックをする必要がなくなる。OptionalのときはisPresentなどのメソッドを使えばよい。

enumを使おう

定数的な値に意味のない値(SUNDAY =1など)を避けることができ、保守性を向上させることができる。

-Java
-

執筆者:


comment

メールアドレスが公開されることはありません。 * が付いている欄は必須項目です

関連記事

no image

小ネタ集

最近覚えた小ネタをのせていきます。 熟練のプログラマからすると当たり前のことが多いと思いますが・・・ Contents1 {}でスコープ限定2 リダイレクト:face-redirect=true3 H …

no image

複合コンポーネント

JSFではラベルやテキストといったコンポーネントをグループ化し、一つのまとまりとすることができます。 メリットとしては似たような部分をテンプレート化しておくことで保守性を向上させることができます。 さ …

no image

Mapからインスタンス変数、インスタンス変数からMapへ

Javaで変数を引き回すときにMapからインスタンス、インスタンスからMap、あるインスタンスから別のインスタンスに変数を差し替える作業って結構多いと思います。 これって普通にやるとMapのget,p …

no image

JSF リダイレクト後のメッセージ&disabled=true

少々細かいですがJSFネタです。 Contents1 リダイレクト後のメッセージ2 h:commandbutton disabled=trueについて リダイレクト後のメッセージ JSFからManag …

no image

JavaEEでのURLマッピング

Contents1 URLマッピング1.1 prettyfaces1.2 導入方法1.2.1 pom.xmlの設定1.2.2 pretty-config.xmlの設定1.3 ManagedBeanでの …