Is this statement valid? - GameCreators Forum

Author

Message

Grook

AGK Backer

11

Years of Service

User Offline

Joined: 22nd Jun 2012

Location: Lancashire, England

Posted: 29th Jun 2012 23:52

Link

Hello all.

I wonder if somebody would be kind enough to tell me if the following assignment is legal and should work ok. It seems to but I'm getting my executable bugging out at variable places at the moment and this was something I started doing recently in the program.

+ Code Snippet

type sItem
	iState as integer
	iType as integer
	iSprite as integer
	iFrameCount as integer
	iFrame as integer
	iTime as integer
	iTimeBetweenNextFrame as integer
endtype

global dim g_aItem [ 5 ] as sItem
global dim g_bItem [ 5 ] as sItem

// Is this statement valid? (Assuming I've actually initialised some stuff into g_aItem[1]
g_bItem[1] = g_aItem[1]

"Happiness is contrary to the human condition" --White (to Black)

Back to top

Profile PM

baxslash

Valued Member

Bronze Codemaster

17

Years of Service

User Offline

Joined: 26th Dec 2006

Location: Duffield

Posted: 30th Jun 2012 02:13

Link

No, you need to specify the type parameter (I think). IE:

+ Code Snippet

g_bItem[1].iState = g_aItem[1].iState
g_bItem[1].iType = g_aItem[1].iType 
g_bItem[1].iSprite = g_aItem[1].iSprite 
g_bItem[1].iFrameCount = g_aItem[1].iFrameCount 
g_bItem[1].iFrame = g_aItem[1].iFrame 
g_bItem[1].iTime = g_aItem[1].iTime 
g_bItem[1].iTimeBetweenNextFrame = g_aItem[1].iTimeBetweenNextFrame

this.mess = abs(sin(times#))

Back to top

Profile PM Email

Marl

12

Years of Service

User Offline

Joined: 19th Nov 2011

Location: Bradford, UK

Posted: 30th Jun 2012 05:21

Link

The statement is legal and should work OK, however it doesn't.

It can leave arrays with unpredictable contents which could well lead to program crashes.

I had this logged as an issue on the bug board. It was closed off recently with the comment "Fixed for 1077".

So until we get 1077, baxslash's solution is the one to use.

You could make it into a function, something like

+ Code Snippet

function CopyAtoB( thisIndex )
   g_bItem[ thisIndex ].iState = g_aItem[ thisIndex ].iState
   g_bItem[ thisIndex ].iType = g_aItem[ thisIndex ].iType 
   g_bItem[ thisIndex ].iSprite = g_aItem[ thisIndex ].iSprite 
   g_bItem[ thisIndex ].iFrameCount = g_aItem[ thisIndex ].iFrameCount 
   g_bItem[ thisIndex ].iFrame = g_aItem[ thisIndex ].iFrame 
   g_bItem[ thisIndex ].iTime = g_aItem[ thisIndex ].iTime 
   g_bItem[ thisIndex ].iTimeBetweenNextFrame =  g_aItem[ thisIndex ].iTimeBetweenNextFrame 

endfunction

Then when the fix is in, you can just replace the function call with the original statement.

Back to top

Profile PM Email

Grook

AGK Backer

11

Years of Service

User Offline

Joined: 22nd Jun 2012

Location: Lancashire, England

Posted: 30th Jun 2012 10:59

Link

Excellent, thank you both for your answers.

"Happiness is contrary to the human condition" --White (to Black)

Back to top

Profile PM

Sorry your browser is not supported!

AppGameKit Classic Chat / Is this statement valid?