c# - 你如何标记代码，以便以后可以回来处理它？

Question

在 C# 中，我使用#warningand#error指令，

#warning This is dirty code...
#error Fix this before everything explodes!

这样，编译器会让我知道我还有工作要做。你用什么技术来标记代码，这样你就不会忘记它？

score 94 · Accepted Answer

用或其他注释标记标记它们// TODO，这些// HACK标记将显示在 Visual Studio 的任务窗格中。

请参阅使用任务列表。

score 28 · Accepted Answer

待办事项评论也是如此。

我们还添加了一个特殊的关键字 NOCHECKIN，我们在源代码控制系统中添加了一个提交挂钩（至少使用 cvs 或 svn 很容易做到），它会扫描所有文件并拒绝签入文件，如果它找到任何地方的文本 NOCHECKIN。

如果您只是想测试一些东西并确保它不会被意外签入（在提交到源代码控制的所有内容的差异期间通过监视的眼睛），这将非常有用。

score 15 · Accepted Answer

//TODO: //HACK:我在我的方法上使用和的组合throw new NotImplementedException();来表示未完成的工作。此外，我在 Visual Studio 中为不完整的行添加了书签。

score 10 · Accepted Answer

//TODO: 人名 - 请解决这个问题。

这是在 Java 中，然后您可以查看 Eclipse 中的任务，这些任务将找到对该标记的所有引用，并且可以按人对它们进行分组，以便您可以将 TODO 分配给其他人，或者只查看您自己的。

score 7 · Accepted Answer

如果我必须在更改过程中放弃所有内容，那么

#error finish this

如果这是我以后应该做的事情，它会进入我的错误跟踪器（用于所有任务）。

score 6 · Accepted Answer

“待办事项”评论在理论上很好，但在实践中却不是很好，至少在我的经验中是这样。如果您要被拉走足够长的时间以需要它们，那么它们往往会被遗忘。

我赞成 Jon T 的一般策略，但我通常只是暂时简单地破坏代码来做到这一点——我经常插入一个故意未定义的方法引用，让编译器提醒我需要返回的内容：

PutTheUpdateCodeHere();

score 6 · Accepted Answer

我非常喜欢的一种方法是“Hack Bombing”，正如 Oren Eini在这里所展示的那样。

try
{
   //do stuff
   return true;
}
catch // no idea how to prevent an exception here at the moment, this make it work for now...
{
  if (DateTime.Today > new DateTime(2007, 2, 7))
    throw new InvalidOperationException("fix me already!! no catching exceptions like this!");
  return false;
}

score 5 · Accepted Answer

添加一个处于禁用状态的测试。它们出现在所有构建报告中。

如果这不起作用，我会提交一个错误。

特别是，我还没有看到 TODO 评论的数量以任何有意义的方式减少。如果我写评论时没有时间做，我不知道为什么我以后会有时间。

score 4 · Accepted Answer

//TODO: Finish this

如果您使用 VS，您可以在工具>选项>环境>任务列表下设置自己的任务标签

score 3 · Accepted Answer

gvim 用黄色突出显示“// XXX”和“// TODO”，当我第一次用这种方式标记一些代码以提醒自己回到它时，这让我很惊讶。

score 3 · Accepted Answer

我是一名 C++ 程序员，但我想我的技术可以很容易地用 C# 或任何其他语言实现：

我有一个ToDo(msg)宏，可以扩展为在本地范围内构造一个静态对象，其构造函数输出一条日志消息。这样，当我第一次执行未完成的代码时，我会在日志输出中收到一条提醒，告诉我不能再推迟任务。

它看起来像这样：

class ToDo_helper
{
  public:
     ToDo_helper(const std::string& msg, const char* file, int line)
     {
       std::string header(79, '*');
       Log(LOG_WARNING) << header << '\n'
                        << "  TO DO:\n"
                        << "    Task:  " << msg << '\n'
                        << "    File:  " << file << '\n'
                        << "    Line:  " << line << '\n'
                        << header;
     }
};

#define TODO_HELPER_2(X, file, line) \
  static Error::ToDo_helper tdh##line(X, file, line)

#define TODO_HELPER_1(X, file, line) TODO_HELPER_2(X, file, line)
#define ToDo(X) TODO_HELPER_1(X, __FILE__, __LINE__)

...你像这样使用它：

 void some_unfinished_business() {
   ToDo("Take care of unfinished business");
 }

score 3 · Accepted Answer

这不是一个完美的世界，我们并不总是有无限的时间来重构或思考代码。

//REVIEW如果我想稍后再回来，我有时会输入代码。即代码正在工作，但也许不相信这是最好的方法。

// REVIEW - RP - Is this the best way to achieve x? Could we use algorithm y?

同样适用//REFACTOR

// REFACTOR - should pull this method up and remove near-dupe code in XYZ.cs

score 2 · Accepted Answer

我使用// TODO: 或// HACK: 来提醒某些事情未完成，并附上解释原因的注释。由于时间限制，我经常（读“很少”）回去完成这些事情。但是，当我查看代码时，我记录了未完成的内容，更重要的是为什么。

