Seems fine for the most part, you should also make this class client only, @OnlyIn does literally nothing for modders, it's just an ugly ASM hack for Forge and Minecraft internal use.
Edit: Well...upon further inspection I think you're code won't function on the server at all, you can't just store static variables in an event handler, remember, there is only one instance of your mod on the server, so every player is going to try to share those variables, breaking everything.