Javascript adding/removing values

Question

I have this JS Code:

$("#submit").on('click',function() {
    //work out number of days between the two dates
    var days_between = $("#todate").val() - $("#fromdate").val()

    //do the cost per month times 12 (months)
    var year_cost = $("#cost_per_month").val() * 12

    //do the yearly cost / 365
    var daily_cost = year_cost / 365
    var daily_cost = parseFloat( daily_cost.toFixed(2) )

    //now do the daily cost times cost_per_month
    var total_cost = daily_cost * days_between

    $(".total_days").html(days_between);
    $(".total_cost").html(total_cost);
})

I am getting an error saying NaN though.

i am entering the following:

#from_date = 2014-08-19
#to_date = 2014-08-31
#cost_per_month = 2.60

Not suprisingly, "2014-08-19" - "2014-08-31" is not a number ? — adeneo
– adeneo, Commented Aug 22, 2014 at 10:21
It would be useful to know if you're using a datepicker or just typing the dates. If you're using something like jQuery's datepicker it would give you totally different answers. — adeneo
– adeneo, Commented Aug 22, 2014 at 10:31

Community · Accepted Answer · 2017-05-23 12:29:46Z

2

The way you are calculating number of dayes between the dats is wrong. look into this How do I get the number of days between two dates in JavaScript? This could be helpful!

edited May 23, 2017 at 12:29

CommunityBot

11 silver badge

answered Aug 22, 2014 at 10:29

Navaneeth

2,5721 gold badge20 silver badges39 bronze badges

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

Comments

cracker · Accepted Answer · 2014-08-22 10:27:32Z

1

This Will Work

    //work out number of days between the two dates
var oneDay = 24 * 60 * 60 * 1000; 
//var date1 = new Date("2014,08,31");
//var date2 = new Date("2014,08,19");

var date1 = new Date("2014-08-31");
var date2 = new Date("2014-08-19");

var Daysd = date1.getDate() - date2.getDate();

    //var days_between = date1 - date2
    var diffDays = Math.round(Math.abs((date1.getTime() - date2.getTime()) / (oneDay)));

    //do the cost per month times 12 (months)
    var year_cost = parseInt(2.60) * 12

alert(Daysd);
alert(year_cost);

DEMO

answered Aug 22, 2014 at 10:27

cracker

4,9043 gold badges26 silver badges41 bronze badges

4 Comments

adeneo Over a year ago

Close, but no cigar, the OP is getting the values as strings from inputs.

cracker Over a year ago

@adeneo, how about now?

adeneo Over a year ago

That is closer, and the OP could insert the strings from the values, so I'll give it a +1

cracker Over a year ago

@user3843997, you are wel come to accept the answer. plz do it :) ;)

Amit Kumar · Accepted Answer · 2014-08-22 10:28:31Z

0

you can't get total days directly

   var tDate = new Date($("#todate").val());
    var fDate = new Date($("#fromdate").val());
    var diff=tDate-fDate;

    This would give you the difference in milliseconds between the two dates.

    var DaysNo= diff / 1000 / 60 / 60 / 24;

answered Aug 22, 2014 at 10:28

Amit Kumar

5,96212 gold badges49 silver badges88 bronze badges

Comments

Milind Anantwar · Accepted Answer · 2014-08-22 10:21:38Z

-1

You have not parsed textbox values to int:

var days_between = parseInt($("#todate").val()) - parseInt($("#fromdate").val())

and

var year_cost = parseInt($("#cost_per_month").val()) * 12

answered Aug 22, 2014 at 10:21

Milind Anantwar

82.3k26 gold badges97 silver badges128 bronze badges

4 Comments

adeneo Over a year ago

That's like writing parseInt("2014-08-19") - parseInt("2014-08-31"), I'm not sure you'd get the expected result ?

Luca B. Over a year ago

Yep parseInt("2014-08-19" ) yields just the first int it finds

Milind Anantwar Over a year ago

I dont think that they are in date format. OP would have not done the operation of multiplying with integer values.

Luca B. Over a year ago

You sould use new Date().getTime(string); to get the date parsed in ms

Collectives™ on Stack Overflow

Javascript adding/removing values

4 Answers 4

Comments

DEMO

4 Comments

Comments

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

4 Comments

Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related