Help with remote event script

Help and Feedback Scripting Support
, ,
#44

Did you do this? (Add on line 11)

#45

Why would he want to fire it to all clients?

It shouldn’t try to take the order of every player in the server. Just the one which triggered it…

#46

its just giving me an error i resolved earlier

#47

Print shouldn’t give you an error

#48

i mean the fire all clients line

#49

i put the print line in the script

#50

Try just putting

print(Buttons)

And if that errors replace it with

print("yes it worked")

And check the output to make sure it printed a button or the phrase “yes it worked”.

Make sure it is after you define the variable Buttons

#51

i put the print(buttons) line but im not getting anything in the output

#52

That means the event is not even connecting to the client.

#53

I think the error is that the server script is doing FireClient but the cash register script is a server script itself and you need a local script for a remote event. The issue with that is that a LocalScript won’t run in workspace. What I suggest is changing the RemoteEvent to a BindableEvent

  • Some corrections:

I tested in studio can do either

Material = Enum.Material.Neon
-- or
Material = ("Neon")

And

math.random(5)
-- is the same as
math.random(1,5)
  • More corrections:

I ran the code

for i=1,10000 do if math.random(1)~=1 then print("No") end end

And I ran the code

for i=1, 10000000 do local r = math.random(1,5); if not (r==1 or r==2 or r==3 or r==4 or r==5) then print("No") end end

And it doesn’t print anything.

#54

local BloxyColaButton = script.Parent.BloxyColaButton
local BurgerButton = script.Parent.BurgerButton
local FriesButton = script.Parent.FriesButton
local PizzaButton = script.Parent.PizzaButton
local TacoButton = script.Parent.TacoButton

game.ReplicatedStorage.CustomerOrder1.OnClientEvent:Connect(function()

local Buttons = math.random(1,5)
print(Buttons)

if Buttons == 1 then
	BloxyColaButton.Material = Enum.Material.Neon
end

if Buttons == 2 then
	BurgerButton.Material = Enum.Material.Neon
end

if Buttons == 3 then
	FriesButton.Material = Enum.Material.Neon
end

if Buttons == 4 then
	PizzaButton.Material = Enum.Material.Neon
end

if Buttons == 5 then
	TacoButton.Material = Enum.Material.Neon
end

end)

Heres the script tell me if its good

#55

Now I said BindableEvent but if you use a server script it will show on all the other clients, which may be undesirable. Finally in that case you would want to move your code to a LocalScript and place it inside the PlayerScripts folder. And in the LocalScript you need to locate the cash register model in workspace.

#56

Yeah, how didn’t I catch that …

Why are you sending a remote event from a server script to another server script?

1 Like
#57

there fire event script works

fireclient works
fireclient works1981×1255 31.6 KB

#58

Yeah that’s fine, but you should use Fire client to send information to a Local Script. Not a server script.

1 Like
#59

This


local BloxyColaButton = script.Parent.BloxyColaButton
local BurgerButton = script.Parent.BurgerButton
local FriesButton = script.Parent.FriesButton
local PizzaButton = script.Parent.PizzaButton
local TacoButton = script.Parent.TacoButton

game.ReplicatedStorage.CustomerOrder1.OnClientEvent:Connect(function()

local Buttons = math.random(1,5)
print(Buttons)

if Buttons == 1 then
	BloxyColaButton.Material = Enum.Material.Neon
end

if Buttons == 2 then
	BurgerButton.Material = Enum.Material.Neon
end

if Buttons == 3 then
	FriesButton.Material = Enum.Material.Neon
end

if Buttons == 4 then
	PizzaButton.Material = Enum.Material.Neon
end

if Buttons == 5 then
	TacoButton.Material = Enum.Material.Neon
end
end)

Will not work if it’s a server script. Make it a local script and it will work. But be aware other players won’t see the color change. Only the player which clicked on the register will see the color change.

#61

Thank you it worked, but is there a way to make it so only one can be triggered at the time

#62

After where you put print(Buttons) set all the buttons materials to the normal material (I assume which is Plastic or SmoothPlastic)

#64

add debounce:

-- in LocalScript
local BloxyColaButton = script.Parent.BloxyColaButton
local BurgerButton = script.Parent.BurgerButton
local FriesButton = script.Parent.FriesButton
local PizzaButton = script.Parent.PizzaButton
local TacoButton = script.Parent.TacoButton

local ordering = false -- debounce variable

game.ReplicatedStorage.CustomerOrder1.OnClientEvent:Connect(function()
	if ordering then return end -- if player is currently ordering, don't do anything by exiting the function using return
	ordering = true -- set ordering to true so it will exit early if the function runs again
	
	local Buttons = math.random(1,5)
	
	if Buttons == 1 then
		BloxyColaButton.Material = Enum.Material.Neon
	end

	if Buttons == 2 then
		BurgerButton.Material = Enum.Material.Neon
	end

	if Buttons == 3 then
		FriesButton.Material = Enum.Material.Neon
	end

	if Buttons == 4 then
		PizzaButton.Material = Enum.Material.Neon
	end

	if Buttons == 5 then
		TacoButton.Material = Enum.Material.Neon
	end
end)
#65

Okay after reviewing again there are several things to do here.

  1. You need to place your material changing code in a LocalScript inside of StarterPlayer.StarterPlayerScripts
  2. In the code, you need to locate each register you want to be watching (in workspace)
  3. You need to add a debounce to prevent other cash registers from acting.
  4. You need to take the code connected to .OnClientEvent and put in a function that takes the register as a parameter so you can use the same code for each cash register.