您如何处理遗留应用程序中未使用的代码?

Sri*_*ddi 18 .net javascript c# asp.net

在巨大的遗留应用程序中,通常会看到业务规则的变化导致未使用的代码.删除是最好的方法吗?或者是否有标记未使用代码的标准?如果需要,SCM确实有助于恢复旧代码.这也是特定于.NET代码库的.

Joh*_*zen 55

  1. 将旧项目检入源代码管理.
  2. 删除未使用的代码
  3. 利润!!

  • 你忘记了第3步.1.)检查旧项目到Source Control.2.)删除未使用的代码.3.)... 4.)利润!! (9认同)
  • @Caladain我相信它应该是3.)??? 4.)利润!! (8认同)

cod*_*ife 36

  1. 标记为过时. http://msdn.microsoft.com/en-us/library/aa664623%28VS.71%29.aspx
  2. 下一个周期,注释掉代码并做出适当的评论.在代码块周围设置一个区域,以便它可以很好地折叠并且不会占用大量的可视空间.
  3. 下一个循环,删除代码.

这样,如果您需要弃用某些已分配给他们但需要拨打电话的代码,您就不会对队友造成太大的惊喜.但是允许你做出那些必要的改变!

  • 我会跳过第1步和第2步以及第3步的第一部分. (2认同)

Cha*_*ion 15

总是删除未使用的代码.这是源代码控制的好处之一.


nas*_*nas 11

听取迈克尔杰克逊的一些建议:

只需删除它...删除它...删除它...删除它...

没有人想被击败

Showin'你的斗争有多么时髦和强大

哪个错误或正确无关紧要

只需删除它...删除它...只需删除它...删除它...


Dar*_*der 5

我相信删除是保持代码清洁的好方法.

如果你不删除,例如:注释掉,或创建obselete实体来存储它们,我已经看到了这样的混乱.

这就是为什么你有存储库,分支,标签等.

我觉得最好删除它.如果代码已经存储在存储库中


Chi*_*joo 5

我通常标记为过时:

    /// <summary>
    /// Purpose of this method
    /// </summary>
    /// <param name="args">Argument 1</param>
    [Obsolete("This method is obsolete, use NewMethod instead")]
    public void SampleMethod(string args)
    {
        //code
    }

    /// <summary>
    /// Purpose of this method
    /// </summary>
    /// <param name="args">Argument 1</param>
    public void NewMethod(string args)
    {
        //code
    }
Run Code Online (Sandbox Code Playgroud)

现在,只要使用标记为Obsolete的方法,编译器就会发出警告.