The Daily WTF: Curious Perversions in Information Technology

2022-03-23 Reply Admin

Even the compare is not needed. The "checked" is always true so it's enough to look at the "IsFlagged" itself. And it would be more readable too.

2022-03-23 Reply Admin

var isFlagged = FilterCheckBox.Checked; FilteredSortedItems = isFlagged ? CurrentItemList.Where(info => info.IsFlagged == isFlagged).ToList() : CurrentItemList.ToList();

Smells like a serious case of over engineering or trying to write clean code without understand what clean code is all about :-)

2022-03-23 Reply Admin

I wonder whether a developer was bored one Friday afternoon? Or wanted to drag a job out a little longer so as not to be in the frame for another project for a while? Can't see how this sort of thing is not at least partly deliberate over-engineering for the sake of it.

2022-03-23 Reply Admin

The intern needed to do some CS homework, and accidentally committed it.

2022-03-23 Reply Admin

@MaxiTB ref

var isFlagged = FilterCheckBox.Checked; FilteredSortedItems = isFlagged ? CurrentItemList.Where(info => info.IsFlagged == isFlagged).ToList() : CurrentItemList.ToList();

I suspect that could be further simplified to

FilteredSortedItems = FilterCheckBox.Checked ? CurrentItemList.Where(info => info.IsFlagged).ToList() : CurrentItemList.ToList();

Andre Alexin · 2022-03-23 Reply Admin

The developer was paid by line of code.

2022-03-23 Reply Admin

"The developer was paid by line of code." I once had a job where management said they were going to start rating programmers by lines of code produced per day. And I thought, great! I can boost my productivity by that measure easily! No more "x=x+4". Instead I'll write "x=x+1" four times. And loops? No, just copy and paste the same code as many times as you need. I had lots of ideas like that. Sadly, the company never followed through on the policy.

2022-03-23