[Flash CS3] AS 1 in AS2 Umwandeln

[HC-Flasher]

Hallo zusammen,

Ich habe ein Ur-Alt-Flash hervorgeholt (ein Karussell) und krieg das Ding einfach nicht AS2 Fit. Habe schon dieverse Änderungen vorgenommen und finde einfach den Fehler nicht.
Kann sich das mal jemand angucken - Vielen Dank!

PHP-Code:

// Anzahl Boxen
var anzahl = 11;
var radius = 200;
var posX = 395;
var posY = 70;

// Global Signal
_global.signal = true;

// Clips auf Umlaufbahn bringen
this.onEnterFrame = function() {
    if (signal) {
        mausPos += ((_root._xmouse-posX)/300);// Geschwindigkeit
        for (var i = 1; i<anzahl+1; i++) {
            winkel = ((mausPos+(360/anzahl*i))*Math.PI)/180;
            this["clip"+i]._x = posX+(Math.sin(-winkel)*300);
            this["clip"+i]._y = posY+(Math.cos(-winkel)*30);
            this["clip"+i]._xscale = 80+(Math.cos(winkel)*20);
            this["clip"+i]._yscale = 80+(Math.cos(winkel)*20);
            this["clip"+i]._alpha = 100+(Math.cos(winkel)*30);
            this["clip"+i].swapDepths(1000+(Math.cos(winkel)*100));
            this["clip"+i].txtName = links[i][1].toUpperCase();
            //Stopt die Rottation
            this["clip"+i].onRollOver = function() {
                signal = false;
            };
            this["clip"+i].onRollOut = function() {
                signal = true;
            };
        }
    }
};

// Logo in Vordergrund
ObjektMitte.swapDepths(999+1); 


Gruss und Dank

[sonar]

Liegt an 'mausPos', das ist NaN und muss oben erst einmal deklariert werden dann geht’s.

[HC-Flasher]

Vielen Dank, jetzt funktioniert es wieder.
Habe oben bei der Variabel Deklaration noch folgendes hinzugefügt:

var mausPos = 0;

[readme.txt]

is aber trotzdem kein AS2, zu AS2 gehört auch eine ordentliche Typisierung.

[HC-Flasher]

Okay, danke für den Hinweis. Habe die Typisierung vorgenommen.
Ist es nun korrekt?

PHP-Code:

//Anzahl Boxen
var anzahl:Number;
var radius:Number;
var posX:Number;
var posY:Number;
var mausPos:Number;
anzahl = 11;
radius = 200;
posX = 395;
posY = 70;
mausPos = 0;

// Global Signal
_global.signal = true;

// Clips auf Umlaufbahn bringen
this.onEnterFrame = function() {
    if (signal) {
        mausPos += ((_root._xmouse-posX)/300);// Geschwindigkeit
        for (var i = 1; i<anzahl+1; i++) {
            winkel = ((mausPos+(360/anzahl*i))*Math.PI)/180;
            this["clip"+i]._x = posX+(Math.sin(-winkel)*300);
            this["clip"+i]._y = posY+(Math.cos(-winkel)*30);
            this["clip"+i]._xscale = 80+(Math.cos(winkel)*20);
            this["clip"+i]._yscale = 80+(Math.cos(winkel)*20);
            this["clip"+i]._alpha = 100+(Math.cos(winkel)*30);
            this["clip"+i].swapDepths(1000+(Math.cos(winkel)*100));
            this["clip"+i].txtName = links[i][1].toUpperCase();
            //Stopt die Rottation 
            this["clip"+i].onRollOver = function() {
                signal = false;
            };
            this["clip"+i].onRollOut = function() {
                signal = true;
            };
        }
    }
};

// MX (Logo)
ObjektMitte.swapDepths(999+1); 


[the binary]

was ist, wenn (durch zufall) eine anderes stueck code auch eine '_global.signal' variable fuer einen anderen zweck benutzt.. ?

1. das _global solltest du noch an den 'clip' heften, der die oEF bekommt, und dann per 'this.signal' in der function darauf zugreifen..

2. du koenntest dir die redundante evaluierung deines 'clip+i' sparen, indem du sie am anfang innerhalb der schleife einmal einer lokalen variable zuweist, und diese dann fuer den zugriff benutzt.

3. die zuweisung der over/out methoden in jedem oEF ist auch ueberfluessig..

// _global.signal
 
this.onEnterFrame = function() {
 
    //-- true fuer auto-start
    if (this.signal == undefined)
        this.signal = true;
 
    var target:MovieClip;
 
    if (signal) {
 
        mausPos += ((_root._xmouse-posX)/300);// Geschwindigkeit
 
        for (var i = 1; i<anzahl+1; i++) {
 
            winkel = ((mausPos+(360/anzahl*i))*Math.PI)/180;
 
            target = this['clip' + i];
            target.host = this;           // referenz auf den 'hosting' movieclip
 
            target._x = posX+(Math.sin(-winkel)*300);
            target._y = posY+(Math.cos(-winkel)*30);
 
            // ... rest der zuweisungen..
            
            //-- mouse-events nur einmal zuweisen
 
            if (target.onRollOver == undefined) {
            
               //-- Stopt die Rottation 
               target.onRollOver = function() {
                  this.host.signal = false;
               };
               target.onRollOut = function() {
                 this.host.signal = true;
               };
            }
        }
    }
}; 

sonst siehts soweit gut aus
gruss

//b