if (customer.id == null)
has to be if (null == customer.id)
I understand the value, but ... tests help catch those issues as well. But... my habit over the last 6 months has been to default to that. if (null == customer.name)
"this is confusing - none of the other code uses this style, it's hard to read, stop doing it".
However, perhaps they just couldn't get the right killer at the right price? Or maybe they're scarcer than you think? We keep hearing immigrants are taking American jobs, but in reality they're taking the jobs no American would take in the first place, so this may be more simple supply/demand...