New Vec2f not working inside New Constructor ?

[gcmartijnParticipant]

I’m trying to convert a Monkey code to  MX2 but, I get a strange error.
In the old Monkey code I was using a Vector class, but I thought this is overkill so i’m was trying the Vec2f instead.

What I have done is strip all the code to make it readable.
When I run this code the program hangs at the point that inside the Player Constructor a method Move(x,y) is creating a New Vec2f

In other words, I can’t create a Vec2f inside a method called by a Method New

Now I’m thinking or its a bug, or its because its a struct thing ?
Maybe Vec2f is not a replacement for the Vector class i’m using ?

Monkey

Namespace myapp #Import "<std>" #Import "<mojo>" Using std.. Using mojo.. Class MyWindow Extends Window Method New() Local FrameSetCollection:StringMap<FrameSet> = New StringMap<FrameSet>() Local player:Player = New Player(FrameSetCollection, "Bla", 200, 300) End Method OnRender( canvas:Canvas ) Override canvas.DrawText( "Hello World",Width/2,Height/2,.5,.5 ) End End Function Main() New AppInstance New MyWindow App.Run() End Class FrameSet Field Position:= New Vec2f Method Move:Void(_mx:Float, _my:Float) ' Print _mx ' Print _my ' HERE IS THE ERROR, CREATING NEW VEC2F ' ' ' Position=New Vec2f - CRASH Position = New Vec2f(_mx,_my) End End Class Class Sprite Field FrameSetCollection:StringMap<FrameSet> Field FrameSetColKey:String Method New(_FrameSetCollection:StringMap<FrameSet>,_FrameSetColKey:String) FrameSetCollection = _FrameSetCollection FrameSetColKey = _FrameSetColKey End End Class Class Player Extends Sprite Method New(_FrameSetCollection:StringMap<FrameSet>, _FrameSetColKey:String, _initX:Float, _initY:Float) Super.New(_FrameSetCollection,_FrameSetColKey) _FrameSetCollection.Get(_FrameSetColKey).Move(_initX,_initY) End End Class

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66

Namespace myapp   #Import "<std>" #Import "<mojo>"   Using std.. Using mojo..   Class MyWindow Extends Window   Method New() Local FrameSetCollection:StringMap<FrameSet> = New StringMap<FrameSet>() Local player:Player =  New Player(FrameSetCollection, "Bla", 200, 300) End Method OnRender( canvas:Canvas ) Override canvas.DrawText( "Hello World",Width/2,Height/2,.5,.5 ) End End   Function Main() New AppInstance New MyWindow App.Run() End     Class FrameSet   Field Position:= New Vec2f   Method Move:Void(_mx:Float, _my:Float) ' Print _mx ' Print _my ' HERE IS THE ERROR,  CREATING NEW VEC2F ' ' ' Position=New Vec2f - CRASH Position =  New Vec2f(_mx,_my) End End Class   Class Sprite Field FrameSetCollection:StringMap<FrameSet> Field FrameSetColKey:String   Method New(_FrameSetCollection:StringMap<FrameSet>,_FrameSetColKey:String) FrameSetCollection = _FrameSetCollection FrameSetColKey = _FrameSetColKey End End Class   Class Player Extends Sprite   Method New(_FrameSetCollection:StringMap<FrameSet>, _FrameSetColKey:String, _initX:Float, _initY:Float) Super.New(_FrameSetCollection,_FrameSetColKey) _FrameSetCollection.Get(_FrameSetColKey).Move(_initX,_initY) End End Class

[Simon ArmstrongParticipant]

If you look over at your debug window you will see Self is null so I think your issue has nothing to do with Vec2f.

Monkey2 does not treat null method calls as an error in debug mode.

[peterigzParticipant]

Think you’re just forgetting to add a new FrameSet to the map. Interesting that the method can be called on a null object though.

Monkey

Namespace myapp #Import "<std>" #Import "<mojo>" Using std.. Using mojo.. Class MyWindow Extends Window Method New() Local FrameSetCollection:StringMap<FrameSet> = New StringMap<FrameSet>() Local player:Player = New Player(FrameSetCollection, "Bla", 200, 300) End Method OnRender( canvas:Canvas ) Override canvas.DrawText( "Hello World",Width/2,Height/2,.5,.5 ) End End Function Main() New AppInstance New MyWindow App.Run() End Class FrameSet Field Position:= New Vec2f Method Move:Void(_mx:Float, _my:Float) ' Print _mx ' Print _my ' HERE IS THE ERROR, CREATING NEW VEC2F ' ' ' Position=New Vec2f - CRASH Position = New Vec2f(_mx,_my) End End Class Class Sprite Field FrameSetCollection:StringMap<FrameSet> Field FrameSetColKey:String Method New(_FrameSetCollection:StringMap<FrameSet>,_FrameSetColKey:String) FrameSetCollection = _FrameSetCollection FrameSetColKey = _FrameSetColKey FrameSetCollection.Add(FrameSetColKey, New FrameSet) '<<<<< End End Class Class Player Extends Sprite Method New(_FrameSetCollection:StringMap<FrameSet>, _FrameSetColKey:String, _initX:Float, _initY:Float) Super.New(_FrameSetCollection,_FrameSetColKey) _FrameSetCollection.Get(_FrameSetColKey).Move(_initX,_initY) End End Class

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67

Namespace myapp   #Import "<std>" #Import "<mojo>"   Using std.. Using mojo..   Class MyWindow Extends Window   Method New() Local FrameSetCollection:StringMap<FrameSet> = New StringMap<FrameSet>() Local player:Player =  New Player(FrameSetCollection, "Bla", 200, 300) End Method OnRender( canvas:Canvas ) Override canvas.DrawText( "Hello World",Width/2,Height/2,.5,.5 ) End End   Function Main() New AppInstance New MyWindow App.Run() End     Class FrameSet   Field Position:= New Vec2f   Method Move:Void(_mx:Float, _my:Float) ' Print _mx ' Print _my ' HERE IS THE ERROR,  CREATING NEW VEC2F ' ' ' Position=New Vec2f - CRASH Position =  New Vec2f(_mx,_my) End End Class   Class Sprite Field FrameSetCollection:StringMap<FrameSet> Field FrameSetColKey:String   Method New(_FrameSetCollection:StringMap<FrameSet>,_FrameSetColKey:String) FrameSetCollection = _FrameSetCollection FrameSetColKey = _FrameSetColKey FrameSetCollection.Add(FrameSetColKey, New FrameSet) '<<<<< End End Class   Class Player Extends Sprite   Method New(_FrameSetCollection:StringMap<FrameSet>, _FrameSetColKey:String, _initX:Float, _initY:Float) Super.New(_FrameSetCollection,_FrameSetColKey) _FrameSetCollection.Get(_FrameSetColKey).Move(_initX,_initY) End End Class

[Mark SiblyKeymaster]

Interesting that the method can be called on a null object though.

Yeah, I might yet change this, but mx2cc itself currently depends on this behaviour so it wont be for a while yet.

[gcmartijnParticipant]

@peterigz
Yes that’s it, but have to re-check why monkey1 is doing it oke something like this.
Thanks

[Author]

Posts