Java私有方法参数最佳实践

Question

public boolean validate(Map<String, String> content) {
    method1(content);
    method2(content);
}

private boolean method1(Map<String, String> content) {
    // Use of content
}

private boolean method2(Map<String, String> content) {
    // Another use of content
}

或

private Map<String, String> content;

public boolean validate(Map<String, String> content) {
    this.content = content;
    method1();
    method2();
}

private boolean method1() {
    // Use of content
}

private boolean method2() {
    // Another use of content
}

也许在这个简单的例子中，它是一样的，但是如果我有10个私有方法，所有的方法都处理10个属性，那么让所有这些方法都有10个参数，这看起来就更难以理解了。

Answer 1

如果从多个线程中使用验证器，则在实例变量中设置内容不是线程安全的。

例如：

// Thread 1
validator.validate(a)
thread 1 calls validator.method1() using this.content=a

// Thread 2
validator.validate(b)

// Thread 1
thread 1 calls validator.method2(), using this.content=b

如果您重复将同一组局部变量传递给一组方法，您可能会考虑“引入参数对象”重构。参见https://blog.novoda.com/refactoring-indrode-parameter-object或https://www.jetbrains.com/help/idea/2021.1/extract-into-class-refactorings.html#extract_parameter_object