Need help with a JavaScript flight calculator please

Hi Steven, I was taking some treehouse courses a while back (HTML & CSS) but had other studies so I had to put my Treehouse subscription on hold, Iv'e tried to just get the location and destination total by writing:

const totalCost = getSelectValue() + getSelectValue2(); and

const totalCost = selectedValue + selecedtValue2;

but none of them seem to work. Like you said I have a good understanding, I know what needs to be done, it's just how...

Your select functions (like "getSelectValue2") currently only log the values to the console. To be able to use them in a formula they need to return the values instead of (or in addition to) logging them.

function getSelectValue2() {
    const selectedValue2 = document.getElementById('destination').value;
//  console.log(selectedValue2);  // optional
    return selectedValue2;        // return the value to the caller
}

Hi Steven. I've been playing around with things and returned the functions as you suggested. But it only seems to pull the option value from the first option in each list. It also doesn't add them together, it just puts them together. So when I click the calculate button it shows 'Your total flight cost is 00'.

/* Total Cost */

const totalCost = getSelectValue() + getSelectValue2();

function total(){ window.alert ("Your total flight cost is " + totalCost); }

/* Value Functions */

function getSelectValue(){ const selectedValue = document.getElementById('location').value; //console.log(selectedValue) return selectedValue; }

function getSelectValue2(){ const selectedValue2 = document.getElementById('destination').value; //console.log(selectedValue2) return selectedValue2; }

Well, you can't perform the calculation when the program loads, because nothing has been selected yet. It has to be done inside the click handler:

function total() {
  const totalCost = getSelectValue() + getSelectValue2();
  window.alert("Your total flight cost is " + totalCost);
}

Thank you Steven! I'm almost there, I just need them to mathematically add together, instead of just being placed next to each other. I was told in college to look into parseInt but I cannot seem to get it to work. Sorry for being a pain aha. But I highly appreciate your help. Thanks again.

I wonder where you placed the parseInt. One way to do it would be to use it when you return a value:

    return parseInt(selectedValue);

Thank you Steven. It's now functioning properly :) Now i've gotta figure out how to multiply the value by the amount of adults, minus £.25.00 for each child selected and add 10 if the meal and legroom radio buttons are selected yes...

Now it's just plain math:

function total() {
  const totalCost = 
        // location + destination + class for each adult
        (getSelectValue() + getSelectValue2() + getSelectValue5()) * getSelectValue3()
        // location + destination + class - discount for each child
      + (getSelectValue() + getSelectValue2() + getSelectValue5() - 25) * getSelectValue4()
        // legroom for each passenger
      + legroom() * (getSelectValue3() + getSelectValue4())
        // meal for each passenger
      + meal() * (getSelectValue3() + getSelectValue4());
  window.alert("Your total flight cost is " + totalCost);
}

Note this relies on "legroom" and "meal" functions that will return either 10 or 0 based on the buttons. I'll leave those to you.

Thank you so much Steven!