在设置器中添加额外的逻辑？

Question

我对jsf的看法如下：

<h:selectOneMenu value="#{myBean.selectedRoleId}">
    <f:selectItems value="#{myBean.roles}" />
    <a4j:ajax event="change" listener="#{myBean.roleChanged}"
                     render="roleFeatures" />
</h:selectOneMenu>

还有 bean 本身：

public class MyBean{
    private Role selectedRole;
    private Integer selectedRoleId;
    private List<SelectItem> rolesSelectItems;
    private RoleService roleService;
    //GET,SET

    public List<SelectItem> getRoles() {
        initRoles();

        return rolesSelectItems;
    }

    private void initRoles() {
        roles = new HashMap<Integer, Role>();
        rolesSelectItems = new LinkedList<SelectItem>();
        //Do some DB operations     
    }

    public void setSelectedRoleId(Integer selectedRoleId) {
        selectedRole = roleService.getBy(selectedRoleId); //Here we are getting 
                                    //actual role by its Id from database
        this.selectedRoleId = selectedRoleId;
    }

}

在 setter 中执行额外的逻辑是否被认为可以？在我的特殊情况下，当用户使用selectOneMenu 选择另一个Role 时，我从数据库中得到了实际的Role-entity。对我来说，setter 是一种只执行设置特定对象的方法，不再执行。这就是我问这个问题的原因。

Answer 1

对我来说，向 setter 添加额外的代码是可以的。例如，如果您知道字符串不能为空，您可以执行以下操作：

public void setMe(String me) {
    if (me == null) {
        this.me = "";
    } else {
        this.me = me;
    }
}

但是在 setter 中添加对数据库的调用是不正确的。这应该在调用 dao 的服务中完成。