根据另一个h:SelectOne菜单值加载h:SelectOne菜单

Question

我有两个h:SelectOne菜单（1：国家，2：城市）。我需要使用ajax在cities SelectOne菜单中加载一个选定国家的所有城市。当我更改countries selectOneMenu的值时，我的cities selectOneMenu从countryBean中获取空值。选择国家。

<h:panelGrid columns="2">
    <h:outputLabel for="countries" value="Countries: " />
    <h:selectOneMenu converter="omnifaces.SelectItemsConverter"
        id="countries" required="true" value="#{countryBean.selectedCountry}">
        <f:selectItem itemLabel="Choose country" />
        <f:selectItems value="#{countriesBB.findAllCountries()}"
            var="country" itemLabel="#{country.name}" />
        <f:ajax event="change" render="cities" />
    </h:selectOneMenu>

    <h:outputLabel for="cities"
        value="Cities: " />
    <h:selectOneMenu converter="omnifaces.SelectItemsConverter"
        id="cities" required="true"
        value="#{cityBean.selectedCity}">
        <f:selectItem itemLabel="Choose city" />
        <f:selectItems value="#{cityBean.findAllCitiesByCountry(countryBean.selectedCountry)}"
            var="city" itemLabel="#{city.name}" />
    </h:selectOneMenu>
</h:panelGrid>

这是找到城市的方法：

public List<city> findAllCitiesByCountry(Country country) {

        List<City> cities = null;
        try {

            cities = citiesService.findAllCitiesByCountry(country);

        } catch (Exception exception) {
            logger.debug("Error finding cities.", exception);
        }

        return cities;

    }

因为countryBean，我得到了一个NullPointerException。selectedCountry始终为空。正确的方法是什么？

Answer 1

JSF初学者需要知道的众多规则之一：

• 不要在getter方法中执行业务逻辑。

一旦您尝试通过保持getter方法为真正的getter方法来解决这个问题（即除了返回属性之外不要做任何其他事情；）并在（post）构造函数和/或action（listener）方法中执行业务逻辑，那么这个特定的问题将消失。

以下是一个启动示例：

<h:selectOneMenu value="#{bean.country}">
    <f:selectItems value="#{bean.countries}" ... />
    <f:ajax listener="#{bean.changeCountry}" render="cities" />
</h:selectOneMenu>
<h:selectOneMenu id="cities" value="#{bean.city}">
    <f:selectItems value="#{bean.cities}" ... />
</h:selectOneMenu>

在@ViewScopedbean中使用类似的内容：

private Country country; // +getter+setter
private City city; // +getter+setter
private List<Countries> countries; // +getter
private List<Cities> cities; // +getter

@EJB
private SomeService service;

@PostConstruct
public void init() {
    countries = service.getCountries();
}

public void changeCountry() {
    cities = service.getCities(country);
}