"I don't understand why FxCop is complaining about this static property on a static class."
"Probably because static classes are stateless, so exposing properties doesn't make a whole lot of sense."
"Yeah, but I want to expose properties so that a caller can set the values that they want."
"But that's really not such a great idea. Either overload methods with the parameters you're trying to read from properties or let callers instantiate the class and set the properties themselves."
Why did I find this so disturbing? Probably because it took two days for the chap to make the change.
The other reason I found it disturbing? I did the math in my head and decided that there would be approximately zero point in trying to get him to understand why globally mutable state can quickly lead to misery (false negatives/positives) when you're dealing with the MTA that NUnit runs in.
No comments:
Post a Comment