Why isn't my array being populated?

Question

I have a javascript function that returns 2 Arrays. Each array element is an array, and each of those arrays contains a coursject object.

I know this is complicated, but the other people on my team are in charge of generating possible schedules from the classes and they want me to return to them like that.

I've run some test on what I have, and for some reason the second level of arrays isn't getting populated or defined. Like if I put one optional class as my input (we have a UI), the OptcourseArray has elements that are undefined.

Below is just the code for creating the OptcourseArray. Does it seem to correctly make an array of arrays of objects? I think I must have messed something up in it.

For code context:

numOptCourses is the number of optional courses.
optCourses is an array of course objects.
courseNumber is the course number of the class, as is catalog_num. This is so that classes with multiple sections go into the same array.

var OptcourseArray = [];  
var catNum = 0;

for(var j = 0; j < numOptCourses; j++){
  catNum = optCourses[j].courseNumber;
  var myArray = [];

  for(var h = 0; h < OptclassList.length; h++){
    if (OptclassList[h].catalog_num === catNum){
      myArray.push(OptclassList[h]);
    }
  } 

  OptcourseArray.push(myArray);
}

IrishGeek82 · Accepted Answer

Acoustic77,

Here is the modified queryCourseData method.

I converted your $.getJSON to $.ajax using async:false (this may or may not have been an issue on my end, so I encourage you to try and set it to true and test whether it works on your end or not).

I then noticed that your startTime and endTime are in the format {hour:#,minute:#} while the item.start_time and item.end_time are time strings in 24 hour format. I wrote some code to convert the former form into 24 hour format ( I am certain there is a more elegant way of doing this than I did).

I also later, after my initial answer, noticed that you were setting myArray to [] every step of your inner for loops where you are constructing the ReqcourseArray and OptcourseArrays. Moving var myArray=[] outside of the inner for was my final fix.

I left my console.log's in place so you could see the results.

function queryCourseData(startTime, endTime, optCourses, reqCourses, numOptCourses, numReqCourses)
{

var numClasses = optCourses.length;
var OptclassList = [];
var i = 0;
for(var m = 0; m < numClasses; m++)
{
    //IrishGeek82@SO, 2014-06-05
    //Your ajax calls were taking too long to call so the code that needed them was
    //getting called before the data was ready.
    //This could be a problem on my end so you can always set async to true and test from your end.
    $.ajax({url:"http://vazzak2.ci.northwestern.edu/courses/?term=4540&subject="+optCourses[m].subject, 
            async: false,
            dataType: 'json',
            success:function(result) 
                    {
                        //IrishGeek82@SO, 2014-06-05
                        //Your start and end times are objects of the format
                        //{hour:x, minute:x}
                        //While your item.start_time and item.end_time are 24 hour time strings.
                        //I am sure there is a more elgant way to do this but here is a dirty conversion
                        //from one to the other.
                        var sTime = (startTime.hour<10?"0"+startTime.hour:startTime.hour) + ":" + startTime.minute+"00";
                        var eTime = (endTime.hour<10?"0"+endTime.hour:endTime.hour) + ":" + endTime.minute+"00";
                        $(result).each(function (index, item) 
                                       {
                                            if (item.start_time > sTime) 
                                            {
                                                if (item.end_time < eTime)
                                                {
                                                    if (item.catalog_num == optCourses[m].courseNumber)
                                                    {
                                                        var coursject = {
                                                                            title: item.title,
                                                                            professor: item.instructor.name,
                                                                            catalog_num: item.catalog_num,
                                                                            section: item.section,
                                                                            subject: item.subject,
                                                                            meeting_days: item.meeting_days,
                                                                            start_time: item.start_time,
                                                                            end_time: item.start_time
                                                                        };

                                                        //IrishGeek82@SO
                                                        //Now Pushing Entries Into Array
                                                        OptclassList.push(coursject);
                                                        i++;
                                                    }
                                                }
                                            }        
                                        });
                    }
          });

}    

var OptcourseArray = [];    
for(var j = 0; j < numOptCourses; j++)
{
    var catNum = optCourses[j].courseNumber;
    //IrishGeek82@SO
    //You were resetting your myArray every time you in the loop below.
    //Subsequently, only the last entry would every get added and you were
    //getting empty arrays.
    var myArray = [];        
    for(var h = 0; h sTime) 
                                            {

                                                if (item.end_time < eTime)
                                                {

                                                    if ($.trim(item.catalog_num) == $.trim(reqCourses[n].courseNumber))
                                                    {
                                                        var coursject = {
                                                                            title: item.title,
                                                                            professor: item.instructor.name,
                                                                            catalog_num: item.catalog_num,
                                                                            section: item.section,
                                                                            subject: item.subject,
                                                                            meeting_days: item.meeting_days,
                                                                            start_time: item.start_time,
                                                                            end_time: item.start_time
                                                                        };
                                                        //IrishGeek82@SO
                                                        //Now Pushing Entries Into Array
                                                        ReqclassList.push(coursject);
                                                        g++;
                                                    }
                                                }
                                            }        
                                        });


                      }
            });
}

