81個あるvalue値を、for文を使って変数に代入したい

Question

前提・実現したいこと

81個あるvalue値を、for文を使って変数に代入したいと思っております。
ID名が
num1_1,num1_2,........num1_9
num2_1,...............num2_9
・
・
・
num9_1,...............num9_9
の9×9の81個の値をそれぞれ
変数に代入したいのですが、上手くいきませんでした。

Uncaught SyntaxError: Unexpected string
というエラーメッセージがでてきてしまいました。

該当のソースコード

function buttonClick(){
    for(let i=1 ; i<=9 ; i++){
        for(let j=1 ; j<=9 ; j++){
            let "Str"+i+"_"+j = document.getElementById("num"+ i + "_" + j).value;
            document.getElementById("otamesi").textContent = "Str"+i+"_"+j;
        }


    }
    
}

試したこと

"Str"+i+"_"+j
ここで変数を定義するときに
iとjをいれなければ、上手くいきました。

Answer 1

let "Str"+i+"_"+j

ここは、配列を使います。

Answer 2

2次元配列を使うといいでしょう。配列に関してはWEB検索で学習してください。

var values=[] 
function buttonClick(){
  for(let i=1 ; i<=9 ; i++){
    values[i-1] = [];
    for(let j=1 ; j<=9 ; j++){
      values[i-1][j-1] = document.getElementById("num"+ i + "_" + j).value;
    }
  }
}

CodePenサンプル

Comments

[shidada]

配列の定義の仕方で[]と要素の数を指定せずに最初においておいて、そのあとに[i-1]＝[]とするのは知りませんでした。すごく便利ですね。

また、アウトプットのところでも
str = ""とはじめにおいて
str = str + 〇○
とすれば、上書きされずに、書かれるというのも感動しました。
これは、プログラミングでは当たり前に使う手法みたいですね。

その他のところもよく分かりましたが、自分でこのコードが書けるようになるのには、まだまだだと感じました。ありがとうございます。

Answer 3

変数を介して別に渡すだけならいちいちユニークな変数を宣言する必要はないでしょう

let num;
for(let i=1 ; i<=3 ; i++){
  for(let j=1 ; j<=3 ; j++){
    num = document.getElementById("num"+ i + "_" + j).value;
    document.getElementById("otamesi").textContent += num
  }
}

Answer 4

class属性

9x9個の要素ノード参照が目的なら、class="num" の方が都合が良いのては、

<select class="num" name="num-1-1" data-x="1" data-y="1">
<select class="num" name="num-1-2" data-x="2" data-y="1">
...
<select class="num" name="num-2-1" data-x="1" data-y="2">
<select class="num" name="num-2-2" data-x="2" data-y="2">

ということは、前質問の時から感じていました(目に付いた時には既にクローズされてましたが)。
https://teratail.com/questions/326497

HTMLTableElement

数独の9x9マスを作るのなら、<table>を使うので、x,y座標値はセル参照時のtable API(rows,cells)に任せる形にすると、
https://developer.mozilla.org/ja/docs/Web/API/HTMLTableElement/rows
https://developer.mozilla.org/ja/docs/Web/API/HTMLTableRowElement
https://docs.oracle.com/javase/jp/1.5.0/guide/plugin/dom/org/w3c/dom/html/HTMLElement.html

table,select要素の

<select data-x="1" data-y="1">

data-*属性を省略出来ます。
更にいえば、createElement でtable,select要素ノード生成すると、面倒なHTMLを書かなくて済みます。

Selectors API

フラグメント識別子など、何かしらのid属性値を使用する理由があるなら、

document.querySelector('select[id^=num-')

で目的のノードを参照可能(誤検出を防ぐ為、id="num-1-1" 指定推奨)ですが、その場合もclass属性併用が無難と考えます。

Re: shidada さん

Comments

[shidada]

とても具体的な回答ありがとうございます。
前回の質問から目を通してくださり、また詳細なご回答をありがとうございます。
今は理解が追い付いていないのですが、必ずthink49様の回答の意味が分かるまで勉強します。