Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
2
Trying to get a clock to chime every hour, not detecting hour correctly?
Basically, I want a sound to play every hour, however, it keeps looping (it is played when it ends, due to a debounce preventing it spam playing) I use ClockTime in the Lighting space to detect whenever it is dead on an hour, but it doesn't seem to read ClockTime correctly. The code is as follows:
01 | Lighting = game.Lighting |
02 | Time = Lighting.ClockTime |
03 | Bell = script.Parent.NewtonBells |
04 | IsPlaying = false |
05 |
06 | while true do |
07 | wait() |
08 | if Time == 1 or Time == 2 or Time == 3 or Time == 4 or Time == 5 or Time == 6 or Time == 7 or Time == 8 or Time == 9 or Time == 10 or Time == 11 or Time == 12 or Time == 13 or Time == 14 or Time == 15 or Time == 16 or Time == 17 or Time == 18 or Time == 19 or Time == 20 or Time == 21 or Time == 22 or Time == 23 or Time == 24 then |
09 | if IsPlaying == true then |
10 |
11 | else |
12 | Bell:Play() |
13 | IsPlaying = true |
14 | wait(50) |
15 | print("BellFinished") |
16 | Bell:Stop() |
17 | IsPlaying = false |
18 | end |
19 | end |
20 | end |
0
Stop spamming the link in the text chat JLKMaster 0 — 4y
0
I haven't spammed anything but ok MacauleyP_Plays 15 — 4y
0
what's this abomination DiamondComplex 285 — 4y
0
its an abomination because people are refusing to help... MacauleyP_Plays 15 — 4y
View all comments (11 more)
0
u don't need to do a whole lot of ors just do if Time == Time + 1 then wizz_zz 3 — 4y
0
actually no, just make a variable for both the previous value and the new value so local previousvalue = 0 then while true do thing then make sure the new value is = to the new value then check if the new value is equal to the previous value + 1 and then set the previous value to the new value wizz_zz 3 — 4y
0
no. DiamondComplex 285 — 4y
0
yes. if he is trying to play something every hour wizz_zz 3 — 4y
0
my problem is that its not actually detecting the ClockTime as the correct value, its constantly thinking the clocktime is everytime, meaning the script repeatedly loops. I get that checking if it was +1 of the previous value, but if the current code doesn't read ClockTime correctly, that won't be of much help MacauleyP_Plays 15 — 4y
0
New Code: Lighting = game.Lighting Time = Lighting.ClockTime Bell = script.Parent.NewtonBells IsPlaying = false while true do L = game.Lighting.ClockTime wait() if L == 12 or L == 24 then if IsPlaying == false then Bell:Play() print("Bells Ringing") IsPlaying = true wait(50) IsPlaying = false else --Bell Doesn't Ring print("Bell didn't ring ,isRinging") end else p MacauleyP_Plays 15 — 4y
0
(This new code doesn't seem to detect the time correctly, it doesn't ring at all, and I don't know why) MacauleyP_Plays 15 — 4y
0
lmao I posted a working code for your problem but I realized that I'm a bad person so I deleted it DiamondComplex 285 — 4y
0
it wasn't remotely working, but yes you are a bad person, so no one cares anyway MacauleyP_Plays 15 — 4y
0
Instead of all the "or"s try something like if if Time >= 2 and <= 10 then soccerstardance251 29 — 4y
0
nope, that wouldn't work. MacauleyP_Plays 15 — 4y
3 answers
Log in to vote
0
instead of a while loop use runservice heartbeat this is so it can check every frame if it is dead on the hour (meaning it will no matter what)
Also to check if the value is dead on an hour you can just check if value == math.round(value)
ex:
01 | local IsPlaying = false |
02 |
03 | game:GetService("RunService").Heartbeat:Connect(function() |
04 | local Time = game.Lighting.ClockTime -- I noticed that you only set the value once making this value never on the hour. Make sure it updates in the loop |
05 | if Time == math.round(Time) then |
06 | if not IsPlaying then -- putting "not" before hand will allow the code to run if IsPlaying is either nil or false. |
07 | -- Code to play the sounds |
08 | end |
09 | end |
10 | end) |
0
ah stupid me, I should've remembered variables don't automatically update! I'll try your code, but it does seem it should work anyway! MacauleyP_Plays 15 — 4y
0
How can I edit the code so I can setup multiple times for it to ring? I presume the math.round(Time) is where I put the time I want it to ring in right? MacauleyP_Plays 15 — 4y
0
this should be set up for any time thats exactly on the hour, but yes, of course you could set it up for any time DollorLua 235 — 4y
0
and also yes math.Round(Time) is where you would add in other times DollorLua 235 — 4y
0
So it can take multiple values? How many exaclty? MacauleyP_Plays 15 — 4y
Log in to vote
0
There it should ring every hour in Real Time when it works so next one will be in around 32 Minutes, ask me if there are any errors or questions
01 | Lighting = game.Lighting |
02 | Time = Lighting.ClockTime |
03 | Bell = script.Parent.NewtonBells |
04 | IsPlaying = false |
05 |
06 | while true do |
07 | wait(0.5) |
08 | local currentMinute = os.date("*t")["min"] -- gets real minute time |
09 | if currentMinute == 0 then -- will activate in real time when its like 0 minutes so it will ring at 10:00 11:00 12:00 etc. |
10 | if IsPlaying == true then |
11 |
12 | else |
13 | Bell:Play() |
14 | IsPlaying = true |
15 | wait(60.5) -- minute passed so it wont repeat |
View all 21 lines...
0
If you want it to ring at like every 9:30 10:30 and so on put currentMinute == 30 sayer80 457 — 4y
0
If you want it to ring at serverstart tell me you have to get the currentMinute of the server and set it as a variable and then check if currentMinute = starttime sayer80 457 — 4y
0
I was wanting it to follow the in-game time instead of real-life time. Do you think you could alter the script so that it runs via the game-time please? MacauleyP_Plays 15 — 4y
0
Macauley i think the issue is that your ingame time is like value = 14 value = 15 but you have to set little intervals like 14.2 so it wont loop sayer80 457 — 4y
Log in to vote
-1
Or you can do
01 | local PreviousValue = 0 |
02 | local Value = ([[intvalue]]).Value |
03 | while true do |
04 | Value = ([[intvalue]]).Value |
05 | if Value == PreviousValue+1 then |
06 | PreviousValue = Value |
07 | -- do stuff |
08 | end |
09 | wait(0.25) |
10 |
11 | end |
Be sure to define lighting and where you put [[intvalue]] is where you put the time's value.