LÖVE

Sam Storms wrote:

undef wrote:Nice idea!
I think apples shouldn't spawn in the same position twice in a row, what do you think?
A temporary speed up item could be interesting as well.

Very nice idea, I added it to my list for improvements for the next update

Ah, bug spotted, I've seen it a few times, but I wasn't able to properly fix it, so I thought instead of fixing it, I'll just change the color of the apple when that happens and it would add more than one point if the player eats it or subtract more than one point if the enemy eats it. So if I can't fix it, I'll probably end up doing that Thanks for trying it out. Cheers!

function Food:respawn()
    math.randomseed(os.time())
    f.x = math.random(gr.getWidth() - f.w)
    f.y = math.random(gr.getHeight() - f.h)
    if f.y < 32 then -- does not let food spawn outside the boundaries of the playing area
        f.y = f.y + 32
    end
end

function Food:respawn()
    --math.randomseed(os.time()) I would only do this once, when you load the game
    local lastX, lastY = f.x, f.y
    f.x = math.random(gr.getWidth() - f.w)
    f.y = math.random(32, gr.getHeight() - f.h) -- is at least 32
    if f.x==lastX and f.y==lastY then
        f.x, f.y = -- put something deterministic here compared to the curent position, so that it is not only unlikely but IMPOSSIBLE 
                      -- that the same position repeats twice in a row (if you just run math.random again it is unlikely but possible)
    end
end

undef wrote: This can be easily fixed:

old:

Code: Select all

function Food:respawn()
    math.randomseed(os.time())
    f.x = math.random(gr.getWidth() - f.w)
    f.y = math.random(gr.getHeight() - f.h)
    if f.y < 32 then -- does not let food spawn outside the boundaries of the playing area
        f.y = f.y + 32
    end
end

suggestion:

Code: Select all

function Food:respawn()
    --math.randomseed(os.time()) I would only do this once, when you load the game
    local lastX, lastY = f.x, f.y
    f.x = math.random(gr.getWidth() - f.w)
    f.y = math.random(32, gr.getHeight() - f.h) -- is at least 32
    if f.x==lastX and f.y==lastY then
        f.x, f.y = -- put something deterministic here compared to the curent position, so that it is not only unlikely but IMPOSSIBLE 
                      -- that the same position repeats twice in a row (if you just run math.random again it is unlikely but possible)
    end
end

I hope that helps!

if f.x == lastX and f.y == lastY then
    f.x, f.y = f.x + math.random(gr.getWidth() - f.w), f.y + math.random(32, gr.getHeight() - f.h)
end

if f.x == lastX and f.y == lastY then
    f.x, f.y = f.x + math.random(gr.getWidth() - f.w), f.y + math.random(32, gr.getHeight() - f.h)
end

EDIT: I just removed all the math.randomseed() calls (two, actually, one to respawn the apples and the second to reposition the enemy on the Y axis when he goes off screen) and replaced them by one single math.randomseed() call when love loads, and previously I did experience two or more apples appearing simultaneously at the same position, but now it didn't happen, I played a lot but it never happened Thanks again!

if f.x == lastX and f.y == lastY then
    f.x = (f.x + math.random(gr.getWidth() - f.w) + 5)%GRIDWIDTH
end

gestaltist wrote:Kudos, it’s a nice little game.

I like the polished starting menu - I never get around to making menus because I find making them tedious.

undef wrote: Yeah, with two random calls on a it becomes very unlikely that two apples appear at the same position in a row, on a 10x10 grid the probability is 0.01 percent.
But I think the following is a good rule of thumb:

If there is something in your program that should never happen, make sure it's impossible.

Code: Select all

if f.x == lastX and f.y == lastY then
    f.x = (f.x + math.random(gr.getWidth() - f.w) + 5)%GRIDWIDTH
end

This way in the 1 percent occasion that your the new position is equal to the old, the new apple will just move 5 gridpositions to the right (with periodic boundry conditions).