Move buttons

Solar2D Lua Language
#1

I am making a game that involves moving a player (.png image) using four buttons: up, down, left, and right. Each button works, but when the user presses anywhere on the screen the player will only be moved in the direction of the first button that was pressed.

Here are two examples:

If I touch the up button, the player image moves up once. Then if I press the left button, the player image moves up once again.

If I first touch the left button, the player image moves left once Then if I press the up button, the player image moves left once again.

Here’s my current code involving moving the player image:

local function move(marker)

    if(marker == 1) then

        player.y = player.y - tile

    elseif marker == 2 then

        player.y = player.y + tile

    elseif marker == 3 then

        player.x = player.x - tile

    elseif marker == 4 then

        player.x = player.x + tile

    end

end

local function makeMoveButtons()

    local buttonImages = {{

            “up.png”,

            tile * 2,

            _H - (tile * 5),

        }, {

            “down.png”,

            tile * 2,

            _H - (tile * 2)

        }, {

            “left.png”,

            _W - (tile * 3.5),

            _H - (tile * 3.5)

        }, {

            “right.png”,

            _W - (tile / 2),

            _H - (tile * 3.5)

        }

    }

    buttons = display.newGroup()

    for i = 1, #buttonImages do

        local button = display.newImage(buttonImages[i][1])

        button.width = tile * 2

        button.height = tile * 2

        button.x = buttonImages[i][2]

        button.y = buttonImages[i][3]

        function button:touch(event)

            if event.phase == “began” then

                display.getCurrentStage():setFocus(self, event.id)

                self.isFocus = true

                return true

            elseif event.phase == “moved” then

                self.isFocus = false

                return true

            elseif self.isFocus == true then

                if event.phase == “ended” then

                    move(i)

                end

                return true

            end

        end

        buttons:insert(button)

        buttons[i]:addEventListener(“touch”)

    end

end

Any ideas on what’s going on?

#2

I think you want to pass in a identifier for which button is being pressed. If you add a line like:

button.id = i

and then change your move() function to:

move(self.id)
#3

I just tried that and it also didn’t work. I’ve tried a variety of different ways, including making each button separately rather than using a loop, but they all behave the same.

I think the source of my problem is that for some reason it doesn’t matter where I touch the screen. Once I’ve touched the first button, anywhere I touch on the screen will trigger the move event, which I don’t think it should do. It seems like I need the program confirm that the previous button’s event has ended so a new one can be started, but I don’t know how to do that.

#4

Okay, I fixed the problem! It turns out the error was here:

                if event.phase == “ended” then

                    move(i)

and I fixed it by adding one line:

                if event.phase == “ended” then

                    display.getCurrentStage():setFocus(self, nil)

                    move(i)

#5

I think you want to pass in a identifier for which button is being pressed. If you add a line like:

button.id = i

and then change your move() function to:

move(self.id)
#6

I just tried that and it also didn’t work. I’ve tried a variety of different ways, including making each button separately rather than using a loop, but they all behave the same.

I think the source of my problem is that for some reason it doesn’t matter where I touch the screen. Once I’ve touched the first button, anywhere I touch on the screen will trigger the move event, which I don’t think it should do. It seems like I need the program confirm that the previous button’s event has ended so a new one can be started, but I don’t know how to do that.

#7

Okay, I fixed the problem! It turns out the error was here:

                if event.phase == “ended” then

                    move(i)

and I fixed it by adding one line:

                if event.phase == “ended” then

                    display.getCurrentStage():setFocus(self, nil)

                    move(i)