加载h:selectOneMenu取决于另一个h:selectOneMenu值

Question

我有两个h:selectOneMenu (1:国家，2:城市)。我需要使用ajax从城市中的一个国家加载所有城市的selectOneMenu。当我更改国家selectOneMenu的值时，我的城市selectOneMenu从countryBean.selectedCountry获得一个空值。

<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;

    }

我得到一个NullPointerException，因为countryBean.selectedCountry总是空的。怎样才是正确的方法？

Answer 1

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

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

一旦您试图通过保持getter方法真正的getter方法(即只做return property;以外的任何事情)并在(Post)构造函数和/或操作(侦听器)方法中执行业务逻辑来修复这个问题，那么这个特定的问题就会消失。

下面是一个开球的例子：

<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>

在@ViewScoped bean中使用这样的内容：

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);
}