My script is giving me an error that i dont know how to fix on math.random?

Hello, SitaruDaniel!

local brick = script.Parent

brick.CanCollide = false 
brick.Parent = game.Workspace

local key = math.random(1,4) -- Arguments of math.random: min, max 
if key == 1 then 
    brick.Position = Vector3.new(-116.75,18.859,387.5) 
    print("first") 
elseif key == 2 then 
    brick.Position = Vector3.new(-156.75,18.859,389.5) 
    print("second") 
elseif key == 3 then 
    brick.Position = Vector3.new(-135.25,18.859,368.5) 
    print("third") 
elseif key == 4 then 
    brick.Position = Vector3.new(-135.75, 18.859, 387.5) 
    print("fourth") 
end

Hi Sitaru, There is an issue with your script, and also an enhancement that can be made to your script. So, let's get right into it!

Issue

First off, the issue is the arguments for math.random(). Math.random() takes two arguments, one being the minimum value that can be produced by the random generator and one is the maximum.

Go here to find out more about math.random()

Enhancement

I don't recommend you use so many if statements for different conditions. Rather, you should use a table of vectors and just access that table for each condition. That way you don't have to write as much (Us programmers tend to be lazy.)

Here is how you could go about that:

local vectors = {  -- Contains all of your vectors
    Vector3.new(-116.75,18.859,387.5),
    Vector3.new(-156.75,18.859,389.5),
    Vector3.new(-135.25,18.859,368.5),
    Vector3.new(-135.75, 18.859, 387.5),
}

local key = math.random(1, 4)
brick.Position = vectors[key] -- Accesses the vector in the table, using the key as an index.

Go here to find out more about tables.

Thanks,

Best regards,

IdealstDeveloper