Как правильно передать параметры в конструктор объекта?

Question

[ppxy]

Есть вот такой участок кода:

function ODESolver(equations) {
        this.f = [];
        this.initY = [];
        this.initX = [];
        this.dt = [];
        this.t = [];

        var i = 0;
        
        for (var eq in equations) {
            for (var param in equations[eq]) {
                if (param === "initX")
                    this.initX[i] = equations[eq][param];
                if (param === "initY")
                    this.initY[i] = equations[eq][param];
                if (param === "f")
                    this.f[i] = equations[eq][param];
                if (param === "dt")
                    this.dt[i] = equations[eq][param];
                if (param === "t")
                    this.t[i] = equations[eq][param];
            }
            i++;
        }
    }

Необходимо передавать в данный конструктор объекта параметры описывающие различные начальные условия для дифференциальных уравнений (конечно не важно, что будет передаваться. Просто на примере понятнее).
На данный момент, задание и передача параметров происходит следующим образом:

var params = {
            eq1: {
                initX: 1,
                initY: 2,
                dt: 3,
                t: 4,
                f: function() {
                    return 1;
                }
            },
            eq2: {
                initX: 5,
                initY: 6,
                dt: 7,
                t: 8,
                f: function() {
                    return 2;
                }
            }
        };
        
        var odes = new ODESolver(params);

Возник вопрос, можно ли делать данную операцию более "красиво" и правильнее?

Answer 1

[Денис Пушкарев]

На вскидку: "объект параметры" лучше заменить на "массив параметры" - как вижу, его ключи тут не важны, соответственно и первый цикл на обычный - тем более всё равно используете индекс i. Второй, вложенный, цикл вообще не нужен. Хотя если нужна краткость, ужал бы вообще до такого:

function ODESolver(equations){
  var keys = ['f', 'initY', 'initX', 'dt', 't'];
  equations.forEach(function(eq){
    keys.forEach(function(key){
      key in this || (this[key] = []);
      this[key].push(eq[key]);
    }, this);
  }, this);
}

new ODESolver([
  {
    initX: 1,
    initY: 2,
    dt: 3,
    t: 4,
    f: function(){
      return 1;
    }
  }, {
    initX: 5,
    initY: 6,
    dt: 7,
    t: 8,
    f: function(){
      return 2;
    }
  }
]);

Comments

[ppxy]

Ключи могут быть важны, когда я захочу передавать параметры в произвольном порядке. Так наверное делать не стоит, но лучше наверное предусмотреть такой вариант.

[Денис Пушкарев]

ppxy: массив мешает передать параметры в произвольном порядке? :)

[ppxy]

нет, но я буду считывать их синхронно и заносить в каждый определенный массив. не возникнет ли путаница на этом этапе, если использовать массивы. Сейчас проверю.

Answer 2

[aen]

Как вариант:

function ODESolver(equations) {
  this.f = [];
  this.initY = [];
  this.initX = [];
  this.dt = [];
  this.t = [];

  var i = 0;

  for (var eq in equations) {
    for (var param in equations[eq]) {
      this[param][i] = equations[eq][param];
    }
    i++;
  }
}

Comments

[ppxy]

данный вариант приемлем, если свойства объекта идут в строгой последовательности. Если они будут идти в произвольном порядке, данный способ будет провальным. Так ведь?