javafx ComboBox с cellfactory не отображает выбранный элемент

У меня есть небольшое пробное концептуальное приложение, которое содержит 6 Labels, ComboBox и Button, созданных с помощью SceneBuilder.

Нажав Button, приложение выполняет вызов API Json, чтобы вернуть список стран и связанные с ними данные (apla2code, apla3code, название и т. д.). Я создал объект CountryDetails, который содержит 3 элемента String. Я использую это, чтобы вернуть массив CountryDetails, который затем загружаю в массив ObserbavleList. Затем я применяю это к ComboBox и загружаю элементы CountryDetails в 3 метки каждый раз, когда элемент выбирается в ComboBox. Все это прекрасно работает (хотя, вероятно, есть гораздо лучший способ сделать это).

У меня проблема в том, что ComboBox не отображает выбранный элемент, и я не могу понять, как это исправить. На изображении ниже показано, в чем проблема.

Код, который делает вызов API, выглядит следующим образом:

import com.google.gson.Gson;
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.net.HttpURLConnection;
import java.net.URL;

public class GetCountries {

    public CountryDetails[] getDetails() {

        String inputLine            = "";
        StringBuilder jsonString    = new StringBuilder();

        HttpURLConnection urlConnection;

        try {
            URL urlObject = new URL("https://restcountries.eu/rest/v2/all");

            urlConnection = (HttpURLConnection) urlObject.openConnection();
            urlConnection.setRequestMethod("GET");

            BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(urlConnection.getInputStream()));
            while ((inputLine = bufferedReader.readLine()) != null) {
                jsonString.append(inputLine);
            }
            urlConnection.getInputStream().close();

        } catch(IOException ioe) {
            System.out.println(ioe.getMessage());
        }

        Countries[] countries = new Gson().fromJson(jsonString.toString(), Countries[].class);

        CountryDetails[] countryDetails = new CountryDetails[countries.length];

        for(int i = 0; i < countries.length; i++){

            countryDetails[i] = new CountryDetails(
                    countries[i].getAlpha2Code(),
                    countries[i].getAlpha3Code(),
                    countries[i].getName()
            );
        }
        return countryDetails;
    }
} 

Код для объекта CountryDetails выглядит следующим образом:

public class CountryDetails {

    private String alpha2Code;
    private String alpha3Code;
    private String name;

    public CountryDetails(String strAlpha2Code, String strAlpha3Code, String strName) {
        this.alpha2Code = strAlpha2Code;
        this.alpha3Code = strAlpha3Code;
        this.name = strName;
    }

    public String getAlpha2Code() { return alpha2Code; }

    public void setAlpha2Code(String alpha2Code) { this.alpha2Code = alpha2Code; }

    public String getAlpha3Code() { return alpha3Code; }

    public void setAlpha3Code(String alpha3Code) { this.alpha3Code = alpha3Code; }

    public String getName() { return name; }

    public void setName(String name) {this.name = name; }
}

Код, который загружает ObservableList, выглядит следующим образом:

GetCountries countries = new GetCountries();

        CountryDetails[] countryDetails = countries.getDetails();

        for (CountryDetails countryDetail : countryDetails) {
            countriesObservableList.add(new CountryDetails(
                    countryDetail.getAlpha2Code(),
                    countryDetail.getAlpha3Code(),
                    countryDetail.getName())
            );
        }

Код, который загружает ComboBox и отображает элементы в Labels, выглядит следующим образом:

    cbCountryList.setCellFactory(new Callback<ListView<CountryDetails>, ListCell<CountryDetails>>() {
            @Override public ListCell<CountryDetails> call(ListView<CountryDetails> p) {
                return new ListCell<CountryDetails>() {
                    @Override
                    protected void updateItem(CountryDetails item, boolean empty) {
                        super.updateItem(item, empty);
                        if (empty || (item == null) || (item.getName() == null)) {
                            setText(null);
                        } else {
                            setText(item.getName());
                        }
                    }
                };
            }
        });

    public void comboAction(ActionEvent event) {
        lblAlpha2Code.setText(cbCountryList.getValue().getAlpha2Code());
        lblAlpha3Code.setText(cbCountryList.getValue().getAlpha3Code());
        lblCountryName.setText(cbCountryList.getValue().getName());
    }

