[AS3] Shooting

[Platypus2008]

I'm have a mc exported for actionscript as "laser" I want to create a new instance of it everytime someone presses the shooting key and then have it move horozontaly. Anyone know how to do that?

[amarghosh]

shootingKeyInstanceName.addEventListener(MouseEvent.CLICK, shootIt);
public function shootIt(e:MouseEvent):void
{
    var t:Laser = new Laser();//export it as Laser (UseInitialCaps), thats the practice;
    addChild(t);
    t.addEventListener(Event.ENTER_FRAME, moveHoriz);
}
public function moveHoriz(e:Event):void
{
    var t:MovieClip = MovieClip(e.currentTarget);
    t.x++;
    if(t.x > maxHorizPosition)//if u wanna remove it 
    {
        t.removeEventListener(Event.ENTER_FRAME, moveHoriz);
        removeChild(t);
    }
}

[Platypus2008]

It sais "The public attribute can only be used inside a package"

[amarghosh]

declare the methods as class's instance methods instead of declaring them within other methods.
move them out of all the method definitions

[Platypus2008]

Originally Posted by amarghosh

declare the methods as class's instance methods instead of declaring them within other methods.
move them out of all the method definitions

I just took the word public out. But is there some benefit in doing it the way you said?

[amarghosh]

are u coding on timeline?

[Platypus2008]

Originally Posted by amarghosh

are u coding on timeline?

Yeah.

[amarghosh]

in that case am not sure about the differences.
if your code is in an AS file, declaring methods as class's instance method is better practice than declaring them inside a function.
it is particularly important when u add these methods as listeners to some external objects.
u can't reference inner methods outside the containing function.

[Platypus2008]

I have the problem with the hitTest. The var "u" does not exist withen the moveHoriz function so the hitTest creates an error. However when I move the line "var u:UFO = new UFO();" into the main code. The hitTest works but it will not let there be more than one instance of UFO on stage at one time. Another thing I tried was add the same line of code into the moveHoriz funtion and that does let there be more than one instance on stage at one time but the hitTest does not seem to work even though there is no reported error. Any ideas?

function shootit(){
    milk.width -= 30;
    var t:Laser = new Laser(); addChild(t);
    t.x = cow.x, t.y = cow.y;
    t.addEventListener(Event.ENTER_FRAME, moveHoriz);
}
function addufo() {
     var u:UFO = new UFO(); addChild(u);
        u.y = 200;
    u.addEventListener(Event.ENTER_FRAME, umh);
}
function moveHoriz(e:Event):void
{
    var t:MovieClip = MovieClip(e.currentTarget);
    t.x -= 8;
    t.y += 1;
    if (t.hitTestObject(u)){t.x = 0, pwns+=1, ufomsg.text = "UFO pwns " + pwns};
    if (t.x < 0)
     {
        t.removeEventListener(Event.ENTER_FRAME, moveHoriz);
        removeChild(t);
    }
}
function umh(e:Event):void
{
  var u:MovieClip = MovieClip(e.currentTarget);
    u.x +=4;
    if (u.x > 600) {
        removeChild(u);
        u.removeEventListener(Event.ENTER_FRAME, umh);
        
    }
}