Comment

Comment by WalterBright

> You have a tendency to overengineer things.

Overengineering is an actual problem. For a tiny example, I'll see things like:

    enum MAGIC = 67;  // explanation
    ...
    foo(MAGIC);
The use of MAGIC is the only one, and is far removed. A better solution is:

    foo(67);  // explanation
because it improves locality.

I also see things like an object fleshed out with all kinds of member functions that are never used.

Discussion

Live from the official Hacker News API

A modern reader for Hacker News · SvelteKit + Skeleton