Question

I need to create a certain number of objects based on user-selected drop-down list. Now I have this code.

var driversFunc = function () {

var driversCount = osagoViewModel.fields.driversCount.selectedValue();

switch (driversCount) {

    case "many":
        return [];
    case "1":
        return [new driver()];
    case "2":
        return [new driver(), new driver()];
    case "3":
        return [new driver(), new driver(), new driver()];
    case "4":
        return [new driver(), new driver(), new driver(), new driver()];
    case "5":
        return [new driver(), new driver(), new driver(), new driver(), new driver()];
    }
};

(knockout.js is used)

Answer 1

Your switch might be able to be replaced with:

if (driversCount === "many") {
  return [];
} else {
  var count = parseInt(driversCount, 10);
  var l = [];
  while (count--) l.push(new driver());
  return l;
}

I don’t understand why many maps to [].