JS - naplnění assoc array pomocí proměných

[ZAJDAN]

Ahoj,
nejsem si jist jak v JS vytvořit/naplnit assoc array. Napadají mě dva způsoby, ale nechám si doporučit jak to dělat správně.

Kód [Vybrat] Rozbalit


METODA 1

var Commission = $("#Commission").val();
var Letter = $("#Letter").val();
var Status = $("#Status").val();
var DeliveryDate = $("#DeliveryDate").val();
//Array
var dataString = {Commission: Commission, Letter: Letter, Status: Status, DeliveryDate: DeliveryDate};

METODA 2
var Array = [];
      Array["Letter"] = $("#Letter").val();
      Array["Commission"] = $("#Commission").val();
      Array["Status"] = $("#Status").val();
      Array["DeliveryDate"] = $("#DeliveryDate").val();



[Reklama]

[Filip Jirsák]

Ve druhém případě zbytečně vytváříte pole, když pak objekt jako pole nepoužíváte. K výsledku povedou oba způsoby, ale oba jsou zbytečně ukecané. Stačí tohle:

Kód [Vybrat] Rozbalit


var dataString = {
  Commission: $("#Commission").val(),
  Letter: $("#Letter").val(),
  Status: $("#Status").val(),
  DeliveryDate: $("#DeliveryDate").val()
};


A mimochodem, nejde o čistý JavaScript, máte tam použité jQuery (ta funkce $(...)).

[gill]

Asi bych volil prvni verzi bez tech nadbytecnych promenych.

Jinak bych doporucil uzavrit to do formulare a misto id pouzivat name.

[ZAJDAN]

díky hoši

[to_je_jedno]

var Array = [];

tohle uz priste nedelej, tohle je fakt spatny pojmenovani.

[Reklama]

[ZAJDAN]

Ten název byl pouze pro studijní účely! Samozřejmě že v provozu si to pojmenuju dle potřeby.

[gill]

var Array = [];

tohle uz priste nedelej, tohle je fakt spatny pojmenovani.

Hlavne by to mel byt objekt {}. Pojmenovani by nevadilo, kdyby se to pouzivalo jako pole.