1

我从来没有在我周围的任何 Java 代码中看到这种常量声明......所以我想知道你是否看到使用非静态最终常量的任何缺点。

例如，我已将 Guava 函数声明为给定 MaintenanceMode 实例的公共常量。我认为这更好，因为如果我创建了一个 getDecoratorFunction() 它每次都会创建一个新的函数实例......

或者 get 函数可以返回在类中保持私有的单个实例函数，但是它有无用的代码......当我们在类级别声明常量时，我​​们直接声明常量是公共的，我们不将它们设为私有并提供访问它们的公共吸气剂...

public class MaintenanceMode {

/**
 * Provides a function to decorate a push service with the appropriate decorator
 */
public final Function<PushService,PushService> MAINTENANCE_DECORATION_FUNCTION = new Function<PushService,PushService>() {
    @Override
    public PushService apply(PushService serviceToDecorate) {
        return new PushServiceMaintenanceDecorator(serviceToDecorate,MaintenanceMode.this);
    }
};

private final EnumMaintenanceMode maintenanceMode;
private final long milliesBetweenMaintenances;
private final Optional<ExecutorService> executorService;


public EnumMaintenanceMode getMaintenanceMode() {
    return maintenanceMode;
}

public long getMilliesBetweenMaintenances() {
    return milliesBetweenMaintenances;
}

public Optional<ExecutorService> getExecutorService() {
    return executorService;
}


private MaintenanceMode(EnumMaintenanceMode maintenanceMode, long milliesBetweenMaintenances, ExecutorService executorService) {
    Preconditions.checkArgument(maintenanceMode != null);
    Preconditions.checkArgument(milliesBetweenMaintenances >= 0);
    this.maintenanceMode = maintenanceMode;
    this.milliesBetweenMaintenances = milliesBetweenMaintenances;
    this.executorService = Optional.fromNullable(executorService);
}

}

我可以通过以下方式访问此变量：

  pushServiceRegistry.decoratePushServices(maintenanceMode.MAINTENANCE_DECORATION_FUNCTION);

我想如果我的 maintenanceMode 是可变的并且可以被多个线程访问，它可能会导致奇怪的行为，但这里不是。

你看到使用这种代码有什么缺点吗？

编辑：我可以有多个 MaintenanceMode 实例，并且所有实例都应该能够根据 MaintenanceMode 状态提供不同的常量功能。所以我不能使用不会访问 MaintenanceMode 状态的静态变量。

4

2 回答 2

2

getter 的重点是动态调度。如果您不需要它，则使用public final字段非常好。我什至经常编写没有 getter 的类 bean 对象，只有public final字段。

于 2012-09-03T13:22:55.890 回答
1

通过使常量非静态，您基本上是在说只有当您拥有该类的实例时才能访问该常量。但它是公共的（在 的情况下MAINTENANCE_DECORATION_FUNCTION）并且它是该类的一部分，所以为什么不让它成为静态的呢？毕竟，常量是一个常量，它不需要在其他地方使用该类的实例。该变量maintenanceMode很好，因为它是一个私有常量。

于 2012-09-03T13:18:28.000 回答