[torginus]

It's kind of weird how terrible this code is (according to software design 'best practices'). 5k lines in a single file, tons of repetition that could be replaced with more compact code etc.

I would think the author was a novice programmer, but looking at the creator's history they've already made Towerfall, another hugely acclaimed game (and one I still play to this day), about 5 years before this was released.

Yet the game itself is extremely polished.

I kind of have no idea where to take this thought, except that successful auters have very different sensibilities than garden variety programmers.

[chii]

> (according to software design 'best practices')

that's because these best practices aren't geared towards producing something good - it's geared towards making something easy to understand for someone else, and easy to maintain when requirements change.

It is also designed for the lowest common denominator programmers - so that many can work on the same code base, over a long period of time.

For games, none of these goals matter at all.

[sirtomato]

Yep, absolutely. A lot of game code has really big files. Although not exactly the gold standard for code style, the Source Engine does have a significant level of even though almost every file is 1k+ SLOC.

Here's the base Source Engine movement code for example: https://github.com/VSES/SourceEngine2007/blob/master/src_mai...

[torginus]

I mean, stuff like this isn't really defensible:

    switch (index)
    {    
        case 0: return sound0;
        case 1: return sound1;
        case 2: return sound2;
        case 3: return sound3;
        case 4: return sound4;
    }

[32gbsd]

Look perfectly readable and easy to understand.

[chii]

only because there's no `default` case.

You could argue that having `sound` as an array, and index into that array is better, but once again, what does "better" mean here?

this code works as is, you will not get an array index out of bounds error. If this is quick and easy to write, what else could you ask for?

[32gbsd]

Its because best practices are written by people who dont write alot of code. Most of them are lowest common denoninator bs.

[account42]

Games are (sadly IMO) mostly write-only software.