|
|
|
|
|
by henning
705 days ago
|
|
The stupid thing about object-oriented programming and modern software is that code like this would never make it through code review on an "agile" team. - "Oh, why are you doing procedural case statements? that's a SOLID anti-pattern, please refactor to polymorphism" - "Oh, why is the Ascii Renderer class meddling in cell mine logic? `cell.neighbour_mines.zero? ? "_" : cell.neighbour_mines` should be its own method in a `Mine` class that is properly tested" You're never allowed to just write code that solves a problem. It never ends. Your team has unlimited authority to slow you down with completely useless "feedback" that does not help anyone. |
|
When I’ve found myself on teams like this, as a junior engineer I just did what was asked to get along and get experience. As a senior engineer, I often looked for a new job. As a staff engineer, I push back with as much politeness as I can.
Many engineers out there have traded getting it done for perfection. That’s not good judgement.