每行填充二维数组

Question

我有一个函数（ getCoeff() ），它返回一维数组。 我尝试用它来填充二维数组：

//set up an 3x3 array for A
A = new Array(3);
for (var i=0; i<3; i++) {
    A[i] = new Array(3);
}

//fill it per row using getCoeff()
for (var i=0; i<3; i++) {
        A[i] = getCoeff(i+1);
}

console.log(A);
console.log(getCoeff(1));
console.log(getCoeff(2));
console.log(getCoeff(3));

但是我只填充了第一行，而其他两行仍然为空：

[ [ -3, 2, -1 ], [ , ,  ], [ , ,  ] ]
[ -3, 2, -1 ]
[ 2, -3, 2 ]
[ 1, -1, 3 ]

如您所见，该函数正确返回行，但是由于某种原因，该函数在循环内不起作用。

另一方面，如果我尝试这样的话：

for (var i=0; i<3; i++) {
        A[i] = [1,2,3];
}
console.log(A);

它工作正常！

[ [ 1, 2, 3 ], [ 1, 2, 3 ], [ 1, 2, 3 ] ]

我的代码有什么问题？

更新：

编辑之前我的原始完整代码：

    var fs = require('fs');
    var input = "LPinput.txt";
    var c = new Array();
    var A = new Array();
    var b = new Array();
    var Eqin = new Array();
    var MinMax;

    open(input);

    console.log(c);
    console.log(A);
    console.log(b);
    console.log(Eqin);
    console.log(MinMax);

    function open(filename) {
        if (fs.existsSync(filename)) {
           var data = fs.readFileSync(filename).toString().split("\n");
           analyse(data);
        } else {
           console.log("ERROR: File doesnt exist!");
        }
    }

    function analyse(data) {

        //clean up whitespaces
        for (i in data) {
            data[i] = data[i].replace(/\s/g, '');       
        }   

        //check LP type & clean up
        if (data[0].substring(0,3) == "max") {
            MinMax = 1;
            data[0] = data[0].replace("max","");
        } else if (data[0].substring(0,3) == "min") {
            MinMax = -1;
            data[0] = data[0].replace("min","");
        } else {
            console.log("ERROR: Invalid format!");
            return;
        }

        //check constraints format & clean up
        if ( data[1].substring(0,4) != "s.t.") {
            console.log("ERROR: Invalid format!");
            return;
        } else {
            data[1] = data[1].replace("s.t.","");
        }

        //get variables 
        var variables = data[data.length-1].split(",");
        var last = variables[variables.length-1];
        variables[variables.length-1] = last.substring(0,last.indexOf(">"));

        //get number of constraints
        var constraints = data.length-2;

        c = getCoeff(0);

            //===============================
            //I JUST NEED TO POPULATE A TO FINISH THIS
        for (var i=0; i<constraints; i++) {
            A[i] = getCoeff(i+1);
        }   
            //===============================

        for (var i=1; i<data.length-1; i++) {
            var end = data[i].length;
            var start = end;
            while (data[i].charAt(start) != "=") {
                 start = start - 1;
            }
            b[i-1] = parseInt(data[i].substring(start+1,end));

            if (data[i].charAt(start-1) == "<") {
                Eqin[i-1]=-1;
            } else if (data[i].charAt(start-1) == ">") {
                Eqin[i-1]=1;
            } else {
                Eqin[i-1]=0;
            }

        }

        function getCoeff(row) {
            var coeff = new Array();
            for (i in variables) {
                var pos = data[row].indexOf(variables[i]);
                if ((data[row].charAt(pos-1) == "+") || (pos-1 < 0)) {
                    coeff[i]=1;
                } else if (data[row].charAt(pos-1) == "-") {
                    coeff[i]=-1;
                } else if (data[row].charAt(pos-1) == "*") {
                    var end = pos-1;
                    var start = end;
                    while ( (start > -1) && (data[row].charAt(start) != "+") && (data[row].charAt(start) != "-") ) {
                        start = start - 1;
                    }
                    coeff[i] = parseInt((data[row].substring(start,end)));
                }
            }
            return coeff;
        }

    }

LPinput.txt：

max 2*x1+x2-4*x3-15
s.t.-3*x1+2*x2-x3>=5
    2*x1-3*x2+2*x3<=9
    x1-x2+3*x3<=5
    x1,x2,x3>=0

更新＃2：

控制台输出：

[ 2, 1, -4 ]
[ [ -3, 2, -1 ] ]
[ 5, 9, 5 ]
[ 1, -1, -1 ]
1

应该是：

[ 2, 1, -4 ]
[ [ -3, 2, -1 ],[ 2, -3, 2 ],[ 1, -1, 3 ]]
[ 5, 9, 5 ]
[ 1, -1, -1 ]
1

Answer 1

这是真正的问题：您正在外部范围中使用i变量。

    for (var i=0; i<constraints; i++) {
        A[i] = getCoeff(i+1);
    }   

当您进入getCoef ，便有了这个for循环

for (i in variables) {

因为你还没有宣布i在这里，它使用相同的i在外部声明。 在第一次运行for循环（填充A的第一行）之后， i更改为"0" ，而不是数值0。因此， for循环的条件不再有效，并且退出循环。

你去！

Answer 2

就我而言，我得到了最后2个。 无论如何，如果您从0开始运行循环，则最好编写

A[i] = getCoeff(i+1);

要么

您运行从1到小于等于3的循环。

Answer 3

在您的代码中：

A = new Array(3);

您应该声明变量，通常不需要初始化具有长度的数组。 同样，按照惯例，以大写字母开头的变量保留给构造函数使用（尽管有些变量在所有大写字母中都使用变量名来表示常量）。

考虑使用数组初始化程序：

var a = [];   
for (var i=0; i<3; i++) {

在下一个循环中初始化数组有点用，因为在下一个循环中替换它们：

    a[i] = [];
}

在下一个循环中，无需再次声明我 （javascript中没有块作用域）。 在这些情况下，通常使用不同的变量（尽管重复使用对我没有不良影响，只是不受欢迎）：

for (var j=0; j<3; j++) {
        a[j] = getCoeff(j + 1);
}

因此，创建一个getCoeff函数，该函数仅返回传递给它的值的数组（纯粹用于测试）：

function getCoeff(v){
  return [v,v,v]; 
} 

“作品”：

console.log(a.join(' - ')); // 1,1,1 - 2,2,2 - 3,3,3