如何（或者应该）避免 jsf 中的长方法/类

Question

我主要在基于 JSF 的项目中处理遗留代码，并且支持 bean 中有很多相当长的类和方法。

这一直困扰着我，但是当我寻找可以做什么时，大多数时候我能想到的就是将一个长方法分成 n 个小方法。这会给你带来很长的课程，有时也更难阅读。

那么，如何才能让你的支持 bean 变得简短明了呢？或者你会在一页上保留一个大的支撑豆吗？有没有最佳实践？

我认为这与 jsf 没有直接关系，而是与您使用控制器“备份”视图的任何模型相关。因此，一般建议也会有所帮助。

Answer 1

将所有字段放在自己的类中，也称为实体或 DTO 类（例如 User、Product、Order 等）并引用它。这些可以是 JDBC/JPA 实体。将所有业务方法放在自己的类中，也称为服务或域对象（例如UserService、ProductService等）并引用它。这些可以是 EJB 的。

例如，因此不是

public class Bean {

    private Long id;
    private String field1;
    private String field2;
    private String field3;
    // Etc... All model fields.

    @PostConstruct
    public void init() {
        // >20 lines of business/domain code to prefill the fields from DB.
    }

    public void save() {
        // >20 lines of business/domain code to save the fields in DB.
    }

    // Getters/setters for all fields and possibly also all nested fields.
}

但更进一步

public class Bean {

    private Long id;
    private Entity entity;

    @EJB
    private EntityService service;

    @PostConstruct
    public void init() {
        entity = service.find(id);
    }

    public void save() {
        service.save(entity);
    }

    // Getter/setter for id and getter for entity.
}

，我还看到了代码，其中嵌套对象/实体是通过 bean 中的附加 getter/setter 委托的，就像这样

private Entity entity;

public String getField1() {
    return entity.getField1();
}

public void setField1(String field1) {
    entity.setField1(field1);
}

// Etc...

这是完全没有必要的。实体只需一个 getter 就足够了（setter 不是强制的！），结合

<h:inputText value="#{bean.entity.field1}" />

实体本身还可以进一步划分。例如 的 street、houseNumber、zipCode、city、country User 可以被同一 User 内的另一个实体/DTO Address 替换。

---

如果您运气不好，代码已由可视化编辑器（例如 Netbeans + Woodstock）自动生成。无论如何，如果不完全重新设计它，就没有太多需要重构的地方，我宁愿寻找另一个项目。

Answer 2

恕我直言，每个方法应该是 1 个步骤，如果该 1 个步骤包含多个内部步骤，您应该将它们分解为更小的方法，以便更容易阅读。现代 IDE 的好处是它们可以毫不费力地为您折射代码。

Answer 3

不仅适用于支持 bean，而且适用于一般的所有对象：

您应该始终通过以下方式重构 长方法 将它们提取成多个较小的。