Есть ли способ создать объект, в котором класс объектов определяется другой переменной? - PullRequest
0 голосов
/ 18 января 2020

У меня есть два класса, у которых есть несколько идентичных методов, метод, приведенный ниже, должен обращаться к методам одного из двух классов, в зависимости от того, какой флажок установлен, однако у меня возникают проблемы при передаче объектов, созданных в инструкциях IF, в основное тело. метода. Есть ли способ достичь этого, не копируя основную часть метода в обе области? Вот код:

public void populateSupplyChainTextFields() {

    if (jDeliveryCheckBox.isSelected() == true) {
        DeliveryCompany supplyChainMember = new DeliveryCompany();
        supplyChainMember = theDeliveryCompanyList.Find(jSupplyChainSearchBox.getSelectedItem().toString());
    } else {
        Supplier supplyChainMember = new Supplier();
        supplyChainMember = theSupplierList.Find(jSupplyChainSearchBox.getSelectedItem().toString());

    }

    jCategoryTextField.setText(supplyChainMember.getCategory());
    jPriceTextField.setText(String.valueOf(supplyChainMember.getPrice()));
    jUnitCostTextField.setText(String.valueOf(supplyChainMember.getUnitCost()));

РЕДАКТИРОВАТЬ **

На основе ответа Салима, вот сообщение об ошибке, которое я получаю: enter image description here

Для подтверждения, метод getBusinessName () используется в классах supplier и deliveryCompany.

Приношу свои извинения, если это вопрос новичка, я довольно новичок в Java, любая помощь с этим приветствуется.

Ответы [ 2 ]

0 голосов
/ 18 января 2020

Как уже упоминали многие, лучше всего сделать так, чтобы Supplier и DeliveryCompany реализовали интерфейс комментариев или расширили общий базовый класс.

Если вы действительно не можете этого сделать, и действительно имеете чтобы избежать дублирования в ваших блоках, и действительно, если ваш реальный код более сложен, чем тот, который вы опубликовали, вы можете сделать так, чтобы ваши блоки совместно использовали переменную

Использование локального класса для хранения данных

public void populateSupplyChainTextFields() {

    class Holder {
        String category, price, unitCost;
        Holder(String category, double price, double unitCost){
            this.category = category; 
            this.price = String.valueOf(price); 
            this.unitCost = String.valueOf(unitCost);
        } 
    }

    Holder data;

    if (jDeliveryCheckBox.isSelected()) {
        DeliveryCompany supplyChainMember = new DeliveryCompany().Find(jSupplyChainSearchBox.getSelectedItem().toString());
        data = new Holder(supplyChainMember.getCategory(), supplyChainMember.getPrice(), supplyChainMember.getUnitCost());
    } else {
        Supplier supplyChainMember = new Supplier().Find(jSupplyChainSearchBox.getSelectedItem().toString());
        data = new Holder(supplyChainMember.getCategory(), supplyChainMember.getPrice(), supplyChainMember.getUnitCost());
    }

    jCategoryTextField.setText(data.category);
    jPriceTextField.setText(data.price);
    jUnitCostTextField.setText(data.unitCost);

Использование локального класса - это только мои предпочтения, вместо этого вы можете использовать что-то вроде карты (конечно, записи будут очень полезны):

Map<String, String> data;

if (jDeliveryCheckBox.isSelected()) {
    DeliveryCompany supplyChainMember = ...
    data = Map.of("category, supplyChainMember.getCategory(), 
                  "price", String.valueOf(supplyChainMember.getPrice()), 
                  "unitCost", String.valueOf(supplyChainMember.getUnitCost()));
} else {
    Supplier supplyChainMember = ...
    data = Map.of("category, supplyChainMember.getCategory(), 
                  "price", String.valueOf(supplyChainMember.getPrice()), 
                  "unitCost", String.valueOf(supplyChainMember.getUnitCost()));
}

jCategoryTextField.setText(data.get("category"));
jPriceTextField.setText(data.get("price"));
jUnitCostTextField.setText(data.get("unitCost"));
0 голосов
/ 18 января 2020

Лучше наследовать DeliveryCompany и Supplier от общего интерфейса или абстрактного класса. Если это не сработает, тогда вы можете использовать Object тип в качестве последнего средства.

    public Object populateSupplyChainTextFields() {
    Object result = null;

        if (jDeliveryCheckBox.isSelected() == true) {
            DeliveryCompany supplyChainMember = new DeliveryCompany();
            supplyChainMember = theDeliveryCompanyList.Find(jSupplyChainSearchBox.getSelectedItem().toString());
    result = supplyChainMember
        } else {
            Supplier supplyChainMember = new Supplier();
            supplyChainMember = theSupplierList.Find(jSupplyChainSearchBox.getSelectedItem().toString());
    result = supplyChainMember
        }


    return result;

    }
//Call the method
Object r = populateSupplyChainTextFields();

//Check type and take action
if (r instanceof DeliveryCompany){
    DeliveryCompany supplyChainMember = (DeliveryCompany)r;

        jCategoryTextField.setText(supplyChainMember.getCategory());
        jPriceTextField.setText(String.valueOf(supplyChainMember.getPrice()));
        jUnitCostTextField.setText(String.valueOf(supplyChainMember.getUnitCost())
}

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...