Ниже приведено изображение приложения:

введите здесь описание изображения


person Yastafari    schedule 07.01.2020    source источник
comment
Задайте ComboBox#buttonCell экземпляр возвращенный вашей пользовательской фабрикой ячеек.   -  person Slaw    schedule 07.01.2020


Ответы (1)


У меня проблема в том, что ComboBox не отображает выбранный элемент, и я не могу понять, как это исправить.

Вам необходимо установить StringConverter для вашего cbCountryList.

cbCountryList.setConverter(new StringConverter<CountryDetails>() {
    @Override
    public String toString(CountryDetails object) {
        return object.getName();
    }

    @Override
    public CountryDetails fromString(String string) {
        return null;
    }
});

Все это прекрасно работает (хотя, вероятно, есть гораздо лучший способ сделать это).

Вы можете рассмотреть возможность обновления следующих вещей,

  • Асинхронный вызов HTTP-запроса и загрузки элементов
  • Вы можете кэшировать свой список fetched-country, поскольку вы звоните каждый раз, когда срабатывает кнопка. Вы можете создать объект Singleton для GetCountries.

Изменен класс GetCountries,

Он кэширует список стран и использует кэшированные данные для нескольких запросов,

 public static class GetCountries {

    private static final String API_URL = "https://restcountries.eu/rest/v2/all";
    private static CountryDetails[] countryDetails;

    public static CountryDetails[] getDetails() {

        //uses cached countryDetails once it gets loaded
        if (countryDetails != null) {
            return countryDetails;
        }

        StringBuilder jsonString = new StringBuilder();
        HttpURLConnection urlConnection;
        try {
            URL urlObject = new URL(API_URL);

            urlConnection = (HttpURLConnection) urlObject.openConnection();
            urlConnection.setRequestMethod(HttpMethod.GET.name());

            String inputLine = "";

            BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(urlConnection.getInputStream()));
            while ((inputLine = bufferedReader.readLine()) != null) {
                jsonString.append(inputLine);
            }
            urlConnection.getInputStream().close();

        } catch (IOException ioe) {
            System.out.println(ioe.getMessage());
        }

        Countries[] countries = new Gson().fromJson(jsonString.toString(), Countries[].class);

        countryDetails = new CountryDetails[countries.length];
        for (int i = 0; i < countries.length; i++) {
            countryDetails[i] = new CountryDetails(
                    countries[i].getAlpha2Code(),
                    countries[i].getAlpha3Code(),
                    countries[i].getName()
            );
        }
        return countryDetails;
    }
}

Используйте Задание для асинхронного получения данных о странах.

Task<CountryDetails[]> fetchCountryTask = new Task<CountryDetails[]>() {
    @Override
    protected CountryDetails[] call() throws Exception {
        return GetCountries.getDetails();
    }
};

fetchButton.setOnAction(event -> new Thread(fetchCountryTask).start());

fetchCountryTask.setOnRunning(event -> cbCountryList.setDisable(true));

fetchCountryTask.setOnSucceeded(e -> {
    cbCountryList.getItems().addAll(fetchCountryTask.getValue());
    cbCountryList.setDisable(false);
});
person Shekhar Rai    schedule 07.01.2020
comment
Сехар Рай: Спасибо за это. StringConverter прекрасно работает. Что касается асинхронного вызова и кэширования, это всего лишь доказательство концепции, которое в конечном итоге станет частью мастера настройки для более крупного приложения. Список будет сгенерирован только один раз при загрузке мастера и больше не будет вызываться. Однако это определенно будет полезно для других частей приложения, которым потребуется более частый доступ к аналогичным возвращаемым данным Json и в том же сеансе. - person Yastafari; 08.01.2020