Machen Funktionen warten, bis AJAX-Aufruf mit jQuery

Folgende änderungen können Sie machen, der AJAX-request nur einmal und rufen Sie Ihre Funktion wie

courses.getCourse('xyz', function(course){
    //Use course here
});

Hier sind die änderungen

var model_courses = (function() {

    //This is what gets returned by the $.ajax call
    var xhr;
    var _storage = {}; //Used for storing course related info
    _storage.courses = {}; //Used for accessing courses directly
    _storage.references = []; //Stores all available course IDs

    var cls = function () {
        xhr = $.ajax({
            type: "GET",
            url: "data/courses.xml",
            dataType: "xml",
            success: function(xml) {
                $(xml).find("course").each(function() {

                    _storage.courses[$(this).attr('id')] = {
                        title       : $(this).find('title').text(),
                        description : $(this).find('description').text(),
                        points      : $(this).find('points').text()
                    }
                    _storage.references.push($(this).attr('id'))

                });
            }
        });
    }
    cls.prototype = {
            //Made changes here, you'd have to make the same 
            //changes to getCourses and getReferences
            getCourse: function (courseID, callback) {
                if (xhr.readyState == 4) {
                     callback(_storage.courses[courseID]);
                }
                else {
                   xhr.done(function(){
                      callback(_storage.courses[courseID]);
                   })
                }

            },
            getCourses: function () {
                return _storage.courses
            },
            getReferences: function (),
                return _storage.references
            }

    }
    return cls
})()

Als eine Randnotiz, dass Ihr Modul Muster wird nicht sehr gut funktionieren, wenn Sie brauchen, um zu instanziieren zwei dieser model_courses Objekte, da die storage-Objekte werden alle gemeinsam in Ihrem selbst aufrufende Funktion die Schließung. Sie in der Regel nicht mischen, das Modul-Muster mit Prototypen (gibt einen Konstruktor aus einem Modul), es sei denn, Sie wirklich wissen, was Sie tun, ist, dass die shared closure-Variablen arbeiten, die als statische Eigenschaften Ihrer Klasse.

Dies ist, was ich tun würde, wenn ich du wäre (da Sie wirklich wollen, private Variablen)

function ModelCourses() {
    var storage = {
      courses: {},
      references: []
    };

    var xhr = $.ajax({
        type: "GET",
        url: "data/courses.xml",
        dataType: "xml",
        success: function(xml) {
            $(xml).find("course").each(function() {   
                storage.courses[$(this).attr('id')] = {
                    title       : $(this).find('title').text(),
                    description : $(this).find('description').text(),
                    points      : $(this).find('points').text()
                }
                storage.references.push($(this).attr('id'))
            })    
        }
    });

    this.getCourse = function(courseId, callback) {
        function getCourse() {
            callback(storage.courses[courseID])
        }
        if (xhr.readyState == 4) {
            getCourse();
        }
        else {
            xhr.done(getCourse);
        }
    };   
}

in getStorage entweder einen check, um zu sehen, ob es irgendwelche Daten zu stehlen (bevorzugt), oder das "eigentliche" Methode als private zu veröffentlichen ist, wenn es hat Elemente, die es zugreifen kann. (Ich würde empfehlen, die erste aber sonst bekommen Sie die Ausnahmen über eine Methode aufrufen, die nicht existiert, auf ein Objekt).