Why is my class undefined at runtime. Javascript module pattern

Question

When i run my application i get: "Uncaught TypeError: Cannot read property 'init' of undefined" The alert in the document reasdy function runs, so i know jQuery is loaded at that point. But i don't know why Task is undefined.

This is in a MVC5 application, and running jQuery 1.10.2

Please help, jsFiddle works fine, and JsHint shows no errors.

    var Task = (function () {

    var initialize = function () {
        console.log($);
        bindEvents();
    };
    var postFilter = function () {
        console.log("clicked");
        var postData = {
            UserID: $('#user_select').val,
            TaskTypeID: $('#taskTypeSelect').val,
            TaskStatusID: $('#status_select').val,
            PriorityID: $('#priority_select').val
        };

        $.ajax({
            url: "/Tasks/_AllTaskView",
            data: postData,
            success: function (response) {
                $('#results').html(response);
            }

        });
    };

    var bindEvents = function () {
        $('#submit_filter').on('click', postFilter);
    };

    return
    {
        init : initialize

    };

})();

$(document).ready(function () {
    alert()
    Task.init();
});

I am upvoting simply because I had no idea that return does not allow its value starting on subsequent lines (something every JS coder should be made aware of) :) — iCollect.it Ltd
– iCollect.it Ltd, Commented Dec 11, 2014 at 14:27

Mike · Accepted Answer · 2014-12-11 14:21:25Z

6

Put the curly brace of your return on the same line as return.

Instead of:

return
{
    init : initialize

};

Do this:

return {
    init : initialize
};

JavaScript has automatic semi-colon insertion which will automatically end the return statement before it hits the curly brace.

answered Dec 11, 2014 at 14:21

Mike

4,09125 silver badges36 bronze badges

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

2 Comments

iCollect.it Ltd Over a year ago

My bad (stupid inconsistent language.. give me C# any day ) - comment removed. Hey, it encouraged 3 other upvotes... don't complain :)

A. Wolff Over a year ago

As a completely useless side note, this one works too: jsfiddle.net/3s6n4d5y/2

SilentTremor · Accepted Answer · 2014-12-11 15:16:01Z

0

Here try this:

return{ init : function(){
                 initialize();
               }
      }

answered Dec 11, 2014 at 15:16

SilentTremor

4,9622 gold badges25 silver badges38 bronze badges

Collectives™ on Stack Overflow

Why is my class undefined at runtime. Javascript module pattern

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related