C#反模式

Question

简而言之:我发现Java反模式是不可或缺的资源.适合初学者和专业人士.我还没有为C#找到这样的东西.所以我将这个问题作为社区维基开放,并邀请所有人分享他们对此的了解.由于我是C#的新手,我对此很感兴趣,但不能从一些反模式开始:/

以下是我发现C#而不是其他语言的答案.

我只是复制/粘贴这些!考虑看看这些评论.

---

投掷 NullReferenceException

抛出错误的异常:

if (FooLicenceKeyHolder == null)
    throw new NullReferenceException();

---

属性与公共变量

类中的公共变量(改为使用属性).

除非该类是一个简单的数据传输对象.

---

不理解bool是一个真正的类型,而不仅仅是一个约定

if (myBooleanVariable == true)
{
    ...
}

或者,甚至更好

if (myBooleanVariable != false)
{
    ...
}

这样的结构通常使用C和C++开发,其中一个布尔值的想法只是一个约定(0 ==假的,什么都为true); 在C#或其他具有真正布尔值的语言中,这不是必需的(或可取的).

---

运用 using()

没有using在适当的地方使用:

object variable;
variable.close(); //Old code, use IDisposable if available.
variable.Dispose(); //Same as close.  Avoid if possible use the using() { } pattern.
variable = null; //1. in release optimised away.  2. C# is GC so this doesn't do what was intended anyway.

Answer 1

错误地重新抛出异常.要重新抛出异常:

try
{
    // do some stuff here
}
catch (Exception ex)
{
    throw ex;  // INCORRECT
    throw;     // CORRECT
    throw new Exception("There was an error"); // INCORRECT
    throw new Exception("There was an error", ex); // CORRECT
}

Answer 2

GC.Collect() 收集而不是信任垃圾收集器.

Answer 3

在Java和C#中我都看到了这种方式太多了......

if(something == true){
  somethingelse = true;
}

如果它也有奖励积分

else{
  somethingelse = false;
}

Answer 4

using Microsoft.SharePoint;

'努夫说

Answer 5

我看到以下代码:

if (i==3)
       return true;
else
       return false;

应该:

       return (i==3);

Answer 6

侮辱德米特法则:

a.PropertyA.PropertyC.PropertyB.PropertyE.PropertyA = 
     b.PropertyC.PropertyE.PropertyA;

Answer 7

投掷NullReferenceException:

if (FooLicenceKeyHolder == null)
    throw new NullReferenceException();

Answer 8

这是真的,我亲眼看到了它.

public object GetNull()
{
     return null;
}

它实际上是在应用程序中使用的,甚至还有一个存储过程也可以使用它,sp_GetNull将返回null ....

这使我的一天.

我认为sp用于经典的asp站点......与结果集有关..net的一个想法是将代码"转换"成.net ...

Answer 9

int foo = 100;
int bar = int.Parse(foo.ToString());

或者更一般的情况:

object foo = 100;
int bar = int.Parse(foo.ToString());

Answer 10

我在我们的项目中发现了这一点,几乎打破了主席......

DateTime date = new DateTime(DateTime.Today.Year, 
                             DateTime.Today.Month, 
                             DateTime.Today.Day);

Answer 11

我经常偶然发现这种滥用:

var ok = Bar();

甚至更好:

var i = AnyThing();

以这种方式使用var毫无意义,也没有任何收获.它只是让代码更难以遵循.

Answer 12

• 在委托调用之前缺少空测试.
• 不知道何时以及如何使用'as'进行空检查与使用try/catch进行强制转换.
• 在catch块中"抛出异常"与"抛出".
• 实例化大量字符串而不是使用StringBuilder.
• 深度嵌套使用块.

Answer 13

不理解bool是一个真正的类型,而不仅仅是一个约定

if (myBooleanVariable == true)
{
    ...
}

或者,甚至更好

if (myBooleanVariable != false)
{
    ...
}

这样的结构通常使用C和C++开发,其中一个布尔值的想法只是一个约定(0 ==假的,什么都为true); 在C#或其他具有真正布尔值的语言中,这不是必需的(或可取的).

更新:重新描述最后一段以提高其清晰度.

Answer 14

类中的公共变量(改为使用属性).

除非该类是一个简单的数据传输对象.

请参阅下面的评论以供讨论和澄清.

Answer 15

我实际看到过这个.

bool isAvailable = CheckIfAvailable();
if (isAvailable.Equals(true))
{ 
   //Do Something
}

击败isAvailable == true反模式的手!
使这成为一个超级反模式!

Answer 16

object variable;
variable.close(); //Old code, use IDisposable if available.
variable.Dispose(); //Same as close.  Avoid if possible use the using() { } pattern.
variable = null; //1. in release optimised away.  2. C# is GC so this doesn't do what was intended anyway.

Answer 17

私人自动实施的属性:

private Boolean MenuExtended { get; set; }

Answer 18

在每个方法的顶部声明和初始化所有局部变量是如此丑陋!

void Foo()
{
    string message;
    int i, j, x, y;
    DateTime date;

    // Code
}

Answer 19

两个字符串反模式
Anti-Pattern#1
检查字符串是否为null或为空

//Bad
if( myString == null || myString == "" )
OR
if( myString == null || myString.Length == 0 )

//Good
string.IsNullOrEmpty(myString)

反模式#2(仅适用于.NET 4.0)
检查字符串是否为空或空或空格

//Bad
if( myString == null || myString == "" || myString.Trim() == "")

//Good
string.IsNullOrWhiteSpace(myString) 

Answer 20

不用铸造(请相信编译器):

foreach (UserControl view in workspace.SmartParts)
{
  UserControl userControl = (UserControl)view;
  views.Add(userControl);
}

Answer 21

if(data != null)
{
  variable = data;
}
else
{
  variable = new Data();
}

可以写得更好

variable = (data != null) ? data : new Data();

甚至写得更好

variable = data ?? new Data();

最后的代码清单适用于.NET 2.0及更高版本