フォールを改善する方法Javaコード

Question

フォールを改善する方法はありますか。コードブロック：

public class MyUnits { 
    public static String MILLSECONDS = "milliseconds"; 
    public static String SECONDS = "seconds"; 
    public static String MINUTES = "minutes"; 
    public static String HOURS = "hours"; 

    public int quantity; 
    public String units; 

    public MyUnits(int quantity, String units) { 
     this.quantity = quantity; 
     this.units = units; 
    } 

    public String toString() { 
     return (quantity + " " + units); 
    } 

    // Test code 
    public static void main(String[] args) { 
     System.out.println(new MyUnits(1, MyUnits.MILLSECONDS)); 
     System.out.println(new MyUnits(2, MyUnits.SECONDS)); 
     System.out.println(new MyUnits(3, MyUnits.MINUTES)); 
     System.out.println(new MyUnits(4, MyUnits.HOURS)); 
    } 
} 

助けてください。

Answer 1

enumをクラス内のユニットに使用します。

public enum MyUnit { 
MILLSECONDS(1, "milliseconds"), SECOND(2, "seconds"), 
MINUTES(3,"minutes"), HOURS(4, "hours"); 



private MyUnit(int quantity, String units) { 
    this.quantity = quantity; 
    this.units = units; 
} 

private int quantity; 
private String units; 

public String toString() { 
    return (quantity + " " + units); 
} 
    /*getters setters*/ 
} 

と、ちょうどその

MyUnit.HOURS.getUnits(); 
MyUnit.HOURS.getCuantity(); 

Answer 2

追加クローン（）、hashCode（）およびequals（）のように呼び出して、匹敵する、直列化およびCloneableを実装。

テストコードを削除し、テストクラスに移動します。テストにはJUnitを使用します。

public Date addToDate(Date date)を書きます。

Answer 3

static intはfinalである必要があります。列挙型で、静的なint型を交換：

public enum Unit {MILLISECOND, SECOND, MINUTE, HOUR} 

新しいコンストラクタのシグネチャは次のようになります。

public MyUnits(int quantity, Unit unit) { ... } 

Javaで "定数" のための通常の修飾子は

public static final <Type> <name> = <value>; 

大きな機能強化ですMyUnitの非静的フィールドはプライベートにする必要があります。アクセスのためのgetter/setterメソッドを追加してください。

最後に（深刻なコードの場合のみ）、テストコードをクラスから分離します。 junitのようなテストフレームワークを見て、別々のテストクラスを実装してください。

Answer 4

1. マークfinalとしてごpublic static変数 。より良いですが、enumを使用します。

2. インスタンス変数（quantity,units）をpublicにしないでください。その値を読み取るための「ゲッター」メソッドを提供します。 は、値を変更するための "setter"メソッドを提供するではありません。これは、簡単に使用することができます不変クラス、（不変オブジェクトの状態がはるかに予測可能な可変オブジェクトのそれよりもです！）になり

3. コード が何を意図しているかについて具体的に。それを としてください。 （これはまた あなたのより良い答えを得るでしょう 、より具体的な質問を、お願いすることができます。）

4. 、特にjavadoc commentsをコメントを追加します。