java - 改善follow的方法。 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

静态整数应该是最终的。 Java 中“常量”的常用修饰符是

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

更大的增强:将 static int 与 enum 交换:

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

新的构造函数签名将是:

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

MyUnit 中的非静态字段应设为私有(private)。添加 getter/setter 方法进行访问。

最后(仅对于严肃的代码)，我将测试代码与类分开。查看测试框架，例如 junit 并实现单独的测试类。