Aren't things supposed to be on new lines to help with readability as well as keeping everything consistent?
if(!name)
(tab)name = etc.

Whereas right now it's just on one line.