Ein Problem mit Klasse

[DoTheSinWave]

Die Aufgabe, die ich mir gestellt habe: eine Klasse zu machen, die Matrizen definiert, und diese durch eine Funktion miteinander multiplizieren kann.
da das mein erster Versuch mit Klassen überhaupt ist, funktioniert natürlich gar nichts.

code in der fla:

m1=new matrix
m2=new matrix
m1.array=[[1,1,1],[1,1,1],[1,1,1]]
m2.array=[[2,2,2],[2,2,2],[2,2,2]]
//die multiplizierte Matrix ausgeben:
trace(MMply(m1,m2)) 

code in der Datei "matrix.as":

class matrix {
    var array:Array;
    private var vert, i, hor:Number;
    private var multi:Array;
    function matrix() {
//als einheitsmatrix initialisieren:        
array = [[1, 0, 0], [0, 1, 0], [0, 0, 1]];
        
    }
    function MMply(m1:matrix, m2:matrix) {
        for (hor=0; hor<3; hor++) {
            for (vert=0; vert<3; vert++) {
            for (i=0; i<3; i++) {
                    multi[hor][vert] += m1[hor][i]*m2[vert][i];
                }
            }
        }
        return multi;
    }
} 

ich bitte um Korrektur der Fehler.

[Alphanimal]

m1=new matrix()
m2=new matrix() 

klammern müssen sein, da der Konstruktor aufgerufen werden muss!

[flash bier]

und muss das nicht heissen:
trace(m1.MMply(m1,m2));
oder
trace(m2.MMply(m1,m2));
?
Die Methode MMply kenn doch nur eine Instanz des Objekts matrix (also m1 oder m2) .....

MfG flash bier

[DoTheSinWave]

aber von der funktionsweise her müsste eigentlich alles klappen, bis auf solche speziellen Fehler wie ihr sie genannt habt?
ich meine, stimmen das variablen-handling und die methode, mit der die matrizen multipl. werden?

stimmt die zuweisung der werte für die Matrix in der .fla so?

m1.array=[[1,1,1],[1,1,1],[1,1,1]] 

und muss das nicht heissen:
trace(m1.MMply(m1,m2));
oder
trace(m2.MMply(m1,m2));
?
Die Methode MMply kenn doch nur eine Instanz des Objekts matrix (also m1 oder m2) .....

ergebnisse:

ausgabe bei trace(m1.MMply(m1,m2))
oder trace(m2.MMply(m1,m2)): "[object Object]"

bei trace(MMply(m1,m2)): undefined

hmmm...zumindest ein kleiner Fortschritt. vielleicht.

[Alphanimal]

Zitat von flash bier

Die Methode MMply kenn doch nur eine Instanz des Objekts matrix

stimmt!
aber da diese methode nix mit der instanz zu tun hat, kannst du sie statisch machen... dh der aufruf direkt über den klassen-prototype:

matrix.MMply(m1,m2); 

deklaration so:

static function MMply(m1:matrix, m2:matrix){
...
} 

[DoTheSinWave]

static function MMply(m1:matrix, m2:matrix){

funzt nicht, ich bekomme folgende meldung:Instance variables cannot be accessed in static functions.

[Alphanimal]

die zuweisung stimmt so...
du kannst deiner matrix-klasse auch eine toString methode geben, um die werte anzuzeigen

function toString(){
return array[0][0] +"\\t"+ array[0][1] +"\\t"+ array[0][2] + "\\n"+ array[1][0] +"\\t"+ ...;
} 

die art wie du matrizen multiplizierst müsste auf den ersten blick stimmen.
mein 4x4 Matrix-Mply algo für die 3D Engine sieht so aus:

for(j=1;j<=4;j++){
for(i=1;i<=4;i++){
MTemp["_"+i+""+j] = 0;
for(n=1;n<=4;n++){
MTemp["_"+i+""+j] += M1["_"+i+""+n] * M2["_"+n+""+j];
}
}
} 

mfg

[Alphanimal]

Zitat von DoTheSinWave

funzt nicht, ich bekomme folgende meldung:Instance variables cannot be accessed in static functions.

ja... spricht für sich selbst!
statische funktionen können auf keine variablen zugreifen die in einer instanz deklariert werden...

also alle vars die du ein er function benutzt (hor, vert, i, multi, m1, m2) müssen in der funktion deklariert werden, oder auch statisch (dann bleiben die werte erhalten!) oder als parameter übergeben werden.

dh. so müsste es gehn: (alle vars in der function deklariert)

static function MMply(m1:matrix, m2:matrix) {
var hor, vert, i, multi = new Array();
for (hor=0; hor<3; hor++) {
for (vert=0; vert<3; vert++) {
for (i=0; i<3; i++) {
multi[hor][vert] += m1[hor][i]*m2[vert][i];
}
}
}
return multi;
} 

da fällt mir auf, du hast multi gar nicht gesetzt, wenn du darauf zugreifen willst...
also:
[as]multi = new Array();[as]

die deklarationen dieser vars in der klasse kannst du weg lassen!

[DoTheSinWave]

so schaut jetzt die fla aus:

m1 = new matrix();
m1.array = [[1, 1, 1], [1, 1, 1], [1, 1, 1]];
m2 = new matrix();
m2.array = [[2, 2, 2], [2, 2, 2], [2, 2, 2]];
 
trace(toString()) 

bei der klasse ist das dazugekommen:

function toString() {
        return array[0][0]+"\t"+array[0][1]+"\t"+array[0][2]+"\n"+array[1][0]+"\t"+array[1][1]+"\t"+array[1][2]+"\n"+array[2][0]+"\t"+array[2][1]+"\t"+array[2][2];
    } 

trotzdem ergibt trace(toString()) "[object Object]"

ich habs mal hochgeladen:hier

edit: du warst schneller... ich schau gleich nach

Zitat von DoTheSinWave

...bei der klasse ist das dazugekommen...

aber wenn du in der fla die function tostring aufrufst, weiß er so ja nicht, das die in der klasse gemeint ist.
also dann auch

m1.toString(); 

schreiben...

[Alphanimal]

toString() is eine methode der klasse! also instanz angeben!
trace(m1.toString());

aber da trace ja einen string verlangt, kannst du das toString weglassen (ist schon in der seehr tiefen Object class angegeben)

also nur
trace(m1);

müsste auch den text aus der toString methode ausgeben

[DoTheSinWave]

thx erst mal für die schnellen antworten - die Ausgabe der Matrix funzt bereits^^
bis morgen dann
mfgDTSW