Simple JavaScript empty string check, what does (an empty string) mean and why is it failing?

Question

Here's my code:

$("#ddlCiudad").change(function () {
    var idCity = $("#ddlCiudad").val();
    $.getJSON("/ProductCheckout/GetPriceForLocation", { cityId: idCity, productId: idProduct, type: "land" },
        function (cityData) {

            console.log("Recieved json data.");                    

            landCost = cityData.LandCost;
            $("#billingshippingcost").text(landCost);
            console.log("Assigned value of LandCost");

            airCost = cityData.AirCost;
            console.log("Assigned value of AirCost");

            console.log(landCost); //Logs: 25,00
            console.log(airCost); //Logs: "(an empty string)"

            if (landCost == "") {
                $(".land").removeClass("land").addClass("land-disabled");
            }
            else {
                $(".land-disabled").removeClass("land-disabled").addClass("land");
            }

            if (airCost = "") {
                $(".air").removeClass("air").addClass("air-disabled");
            }
            else {
                $(".air-disabled").removeClass("air-disabled").addClass("air");
            }

        }
    );
});

That if statement is not being fired, any suggestions on why it's not firing?

Maybe an empty string isn't the same as "" in Javascript.

@NullUserExceptionఠ_ఠ Yeah I fixed that already. :) The error still stands however. That condition is not passing. — Only Bolivian Here
– Only Bolivian Here, Commented Nov 4, 2011 at 13:53
Are you sure? Add something like console.log('airCost is empty') in the if block just to check if it gets there (and it's just the classes that aren't being added and removed correctly) — NullUserException
– NullUserException, Commented Nov 4, 2011 at 13:54
Btw, that edit doesn't apply. It completely changes my questions intent. — Only Bolivian Here
– Only Bolivian Here, Commented Nov 4, 2011 at 13:54

Richard Dalton · Accepted Answer · 2011-11-04 13:56:26Z

1

Try:

if (!airCost) {
    $(".air").removeClass("air").addClass("air-disabled");
}
else {
    $(".air-disabled").removeClass("air-disabled").addClass("air");
}

answered Nov 4, 2011 at 13:56

Richard Dalton

35.8k6 gold badges75 silver badges92 bronze badges

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

Collectives™ on Stack Overflow

Simple JavaScript empty string check, what does (an empty string) mean and why is it failing?

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related