場合によって関数の戻り値が違う

Question

自分で定義した関数を、引数を変えて正誤判定しています。
正解はすでに配列に入っていて、関数の戻り値と配列の要素が一致しないときだけ出力しています。

for( var a = 0; a < 10; a++ ) {
  for( var b = 0; b < 10; b++ ) {
    for( var c = 0; c < 10; c++ ) {
      if( test(a, b, c) != answer[ a*100 + b*10 + c ] ) {
　　　　//test:自分で定義した関数  answer:正解の文字列が入っている配列
        console.log( test(a, b, c), answer[ a*100 + b*10 + c ], a, b, c );
        //関数による誤答、正解、その時の引数を出力
      }
    }
  }
}
//例えば あいうえお かきくけこ 0 1 2 と出力されたとする
--------------------------
//しかし...

console.log( test(0, 1, 2) );
//繰り返しがないと　かきくけこ と出力される、つまり正しい答えが出ている

誤答として出力されるのは数百通りの内の数十通りで、データ型によるエラーではないと考えています。関数の部分は膨大でお見せできませんが、何か原因として考えられることはありますか。実際は「あいうえお」などではなく、漢字だけの文字列を比較しています。

Answer 1

考えられる原因は2つあります。

• test() 関数の処理が間違っている
• answer 配列の要素値が間違っている

切り分けとしては、相違がある要素を全て出力してみれば良いと思います。

'use strict';
var errors = [], testResults = [];

for( var a = 0; a < 10; a++ ) {
  for( var b = 0; b < 10; b++ ) {
    for( var c = 0; c < 10; c++ ) {
      testResults[a*100 + b*10 + c] = test(a, b, c);

      if ( test(a, b, c) !== answer[ a*100 + b*10 + c ] ) {
        errors[a*100 + b*10 + c] = [test(a, b, c), answer[ a*100 + b*10 + c ]];
      }
    }
  }
}

console.log(JSON.stringify(testResults));
console.log(JSON.stringify(answer));
console.log(JSON.stringify(errors));

変数 errors に相違がある要素値が出力される為、「test() の返り値」と「answer の要素値」のどちらが正しいか、を確認して下さい。
testResults が正しいのであれば、answer を testResults の値に置き換えるだけで解決します。

chromeを使っていますが、そういった類のバグなどあるのでしょうか...?

「ブラウザのバグ」を疑うのはコード上のバグが完全にない事が確認できた時だけで、通常は最後に疑うべき部分です。
今回のケースでは「test()の返り値」と「answer の配列要素値」のどちらかが間違っている事が確定していますが、その間違いがブラウザのバグによるものである事は確定していません。
私の経験上では、ブラウザのバグ起因である事はほとんどなく、ロジック上の問題である事が大半ですね。

Re: rururururururu さん

Answer 2

ご質問の内容からは、test関数の処理が誤っているのか、answer配列の構築が誤っているのかを判断することはできません。

すべての組み合わせを出力してもたかだか1000通りですので、一度全出力を行って、test関数の振る舞いを確認されるとよいのではないかと思います。

for( var a = 0; a < 10; a++ ) {
  for( var b = 0; b < 10; b++ ) {
    for( var c = 0; c < 10; c++ ) {
      console.log(test(a, b, c), answer[ a*100 + b*10 + c ], a, b, c );
    }
  }
}

Comments

[rururururururu]

ご回答ありがとうございます。試しましたところ、正誤判定が無くても繰り返しの回数によって関数の戻り値が変わります（一定の回数を超えると間違い始めます）。chromeを使っていますが、そういった類のバグなどあるのでしょうか...?

[退会済みユーザー]

test関数での処理が不明なため、回答いたしかねます。
可能であれば処理内容を質問欄に記載ください。

Answer 3

console.logは「表示された時の値を表示する」というlazyな実装になっているようなので、もしかしてそれが原因では。

JavaScript - JavaScript Arrayオブジェクトが正しく表示されない(5420)｜teratail
JavaScript - console.log()で出力した値が、変わるのはなぜ？(61338)｜teratail

もしこれが原因なら、letを使うなどスコープを活用すれば上手くいくのではないでしょうか。

Comments

[think49]

オブジェクトを展開すると仰る通りの挙動ですが、現在の Google Chrome ではコンソールログ上に出力される「オブジェクト初期化子の文字列」には console.log 呼び出し当時のオブジェクト初期化子が出力されるように修正されています。
var obj = {a:1};
console.log(obj); // {a: 1}
obj.b = 2;

それと、質問コード上の例示では、"あいうえお" と "かきくけこ" の比較で配列内のプリミティブ値を比較しているように見えました。

[Lhankor_Mhy]

なるほど、勉強になりました。ありがとうございます。