Scripting Helpers is winding down operations and is now read-only. More info→

elseif problem i would like some help please? Your title should be specific?

Asked by

raid6n 2196

5 years ago

Edited 5 years ago

First, I typed in a local script into a text button.

Here's the script:

01LocalPlayer = game.Players.LocalPlayer
02tableLots = script.Parent.Parent.Parent.TableLots
03 
04script.Parent.MouseButton1Click:connect(function()
05    local text = script.Parent.Parent.TableHandler.Text
06    local number = tonumber(text)
07    print("Number ", number)
08 
09    if number >= 1 and number <= 18 then
10        local chosenTable = tableLots["Table"..tostring(number)]
11        print("Table ", chosenTable)
12        chosenTable.BackgroundColor3 = Color3.fromRGB(255,0,0)
13elseif
14chosenTable.BackgroundColor3 == Color3.fromRGB(255,0,0) then
15    script.Parent.Text = "Table already taken."
16    wait(5)
17    script.Parent.Text = "Please try again."
18    wait(5)
19    script.Parent.Text = "Submit"
20    end
21end
 View all 21 lines...

As I type in elseif and the continued script, nothing works.

It would be wonderful if someone explained the elseif part.

Else if is written as "elseif" not "else if" MakeYourEscape 334 — 5y

I tried that, didn't work, just a typo in the code block. raid6n 2196 — 5y

nigger fangsrevive 0 — 2y

fuck you NIGGER raid6n 2196 — 2y

2 answers

Answered by

royaltoe 5144

5 years ago

Edited 5 years ago

chosenTable was out of scope (meaning the variable doesn't exist in the current context.)

Looking at the rest of your code, it seems like you're trying to alert the player that a table is taken.

If you were to put the code in an elseif, it would only run the code when the table number is either less than 1 or greater than 18, which isn't what we want to do.

We instead need to put the check to see if the table is empty inside the first if statement.

The logic would go like this:

player inputs a number

if the number is between 1-18, the table number is valid then check if the valid table is taken, alert the player that it's taken and let them try again.

if the table is not not taken, give that player that table

if the number is greater than 18 or less than one, the player typed in an invalid table. we tell them that it is invalid and let them try again.

Working Script:

01LocalPlayer = game.Players.LocalPlayer
02tableLots = script.Parent.Parent.Parent.TableLots
03 
04script.Parent.MouseButton1Click:Connect(function()
05    local text = script.Parent.Parent.TableHandler.Text
06    local number = tonumber(text)
07    print("Number ", number)
08 
09    if number >= 1 and number <= 18 then
10        local chosenTable = tableLots["Table"..tostring(number)]
11 
12        if(chosenTable.BackgroundColor3 == Color3.fromRGB(255,0,0))then
13           script.Parent.Text = "Table already taken."
14           wait(3)
15           script.Parent.Text = "Please try again."
16           wait(3)
17           script.Parent.Text = "Submit"
18        else
19            chosenTable.BackgroundColor3 = Color3.fromRGB(255,0,0)
20        end
21    else
22        script.Parent.Text = "There are only 18 tables.\n Please enter another table number."
23    end
24end)
 View all 24 lines...

Answered by

MakeYourEscape 334

5 years ago

There are a few things wrong.

01-- these variables should be local because i'm assuming this isn't inside a function
02local LocalPlayer = game.Players.LocalPlayer
03local tableLots = script.Parent.Parent.Parent.TableLots
04 
05-- changed "connect" to "Connect", since "connect" is deprecated.
06script.Parent.MouseButton1Click:Connect(function()
07    local text = script.Parent.Parent.TableHandler.Text
08    local number = tonumber(text)
09    print("Number ", number)
10 
11    if number >= 1 and number <= 18 then
12        local chosenTable = tableLots["Table"..tostring(number)]
13        print("Table ", chosenTable)
14        chosenTable.BackgroundColor3 = Color3.fromRGB(255,0,0)
15-- elseif should be put together.
16    elseif chosenTable.BackgroundColor3 == Color3.fromRGB(255,0,0) then
17        script.Parent.Text = "Table already taken."
18        wait(5)
19        script.Parent.Text = "Please try again."
20        wait(5)
21        script.Parent.Text = "Submit"
22        end
23end)
24-- You forgot the closed parenthesis
 View all 24 lines...

Haven't tested it, but fixed a lot of the formatting of this script. I'm assuming it's very old code. I also recommend using Activated instead of MouseButton1Click and Activated makes it compatible across all devices.

Accept this answer if it helped.

Only the Text part didn't work, everything else is fine. raid6n 2196 — 5y

Stop begging for accepts. Trollapse 89 — 5y

Good job pointing out the end parenthesis. The else if does not need to be together. Logically, it works fine as else if. Having the variables at the top is fine as he might need to reference the local player in another function. It is nicer to have it defined at the top rather than having to define the local player in every function. tableLots could be a local variable, but how it is is fine. royaltoe 5144 — 5y

Activated is good, I agree. royaltoe 5144 — 5y

elseif problem i would like some help please? Your title should be specific?

2 answers

The logic would go like this:

Working Script:

Answer this question