[jfriends] 関数と引数とメンテナンス効率、とか

[inami <inami@xxxxxxxxxx>]

元々の staticがどうたら、と言う話とは関係無いです(^^;

On Wed, 28 Apr 1999 11:51:22 +0900
「[jfriends] Re: [jfriends] Re: staticの使い方（メソッド）」にて
Kazuya Maebashi <maebashi@xxxxxxxxxx> wrote:

> (1)引数なしのメソッドにする。プライベートメンバの勤続年数・等
>   級などを参照する。
> (2)引数として勤続年数・等級などを受け取るようにする。
> 
> (2)の方が、「算出ロジック」という趣旨に合致しているのではな
> いかと。


> 関数といえば、引数を与えて、戻り値を返すモノであり、同じ引数
> を与えれば必ず同じ値を返します(Cの関数はそうではないですが)。
> 関数が状態を持たないものであるなら、それはオブジェクトと結び
> 付いているべきではないだろう、と思うわけです。

「引数を渡すから」オブジェクトと結びつかない、と言う反対の見
方もあるわけで。

class 社員CLASS {
  private int 入社年;
  private int 役職;
    :
  public static int 給与計算(int 勤続年数, int 役職コード ...)
  {...};
    :
}
な関数が有ったとして、結局「勤続年数」「役職コード」は何から
取得するか考えると、社員の情報から取得して渡す、ってことに
なるんだと思います。

擬似コードで書くと、
---
 社員CLASS したっぱー井浪 = new 社員CLASS( ....);
 int salary = 社員CLASS.給与計算( したっぱー井浪.get勤続年数(),
                                  したっぱー井浪.get役職(), ..);
---
こんな感じになるんでしょう。
この、引数に渡すために、社員の情報を1個ずつ取り出すほうが、
面倒と言うか、マヌケ臭い気がします。

さらに
> (2)引数として勤続年数・等級などを受け取るようにする。
な方法は、私の経験でいけば、間違いなくメンテナンス時に破綻し
ます。
「今年から(情報処理等の)有資格者は、資格手当てが出るように
なるぞ」っと言うような、給与体系に変更があった場合、引数に
渡さなければならない情報が増えた結果、あれも直してこれも直
して……あぁあっちでも給与計算の関数を呼んでる〜、と、どこ
でこの関数を呼び出してるか調べまくって、直しまくる、と言う
やや不毛な作業が待ってます。
# 現状の COBOLなんかでは当たり前におこってますが、空しいです。

この点から考えれば、(個人的には)
> (1)引数なしのメソッドにする。プライベートメンバの勤続年数・等
>   級などを参照する。
の方が、作る時も、メンテナンス時も非常に楽です。
# COBOL,Fortran等に比べて、オブジェクト指向のありがたみが
# 最初に理解できた例でした。

------------------------------
あとは好みの問題かもしれませんが、

class 経理部 {
  public static int 給与計算 (社員CLASS 社員) {
     return 給与計算( 社員.get勤続年数(),
                      社員.get役職(),
                      社員.get資格(), ....
  }

  private stataic int 給与計算(int 勤続年数....) {
    :
  }
}

のような、社員とは関係無い classに給与計算のprivateな「関数」
を用意しておいて、publicな方で「社員」を受け取りその社員情報
で関数呼出し〜、みたいな方が好みです。
# 一目で理解できるかどうかは別の話 :-)

---
井浪@xxxxxxxxxx % COBOLer会社な空しい人