質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.50%

  • JavaScript

    16444questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

javascriptで配列にwhileで代入するとループが抜けられなかった。

解決済

回答 4

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 232

liflect

score 8

二次元配列にmarkを5個入れようと思ったら、下記のコードではループが抜けられなかったのです。console.logで出力すると、全部markで埋まった配列が出力されるんですが、なんでそうなるのかが、よく分かりません。どこに問題があるのでしょうか?

let mark = "O";
let length = 3;
function makeArray(){
    let i = 0;
    let x;
    let y;
    let markArray = (new Array(length)).fill((new Array(length)).fill(""));
    while(i < 5){
        x = Math.floor(Math.random()*length);
        y = Math.floor(Math.random()*length);
        if(!markArray[x][y]){
            markArray[x][y] = mark;
            i++;
        }
        console.log(markArray);
    }
    return markArray;
}
let markArray = makeArray();
console.log(markArray);
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • liflect

    2018/06/07 18:15

    ループを抜けてmarkが5個入った二次元配列を返すことです。

    キャンセル

  • mts10806

    2018/06/07 18:45

    文字での説明ではなく例えば配列をコンソールで出力した結果のような形とかそういう意味です。「5個はいった二次元配列」だけではほぼ無限に答えがあります。

    キャンセル

  • mts10806

    2018/06/07 18:50

    解決されたんですね。良かったです。今後は「確実に伝わる表現」を心掛けてください。文章で伝えるには限界があることも多いです。

    キャンセル

回答 4

checkベストアンサー

+4

面白いことに、markArray[0]とmarkArray[1]は同一オブジェクトになっています。

let markArray = (new Array(length)).fill((new Array(length)).fill(""));

console.log(markArray[0] === markArray[1]); // true
console.log(markArray[0] === markArray[2]); // true

↑なので、

markArray[x][y] = mark;

↑この処理はxの値に関係なく、yの値にのみ依存して実行されます。
その結果、

if(!markArray[x][y]){
    markArray[x][y] = mark;
    i++;
}

↑このブロック内に到達できるのは最大3回。
whileループの継続条件「while(i<5)」からは絶対に抜け出せない、ということになります。

let markArray = (new Array(length)).fill((new Array(length)).fill(""));

↑「Array.fill(object)」を実行すると配列の各要素に「object」そのものを埋め込むようです。
stackoverflowでもトピックに上がってました)
一度、空要素で埋めてからmapするとobjectのコピーが埋め込まれます。

let markArray = (new Array(length)).fill().map(v=>(new Array(length)).fill(""));

console.log(markArray[0] === markArray[1]); // false
console.log(markArray[0] === markArray[2]); // false

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/07 18:18

    分かりやすく教えてくれてありがとうございました。

    キャンセル

+1

ただ単に

if(!markArray[x][y]){
            markArray[x][y] = mark;
            i++;
}

この部分が実行できていないだけでは?
if文の条件を見直すと解決すると思います

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/07 07:27

    console.logで全部markで埋まった配列が出力されるのでその部分は実行されているはずだと思ってるんですが?

    キャンセル

  • 2018/06/07 07:38 編集

    ソースを確認したところ件のブロックは3回しか実行できていないようです。つまりiが3で止まりi>5とならない。

    キャンセル

  • 2018/06/07 07:49 編集

    おそらくジャグ配列の使い方が間違っています

    ```
    et mark = "O";
    let markArray = (new Array(length)).fill((new Array(length)).fill(""));
    markArray[2][0] = mark;
    console.log(markArray)
    ```

    上記のコードを実行したところ

    ```
    (3) [Array(3), Array(3), Array(3)]
    0
    :
    (3) ["O", "", ""]
    (3) ["O", "", ""]
    (3) ["O", "", ""]
    ```

    となってしまいました。
    多分`markArray[2][0] = mark;`で2,0を指定しているから

    ```
    (3) ["", "", ""]
    (3) ["", "", ""]
    (3) ["O", "", ""]
    ```

    となってほしいはずですよね

    3回の書き込みで全ての要素がmarkになってしまい、それ以上、件のifブロックを通らないことによりiが5以上にならないことが原因で無限ループになっているようです。

    キャンセル

+1

多次元配列の宣言がよくなかったようです。
書きか出たら、おそらく意図しているような動作になりました。

改修前

    let markArray = (new Array(length)).fill((new Array(length)).fill(""));

改修後

    let markArray= new Array(length);
    for(let i=0;i<length;i++){
      markArray[i]=new Array(length).fill("");
    }

改修後(全体)

let mark = "O";
let length = 3;
function makeArray(){
    let i = 0;
    let x;
    let y;
    //let markArray = (new Array(length)).fill((new Array(length)).fill(""));
    let markArray= new Array(length);
    for(let i=0;i<length;i++){
      markArray[i]=new Array(length).fill("");
    }
    while(i < 5){
        x = Math.floor(Math.random()*length);
        y = Math.floor(Math.random()*length);
        if(!markArray[x][y]){
            markArray[x][y] = mark;
            i++;
        }
        console.log(markArray);
    }
    return markArray;
}
let markArray = makeArray();
console.log(markArray);

出力結果

(3) ["", "", "O"]
(3) ["O", "O", "O"]
(3) ["", "O", ""]

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/07 18:19

    確かに動くようになりました。

    キャンセル

+1

一応参考までに

let mark = "O";
let length = 3;
let tmp = makeArray(mark,length);
console.dir(tmp);
function makeArray(mark,length){
  let i = 0;
  let ret = (new Array(length)).fill(null).map(function(x){return new Array(length).fill("")});
  while(i < 5){
    let x = Math.floor(Math.random()*length);
    let y = Math.floor(Math.random()*length);
    ret[x][y] =mark;
    i++;
  }
  return ret;
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • JavaScript

    16444questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。