2

配列 (CritPath) を取得し、配列内の各項目に対して; テーブルをチェックし、列を別々の配列 (SysDate & SysTime) に分割します。

(CritPath) 名に (SysTime) 名を加えたものを返し、SysTime 配列値を持つ新しい変数名を形成します (つまり、ATMCXPSysTime = [2,2])。CritPath 配列のそれぞれに対してこれを行います。

これまでのところ、配列値を出力することしかできませんでしたが、(CritPath) から (SysDate) や (SysTime) の値を使用して変数名を作成することはできませんでした。

最終的には、ATMCXPSysTime 変数名を呼び出して、値 [2,2] を与え、(CritPath) の他の値も同じようにしたいと考えています... CCSysTime は [6,5] を出力します。

脚本は…

var CritPath = []
CritPath = ['ATMCXP','CC']
var SysDate = [];
var SysTime = [];

$(function(Sys){
    for (i in CritPath) {
        var Sys = CritPath[i];
        var SysDate = Sys+'Date';
        var SysTime = Sys+'Time';

        for (i in Sys) {
            $('#' + CritPath[i] + ' tbody tr').each(function(index) {
                var $this = $(this);
                var str = $(this).find(":nth-child(2)").html()
                var parts = str.split(":");
                var minutes = parseInt(parts[0], 10) * 60 + parseInt(parts[1], 10);

                SysDate[index] = [$(this).find(":nth-child(1)").html()];
                SysTime[index] = [index] = [minutes];
            });
        }
    }
    return Sys;
});

alert(SysTime);
alert(CritPath);
alert(Sys);

単純な html テーブル...

<table border="1" id="ATMCXP" cellspacing="1" align="center">
    <tbody>
        <tr>
            <td>2013-04-09</td>
            <td>00:02</td>
        </tr>
        <tr>
            <td>2013-04-10</td>
            <td>00:02</td>
        </tr>
    </tbody>
</table>
<table border="1" id="CC" cellspacing="1" align="center">
    <tbody>
        <tr>
            <td>2012-04-09</td>
            <td>00:06</td>
        </tr>
        <tr>
            <td>2012-04-10</td>
            <td>00:05</td>
        </tr>
    </tbody>
</table>

編集:フィドルを追加しました... http://jsfiddle.net/sherman2k2/j3dWS/

4

1 に答える 1

1

更新されたフィドルの更新

私はあなたがやろうとしていることとあなたの問題を理解しているようです。更新されたフィドルには2行のコードがあります

    ATMCXPDate[index] = [$(this).find(":nth-child(1)").html()]; //set dates found in column into array and assign variable name

    ATMCXPTime[index] = [minutes]; //set converted minutes from column into array and assign variable name

これはタイプミスか、index変数の宣言を忘れているかのどちらかです。持っているすべてのテーブルのすべての日付とすべての時間に 1 つの配列が必要なため、それを次のように置き換えます。

   ATMCXPDate.push($(this).find(":nth-child(1)").html());
   ATMCXPTime.push(minutes); 

ここでは 2 つのことを行っています。解析された日付と時刻を配列に入れることと、以前のように配列にラップされていない単純な値を入れることです。配列内に配列は必要ありません。

元のコードのパフォーマンスに関する更新:
論理エラーと思われるものがいくつかあります

  • あなたは余分なサイクルを持っています for ( i in Sys)
  • テーブルを1回選択し、その後行を反復処理できます($('#' + CritPath[i] + ' tbody tr')に置き換えられます$('#'+CritPath[i]).find('tr').each

コードの明らかなエラーの 1 つは、グローバル スコープと関数の両方で同じ名前の変数を作成していることです。すなわち

...
// here you created global variables and assigned them to empty arrays
var SysDate = [];
var SysTime = [];

$(function(Sys){
    for (i in CritPath) {

        // here you created local variables with the same names so global ones are no more accessible within this function
        var SysDate = Sys+'Date';
        var SysTime = Sys+'Time';

    for (i in Sys) {
        $('#' + CritPath[i] + ' tbody tr').each(function(index) {
            ...
            // here you assigned local variables , but
            SysDate[index] = [$(this).find(":nth-child(1)").html()];

            // this line doesnt make sense
            SysTime[index] = [index] = [minutes];
            // you probably meant 
            SysTime[index] = [minutes];
        });
    }
   }
// next line just doesn't make any sense as it is not used anywhere
return Sys;
});
// here you alerting global variables, that wasn't affected by your ondomready    handler
alert(SysTime);
alert(CritPath);
alert(Sys);

全体的に、特定の名前でグローバルスコープに変数を作成したい場合は、ウィンドウオブジェクトを使用してこれを行うことができます

function setVar () {
  var nameOfGlobalVariable = 'SysTime0202'
  window[nameOfGlobalVariable] = 'some value'
}

setVar()

console.log(SysTime0202)  // outputs 'some value'

異なる名前のグローバル変数は実際には必要なく、代わりに日付と時刻の両方の配列のペアだけが必要だと思います。ある日付/時刻でルックアップを行い、別の配列から対応する値を取得したいと考えています。これは、一連のクリティカル パスとは異なるクリティカル パスです。

次のようなことを試すことができます

var CriticalPathData = []
$(function(){ 

  // looping through named tables to grab all date/times
  for (var i in CritPath) {

    var cp = CritPath[i]
      , table = $('#' + cp)
      // this is object which will hold all date/times for particular critical path
      , cpDateTime = { items : [] }


        table.find('tr').each(function(index, row) {
            var $this = $(this)
              , tds = $this.find('td')
              , rowDate = tds[0].innerHTML
              , rowTime = tds[1].innerHTML
              , parsedTime = rowTime.split(':')
            cpDateTime.items.push({ Date: rowDate , Time : +parsedTime[0] * 60 + +parsedTime[1] })  
        });
      // put parsed dates/times for all items with critical path to global object
      CriticalPathData.push(cpDateTime)
   }

   // after you filled in all the data you can access it here
   alert(CriticalPathData[0].items[0].Date)
   alert(CriticalPathData[0].items[0].Time)
})   

// or here
alert(CriticalPathData[0].items[0].Date)
alert(CriticalPathData[0].items[0].Time)
于 2013-05-20T13:38:20.223 に答える