logoalt Hacker News

IshKebabyesterday at 8:48 PM1 replyview on HN

No it isn't. You don't need to know the answer to know that it is bad code. The very fact that it isn't clear shows that.


Replies

tialaramexyesterday at 11:24 PM

Right, the feedback I'd expect in a code review interview is something like "This is unclear or wrong, write what you actually meant".

That's the feedback I would want, and it's the feedback I give to my colleagues in reviews. Actually I tend to be too verbose, so you might get a full paragraph explaining what the ISO document says and that you shouldn't assume it does whatever it is your compiler says.

My actual feelings for this specific case are that the language is defective, but if we're wedded to a defective language then the reviews need to call out such usage.

show 1 reply