Crash when using my function

[NoreoAlles]

Hey, i want to be able to place blocks in my Platformer and wrote this function to have them all on the same grid.

Code: Select all

--v is the value, m is the value that v should be the multiple of and dir says if v should be subtracted by one or if one should be added to it
function roundToMultiple(v, m, dir) 
    local v = v 
    local m = m
    local dir = dir 
    math.floor(v)
    while(v%m < 0 or v%m > 0) do 
        v = v + dir
    end
    return v
end

And this is the function in wich i use the seen above function.

Code: Select all

function Player:placeBlock(key)
    if key == "b" then 
        if self.x%50 == 0 and  self.y%50 == 0 then 
            obj = Object.new(self.x, self.y,50, 50)
            table.insert(Bdraw, obj)
        else 
            local x = roundToMultiple(self.x, 50, -1)
            local y = roundToMultiple(self.y, 50, -1)
            obj = Object.new(x, y,50, 50)
            table.insert(Bdraw, obj)
        end
    end
end

But it crashes

[BrotSagtMist]

Youre not actually using math.floor(v), thats about the only error i see.
It has to be v=math.floor(v).

Further a construction like local v = v is nonsense since having v in the function definition already makes it a local.

[zorg]

If it crashes, post the error and traceback you get; if it hangs, then you just learned why arithmetic with floating point numbers equaling a specific number might not ever work unless you keep your numbers whole.

[NoreoAlles]

If it crashes, post the error and traceback you get; if it hangs, then you just learned why arithmetic with floating point numbers equaling a specific number might not ever work unless you keep your numbers whole.

It just hangs

[NoreoAlles]

Youre not actually using math.floor(v), thats about the only error i see.
It has to be v=math.floor(v).

Further a construction like local v = v is nonsense since having v in the function definition already makes it a local.

Thanks, that fixed it.