var ReqcourseArray = [];    
for(var j = 0; j < numReqCourses; j++)
{
    var catNum = reqCourses[j].courseNumber;
    //IrishGeek82@SO
    //You were resetting your myArray every time you in the loop below.
    //Subsequently, only the last entry would every get added and you were
    //getting empty arrays.
    var myArray = [];
    for(var h = 0; h < ReqclassList.length; h++)
    {
            if ($.trim(ReqclassList[h].catalog_num) == $.trim(catNum))
            {
                myArray.push(ReqclassList[h]);
            }
    }   

    ReqcourseArray.push(myArray);
}

console.log("--REQ--");
console.log(JSON.stringify(ReqcourseArray));
console.log("--REQ--");

return [OptcourseArray, ReqcourseArray];

}

The results of my testing are as follows:

Test Case:

2 courses
EECS 214-0  (optional)
EECS 223-0  (required)

Results:

XHR finished loading: GET "http://vazzak2.ci.northwestern.edu/courses/?term=4540&subject=EECS". 
--OPT--
[[{"title":"Data Structures & Data Management","professor":"Morteza Amir Rahimi","catalog_num":"214-0","section":"21","subject":"EECS","meeting_days":"MoWeFr","start_time":"11:00:00","end_time":"11:00:00"}]]
--OPT--
XHR finished loading: GET "http://vazzak2.ci.northwestern.edu/courses/?term=4540&subject=EECS". jquery.min.js:4
--REQ--
[[{"title":"Fundamentals of Solid State Engineering","professor":"Koray Aydin","catalog_num":"223-0","section":"01","subject":"EECS","meeting_days":"MoTuWeFr","start_time":"09:00:00","end_time":"09:00:00"}]]
--REQ--

Test Case:

5 courses:
EECS 214-0 (optional)
EECS 223-0 (required)
EECS 110-0 (required)
EECS 213-0 (required)
EECS 203-0 (optional)

Results:

XHR finished loading: GET "http://vazzak2.ci.northwestern.edu/courses/?term=4540&subject=EECS". jquery.min.js:4
XHR finished loading: GET "http://vazzak2.ci.northwestern.edu/courses/?term=4540&subject=EECS". jquery.min.js:4
--OPT--
[[{"title":"Data Structures & Data Management","professor":"Amartya Banerjee","catalog_num":"214-0","section":"20","subject":"EECS","meeting_days":"MoWeFr","start_time":"09:00:00","end_time":"09:00:00"},{"title":"Data Structures & Data Management","professor":"Morteza Amir Rahimi","catalog_num":"214-0","section":"21","subject":"EECS","meeting_days":"MoWeFr","start_time":"11:00:00","end_time":"11:00:00"}],[{"title":"Introduction to Computer Engineering","professor":"Hai Zhou","catalog_num":"203-0","section":"01","subject":"EECS","meeting_days":"MoWeFr","start_time":"11:00:00","end_time":"11:00:00"}]]
--OPT--
XHR finished loading: GET "http://vazzak2.ci.northwestern.edu/courses/?term=4540&subject=EECS". jquery.min.js:4
XHR finished loading: GET "http://vazzak2.ci.northwestern.edu/courses/?term=4540&subject=EECS". jquery.min.js:4
XHR finished loading: GET "http://vazzak2.ci.northwestern.edu/courses/?term=4540&subject=EECS". jquery.min.js:4
--REQ--
[[{"title":"Fundamentals of Solid State Engineering","professor":"Koray Aydin","catalog_num":"223-0","section":"01","subject":"EECS","meeting_days":"MoTuWeFr","start_time":"09:00:00","end_time":"09:00:00"}],[{"title":"Introduction to Computer Programming","professor":"Aleksandar Kuzmanovic","catalog_num":"110-0","section":"20","subject":"EECS","meeting_days":"MoTuWeFr","start_time":"10:00:00","end_time":"10:00:00"}],[{"title":"Introduction to Computer Systems","professor":"Peter A Dinda","catalog_num":"213-0","section":"20","subject":"EECS","meeting_days":"TuTh","start_time":"14:00:00","end_time":"14:00:00"}]]
--REQ--

Please let me know if this helps :)

Why isn't my array being populated?

Answers (1)

Related Questions

Why isn&#39;t my array being populated?

Answers (1)

Related Questions

Why isn't my array being populated?