Why isn't my sprite animation working corectly?

Author

Message

Rafaelinio

11

Years of Service

User Offline

Joined: 27th May 2013

Location:

Posted: 24th Dec 2016 11:55

Link

Sorry if my code is terrible, first time i try to use AGK:

// Project: Heads or Tails 
// Created: 2016-12-22

// show all errors
SetErrorMode(2)

// set window properties
SetWindowTitle( "Heads or Tails" )
SetWindowSize( 1024, 768, 0 )

// set display properties
SetVirtualResolution(1080, 1920)
SetOrientationAllowed( 1, 1, 1, 1 )
SetSyncRate( 30, 0 ) // 30fps instead of 60 to save battery
UseNewDefaultFonts( 1 ) // since version 2.0.20 we can use nicer default fonts

CreateSprite(1, 0)

//Heads to Heads: 1-17
for i = 1 to 17
	LoadImage(i, "Heads to Heads\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Heads to Tails: 18-29
for i = 18 to 29
	LoadImage(i, "Heads to Tails\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Tails to Heads: 30-38
for i = 30 to 38
	LoadImage(i, "Tails to Heads\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Tails to Tails: 39-47
for i = 39 to 47
	LoadImage(i, "Tails to Tails\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

SetSpriteSize(1, 400, 400)
SetSpriteOffset(1, 200, 200)
SetSpritePositionByOffset(1, 1080/2, 1920/2)

AddVirtualButton(1, 1080/2,1920/2 + 500, 400)
SetVirtualButtonText(1, "Heads or Tails")
//SetSpritePosition(1, 1080/2, 1920/2)
//PlaySprite(1, 10, 1, 1, 8)

//0-Heads 1-Tails
coin_state = 0
curr = 0
do
	If GetVirtualButtonPressed(1) = 1
		curr = Random(0,1)
		if curr = 0 and coin_state = 0
			PlaySprite(1,10,0,1,17)
			coin_state = 0
		elseif curr = 0 and coin_state = 1
			PlaySprite(1,10,0,18,29)
			coin_state = 1
		elseif curr = 1 and coin_state = 0
			PlaySprite(1,10,0,30,38)
			coin_state = 0
		elseif curr = 1 and coin_state = 1
			PlaySprite(1,10,0,39,47)
			coin_state = 1
		endif
	endif
    Sync()
loop

+ Code Snippet


// Project: Heads or Tails 
// Created: 2016-12-22

// show all errors
SetErrorMode(2)

// set window properties
SetWindowTitle( "Heads or Tails" )
SetWindowSize( 1024, 768, 0 )

// set display properties
SetVirtualResolution(1080, 1920)
SetOrientationAllowed( 1, 1, 1, 1 )
SetSyncRate( 30, 0 ) // 30fps instead of 60 to save battery
UseNewDefaultFonts( 1 ) // since version 2.0.20 we can use nicer default fonts

CreateSprite(1, 0)

//Heads to Heads: 1-17
for i = 1 to 17
	LoadImage(i, "Heads to Heads\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Heads to Tails: 18-29
for i = 18 to 29
	LoadImage(i, "Heads to Tails\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Tails to Heads: 30-38
for i = 30 to 38
	LoadImage(i, "Tails to Heads\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Tails to Tails: 39-47
for i = 39 to 47
	LoadImage(i, "Tails to Tails\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

SetSpriteSize(1, 400, 400)
SetSpriteOffset(1, 200, 200)
SetSpritePositionByOffset(1, 1080/2, 1920/2)

AddVirtualButton(1, 1080/2,1920/2 + 500, 400)
SetVirtualButtonText(1, "Heads or Tails")
//SetSpritePosition(1, 1080/2, 1920/2)
//PlaySprite(1, 10, 1, 1, 8)

//0-Heads 1-Tails
coin_state = 0
curr = 0
do
	If GetVirtualButtonPressed(1) = 1
		curr = Random(0,1)
		if curr = 0 and coin_state = 0
			PlaySprite(1,10,0,1,17)
			coin_state = 0
		elseif curr = 0 and coin_state = 1
			PlaySprite(1,10,0,18,29)
			coin_state = 1
		elseif curr = 1 and coin_state = 0
			PlaySprite(1,10,0,30,38)
			coin_state = 0
		elseif curr = 1 and coin_state = 1
			PlaySprite(1,10,0,39,47)
			coin_state = 1
		endif
	endif
    Sync()
loop

So, the loops at the start set each frame to the sprite. I attached the pictures in this thread if you guys need them(They are terrible, i just quickly made them with photoshop to try making this app :p )
Inside the main loop, i think its pretty self explanatory. If the button is pressed, play the specific animation needed.

My problem is, it always comes out to Heads for some reason. What am i doing wrong?
Sorry for the noob question :p

Trololololo

Attachments

Login to view attachments

Back to top

Profile PM Email

damothegreat

User Banned

Posted: 26th Dec 2016 12:34

Link

Is your radomisation not working correctly

My best bet is probably put a Print statement in each of the IF statements, to see where the logic keeps heading towards

I.E

+ Code Snippet


       if curr = 0 and coin_state = 0
            PlaySprite(1,10,0,1,17)
            coin_state = 0
 
          Print ("Playing Sprite 1")
 
       elseif curr = 0 and coin_state = 1
            PlaySprite(1,10,0,18,29)

            coin_state = 1

Print ("Playing Sprite 2")

        elseif curr = 1 and coin_state = 0
            PlaySprite(1,10,0,30,38)
            coin_state = 0

Print ("Playing Sprite 3")

        elseif curr = 1 and coin_state = 1
            PlaySprite(1,10,0,39,47)
            coin_state = 1

Print ("Playing Sprite 4")

        endif

then if the code always plays the same PRINT SPRITE <num> then its a logic error in the IF statements, if it plays all 4 depending on the random
on each loop then it maybe the implementation of the sprite animation issue

Let us know once you got debugging in place and come back to us

Damian

Using Tier 1 AppGameKit V2
Started coding with AMOS
Anything is possible if put mind to it

Back to top

Profile

damothegreat

User Banned

Posted: 26th Dec 2016 12:57 Edited at: 26th Dec 2016 13:38

Link

Just reliased you popped your PNGs here, I have had a look

Try this:-

// Project: Heads or Tails 
// Created: 2016-12-22
 
// show all errors
SetErrorMode(2)
 
// set window properties
SetWindowTitle( "Heads or Tails" )
SetWindowSize( 1024, 768, 0 )
 
// set display properties
SetVirtualResolution(1080, 1920)
SetOrientationAllowed( 1, 1, 1, 1 )
SetSyncRate( 30, 0 ) // 30fps instead of 60 to save battery
UseNewDefaultFonts( 1 ) // since version 2.0.20 we can use nicer default fonts
 
CreateSprite(1, 0)
 
//Heads to Heads: 1-17
for i = 1 to 17
    LoadImage(i, "Heads to Heads\" + Str(i) + ".png")
    SetImageMinFilter(i, 0)
    SetImageMagFilter(i, 0)
    AddSpriteAnimationFrame(1, i)
next i
 
//Heads to Tails: 18-29
for i = 18 to 29
    LoadImage(i, "Heads to Tails\" + Str(i) + ".png")
    SetImageMinFilter(i, 0)
    SetImageMagFilter(i, 0)
    AddSpriteAnimationFrame(1, i)
next i
 
//Tails to Heads: 30-38
for i = 30 to 38
    LoadImage(i, "Tails to Heads\" + Str(i) + ".png")
    SetImageMinFilter(i, 0)
    SetImageMagFilter(i, 0)
    AddSpriteAnimationFrame(1, i)
next i
 
//Tails to Tails: 39-47
for i = 39 to 47
    LoadImage(i, "Tails to Tails\" + Str(i) + ".png")
    SetImageMinFilter(i, 0)
    SetImageMagFilter(i, 0)
    AddSpriteAnimationFrame(1, i)
next i
 
SetSpriteSize(1, 400, 400)
SetSpriteOffset(1, 200, 200)
SetSpritePositionByOffset(1, 1080/2, 1920/2)
 
AddVirtualButton(1, 1080/2,1920/2 + 500, 400)
SetVirtualButtonText(1, "Heads or Tails")
//SetSpritePosition(1, 1080/2, 1920/2)
//PlaySprite(1, 10, 1, 1, 8)
 
//0-Heads 1-Tails
coin_state = 0
curr = 0
do

If GetVirtualButtonPressed(1) = 1
        curr = Random(0,1)

// Debug stuff
    SetPrintSize(50)
    print(curr)
    print("Coin State = " + str(coin_state))
    
    if coin_state = 0 then print("Should be Heads")
    if coin_state = 1 then print("Should be Tails")
    
    sync()
    sleep(1000)

//END of debug stuff
        
        if curr = 0 and coin_state = 0
            PlaySprite(1,10,0,1,17)
          
			sect=1
            
            coin_state = 1
        else
			if curr = 0 and coin_state = 1
				PlaySprite(1,10,0,18,29)
        
				sect=2
				coin_state = 0
			endif
		endif
		
		if curr = 1 and coin_state = 0
			PlaySprite(1,10,0,30,38)
        
			sect=3
        
			coin_state = 1
		else
				
			if curr = 1 and coin_state = 1
				PlaySprite(1,10,0,39,47)
				coin_state = 0
        
						sect=4 //Debug purposes
			endif
		endif
		
    endif
    
    Sync()
loop

+ Code Snippet


// Project: Heads or Tails 
// Created: 2016-12-22
 
// show all errors
SetErrorMode(2)
 
// set window properties
SetWindowTitle( "Heads or Tails" )
SetWindowSize( 1024, 768, 0 )
 
// set display properties
SetVirtualResolution(1080, 1920)
SetOrientationAllowed( 1, 1, 1, 1 )
SetSyncRate( 30, 0 ) // 30fps instead of 60 to save battery
UseNewDefaultFonts( 1 ) // since version 2.0.20 we can use nicer default fonts
 
CreateSprite(1, 0)
 
//Heads to Heads: 1-17
for i = 1 to 17
    LoadImage(i, "Heads to Heads\" + Str(i) + ".png")
    SetImageMinFilter(i, 0)
    SetImageMagFilter(i, 0)
    AddSpriteAnimationFrame(1, i)
next i
 
//Heads to Tails: 18-29
for i = 18 to 29
    LoadImage(i, "Heads to Tails\" + Str(i) + ".png")
    SetImageMinFilter(i, 0)
    SetImageMagFilter(i, 0)
    AddSpriteAnimationFrame(1, i)
next i
 
//Tails to Heads: 30-38
for i = 30 to 38
    LoadImage(i, "Tails to Heads\" + Str(i) + ".png")
    SetImageMinFilter(i, 0)
    SetImageMagFilter(i, 0)
    AddSpriteAnimationFrame(1, i)
next i
 
//Tails to Tails: 39-47
for i = 39 to 47
    LoadImage(i, "Tails to Tails\" + Str(i) + ".png")
    SetImageMinFilter(i, 0)
    SetImageMagFilter(i, 0)
    AddSpriteAnimationFrame(1, i)
next i
 
SetSpriteSize(1, 400, 400)
SetSpriteOffset(1, 200, 200)
SetSpritePositionByOffset(1, 1080/2, 1920/2)
 
AddVirtualButton(1, 1080/2,1920/2 + 500, 400)
SetVirtualButtonText(1, "Heads or Tails")
//SetSpritePosition(1, 1080/2, 1920/2)
//PlaySprite(1, 10, 1, 1, 8)
 
//0-Heads 1-Tails
coin_state = 0
curr = 0
do

    If GetVirtualButtonPressed(1) = 1
        curr = Random(0,1)

// Debug stuff
    SetPrintSize(50)
    print(curr)
    print("Coin State = " + str(coin_state))
    
    if coin_state = 0 then print("Should be Heads")
    if coin_state = 1 then print("Should be Tails")
    
    sync()
    sleep(1000)    

//END of debug stuff
        
        if curr = 0 and coin_state = 0
            PlaySprite(1,10,0,1,17)
          
			sect=1
            
            coin_state = 1
        else
			if curr = 0 and coin_state = 1
				PlaySprite(1,10,0,18,29)
        
				sect=2
				coin_state = 0
			endif
		endif
		
		if curr = 1 and coin_state = 0
			PlaySprite(1,10,0,30,38)
        
			sect=3
        
			coin_state = 1
		else
				
			if curr = 1 and coin_state = 1
				PlaySprite(1,10,0,39,47)
				coin_state = 0
        
						sect=4 //Debug purposes
			endif
		endif
		
    endif
    
    Sync()
loop

Your code was just flipping in SECT 1 and 3.

So I made the State_Coin flip too in each step, so now it will flip inside all 4 IF statements

Damo

Using Tier 1 AppGameKit V2
Started coding with AMOS
Anything is possible if put mind to it

Back to top

Profile

Rafaelinio

11

Years of Service

User Offline

Joined: 27th May 2013

Location:

Posted: 26th Dec 2016 14:05

Link

Ah alright, i get what i did wrong, thanks for the help!

Trololololo

Back to top

Profile PM Email

Rafaelinio

11

Years of Service

User Offline

Joined: 27th May 2013

Location:

Posted: 26th Dec 2016 14:20

Link

Ok i realised i had some other issues in there with the frames used and some variables not being correctly assigned :p Fixed those too and its working perfectly, thanks

// Project: Heads or Tails 
// Created: 2016-12-22

// show all errors
SetErrorMode(2)

// set window properties
SetWindowTitle( "Heads or Tails" )
SetWindowSize( 1024, 768, 0 )

// set display properties
SetVirtualResolution(1080, 1920)
SetOrientationAllowed( 1, 1, 1, 1 )
SetSyncRate( 30, 0 ) // 30fps instead of 60 to save battery
UseNewDefaultFonts( 1 ) // since version 2.0.20 we can use nicer default fonts

CreateSprite(1, 0)

//Heads to Heads: 1-17
for i = 1 to 17
	LoadImage(i, "Heads to Heads\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Heads to Tails: 18-29
for i = 18 to 29
	LoadImage(i, "Heads to Tails\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Tails to Heads: 30-38
for i = 30 to 38
	LoadImage(i, "Tails to Heads\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Tails to Tails: 39-47
for i = 39 to 47
	LoadImage(i, "Tails to Tails\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

SetSpriteSize(1, 400, 400)
SetSpriteOffset(1, 200, 200)
SetSpritePositionByOffset(1, 1080/2, 1920/2)

AddVirtualButton(1, 1080/2,1920/2 + 500, 400)
SetVirtualButtonText(1, "Heads or Tails")
//0-Heads 1-Tails
coin_state = 0
curr = 0
do
	If GetVirtualButtonPressed(1) = 1
		curr = Random(0,1)
		if curr = 0 and coin_state = 0
            PlaySprite(1,10,0,1,17)
            coin_state = 0
        else
            if curr = 0 and coin_state = 1
                PlaySprite(1,10,0,30,38)
                coin_state = 0
            endif
        endif      
		if curr = 1 and coin_state = 0
            PlaySprite(1,10,0,18,29)
            coin_state = 1
        else    
            if curr = 1 and coin_state = 1
                PlaySprite(1,10,0,39,47)
                coin_state = 1
            endif
        endif
    endif
    
    Sync()
loop

+ Code Snippet

// Project: Heads or Tails 
// Created: 2016-12-22

// show all errors
SetErrorMode(2)

// set window properties
SetWindowTitle( "Heads or Tails" )
SetWindowSize( 1024, 768, 0 )

// set display properties
SetVirtualResolution(1080, 1920)
SetOrientationAllowed( 1, 1, 1, 1 )
SetSyncRate( 30, 0 ) // 30fps instead of 60 to save battery
UseNewDefaultFonts( 1 ) // since version 2.0.20 we can use nicer default fonts

CreateSprite(1, 0)

//Heads to Heads: 1-17
for i = 1 to 17
	LoadImage(i, "Heads to Heads\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Heads to Tails: 18-29
for i = 18 to 29
	LoadImage(i, "Heads to Tails\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Tails to Heads: 30-38
for i = 30 to 38
	LoadImage(i, "Tails to Heads\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

//Tails to Tails: 39-47
for i = 39 to 47
	LoadImage(i, "Tails to Tails\" + Str(i) + ".png")
	SetImageMinFilter(i, 0)
	SetImageMagFilter(i, 0)
	AddSpriteAnimationFrame(1, i)
next i

SetSpriteSize(1, 400, 400)
SetSpriteOffset(1, 200, 200)
SetSpritePositionByOffset(1, 1080/2, 1920/2)

AddVirtualButton(1, 1080/2,1920/2 + 500, 400)
SetVirtualButtonText(1, "Heads or Tails")
//0-Heads 1-Tails
coin_state = 0
curr = 0
do
	If GetVirtualButtonPressed(1) = 1
		curr = Random(0,1)
		if curr = 0 and coin_state = 0
            PlaySprite(1,10,0,1,17)
            coin_state = 0
        else
            if curr = 0 and coin_state = 1
                PlaySprite(1,10,0,30,38)
                coin_state = 0
            endif
        endif      
		if curr = 1 and coin_state = 0
            PlaySprite(1,10,0,18,29)
            coin_state = 1
        else    
            if curr = 1 and coin_state = 1
                PlaySprite(1,10,0,39,47)
                coin_state = 1
            endif
        endif
    endif
    
    Sync()
loop

Trololololo

Back to top

Profile PM Email

Sorry your browser is not supported!

Newcomers AppGameKit Corner / Why isn't my sprite animation working corectly?

Attachments