我经常在一天或一周结束时使用的另一条评论：

// 从这里开始克里斯

^^^^^^^^^^^^^^^^^^^^ 告诉我我在哪里停下来，这样我就可以在星期一早上尽量减少我的引导时间。

score 2 · Accepted Answer

我将 //FIXME: xxx 用于损坏的代码，将 //CHGME: xxx 用于需要注意但有效的代码（可能仅在有限的上下文中）。

score 1 · Accepted Answer

1

待办评论。

于 2008-12-02T20:44:36.717 回答

score 1 · Accepted Answer

// TODO: <explanation>

如果这是我还没有开始实施并且不想忘记的事情。

// FIXME: <explanation>

如果它是我认为不合适的东西，并且想稍后再回来或让其他人看到它。

从未想过#error/#warning 选项。这些也可以派上用场。

score 1 · Accepted Answer

这是我发现有助于标记需要解决的问题的三种不同方法。

在需要查看的代码旁边放置一个注释标志。大多数编译器可以识别常见的标志并以有组织的方式显示它们。通常你的 IDE 有一个专门为这些标志设计的监视窗口。最常见的注释标志是： //TODO 你将如何使用它：

//TODO：在发布之前修复它。这会导致访问冲突，因为它正在使用尚未创建的内存。
在发布之前标记需要解决的问题的一种方法是创建一个无用的变量。如果您有未使用的变量，大多数编译器都会警告您。以下是如何使用此技术：

int This_Is_An_Access_Violation = 0;
IDE 书签。大多数产品都会提供一种在您的代码中放置书签以供将来参考的方法。这是一个好主意，只是它只能被您看到。当您共享代码时，大多数 IDE 不会共享您的书签。您可以查看 IDE 的帮助文件系统，了解如何使用它的书签功能。

score 1 · Accepted Answer

我也使用 TODO：评论。我理解他们很少真正得到修复的批评，并且最好将它们报告为错误。但是，我认为这忽略了几点：

当我不断重构和重新设计事物时，我最常使用它们。所以我一直在看着他们。在这种情况下，他们中的大多数实际上确实得到了解决。另外，搜索 TODO 很容易：确保我没有遗漏任何内容。
对于阅读您的代码的人来说，了解您认为写得不好或被破解的地方非常有帮助。如果我正在阅读不熟悉的代码，我倾向于寻找组织模式、命名约定、一致的逻辑等。如果为了权宜之计不得不违反这种一致性一两次，我宁愿看到一个说明。这样我就不会浪费时间去寻找没有逻辑的地方。

score 1 · Accepted Answer

如果是一些长期的技术债务，你可以这样评论：

// TODO: This code loan causes an annual interest rate of 7.5% developer/hour. Upfront fee as stated by the current implementation. This contract is subject of prior authorization from the DCB (Developer's Code Bank), and tariff may change without warning.

... 呃。我想 TODO 会做到这一点，只要你不简单地忽略它们。

score 0 · Accepted Answer

正如大多数程序员在这里所做的那样，我使用 TODO 注释。此外，我使用 Eclipse 的任务界面Mylyn。当任务处于活动状态时，Mylyn 会记住我打开的所有资源。这样我可以追踪

在文件中我必须做某事（以及什么），
我必须在哪些文件中执行此操作，以及
它们与什么任务相关。

score 0 · Accepted Answer

除了关闭“TODO:”注释之外，许多 IDE 还关闭了“TASK:”注释。一些 IDE 甚至允许您配置自己的特殊标识符。

score 0 · Accepted Answer

这是我使用的临时评论标签列表：

//+TODO  Usual meaning.
//+H     Where I was working last time.
//+T     Temporary/test code.
//+B     Bug.
//+P     Performance issue.

表示不同的优先级，例如：//+B vs //+B+++

好处：

易于从代码中搜索/删除（查找//+）。
易于按优先级过滤，例如：搜索//+B以找到所有错误，搜索//+B+++以仅获得高优先级的错误。

可用于 C++、C#、Java、...

为什么是//+符号？因为+符号看起来有点像t，为暂时的。

注意：这不是标准推荐，只是个人推荐。

score 0 · Accepted Answer

在你的代码库中散布信息量不足的 TODO 可能不是一个好主意，特别是如果你有多个贡献者随着时间的推移。这对于新手来说可能会很困惑。然而，在我看来，在实践中运作良好的是说明作者和 TODO 的编写时间，带有标题（最多 50 个字符）和更长的正文。

无论您在 TODO 评论中包含什么内容，我都建议您系统地跟踪它们。例如，有一项服务可以根据git blame( http://www.tickgit.com ) 检查存储库中的 TODO 注释。

我开发了自己的命令行工具，以使用此处答案（https://github.com/mristin/opinionated-csharp-todos）中的想法来强制执行 TODO 注释的一致风格。将它集成到持续集成中相当容易，因此每次推送到主服务器时都会重新生成任务列表。

当您在与其他人的会议中讨论TODO时，当您想通过电子邮件共享它时，将任务列表与您的 IDE 分开也是有意义的。

c# - 你如何标记代码，以便以后可以回来处理它？

23 回答 23

Related

Reference