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);

OR

你运行该循环从1开始到小于等于3。

Answer 3

在您的代码中：

A = new Array(3);

你应该声明变量，并且通常不需要初始化具有长度的数组。此外，按照惯例，以大写字母开头的变量是为construtors保留的（尽管有些变量在所有大写字母中使用变量名来表示常量）。

考虑使用数组初始化器：

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

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

    a[i] = [];
}

在下一个循环中， i 被不必要地再次声明（javascript中没有块范围）。在这些情况下使用不同的变量是很常见的（尽管重新使用 i 没有任何不良影响，但它只是不喜欢）：

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

每行填充二维数组

3 个答案: