Well the first thing is that you really do not want to do your own GL context rendering in the way you've done it. Making something current on the windows event thread will not get you the context that is current for the drawable you're working with. In fact, what you're doing with that GL call in the event listener handler is very dangerous and likely to cause all sorts of weird things to happen. You are far better of having a local state variable that the key listener sets to the appropriate state, and then at the beginning of your rendering loop set the right GL state at that time.

No, it will be far easier to do so because you don't have to worry about all sorts of oddball states happening. Collecting everything into a single class, and then executing it all at once at exactly the right time is the correct way to do it, and also the easiest to maintain and modify in the future.

java-gaming.org is not responsible for the content posted by its members, including references to external websites,
and other references that may or may not have a relation with our primarily
gaming and game production oriented community.
inquiries and complaints can be sent via email to the info‑account of the
company managing the website of java‑gaming.org