Is there a shorter way

Question

I have to calculate the price of a sandwich with the selected toppings
Is there a shorter way to write the bereken_teBetalen method without changen code outside the method
public class Topping {
     * Alle toppings zijn opgenomen in toppingWaarden.  Toevoeging van
     * een topping en overeenkomstige prijs geeft meteen een extra keuze
     * op overneekomstige schermen.
     * De prijzen zijn achtereenvolgens :
     *  2.0 voor basissandwich zonder topping
     *  + 0.5 voor kaas
     *  + 0.3 voor ham
     *  + 0.2 voor ananas
    String [] toppingsWaarden = {"kaas","ham","ananas"};
    double[] prijzen = {2.0,0.5,0.3,0.2};
    Map<String, Boolean> toppings;
    Topping(){
        toppings = new HashMap <String , Boolean>();
        for(int i= 0;i<2;i++){
            toppings.put(toppingsWaarden,false); }
public boolean is(String topping) {
return (toppings.get(topping)) ? true:false;
* deze methode retourneert de mogelijke toppings als strings.
public String[] getToppingsWaarden() {
return toppingsWaarden;
public void set(String topping,boolean gekozen){
toppings.put(topping, gekozen);
public double bereken_teBetalen() {
double prijs=prijzen[0];
if(is("kaas")) prijs+=prijzen[1];
if(is("kaas")&&is("ham")) prijs+=prijzen[1]+prijzen[2];
if(is("kaas")&&is("ham")&&is("ananas")) prijs+=prijzen[1]+prijzen[2]+prijzen[3];
if(is("kaas")&&is("ananas")) prijs+=prijzen[1]+prijzen[3];
if(is("ham")) prijs+=prijzen[2];
if(is("ham")&&is("ananas")) prijs+=prijzen[2]+prijzen[3];
if(is("ananas")) prijs+=prijzen[3];
return prijs;
public ImageIcon bepaalIcon(){
String bestand = "sandwich images/brood";
for (String s : toppings.keySet()) {
if(toppings.get(s))
bestand += s;
bestand +=".jpg";
return new ImageIcon(bestand);
* Deze methode genereert de tekst overeenkomstige de gekozen sandwich
* bv. : sandwich ham kaas
public String toString(){
return null;

Answer 1

muckv wrote:
I have to calculate the price of a sandwich with the selected toppings
Is there a shorter way to write the bereken_teBetalen method without changen code outside the method
if(is("kaas"))   prijs+=prijzen[1];
if(is("kaas")&&is("ham")) prijs+=prijzen[1]+prijzen[2];
if(is("kaas")&&is("ham")&&is("ananas")) prijs+=prijzen[1]+prijzen[2]+prijzen[3];
if(is("kaas")&&is("ananas")) prijs+=prijzen[1]+prijzen[3];
if(is("ham"))   prijs+=prijzen[2];
if(is("ham")&&is("ananas"))   prijs+=prijzen[2]+prijzen[3];
if(is("ananas"))   prijs+=prijzen[3];
Definitely take previous advice, however, these two things are more or less equivalent (assuming there are no side-effects):
   if(is("kaas"))   prijs+=prijzen[1];
   if(is("kaas")&&is("ham")) prijs+=prijzen[1]+prijzen[2];
   if(is("kaas")&&is("ham")&&is("ananas")) prijs+=prijzen[1]+prijzen[2]+prijzen[3];and
if ( is( "kaas" ) ) {
   prijs += prijzen[1];
   if( is( "ham" ) ) {
      prijs += prijzen[1] + prijzen[2];
      if ( is( "ananas" ) ) {
         prijs += prijzen[1] + prijzen[2] + prijzen[3];
}But note that in both of the above cases, if you have cheese, ham, and pineapple (sorry I don't know your language, so that's just a guess--but I like Hawaiian pizza!), you will be charged prijzen[1] three times, prijzen[2] twice and prijzen[3] once. I'm guessing what you really intend to do is this:
if ( is( "kaas" ) ) { prijs += prijzen[1]; }
if ( is( "ham" ) ) { prijs += prijzen[2]; }
if ( is( "ananas" ) ) { prijs += prijzen[3]; }And, as has been hinted, if you have a class called Topping (as you do), you really should just include a "price" attribute for the class, and add up the price that way.
Edit: Apparently this is for a sandwich! I'm not sure I would like a Hawaiian sandwich.
Edited by: endasil on 6-Jun-2008 12:48 PM