beanに処理を書くことについて

Question

SpringBootとDomaを利用しシステム開発を行っています。
従事しているプロジェクトは5年以上前から開発が行われており、開発メンバーの入れ替わりが激しく重複した処理が度々作成されます。
また、誕生日等、日付を持つプロパティの型がLocalDateやDate、またはStringであったりとバラバラな状態で作られています。
また、今まで作られたUtilクラスもそれぞれの型用に作られています。
なので、システムの改修を行うときに、既存Utilクラスを使う様にすると、
内容によって膨大な量の型変換を行わなければならず、それだけで数十行も使ってしまい
業務ロジックが分かり辛くなってします。

そこで、型変換をgetter部分に纏めてしまおうと思うのですが、
これは、Javaの原則から言えば、良い処理といえるのでしょうか？

import java.time.LocalDate;
import java.time.format.DateTimeFormatter;
public class Human {
    LocalDate birthday;
    public Human(LocalDate birthday) {
        this.birthday = birthday;
    }
    public LocalDate getBirthday() {
        return this.birthday;
    }

    // このメソッドは良いといえる処理ですか？
    public String getBirthdayAsString() {
        DateTimeFormatter dateTimeFormatter = DateTimeFormatter.ofPattern("yyyy/MM/dd");
        return this.birthday.format(dateTimeFormatter);
    }
}

Beansは、値を保持する為に存在すると言えば、ダメな気がするのですが、
重複を防ぐ観点で言えば、Beansに集約したほうが良いと思うのです。

Comments

[y_waiwai]

Bean？Beans?とはなんでしょうか

Answer 1

Javaというよりオブジェクト指向の原則を考えるべきですが、
値の保持と値の変換は全く別の役割なので、ダメです。

値を変換するときの多くは「出力のため」です。
DBに保存するときに型をあわせることもありますが、
あくまでDBへの受け渡しのためであって
Beanに保管するためではありません。

でもこれはあくまで一般論であって、
コンパイルエラーが出るとかコードが全く動かないとかそういうことになはならないので、**「プロジェクトやチームで決め事を作って守ればいいんじゃないか」**というのが直接の回答です。

既にそれなりに保守されてきたチーム内の仕組みなのであれば
尚更今から大きく方向転換するのは逆に色んなものを損ねますしね。
「新しく作る」時に検討することかと思います。

開発メンバーの入れ替わりが激しく重複した処理が度々作成されます。

というところから、既にグチャグチャなのがなんとなく伝わってくるので
おそらく1回捨てて作り直した方が早く確実でしょうし。
決めたうえで徹底できるようにメンバー教育とレビューしっかりしないといけませんしね。

Answer 2

プロジェクトで決めればよいと思いますが、この対応で問題が解決されるかは疑問です。

getBirthdayAsStringを呼ばずに、ずっとgetBirthdayを使い、それを変換するコードが
残っていたら結局意味がないです。
メソッドを用意することと使用を強制させることが直接結びつかないです。

また日付フォーマットの異なるものが、無秩序に作成されそうな懸念もあります。

まずは、日付の型を統一するかどうかの判断をして、それらをリファクタリングで統一するのが
良いとは思うのですが、事情もあると思います・・・

統一するなら、特定のクラスに統一するのが良いと思いますがプロジェクト用に日付クラスを
作るのも手かと思います。
そうるれば件の型変換は、このクラスに集約できます。

統一せずに型変換を許容する方針として、
※たぶん、この質問の前提かとは想像しています
変換コードが散乱するのを防止したいというのであれば、変換用のユーティリティを作って
getBirthdayを使ってる部分を、一律その変換ユーティリティを使ってるかチェックするように
した方がよいかと思います。
※getBirthdayや、getXXXDateをgrepして、「Util.」があるかをチェックします。

もとの質問にあるメソッドを複数用意した場合は、grepで、その１行だけ見ても変換コードなのか
判断ができず、結局そのソースの該当箇所をわざわざ確認する作業が必要になります。

Utilクラスもそれぞれの型用に作られています

私の認識が間違っているかもしれませんが、
型用のUitlクラスでなく、役割用のUtilクラスを作り
オーバーロードで、異なる型を同じように処理する方が良いです。

たとえば、最初のgetBirthdayならUtil.getBirthday(ABean)、Util.getBirthday(CBean)のような
感じすると思います。

使用するBeanが多くメソッドを大量に作る可能性はありますが、使用するメソッドの集約は
のちのちクラスの統一をはかるうえでもメリットはありますし、
型変換のコードを独自に書かせないということを目的としているので多少コストかけてでもやる価値は
あると思います。

ちょっと長くなったのでポイントだけ整理すると

• 型変換のメソッドを用意し、個人が変換コードをかかないようにする。
• 型変換のメソッドを使用しているか、簡単にチェックできる仕組みにしておく

質問では最初の個人が変換コードを書かないは達成できるかもしれませんが、
それを確認する手段が少し面倒なので、別途クラス・メソッドを用意した方が良いと思いました。