0

このスクリプトがブラウザをクラッシュさせている理由を誰かが解明するのを手伝ってくれることを願っていました。最初の 4 つのループには約 20 の要素があり、2 番目のループには 8 つの要素があります。クラッシュするのに十分だとは思いませんでしたが、間違っている可能性があります。

テーブルからデータ値をスクレイピングして、ページ上にカスタム グラフを作成するために使用しようとしています。

設計に関するヘルプは大歓迎です。

ありがとう！

function mainFunc() {
  var b;
  var a;
  var x;
  var y;
  var c;
  var times = [];
  var mainTable = document.getElementsByClassName("reportTable tabularReportTable");
  var rows = mainTable[0].getElementsByClassName("breakRowClass1 breakRowClassBottom");
  for (x = 0; x < rows.length; x++) {
    a = rows[x].children;
    for (y = 0; y < a.length; x++) {
      if (a[y].innerHTML !== "&nbsp;") {
        b = a[y].innerHTML.split("<");
        c  = parseFloat(b[0]);
        times.push(c);
      }
    }

  }
}
4

1 に答える 1

1

x++の代わりに変数 x をインクリメントすることで、ループの y をインクリメントしていますy++。

for (y = 0; y < a.length; x++) {

次のようにする必要があります。

for (y = 0; y < a.length; y++) {
于 2013-08-28T17:40:56.227 に答える