Forum Discussion

jbrave's avatar
jbrave
Channel Surfer
16 years ago

(rant) if then else weirdness & ideal message loops

I want my code to be readable. an IF that isn't terminated with an End if makes for difficult to read code. Yet Brightscript seems to complain if you have every if terminated, for instance: if x=1...
Roku Developer Program
renojim's avatar
renojim
Community Streaming Expert
16 years ago
If you want to always have an 'end if', then don't put the return on the same line:
if x=1 then
   return
end if

I'd say it isn't always necessary to check the msg type if the port is only associated with one type of component, but it does make the code more readable. Also, if your port is associated with more than one component, like a Springboard screen and an audio player, then your code would crash if you received an roAudioPlayerEvent and you had 'if msg.isScreenClosed() then' since there is no isScreenClosed() event for an audio player. There are quite a lot of events where msg.GetMessage() returns an empty string. In fact, it's probably something that is used less often than the other means of testing for the purpose of the message.

-JT