配列内で次の値に変化があった時の処理

Question

多次元配列で配列内の左の文字が同じ時だけ右の数値を足すという処理を考えてます。
多次元配列で配列(0番目は除く)[a,b]の内、aの値が同じ時だけbの値を足し、その計を配列に保管していくという処理を考えています。
途中までは上手く行くのですが最後が足せなくなります。
最初の'a'は必ず入るものとして考えています。

また、下記謎のエラーもでるので構文的に間違えているのかもよくわかりません。
Uncaught TypeError: Cannot read property '0' of undefined

もっと上手い書き方があるよという方、ご教示をお願いしたいです。

let tajigen = [
                [ 'a' ], 
                [ 1,1 ], 
                [ 1,1 ], 
                [ 1,1 ],  
                [ 2,5 ], 
                [ 2,6 ], 
                [ 2,9 ],
                [ 4,3 ], 
                [ 4,5 ], 
                [ 7,10 ],
                [ 7,12 ],
            ] ;
            let hensu1=0;//比べる元を入れる用
            let hensu1v=[];//１と２それぞれ足した値を入れる用
            let syoukei=0;//左が同じ数値の時に足す用
            hensu1 = tajigen[1][0];
            for(let n=1;n<tajigen.length-1;n++){
                let hensu2 = tajigen[n+1][0];
                if(n!==tajigen.length){
                    if(hensu1===hensu2){
                        syoukei += tajigen[n][1];
                    }else{
                        syoukei += tajigen[n][1];
                        hensu1v.push(syoukei);
                        　　console.log('syoukei1:', syoukei);
                        syoukei = 0;
                        hensu1 = tajigen[n+1][0];
                    }
                }else{
                    syoukei += tajigen[tajigen.length-1][1];
                    　　console.log('syoukei2:', syoukei);
                    hensu1v.push(syoukei);
                    syoukei = 0;
                }
            }
            console.log('hensu1v:', hensu1v);

Comments

[yukkuri]

私はjavascriptはほとんど知りませんが、まず調べましょう。「javascript エラー名」みたいな感じですると、情報が出てきます。

[barao]

返信ありがとうございます。調べた結果よく分からないという結論なのです。値がないという意味らしいのですが値があるのに無いとはどういう事なのかが分かりません。ちなみにfor文の条件式を-1するとエラーは起きません。

[x_x]

「左の文字」や「右の数値」は何を指していますか？　「同じ」というのは何と何が同じなのでしょうか？　「足す」というのも何に足すのかわかりません。

[barao]

返信ありがとうございます。配列(0番目は除く)[a,b]の内、aの値が同じ時だけbの値を足し、その計を配列に保管していくという事なのですが、質問の回答になっていますでしょうか？説明が不足して申し訳ありません

[x_x]

「同じ」というのは何と何が同じなのでしょうか？に答えていないと思います。「左の文字」というのがaですか？ 数値のようですがいいのでしょうか？ また、「右の数値」がbですか？

[x_x]

提示コード中のtajigenの場合、どうなるのが理想なのでしょうか？　また、tajigen自体を書き換えるのですか？

Answer 1

多次元配列で配列内の...

言葉の意味としては2次元配列は多次元配列に含まれるものですが、
「2次元配列のみ考慮すればいい」のか、「3次元も考慮すべき」なのかで全く書くべきコードは異なります。
従って、お小言になって心苦しいですが質問者であればどちらかは書くべきでしょう。

コードを読む限り、2次元配列のみで良さそうなのでそう捉えつつ次へ進みます。

Uncaught TypeError: Cannot read property '0' of undefined

意訳: 君、配列から0のキーを取り出したいと言ってるけど、これは配列ではなくundefinedだから0キーなんて無いよ？

要するにtajigen[n+1][0]でしょうね。
配列は0スタート、length-1が終点になりますので、
for文内の最後のループでn+1のキーを取得しにいくと存在しないので、undefinedになるはずです。

...配列内の左の文字が同じ時だけ右の数値を足すという処理を考えてます。

日本語でおｋ良い感じになってますね、対応GJ。
質問文のコードを走らせての感想ですが、
配列の1要素目が「1同士を合計して3」「2同士を合計して20」「4同士を合計して8」を作りたいという意味なんですね。

こういう時はオブジェクトに変換したほうが良いと思います。

もっと上手い書き方があるよという方、ご教示をお願いしたいです。

配列は0から始まるものなので、
for文を扱う時はfor (i = 0; i < arr.length; i++)で始めましょう。
そうすることで例外を減らした綺麗なコードが書けるでしょう。

また、if文の中身は多くて2行、ガード節というelseを繋げずすぐに逃げる書き方があるので簡略化します。

// 変数の宣言時はできるだけconstにすべき
const tajigen = [
  ['a'], 
  [1, 1], 
  [1, 1], 
  [1, 1],  
  [2, 5], 
  [2, 6], 
  [2, 9],
  [4, 3], 
  [4, 5], 
  [7, 10],
  [7, 12],
];
const result = {};
for (let i = 0; i < tajigen.length; i++) {
  const key = tajigen[i][0];
  const val = tajigen[i][1];
  // aを捨てる書き方は色々あるが、今回は2個目の要素が存在しない場合は弾く事にする
  if (typeof val === "undefined") continue;
  if (!result[key]) result[key] = 0;
  result[key] += val;
}
console.log(result); // {1: 3, 2: 20, 4: 8, 7: 22}

// こうやって使う想定
console.log(result[1]); // 3

Comments

[barao]

ありがとうございます。
分かりやすく、きれいなコードかつ、回答ですね。
いつかmiyab-sunのように簡潔なコードを書けるようにがんばります(*^^*)

Answer 2

存在しない要素を参照しようとしてエラーになってます。
(n=10の時にtajigen[n+1][0]のところでtajigen[11][0]を参照しようとしてエラーが発生している)

Comments

[barao]

エラーの内容はなるほど、そういう事なのですね。回答ありがとうございます。