[javascript]配列の中の連想配列に追加

とあるゲームを作成中で、配列の中に　"point"と"name"というキーを持ったユーザーごとの連想配列を追加して、ランキングとして表示しようとしています。

ローカルストレイジから得点と名前を取得し、それを元に連想配列を作ります。

そしてそれを　"users" という配列にゲームごとに追加し、最後にpointを元にソート、表示、という流れを作りたいのですが、2回目行こうの代入により全て同じ名前、得点になってしまいます。

改善するべきとこはどこでしょうか？

よろしくお願いいたします。

js
1var users = [];
2var pArray = new Array();
3
4    var rk = document.getElementById("rankPlace");
5    var strgName = window.localStorage.getItem("player");
6    var getP = document.getElementById("rankPlace");
7    getP.innerHTML = "";
8
9　　//連想配列を作り、それを配列に追加
10    pArray["point"] = a_count;
11    pArray["name"] = strgName;
12    users.push(pArray);
13
14
15    //ソート
16    users.sort(function(a,b){
17      return (a.point < b.point) ? -1 : 1;
18    });
19
20
21    for (var i = 0; i <= pArray.length - 1; i++) {
22      var newRank = "<p class='ranking'>" + (i + 1) + ". Points :   " + users[i].point + "   :   " + users[i].name + "</p>";
23      rk.insertAdjacentHTML(
24        "beforeend",
25        newRank
26      );
27

おそらく問題は代入してる部分？だと思うのですが、私の理解としては

ゲーム1人目
pArray（連想配列）　<- 新しいユーザの情報を突っ込む

users <- pArray(ユーザ１の情報)を格納、即ち

users = [
{user[0]の情報}
];

ゲーム2人目
pArray（連想配列）　<- 次のユーザの情報を突っ込む(pArrayに上書き）

users <- pArray(ユーザ２の情報)を格納（新しい連想配列を追加）、即ち

users = [
{user[0]の情報},
{user[1]の情報}
];

なんですが、違うのでしょうか？

papinianus

2018/09/28 06:41

改善するとしたら毎回pArrayをnewするか、単にusers.push([a_count, strgName]);とするか

行動規範の内容に同意します

回答2件

ベストアンサー

提示されたコードですが、元のコードから関連箇所を抜粋して掲載されてるような気がします。
どうでしょうか？

その体で考えると、おそらく連想配列を使いまわしているのが原因です。
新しいユーザーの情報を突っ込む際、毎回連想配列を作り直すようにすると上手くいくと思います。

作り直さない場合、配列に追加される連想配列は全ておなじものです。
なので、追加済みの連想配列の情報まで書き換わり、すべて同じになってしまうわけです。

改修案の具体的には次のような感じです。

js
1// ここたぶんグローバル変数かメンバ変数ですよね
2var users = [];
3//var pArray = new Array(); // 廃止
4
5// ここからはたぶん関数ですよね
6function xxx(){
7
8   //  ...割愛
9
10　　//連想配列を作り、それを配列に追加
11    var pArray = new Array(); // 連想配列を新しく作成する
12    pArray["point"] = a_count;
13    pArray["name"] = strgName;
14    users.push(pArray);
15
16   //  ...割愛
17
18}