2

はじめに、私たちは小さな組織であり、このシステムはずっと前に誰かによって構築されました。私はjavascriptの初心者なので複雑なことをするのに苦労していますが、あなたの答えを理解するために最善を尽くします. しかし残念なことに、すべてをゼロからやり直すことは、現時点では現実的な選択肢ではありません。

クライアントがログインを使用してメンバー ID を確認するデータ収集システムがあり、システムはこの ID を使用して MS Access データベースから .ASP/html フォームにレコードをプルし、クライアントがデータを更新できるようにします。これらのページの 1 つには、フォームの送信時に実行される次の関数があり、フィールド a/b/c の合計が d/e/f/g/h/i と同じ合計になることを確認します。これは、表示される列ごとに個別に行われます (各列はデータベース内のレコードであり、各 a/b/c/d/e/f はレコード内のフィールドです)。

問題は、関数のこのセクションにあります。

for (var j=0; j<recCnt; j++) {
   sumByType = milesSurf[j] + milesElev[j] + milesUnder[j];
   sumByTrack = milesSingle[j] + milesDouble[j] + milesTriple[j] + milesQuad[j] + milesPent[j] + milesSex[j];

等

JavaScript FOR を使用して各レコードをループし、合計が同じかどうかをテストする必要があります。

Firefox と IE では、これは正しく機能しています。フィールドは「sumByType」と「sumByTrack」に正しく合計されます。以下に示すように、何が問題なのかを把握するために小さなアラートを追加しました。

alert(sumByType + " " + j + " " + recCnt + " " + milesSurf[j] + " " + milesElev[j] + " " + milesUnder[j]);

Chrome では、このアラートは、「sumByType」と「sumByTrack」のコンポーネント (さまざまな「milesXXXXX」変数) が定義されていないことを示しています。

私の質問は: IE と FOX では正常に動作しているのに、なぜ Chrome では正常に動作しないのですか? 何か案は？

以下の完全な関数コード:

function submitCheck(formy, recCnt) {
//2/10/03: added milesQuad
//---------------checks Q#4 that Line Mileage by type is the same as by track     
var milesElev = new Array(); 
var milesSurf = new Array();
var milesUnder = new Array(); 
var milesSingle = new Array();
var milesDouble = new Array(); 
var milesTriple = new Array();
var milesQuad = new Array();
var milesPent = new Array();
var milesSex = new Array();
var sumByType = 0;
var milesLineTrack = new Array();   //this is for Q5 to compare it to mileage by trackage
var j = 0; var sumByTrack = 0; var liney; var yrOp;
//var str = "document.frm.milesElev" + j;
//alert(str.value);
for (var i in document.frm) {
    if (i.substring(0, i.length - 1) == "milesElev") {
        milesElev[parseInt(i.substring(i.length-1, i.length))] = parseFloat(document.frm[i].value); }   
    if (i.substring(0, i.length - 1) == "milesSurf") {
        milesSurf[parseInt(i.substring(i.length-1, i.length))] = parseFloat(document.frm[i].value); }
    if (i.substring(0, i.length - 1) == "milesUnder") {
        milesUnder[parseInt(i.substring(i.length-1, i.length))] = parseFloat(document.frm[i].value); }
    if (i.substring(0, i.length - 1) == "milesSingle") {
        milesSingle[parseInt(i.substring(i.length-1, i.length))] = parseFloat(document.frm[i].value); }
    if (i.substring(0, i.length - 1) == "milesDouble") {
        milesDouble[parseInt(i.substring(i.length-1, i.length))] = parseFloat(document.frm[i].value); }
    if (i.substring(0, i.length - 1) == "milesTriple") {
        milesTriple[parseInt(i.substring(i.length-1, i.length))] = parseFloat(document.frm[i].value); }
    if (i.substring(0, i.length - 1) == "milesQuad") {
        milesQuad[parseInt(i.substring(i.length-1, i.length))] = parseFloat(document.frm[i].value); }
    if (i.substring(0, i.length - 1) == "milesPent") {
        milesPent[parseInt(i.substring(i.length-1, i.length))] = parseFloat(document.frm[i].value); }
    if (i.substring(0, i.length - 1) == "milesSex") {
        milesSex[parseInt(i.substring(i.length-1, i.length))] = parseFloat(document.frm[i].value); }
    if (i.substring(0, i.length -1) == "milesLineTrack") {
        milesLineTrack[parseInt(i.substring(i.length-1, i.length))] = document.frm[i].value; }  //12/13/02 used to be parseFloat(document.frm[i].value)
    if (i.substring(0,5)=="Lines") {
        liney = document.frm[i].value;
    if (parseInt(liney)<1 || isNaN(liney)) {
        alert("Each mode must have at least 1 line.  Please correct the value in question #2."); 
        document.frm[i].select(); return false; }}
    if (i.substring(0,8)=="yearOpen") { 
        yrOp = document.frm[i].value;
        if (parseInt(yrOp)<1825 || isNaN(yrOp)) {
            alert("Please enter a year after 1825 for question #3"); 
            document.frm[i].select(); return false; }
    }
}
for (var j=0; j<recCnt; j++) {
    sumByType = milesSurf[j] + milesElev[j] + milesUnder[j];
    sumByTrack = milesSingle[j] + milesDouble[j] + milesTriple[j] + milesQuad[j] + milesPent[j] + milesSex[j];

    //---------------to round sumByTrack and sumByType from a long decimal to a single decimal place, like frm 7.89999998 to 7.9.
    sumByTrack = sumByTrack * 10;
    if (sumByTrack != parseInt(sumByTrack)) {
        if (sumByTrack - parseInt(sumByTrack) >= .5) {
        //round up
            sumByTrack = parseInt(sumByTrack) + 1; }
        else {  //truncate
            sumByTrack = parseInt(sumByTrack); }}
    sumByTrack = sumByTrack / 10;
    sumByType = sumByType * 10;
    if (sumByType != parseInt(sumByType)) { 
        if (sumByType - parseInt(sumByType) >= .5) {
        //round up
            sumByType = parseInt(sumByType) + 1; }
        else {  //truncate
            sumByType = parseInt(sumByType); }}
    sumByType = sumByType / 10;
    //-------------end of rounding ---------------------------

    if (sumByType != sumByTrack) {
        if (isNaN(sumByType)) { 
            sumByType = "(sum of 4.a., b., and c.) "; }
        else {
            sumByType = "of " + sumByType; }
        if (isNaN(sumByTrack)) { 
            sumByTrack = "(sum of 4.d., e., f., g., h., and i.) "; }
        else {
            sumByTrack = "of " + sumByTrack; }
        alert("For #4, the 'End-to-End Mileage By Type' " + sumByType + " must equal the 'End-to-end Mileage By Trackage' " + sumByTrack + ".");
        alert(sumByType + " " + j + " " + recCnt + " " + milesSurf[j] + " " + milesElev[j] + " " + milesUnder[j]);
        return false;
    }
    //alert (milesLineTrack[j] + "  " + milesSingle[j] + "  " + 2*milesDouble[j] + "  " + 3*milesTriple[j] + "  " + 4*milesQuad[j] + "  " + 5*milesPent[j] + "  " + 6*milesSex[j]);
    var singDoubTrip = (milesSingle[j] + 2*milesDouble[j] + 3*milesTriple[j] + 4*milesQuad[j] + 5*milesPent[j] + 6*milesSex[j])
    //----------round singDoubTrip to one digit after the decimal point (like from 6.000000001 to 6.0)
    singDoubTrip = singDoubTrip * 10;
    if (singDoubTrip != parseInt(singDoubTrip)) { 
        if (singDoubTrip - parseInt(singDoubTrip) >= .5) {
        //round up
            singDoubTrip = parseInt(singDoubTrip) + 1; }
        else {  //truncate
            singDoubTrip = parseInt(singDoubTrip); }}
    singDoubTrip = singDoubTrip / 10;
    //----------end round singDoubTrip-----------------------------------------
    if (parseFloat(milesLineTrack[j]) != singDoubTrip) {
        //var mlt = milesLineTrack[j];
        //if isNaN(milesLineTrack[j]) { mlt = 
        alert("For column #" + (j+1) + ", the mainline passenger track mileage of " + milesLineTrack[j] + " must equal the single track plus 2 times the double track plus 3 times the triple track plus 4 times the quadruple track plus 5 times the quintuple track plus 6 times the sextuple track, which is " + singDoubTrip + ".");
        return false; 
    }
}
//---------------------end of checking Q#4----------------

//return false;

}

4

1 に答える 1

3

が問題だと思いますfor (var i in document.frm)。要素を列挙するべきではありませんform。予想外のプロパティがたくさんあるでしょう -なぜ配列反復で "for...in" を使用するのは悪い考えですか? を参照してください。、特に配列のようなオブジェクトに当てはまります。これが FF で適切に機能するなんて信じられません :-)

これを使って：

var ele = document.frm.elements; // or even better document.getElementById("frm")
for (var i=0; i<ele.length; i++) {
    // use ele[i] to access the element,
    // and ele[i].name instead of i where you need the name
}

また、膨大な数の if ステートメントに対してループを優先する必要があります。

于 2013-01-30T14:09:06.280 に答える