As you keep calling the method in OnGUI(), it gets called at least once per frame. You should handle the question picking somewhere else or put a timer or a checker to restrict it.
Don’t you need to specify the return type in javascript ?
Anyway, the random question is picked twice each frame, and as you don’t change the seed (check out how random works on computer to understand what I am saying), you probably have random question going by pairs each frame if you print them. You need to store the question once (on an event, keyboard, button, or Start/Awake functions etc) and then use that variable.