I was using Resharper 4 and noticed some nice code cleanup advise that it gives. One to note it the compression of:
// Create job if quote is won if (CurrentQuote.Status == "A") { quoteView.FindControl("btnCreateJobCard").Visible = true; } else { quoteView.FindControl("btnCreateJobCard").Visible = false; }
To the one liner (which I prefer)
// Create job if quote is won quoteView.FindControl("btnCreateJobCard").Visible = CurrentQuote.Status == "A";
I like short succinct code. I think the shortened form is better, but I know some people need to do a double take to see what it’s trying to do.
I would love to hear your opinions on this.