The Daily WTF: Curious Perversions in Information Technology

Classic WTF: The Great Code Spawn

by Alex Papadimoulis in CodeSOD on 2021-06-21

We're taking a little summer break this week for our regular articles. Since we're re-using articles, let's start with one about developer efficiency. Why write code when you can automate writing code? Original -- Remy

Several years ago, Dan D’s predecessor, Steve, came to the realization that many of us arrived at one point or another: writing data-access code can get boring and repetitive. To ease the tedium, Steve did what many developers in his position do. He wrote some code to generate a lot of code.

Unfortunately, Steve’s coding skills weren’t all too hot. Worse were his code-writing-code writing skills. In the years since The Great Code Spawn (as it has come to be known), the data-access layer has become an unmaintainable disaster – so much so that, rather than add a new database column, developers have “split” single fields into multiple through bit-shifting and string manipulation.

14 comments - Last comment @ 2021-06-23

Filter Your Kwargs

by Remy Porter in CodeSOD on 2021-06-17

Mark's team needed someone to write a pretty straight-forward report for their Python application. It would take a set of keyword arguments, turn those into a filter, apply the filter, and get the results back.

This was implemented in two methods. First:

25 comments - Last comment @ 2021-06-21

World Class Contracting

by Remy Porter in CodeSOD on 2021-06-16

The time and effort needed to complete a project and the amount of time available rarely line up well. Wayne M's company found themselves in a position where what they needed to deliver and what they could deliver by the deadline simply didn't match up.

The requirements were well specified, so they bundled up a bunch of requirements for search-related functionality, and handed them off to a self-described "world class" developer working on contract.

27 comments - Last comment @ 2021-06-17

A Date With Yourself

by Remy Porter in CodeSOD on 2021-06-15

Once upon a time, someone wanted to add a banner to a web page. They also wanted the banner to only appear after a certain date. Jack stumbled across their implementation when trying to understand why the banner would vanish for two weeks at the start of every month.

// get date
var MyDate = new Date();
var MyDateString;

MyDate.setDate(MyDate.getDate());

MyDateString = ('0' + MyDate.getDate()).slice(-2) + '-' + ('0' + (MyDate.getMonth()+1)).slice(-2) + '-' + MyDate.getFullYear();

if (MyDateString > '13-04-2014') {
    // do stuff...
}

30 comments - Last comment @ 2021-06-20

Experience is Integral

by Remy Porter in CodeSOD on 2021-06-14

Behind every code WTF is a process WTF. For example, Charles W was recently tasked with updating some file-handling code to match changes in the underlying file-format it parses. This is the C code which parses an integer:

		if ((*p == '-' || ('0' <= *p && '9' >= *p)) && retCode == -256)
		{
			retCode = 0;
			p = _tcsrev(p);
			if (*p == ' ')
				p++;
			for (i = 0; '0' <= *p && '9' >= *p; i++)
			{
				retCode += (int)pow(10, (double)i) * ((int)*p - 0x30);
				p++;
			}
			if (*p == '-')
				retCode *= -1;
		}

36 comments - Last comment @ 2021-06-18

Quite the Event

by Remy Porter in CodeSOD on 2021-06-10

A few years back, Alvin was in college, and took his first formal summer job as a programmer. It was supposed to be just some HTML layout work, and the designer handed him a few sample pages. "Just do it like this."

The "sample pages" were a mishmash of random indentation, huge swathes of commented out code, and all those other little traits of "someone's just coding in production, aren't they?" that crop up in files. Still, it was just some HTML layout work, so how hard could it be?

25 comments - Last comment @ 2021-06-21

Getting Overloaded With Details

by Remy Porter in CodeSOD on 2021-06-09

Operator overloading is one of those "dangerous" features. Like multi-parent inheritance, it can potentially create some really expressive, easy to read code, or it can create a disaster of incomprehensible nonsense.

In C++, many core classes use operator overloading, most notably the I/O classes, which reuse (or abuse) the bitshift operators into stream operators. So, for example, one possible way of converting a string into an integer might be to do something like this:

17 comments - Last comment @ 2021-06-13

Contractor Management System

by Remy Porter in CodeSOD on 2021-06-08

Maximillion was hired to maintain a PHP CMS. His new employer, up to this point, had just been contracting out work, but as time went on, contracting rates got higher, the amount of time required to add basic features kept going up. It was time to go ta a full time employee.

"The system's pretty simple," the hiring manager explained during the interview, "as I understand it, it's basically just one PHP file."

33 comments - Last comment @ 2021-06-12

Are You Active Enough?

by Remy Porter in CodeSOD on 2021-06-07

Cornelius was working with some code where the objects might be "active" or "inactive". His code needed to do something different, depending on whether the objects were "active" or not, but fortunately, there was a handy-dandy IsActive method. Weirdly, that method required a bool parameter, but also returned a bool. Since there wasn't any useful documentation, Cornelius checked the C++ code.

    bool SomeActivatedClass::IsActive(bool& active)
    {
        active = true;
        return false;
    }

32 comments - Last comment @ 2021-06-12

A Little Info

by Remy Porter in CodeSOD on 2021-06-03

Matt has plans for the next few years: dealing with the "inheritance" of some legacy systems.

They're written in VB.Net, which isn't itself a problem, but the code quality leaves just a bit to be desired.

24 comments - Last comment @ 2021-06-12

A World Class Programmer

by Remy Porter in CodeSOD on 2021-06-02

Jesse had a "special" co-worker, Rupert. Rupert was the sort of person that thinks they're the smartest person walking the Earth today, and was quite happy to loudly proclaim that everyone else is wrong. Rupert was happy so long as everyone else was ready to bask in his "genius".

Fortunately for Jesse, Rupert left, because he'd received a huge offer for a senior developer role at a different company. Everyone at Jesse's company privately chuckled about it, because this is the kind of code Rupert's genius produced:

40 comments - Last comment @ 2021-06-07

As Authentic as it Gets

by Remy Porter in CodeSOD on 2021-06-01

Virginia N (previously) needed to maintain some authentication logic. The actual rules for authentication weren't actually documented, so her choices were to either copy/paste some authentication code from a different project, or try and refactor the authentication method in this one.

It was a hard choice. We'll dump the whole block of code, but I want to pull out a few highlights, starting with the opening condition:

16 comments - Last comment @ 2021-06-17

Recent CodeSOD