User Prompt Array Question for Javascript

Question

EDIT: Thanks to someone's response, I got a portion working. It now adds topping prices no problem. However, it adds a price even when a user doesn't put toppings into the prompt. I've updated my code below.

EDIT 2: For full code, here's my github link. It's updated with the below code (and the rest of the coding) https://github.com/cas389/m4-hw6-ju-cheryl.

I'm in a JavaScript class, and I am stuck on this portion of my homework assignment. Basically, we are creating a pizza ordering system where a user would fill out if they want thin/thick/pan crust, which toppings they want, if they want extra cheese and finally if they want delivery.

I have everything except the pizza toppings part. This is the information for this section (including the comments the teacher gave us as "hints") along with what I've been trying to make work. I've cut out all the sections I've already finished.

function getPizzaOrder() {
  var extraCheeseUpcharge = 1.5
  var thickCrustUpcharge = 2
  var deliveryFee = 3.5
  var toppingsFee = 1.5
  var basePrice = 10

  var toppings = prompt("Please enter additional toppings (comma separated)")
      // HINT: prompt() will return an empty string "" if the user presses 'OK' without entering a value
      // if the user enters toppings, use .split(",") to separate toppings into an array
      // if no toppings are given, make sure pizza.toppings is set to []
      // if the user has added toppings, add toppingsFee multiplied by
      // the number of toppings added to pizza.cost
      // YOUR CODE HERE
    
       pizza.toppings = [];
       pizza.toppings = toppings.split(',');
       for (let i = 0; i < pizza.toppings.length; i++){
           pizza.cost += toppingsFee;
       }
       console.log(pizza.toppings);




  return pizza
}

pizza.cost =+ pizza.toppings * toppingsFee is using =+ instead of +=. You also need to access the correct topping in the loop using the index (i). — emeraldsanto
– emeraldsanto, Commented Feb 8, 2022 at 19:55
Yeah =+ causes in-flight cast to ineger - stackoverflow.com/questions/10160850/… — Marek Kamiński
– Marek Kamiński, Commented Feb 8, 2022 at 20:06
As hinted by the professor, before trying to split the toppings, you most likely need an if clause to check if the toppings input is an empty string or not. - if it is an empty string, you will not need to use .split(',') and maybe give a message. A technique that might be useful to you is using a debugger, or just plain print debugging with a lot of console.log — Bao Huynh
– Bao Huynh, Commented Feb 8, 2022 at 20:21
@BaoHuynhLam thank you! I thought I needed an if / else statement (but when I asked my teacher he said we didn't) but I just went ahead and added one and I got it to work along with some other changes suggested. Thank you so much!! — Cheryl Ju
– Cheryl Ju, Commented Feb 8, 2022 at 21:02

Rodion · Accepted Answer · 2022-02-08 20:24:55Z

0

Access element through index. And first of all You should declare pizza

let pizza;
pizza.toppings = [];
      pizza.toppings = toppings.split(',');
      for (let i = 0; i < pizza.toppings.length; i++){
       pizza.cost += +pizza.toppings[i] * toppingsFee;
      }

answered Feb 8, 2022 at 20:24

Rodion

376 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Cheryl Ju · Accepted Answer · 2022-02-08 21:05:56Z

0

I figured it out with some help of the comments. I needed to create an if / else statement around my for statement :) I got it to work now!

answered Feb 8, 2022 at 21:05

Cheryl Ju

135 bronze badges

Collectives™ on Stack Overflow

User Prompt Array Question for Javascript

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related