[Qt] add more NULL pointer checks in bitcoingui

- add missing checks for clientModel and optionsModel
- small cleanups for an #ifdef on Mac
- remove an unneeded else
- small layout changes for if clauses
Nonono, that's by no means a certainly. An event handler should use the sequence that makes sense, sometimes that can be very subtle. If we have no need to change this, keep it as it is.