problem with timer cancel

Games
#1

i am following this tutorial http://www.youtube.com/watch?v=6TsDdLY7VXk (with some changes that i made ) and i what to now what is wrong in my code because when he changes the local function to add
the win sound and the timer cancel when you exploit all the orbs
local function trackBalls (obj)
obj:removeSelf()
o = o-1;

if(o == 0) then
audio.play(win_sound);
timer.cancel(gametmr);
end
end

i hope you can help me and understand the problem , i am very newbie
(and sorry for the bad english i am from venezuela )

this is my code

[code]
o = 0;
time_remain = 10;
time_up = false;
ready = false;
total_balls = 25
–set music
local pop_sound = audio.loadSound(“pop.mp3”)
local win_sound = audio.loadSound(“win.mp3”)

local display_txt = display.newText(“Wait”, 0, 0, native.sistemFont, 16*2);
display_txt.xScale = .5; display_txt.yScale = .5;
display_txt:setReferencePoint(display.bottomLeftReferencePoint);
display_txt.x = 20; display_txt.y = display.contentHeight-20;

local countdowntxt = display.newText(time_remain, 0, 0, native.systemFont, 16*2);
countdowntxt.xScale = .5; countdowntxt.yScale = .5 ;
countdowntxt:setReferencePoint(display.BottomRigthReferencePoint) ;
countdowntxt.x = display.contentWidth-20; countdowntxt.y = display.contentHeight-20;

– add physics engine, start up the engine, and apply gravity
local physics = require(“physics”)
physics.start()
– Set gravity to act “down” (ie, toward the bottom of the device )
physics.setGravity (0, 9)
–set wall image and position
local wall= display.newImage(“bara.png”)
wall.x = display.contentWidth/99,9
–turn wall into physics body
physics.addBody(wall,“static”)
–set wally image and position
local wally = display.newImage(“bara.png”)
wally.x = display.contentWidth/1
–turn wally into physics body
physics.addBody(wally,“static”)

–add floor image and position
local floor = display.newImage(“bar.png”)
floor.y = display.contentHeight/1
– turn floor into physics body
physics.addBody(floor,“static”)

local function trackBalls (obj)
obj:removeSelf()
o = o-1;

if(o == 0) then
audio.play(win_sound);
timer.cancel(gametmr);
end
end

local function countDown(e)
if(time_remain == 10) then
ready = true;
display_txt.text = “Go”;
end
time_remain = time_remain-1;
countdowntxt.text = time_remain;
end

–set ball to stage and position
local function spawnBall()
local ball =display.newImage(“bead2.png”)
ball.x = math.random ( 320 )
ball.y = -100

– turn ball into physics body
physics.addBody(ball,{bounce = 0.1 } )

function ball:touch(e)
if(ready == true ) then
if(e.phase == “ended”) then
–play pop
audio.play(pop_sound)
–remove the balls
trackBalls(self);
end
end

return true;

end
–increment o for every ball created
o = o+1;

ball:addEventListener(“touch”, ball) ;

–if all balls created, start the game timer
if(o == total_balls) then
gametmr = timer.performWithDelay(1000, countDown, 10 );
else
ready = false;

end

end

timer.performWithDelay( 25,spawnBall,total_balls)
[import]uid: 179948 topic_id: 31658 reply_id: 331658[/import]

#2

Hey, not sure what kind of error you’re getting but I suspect your problem may be related to the fact that you haven’t included the code that sets “ready = false” when time runs out.

You’ll eventually put this in your countDown() function:

  
if(o ~= 0) then  
 audio.play(fail\_sound)  
 display\_txt.text = "FAIL"  
 ready = false  
end

I suspect that your trackBalls() function MAY be canceling a timer that’s already run its course (since you may have technically “won” after the fail condition should have triggered). Not sure, though. I would complete the code and see if the issue resolves itself :slight_smile:

Brandon [import]uid: 136211 topic_id: 31658 reply_id: 126453[/import]

#3

i try to do ,want you say but didn’t work,but i think that I made wrong thank… brandon
i hope you can tell me hoy fix the problem [import]uid: 179948 topic_id: 31658 reply_id: 126507[/import]

#4

glider says that in the trackBalls function in audio.play and timer.cancel had “possibly uninitialized”

thanks for the help brandon your a very good developer [import]uid: 179948 topic_id: 31658 reply_id: 126511[/import]

#5

Hey, no idea on the audio side but you should probably be forward referencing your variables.

[code]
– Declare your variables at the top
local gametmr

– Then say what they do later
gametmr = timer.performWithDelay(1000, countDown, 10 );[/code]

By declaring the variable first all of the code afterwards will know it exists. As your code currently stands it’s possible that it’s trying to cancel something it can’t see…

If that doesn’t fix the problem we need you to write the error message Corona gives you :slight_smile: [import]uid: 41884 topic_id: 31658 reply_id: 126563[/import]

#6

Hey, not sure what kind of error you’re getting but I suspect your problem may be related to the fact that you haven’t included the code that sets “ready = false” when time runs out.

You’ll eventually put this in your countDown() function:

  
if(o ~= 0) then  
 audio.play(fail\_sound)  
 display\_txt.text = "FAIL"  
 ready = false  
end

I suspect that your trackBalls() function MAY be canceling a timer that’s already run its course (since you may have technically “won” after the fail condition should have triggered). Not sure, though. I would complete the code and see if the issue resolves itself :slight_smile:

Brandon [import]uid: 136211 topic_id: 31658 reply_id: 126453[/import]

#7

i try to do ,want you say but didn’t work,but i think that I made wrong thank… brandon
i hope you can tell me hoy fix the problem [import]uid: 179948 topic_id: 31658 reply_id: 126507[/import]

#8

glider says that in the trackBalls function in audio.play and timer.cancel had “possibly uninitialized”

thanks for the help brandon your a very good developer [import]uid: 179948 topic_id: 31658 reply_id: 126511[/import]

#9

Hey, no idea on the audio side but you should probably be forward referencing your variables.

[code]
– Declare your variables at the top
local gametmr

– Then say what they do later
gametmr = timer.performWithDelay(1000, countDown, 10 );[/code]

By declaring the variable first all of the code afterwards will know it exists. As your code currently stands it’s possible that it’s trying to cancel something it can’t see…

If that doesn’t fix the problem we need you to write the error message Corona gives you :slight_smile: [import]uid: 41884 topic_id: 31658 reply_id: 126563[/import]