You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the Comments section, the styleguide states that:
FIXME points to errors that are too small or too much in-the-making for internal incidents.
TODOs are places where you want to complete something in the near(!) future.
XXX marks code that works but could be better.
My question: what exactly does XXX stand for? Is it an abbreviation of some sort?
It would be nice if this category ("code works but could be better") were to be documented consistently throughout the code. But XXX does not seem all that descriptive to me, causing some unaware developer to maybe even delete these comments, mistaking them for typos!
As per the character of this category, I would most likely see it as a definition of Technical Debt. With that, I would propose an abbreviation that is more telling than XXX, for instance one of the below:
TD marks technical debt (the way it is also defined in literature)
TEDE marks technical debt (harder to be mistaken for ToDo than the above)
The text was updated successfully, but these errors were encountered:
Probably popularized by a developer called Bill Joy in 1971, working on the Berkley Software Distribution (BSD)
Seems to have emerged for unknown reasons as a variant for "fixme" and "###"
Later mostly used as a weaker, not so urgent, or more uncertain alternative to "fixme"
We didn't come up with this abbreviation. We took it over from other projects who used it. This makes me worried a bit whether it's a good idea to mess with it?
I think the XXX codetag is not self-evident. We should drop it. We should go a step further and have broader recommendations for these kinds of comments (codetags), like making them easy to read and understand, which is the essence of clean code. They should also be used rarely.
I don't think TD or TEDE help as well, but each team should be able to decide. We should not dictate which codetags to use (apart from the widespread TODO), but how to use them.
In the
Comments
section, the styleguide states that:FIXME
points to errors that are too small or too much in-the-making for internal incidents.TODO
s are places where you want to complete something in the near(!) future.XXX
marks code that works but could be better.My question: what exactly does
XXX
stand for? Is it an abbreviation of some sort?It would be nice if this category ("code works but could be better") were to be documented consistently throughout the code. But
XXX
does not seem all that descriptive to me, causing some unaware developer to maybe even delete these comments, mistaking them for typos!As per the character of this category, I would most likely see it as a definition of Technical Debt. With that, I would propose an abbreviation that is more telling than
XXX
, for instance one of the below:TD
marks technical debt (the way it is also defined in literature)TEDE
marks technical debt (harder to be mistaken for ToDo than the above)The text was updated successfully, but these errors were encountered: