Question

They both do the same thing. Is one way better? Obviously if I write the code I’ll know what I did, but how about someone else reading it?

if (!String.IsNullOrEmpty(returnUrl)) {     return Redirect(returnUrl); } return RedirectToAction('Open', 'ServiceCall'); 

OR

if (!String.IsNullOrEmpty(returnUrl)) {    return Redirect(returnUrl); } else {     return RedirectToAction('Open', 'ServiceCall'); } 

Answer 1

return String.IsNullOrEmpty(returnUrl) ?              RedirectToAction('Open', 'ServiceCall') :              Redirect(returnUrl); 

I prefer that.

Or the alternative:

return String.IsNullOrEmpty(returnUrl)               ? RedirectToAction('Open', 'ServiceCall')               : Redirect(returnUrl);