Javaのcall-by-valueを「悪用する」ことは賢明でしょうか

Question

次のような方法を検討してください。ユーザーは 30D、90D、180D、360D、1M、3M、6M、12M、1Y （D =日、M =月、Y =年）のようなものを入力できます。

次の2つの方法で月数を計算します。 Javaは値として渡された参照して、コールバイ値で動作しますので

private int getLengthOfPeriodInMonths(Integer lengthOfPeriod, String unitOfPeriod) { 
     int periodInMonths = lengthOfPeriod; 
     if ("D".equals(unitOfPeriod)) { 
      periodInMonths = lengthOfPeriod/30; 
     } else if ("Y".equals(unitOfPeriod)) { 
      periodInMonths = lengthOfPeriod * 12; 
     } 
     return periodInMonths; 
    } 

    private int getLengthOfPeriodInMonths(Integer lengthOfPeriod, String unitOfPeriod) { 
     if ("D".equals(unitOfPeriod)) { 
      lengthOfPeriod = lengthOfPeriod/30; 
     } else if ("Y".equals(unitOfPeriod)) { 
      lengthOfPeriod = lengthOfPeriod * 12; 
     } 
     return lengthOfPeriod; 
    } 

lengthOfPeriod方法の外に変更されません。私は使用するのがより適切であるか分かりません。

このメソッドは、enum Periodsなどを使用してリファクタリングできます。しかしここでこれについて議論してはいけません。

Answer 1

メソッドパラメータを割り当てることは良い考えではありません（また、デフォルトのIMHOによって最終的に行われているはずです）。あなたが本当に余分な変数を避けたいのであれば（実際に違いがないわけではない）、if節に戻り値を入れることができます。

しかし、パラメータを「自動ローカル変数」として使用しないでください。バグが見つからなくなることがあり、コードの実行能力が向上しません。

Answer 2

唯一の違いは、最初の方法で独自の変数int periodInMonths = lengthOfPeriod;を使用することです。しかし、これは必要ではありません！

2番目のものを使用することができます。

int lengthOfPeriodInMonths = getLengthOfPeriodInMonths(lengthOfPeriod, unitOfPeriod); 

、あなたがlengthOfPeriodInMonths

PSに保存されている方法の外で計算int型を持っている：この呼び出し

lengthOfPeriod = lengthOfPeriod/30; 

が

lengthOfPeriod = new Integer(lengthOfPeriod/30); 

と等価である（自動」に見えますボクシング "）。

あなたが言ったように、Javaはcall-bay-valueを使用します。このコールでは、変数lengthOfPeriodに新しい参照が割り当てられます。だから、この計算はメソッドの外で失われるでしょう！だからあなたは新しく計算された値を返さなければなりません！

Answer 3

最初のアドバイス：決してメソッドパラメータを変更しないでください。これらは、メソッドのユーザーが渡す値を常に反映する必要があります。

第2の助言：特に誰かが決して何かをしないことをアドバイスしている場合、特に「今まで決して」というフレーズを使用していない場合は、塩分で助言を受けてください。例外のないルールはありません（これまで決してありません;-)。このようなルールは大雑把なルールですが、常に自分の判断を使用してください。より明確に思えるソリューションを選択してください。私は、メソッドのパラメータを変更することが、代替メソッドよりも読みやすくなっていることが判明した状況を知っています。

たとえば、メソッドのユーザーがnullをメソッドに渡すことを許可することができますが、それをいくつかのデフォルト値に置き換えることができます。

public void doSomething(final String s) { 
    String sOrEmpty = s == null ? "" : s; 
    System.out.println(sOrEmpty); 
}