使用 as 运算符进行转换，即使在使用 is 运算符验证 C# 中的 class 类型之后？

Question

我知道使用as运算符在显式转换上转换 object 通常更可取，因为如果转换失败，引用变量将转到 null 而不是引发异常。

但是，假设我检查了 class 和 object 的类型，这是在列表内部，在使用as运算符之前，像这样，

DrawableGameComponent drawComponent;
foreach (component in Components)
{
     if (component is DrawableGameComponent)
     {
          drawComponent = component as DrawableGameComponent;

         // do something with drawComponent
     }
}

通过首先检查is运算符，使用as运算符会失去其好处吗？ 所以做下面的演员也一样好，因为我们在尝试演员之前首先使用is检查 class 类型？

if (component is DrawableGameComponent)
{
     ((DrawableGameComponent)componet).Visible = true;
}

我只是想知道是否有一些我遗漏的基础部分，或者这是否真的归结为使用哪种模式的品味问题。 后一种模式是否通过显式强制转换产生垃圾？

提前致谢！

Answer 1

更好（节省一个“演员” - 比较生成的 IL）：

DrawableGameComponent drawComponent;
foreach (component in Components)
{
     drawComponent = component as DrawableGameComponent;
     if (drawComponent != null)
     {


         // do something with drawComponent
     }
}

Answer 2

您正在使用 as 运算符检查 class 两次，但我怀疑除非在真正庞大的循环中，否则开销会很明显。

我更喜欢使用 as 和测试 null 并清理您的声明以减少行数：

foreach (component in Components)
{
     var drawComponent = component as DrawableGameComponent;
     if (drawComponent != null)
     {
          // do something with drawComponent
     }
}

Answer 3

我会用

DrawableGameComponent drawComponent;
foreach (component in Components)
{
    drawComponent = component as DrawableGameComponent;
    if (drawComponent != null)
    {
        // do something with drawComponent
    }
 }

FXCop可以帮助您在代码中找到可以从像这样的小优化中受益的地方

Answer 4

通过结合is和as你有效地进行了两次类型检查。 只需在这种情况下使用as （如其他答案所示）即可获得所需的内容，并产生更具可读性的 IMO 代码。

Answer 5

您可以使用 LINQ 来获取您的类型。 这种方式不需要铸造。

public class A
{   
}

public class B : A
{
    public bool Visible { get; set; }
}

public class C : A
{
}

void Main()
{
    var data = new List<A> { new A(), new B(), new C(), new B() };

    data.OfType<B>().ToList().ForEach(x => x.Visible = true);
}