Getting [object] when passing parameter to javascript function

Question

I have the following function that takes three arguments.

initializeDateDropdown("year", "month", "day");

    function initializeDateDropdown(year1,month1,day1) {
        var currentYear = new Date().getFullYear();
        $("#"+year1).append(function () {
            var yearList = '';
            for (var i = 1951; i <= currentYear; i++) {
                yearList += '<option value="' + i + '">' + i + '</option>';
            }
            return yearList;
        }).val(currentYear)
      .add($('#'+month1).val(new Date().getMonth()))
      .bind('change', showDays);

        showDays(year1, month1, day1);

        $('#'+day1).val(new Date().getDate());


    }

In the definition of the showDays() method, when I try to display the year1, I keep getting [object] [object] and month1 and day1 as undefined.

Can anybody tell me what is wrong with this code?

Below is the showDays method defination.

function showDays(year, month, day) {
        alert(year);
        var days = new Date($("#" + year).val(), parseInt($('#' + month).val(), 10) + 1, 0).getDate();

        var prevSelectedDate = $('#' + day).val();
        $('#' + day).empty();
        for (var i = 1; i <= parseInt(days, 10) ; i++) {
            $('#' + day).append($('<option />', { text: i, value: i }));
        }

        $('#' + day).val(prevSelectedDate);
    }

Please help me with this.

Your trying to use that function as an event handler. When called in that case, it'll be passed an event object. — Pointy
– Pointy, Commented Aug 23, 2013 at 13:59
@PM : First initializeDateDropdown will be called then showDays will be called — PaRsH
– PaRsH, Commented Aug 23, 2013 at 14:05

rink.attendant.6 · Accepted Answer · 2013-08-23 14:08:47Z

4

When you do .bind('change', showDays), this is the equivalent:

.bind('change', showDays(event));

Where event is the event object.

You'll need to change that part of the function to:

.bind('change', function(){
    showDays(year1, month1, day1);
});

Since you need to pass parameters to the function (year1, month1, day1), you need to wrap it in an anonymous callback function.

On a secondary note, .bind is deprecated. Use .on():

.on('change', function(){
    showDays(year1, month1, day1);
});

edited Aug 23, 2013 at 14:08

answered Aug 23, 2013 at 14:02

rink.attendant.6

46.7k64 gold badges111 silver badges161 bronze badges

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

Collectives™ on Stack Overflow

Getting [object] when passing parameter to javascript function

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