Increment a value if ... Javascript

Question

I am working on a small piece of code that looks to see if a value is equal to 'December 25 - January 9' and if it does increment the year (that I append to it) by one

However I can not figure out how to find that value. Every time I run the script it skips to the else part of the script when that value is selected. Any guidance is appreciated

if ($('#date').val === 'December 25 - January 9') {
                    var startDates = $('#date').val().split(" - ");
                    var year = $('year').val;
                    var yearDec = parseInt(year, 10) + 1;
                    var payPdStart = startDates[0] + ' ' + year;
                    var payPdEnd = startDates[1] + ' ' + yearDec;
                    var startDate = Date.parse(payPdStart);
                    myStartDates = new Date(startDate);
                    var endDate = Date.parse(payPdEnd);
                    myEndDates = new Date(endDate)
                    while (myStartDates <= myEndDates) {
                        var firstCol = "<td style='border:none;'>" + myStartDates + "</td>";
                        $('#timeTable').append("<tr>" + firstCol + "</td>");

                        var newDate = myStartDates.setDate(myStartDates.getDate() + 1);
                        start = new Date(newDate);
                    }
                }
                else{
                    var startDates = $('#date').val().split(" - ");
                    var year = $('#year').val() ;
                    var payPdStart = startDates[0] + ' ' + year;
                    var payPdEnd = startDates[1] + ' ' + year;
                    var startDate = Date.parse(payPdStart);
                    myStartDates = new Date(startDate);
                    var endDate = Date.parse(payPdEnd);
                    myEndDates = new Date(endDate)

                    console.log(myStartDates);
                    console.log(myEndDates);
                    while (myStartDates <= myEndDates) {
                        var firstCol = "<td style='border:none;'>" + myStartDates + "</td>";
                        $('#timeTable').append("<tr>" + firstCol + "</td>");

                        var newDate = myStartDates.setDate(myStartDates.getDate() + 1);
                        start = new Date(newDate);
                    }

.val() not .val - it's a function, and you have to call it. — Pointy
– Pointy, Commented Jul 9, 2014 at 15:02
Which you seem to do correctly several times through the rest of your code, maybe just a typo? — user400654
– user400654, Commented Jul 9, 2014 at 15:03
It's questions like this that reaffirm my belief that jQuery, as powerful a tool as it is, makes people ask the worst questions... — Niet the Dark Absol
– Niet the Dark Absol, Commented Jul 9, 2014 at 15:07

Matas Vaitkevicius · Accepted Answer · 2014-07-09 15:16:04Z

3

Problem is you are using JQuery but chexking value as in javascript

$('#date')[0].value === 'December 25 - January 9'

or

$('#date').val() === 'December 25 - January 9'

should do it.

edited Jul 9, 2014 at 15:16

answered Jul 9, 2014 at 15:06

Matas Vaitkevicius

61.9k37 gold badges254 silver badges278 bronze badges

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

Comments

Elliott Frisch · Accepted Answer · 2014-07-09 15:06:24Z

1

I see two places where you used val instead of val(), you want to call the function val not access a non-existent val field

if ($('#date').val() === 'December 25 - January 9') { // <-- val()
  var startDates = $('#date').val().split(" - ");
  var year = $('year').val(); // <-- val()

answered Jul 9, 2014 at 15:06

Elliott Frisch

202k20 gold badges166 silver badges265 bronze badges

Comments

tgbrunet · Accepted Answer · 2014-07-09 15:14:55Z

Try this. It should work.

You have var should be var() also year should be #year

if ($('#date').val() === 'December 25 - January 9') {
                    var startDates = $('#date').val().split(" - ");
                    var year = $('#year').val();
                    var yearDec = parseInt(year, 10) + 1;
                    var payPdStart = startDates[0] + ' ' + year;
                    var payPdEnd = startDates[1] + ' ' + yearDec;
                    var startDate = Date.parse(payPdStart);
                    console.log(startDates);
                    console.log(year);
                    console.log(yearDec);
                    console.log(payPdStart);
                    console.log(payPdEnd);
                    myStartDates = new Date(startDate);
                    var endDate = Date.parse(payPdEnd);
                    myEndDates = new Date(endDate)
                    while (myStartDates <= myEndDates) {
                        var firstCol = "<td style='border:none;'>" + myStartDates + "</td>";
                        $('#timeTable').append("<tr>" + firstCol + "</td>");

                        var newDate = myStartDates.setDate(myStartDates.getDate() + 1);
                        start = new Date(newDate);
                    }
                }
                else{
                    var startDates = $('#date').val().split(" - ");
                    var year = $('#year').val() ;
                    var payPdStart = startDates[0] + ' ' + year;
                    var payPdEnd = startDates[1] + ' ' + year;
                    var startDate = Date.parse(payPdStart);
                    myStartDates = new Date(startDate);
                    var endDate = Date.parse(payPdEnd);
                    myEndDates = new Date(endDate)

                    console.log(myStartDates);
                    console.log(myEndDates);
                    while (myStartDates <= myEndDates) {
                        var firstCol = "<td style='border:none;'>" + myStartDates + "    </td>";
                        $('#timeTable').append("<tr>" + firstCol + "</td>");

                        var newDate = myStartDates.setDate(myStartDates.getDate() + 1);
                        start = new Date(newDate);
                    }

Collectives™ on Stack Overflow

Increment a value if ... Javascript

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related