The Weird and The Wonderful

I can't really believe it. I found this code in a string library I had downloaded from code project a few days before. I had not read the previous post. What are the odds? This is spooky.

Even Resharper advises you to use _value.

Almost all generalizations are stupid. "To alcohol! The cause of, and solution to, all of life's problems" - Homer Simpson

I have done things like this in the past, when I wanted to give me or other developers a hint for a possible later change. E.g. The "Name" to "2nd Parameter" correspondence could be just by chance or a quick first implementation. What I mean is: maybe "2nd Parameter" has to be localized or mapped later. If you would shorten the code with a direct assignment, I'd assume as a maintenance developer that "Name" always holds a valid "2nd Parameter to IngGenInvPs constructor. And this is maybe not true. This code is not good for other reasons (use switch, use enum, others mentioned it), but not necessarily for the obvious redundance.

Sorry, no exception even for convenience. An integer is an integer, a pointer is a pointer and a boolean is a boolean.

+5,, Thanks for the link My Tip/Trick[^]

Another reason for NOT using "meaningful nulls". What I actually do for the sake of effortless, by the way. [a meaningful wink to the Nullable "?" class]. Greetings - Jacek

Over the years I have seen lots of bad documentation; from missing specs to APIs that are just plain wrong. I am no longer surprised when the docs do not match what the app actually does. Apparently documentation is an afterthought at many companies. :sigh: Just because the code works, it doesn't mean that it is good code.

and that essential scope modifier, "Unused"... as in public unused Boolean FlagSomething = false;

Yeah, but remember, before xhtml came around, it was used like: <input disabled> which makes a lot of sense.

Sounds like you didn't RTFA: "Only primitive string processing was affected by raised exceptions" I'd describe a conversion from an string to an int as fitting that description. Generally, however, I agree - people are too fixated with the costs of exceptions. If there's any I/O going on at all, its negligible.

Maybe he didn't mean slightly odd, but evenly odd?

Over 9GB of log fileS? Seems like you did a pretty good job. We got a xml log file that's 9GB. Opening it instantly crashes the server :) And seeing what my company logs in other applications I suspect it's 9GB full of crap that wouldn't help any programmer to fix any bug :sigh: It's an OO world. public class Naerling : Lazy<Person>{ public void DoWork(){ throw new NotImplementedException(); } }

For sure who doesn't want Steve to sell their stuff. Someone who can convince a planet full of people that they need some crap that they had never even seen before has to be the g8est Salesman.

Spelling mistake should be "DUH" not "The" ;)

The way I see the engineer who wrote it was a genius and all other engineers in the company are morons who can't find such an obvious fault. The engineer might have gone on to bigger better things for sure. Damn with the company that hired the other engineers, fire them and re-hire this guy.

I have once worked with a customer database that had tables with 40 columns, no FK and PK of type varchar(50). Heck the Names column was primary key, when I asked the client what'll happen if another guy joins the Org with same name as his, he was shell shocked. :laugh:

Did you got the guy fired, cause of such guys the software can't cope up even with hardware advancements. Even with a quad core machines and GBs of RAM we still have the same performance issues we used to have in the days of Pentium I and a few MB of RAM.

I bet a happy programmer isn't one who gets their post "Message Automatically Removed"

CDP1802 wrote: if (this == null) { throw new NullReferenceException(); } You don't know how hard we laughed when we found such one in code library. Regards Vallarasu S | BreakingDotNet.blogspot.com