Сокращение дублирования кода

Question

я в настоящее время есть несколько кода, как это для различных начинок

// Toppings - Egg 
System.out.print("Do you want " + egg.getType() + "?"); 
input = keyboard.nextLine(); 
choice = input.charAt(0); 
if (choice == 'y') { 
    l.add(egg.getType()); 
    c.add((double) egg.getCost()); 
    numberOfToppings = numberOfToppings + 1; 
    totalToppingPrice = totalToppingPrice + egg.getCost(); 
    toppings = toppings + "Egg"; 
} 

Он отлично работает, но я надеялся, что я мог сделать все начинку в только один блок кода. Потому что у меня их около 5, и это слишком много, и мне было рекомендовано это сделать. У кого-нибудь есть идеи, как это можно сделать? благодаря

Answer 1

Я хотел бы предложить вам сделать класс Топпинг, который можно использовать следующим образом:

Toppping egg = new Topping ("egg", 0.5); // Cost 
ArrayList<Topping> toppings = new ArrayList<Topping>(); 
toppings.add(egg); 

Позже вы можете цикл по вектору Начинка похожее на это:

for (Topping current : toppings) { 
    if (wantsTopping(current)) { 
    chosenToppings.addObject(current); 
    } 
} 

Примечание: Это Java как код, но он не будет компилироваться. Есть еще кое-что, что вам нужно найти

Answer 2

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

enum Topping{ 
    EGG("egg", 22),... ; 
    private String type; 
    private double cost; 

    private Topping(String type, double cost){ 
     this.type = type; 
     this.cost = cost; 
    } 
    //getters and setters 

} 

Тогда вы могли бы написать метод внутри вашего класса, содержащего код выше, который должен быть в состоянии обрабатывать Topping экземпляр, как это:

private handleTopping(Topping top){ 
    System.out.print("Do you want "+top.getType() +"?"); 
    input = keyboard.nextLine(); 
    choice = input.charAt(0); 
    if (choice == 'y'){ 
     l.add(top.getType()); 
     c.add(top.getCost()); 
     numberOfToppings = numberOfToppings + 1; 
     totalToppingPrice = totalToppingPrice + top.getCost(); 
     toppings = toppings + " " + top.getType(); 
    } 
} 

Наконец, вызовите метод для всех начинок, доступных

for(Topping top : topping.values()){ 
    handleTopping(top); 
} 

Это все о СУХОЙ (не повторяйте принцип). Это даже не обязательно связано с объектно-ориентированной парадигмой. Даже в процедурном программировании одним из основных принципов является извлечение общих и часто используемых функций в параметризованные процедуры.