Automation for Radio Stations and Internet Streams

[ripdad]

Hello all

Can somebody check this script please?

It shows no errors on syntax and it works good.

But, I need to see if it can be done any simpler

or should I leave it alone?

Thanks

Example of WAAM.au3

[JohnOne]

First thing that jumps out at me, id that your script sleeps for 1000 miliseconds, or 1 second, and its conditional on seconds.

That says to me you are bound at some point to miss the 00 seconds

[ripdad]

The seconds to 00 doesn't matter too much.

It was early on, but the script would max the CPU load if I didn't add 1 second to it.

I'll look into it again.

Thanks for your reply

[ripdad]

I rechecked the 1 second line.

It does indeed need to be included, or else the script will max the CPU.

It's weird. Does anyone know why this happens?

-- Thanks

[JohnOne]

Yes you must have a sleep in there, but my point is this

While 1
Sleep(1000); <================================================= sleeping 1 second
If @MIN = 00 and @SEC = 00 Then; <============================= Relying on in seconds
SoundPlay("c:\Podcast\MCID.mp3", 0) ; Plays ID at the top of every hour
Sleep(1000)
EndIf
If @MIN = 55 and @SEC = 00 Then
Run(@ComSpec & " /c " & 'c:\Podcast\download.exe http://64.26.15.101/feeds/FSNBulletin.mp3 /update /output:c:\Podcast', "",@SW_HIDE) ; Downloads the lastest news spot
Sleep(1000)
EndIf

So if 00 seconds occurs while your code is sleeping that particular second, it may well miss it and your sound will not play.

I could be off the mark here so dont take me at my word, but for me, if the sleep had to be that much it would be Sleep(999)

[ripdad]

Ah - I see what you're saying. It has a 1 in 60 chance of missing 00.

Whats the optimal sleep value at that line?

--Thanks

[ripdad]

Update: fixed minor issues and formatted

Example of WAAM.au3