Weblabor

Rejtélyesen viselkedik az éppen megírt JavaScript programom, és nem találom az okát. Látszólag mindent jól csináltam, és mégsem megy! Egyszer csak leesik… hogy erre nem gondoltam… de úgy látom, más sem, ezért leírom, mindannyiunk okulására.

Adott egy nagy mátrix, amiből kell csinálnom egy kisebb mátrixot úgy, hogy csak a megadott sorokat és oszlopokat veszem bele. Mi sem egyszerűbb:

originalMatrix.getParts = function( selected ) {
	var newMatrix = new Matrix( selected.length, selected.length );
	for( var row in selected ) {
		for( var col in selected ) {
			newMatrix.set( row, col, this.get( selected[row], selected[col] ) );
		}
	}
	return newMatrix;
};

(A selected egy egyszerű tömb, amibe push()-sal beraktam azoknak a soroknak/oszlopoknak a sorszámát, amit szeretnék átvinni a régiből az újba.) Miért ezt a módját választottam az iterációnak? Mert elegánsabbnak tartottam, a klasszikus for ciklust:

    for ( var row = 0; row < selected.length; row++ ) {
        for ( var col = 0; col < selected.length; col++ ) {
            newMatrix.set( row, col, this.get( selected[row], selected[col] ) );
        }
    }

Vagy, hogy mit ne mondjak, a .forEach() használatát:

    var thisMatrix = this;
    selected.forEach( function( oldRow, newRow ) {
        selected.forEach( function( oldCol, newCol ) {
            newMatrix.set(newRow, newCol, thisMatrix.get( oldRow, oldCol ) );
        });
    });

…és mekkorát tévedtem!

A mátrix „osztály”, amit használnom kellett, egyetlen tömbben tárolja az adatokat, az elérés függvényei egyszerűek:

//indexes start at zero
Matrix.prototype.get = function (r,c) {
 return this.data[r*this.cols + c];
}

Matrix.prototype.set = function (r,c,x) {
 this.data[r*this.cols + c]=x;
}

És ezen a ponton történtek a bajok.

Ugyanis a for-in a megkapott objektum felsorolható tulajdonságainak nevein megy végig, nem a megkapott tömb indexein. Egy tulajdonság neve pedig, akárhogy is nézzük, egy string. És ahogyan Török Gábor kifejtette nemrég, JavaScriptben sajnos ugyanazt az operátort használhatjuk összeadásra és string összefűzésre. Így alakult, hogy a newMatrix.set törzsében a következő kifejezés ](string)r * (number)this.cols + (string)c. Persze, hogy a data ezen attribútumait hiába állítgattam.

Sajnos ezt az apróságot nem szokás megemlíteni, sőt a Google-ban rákeresve az első helyeken mind tömbbel mutatják be a for-in használatát. Szomorú.

(Van még egy ok, ami miatt célszerű kerülni a használatát – de ezt néha megemlítik és könnyebb is rájönni a hiba okára, ha belefutunk –, nevezetesen ha az input tömb egyéb tulajdonságokkal is rendelkezik, akkor azok is megjelennek a felsorolásban.)