This customer's dev is going to be the end of me. I had to explain to him why it doesn't make any sense to return a value after throwing an exception.

In a function that was supposed to insert a record amd return its id, in the error handling code:

catch(Exception ex) {
//logs the exception
throw ex;
return - 999; //this code will NEVER be reached, and why the f.... - 999?
}
The customer wants us to develop the project together, but he won't listen and always write whatever he wants. Some might be differences in taste. Like me preferring
if(condition) {
return a;
}
return b;

And him preferring :

Foo result = null;
if(condition) {
result = a;
} else {
result = b;
}
return result;

Ok, that I can accept. But things like the one I'm ranting about... I wont.

I'm starting to wonder what was he doing during his "9 years of coding experience"

When working through a 3rd party over the last 3 weeks to fix a bug when you know that you could fix it in half the time if they gave you access.