Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
1
A error shows up when I am trying to click on a button. Help me?
So I am trying to make a bank ATM. I made the gui so it is like I have a textbox then 2 buttons. 1 Button says 'Deposit' and the other one says 'Withdraw'. So when I enter a amount and then click on any of those buttons this error shows up:
07:45:56.117 - Players.iiLevelMaker.PlayerGui.ATMGui.Container.withdraw.LocalScript:9: attempt to compare string with number
This is the script for deposit:
01 | local player = game:GetService('Players').LocalPlayer |
02 | local gui = player:WaitForChild('PlayerGui'):WaitForChild('ATMGui') |
03 | local button = gui:WaitForChild('Container').deposit |
04 | local amountbox = gui:WaitForChild('Container').TextBox |
05 | local cash = player.leaderstats.Cash |
06 | local bank = player.leaderstats.Bank |
07 |
08 | function MouseClick() |
09 | if cash.Value >= amountbox.Text then |
10 | bank.Value = bank.Value +amountbox.Text |
11 | cash.Value = cash.Value -amountbox.Text |
12 | end |
13 | end |
14 |
15 | button.MouseButton1Click:connect(MouseClick) |
This is the script for withdraw:
01 | local player = game:GetService('Players').LocalPlayer |
02 | local gui = player:WaitForChild('PlayerGui'):WaitForChild('ATMGui') |
03 | local button = gui:WaitForChild('Container').withdraw |
04 | local amountbox = gui:WaitForChild('Container').TextBox |
05 | local cash = player.leaderstats.Cash |
06 | local bank = player.leaderstats.Bank |
07 |
08 | function MouseClick() |
09 | if bank.Value >= amountbox.Text then |
10 | cash.Value = cash.Value +amountbox.Text |
11 | bank.Value = bank.Value -amountbox.Text |
12 | end |
13 | end |
14 |
15 | button.MouseButton1Click:connect(MouseClick) |
1 answer
Log in to vote
0
You need to use the tonumber function so that you can compare the string :)
01 | local player = game.Players.LocalPlayer |
02 | local gui = player:WaitForChild('PlayerGui'):WaitForChild('ATMGui') |
03 | local button = gui:WaitForChild('Container').withdraw |
04 | local amountbox = gui:WaitForChild('Container').TextBox |
05 | local cash = player.leaderstats.Cash |
06 | local bank = player.leaderstats.Bank |
07 |
08 | function MouseClick() |
09 | --Convert string to a number datatype |
10 | local amount = tonumber(amountbox.Text) |
11 | if bank.Value >= amount then |
12 | cash.Value = cash.Value +amount |
13 | bank.Value = bank.Value -amount |
14 | end |
15 | end |
16 |
17 | button.MouseButton1Click:Connect(MouseClick) --'connect' is deprecated |
0
I will try it :) RootEntry 111 — 7y
1
Did it work? bartekrabit 38 — 7y
1
Damnit goul! xD I saw this in chat and was gonn' answer haha. awfulszn 394 — 7y
0
It worked. RootEntry 111 — 7y