C #: 'is' คำหลักและตรวจสอบ Not

นี่เป็นคำถามงี่เง่า แต่คุณสามารถใช้รหัสนี้เพื่อตรวจสอบว่ามีบางอย่างที่เป็นประเภทใด ...

if (child is IContainer) { //....

มีวิธีที่สง่างามกว่านี้ในการตรวจสอบอินสแตนซ์ "NOT" หรือไม่

if (!(child is IContainer)) { //A little ugly... silly, yes I know...

//these don't work :)
if (child !is IContainer) {
if (child isnt IContainer) { 
if (child aint IContainer) { 
if (child isnotafreaking IContainer) { 

ใช่ใช่ ... คำถามงี่เง่า ....

เนื่องจากมีคำถามบางอย่างเกี่ยวกับลักษณะของรหัสมันเป็นเพียงการคืนค่าอย่างง่าย ๆ ในตอนเริ่มต้นของวิธีการ

public void Update(DocumentPart part) {
    part.Update();
    if (!(DocumentPart is IContainer)) { return; }
    foreach(DocumentPart child in ((IContainer)part).Children) {
       //...etc...

if(!(child is IContainer))

เป็นผู้ดำเนินการรายเดียวที่จะดำเนินการ (ไม่มีIsNotผู้ดำเนินการ)

คุณสามารถสร้างวิธีการขยายที่:

public static bool IsA<T>(this object obj) {
    return obj is T;
}

จากนั้นใช้เพื่อ:

if (!child.IsA<IContainer>())

และคุณสามารถติดตามธีมของคุณ:

public static bool IsNotAFreaking<T>(this object obj) {
    return !(obj is T);
}

if (child.IsNotAFreaking<IContainer>()) { // ...

อัปเดต (พิจารณาข้อมูลโค้ดของ OP):

เนื่องจากคุณหล่อมูลค่าหลังจากนั้นคุณสามารถใช้asแทน:

public void Update(DocumentPart part) {
    part.Update();
    IContainer containerPart = part as IContainer;
    if(containerPart == null) return;
    foreach(DocumentPart child in containerPart.Children) { // omit the cast.
       //...etc...

คุณสามารถทำได้ด้วยวิธีนี้:

object a = new StreamWriter("c:\\temp\\test.txt");

if (a is TextReader == false)
{
   Console.WriteLine("failed");
}

ทำไมไม่ลองใช้อย่างอื่นล่ะ?

if (child is IContainer)
{
  //
}
else
{
  // Do what you want here
}

มันเป็นระเบียบที่คุ้นเคยและเรียบง่าย

วิธีที่คุณมีนั้นดี แต่คุณสามารถสร้างชุดของวิธีการขยายเพื่อทำให้ "เป็นวิธีที่หรูหรากว่าในการตรวจสอบอินสแตนซ์ 'NOT'

public static bool Is<T>(this object myObject)
{
    return (myObject is T);
}

public static bool IsNot<T>(this object myObject)
{
    return !(myObject is T);
}

จากนั้นคุณสามารถเขียน:

if (child.IsNot<IContainer>())
{
    // child is not an IContainer
}

เรื่องนี้ยังไม่ได้รับการกล่าวถึง มันใช้งานได้และฉันคิดว่ามันดูดีกว่าการใช้!(child is IContainer)

if (part is IContainer is false)
{
    return;
}

isไวยากรณ์: expr is constantโดยที่ expr คือนิพจน์ที่ใช้ประเมินและค่าคงที่คือค่าที่จะทดสอบ

น่าเกลียด? ฉันไม่เห็นด้วย. วิธีอื่นเท่านั้น (โดยส่วนตัวฉันคิดว่านี่คือ "uglier"):

var obj = child as IContainer;
if(obj == null)
{
   //child "aint" IContainer
}

isประเมินผู้ประกอบการที่จะเป็นผลบูลีนเพื่อให้คุณสามารถทำอะไรที่คุณมิฉะนั้นจะไม่สามารถที่จะทำในบูล เพื่อลบล้างมันใช้!ผู้ประกอบการ ทำไมคุณต้องการมีผู้ประกอบการที่แตกต่างกันเพียงแค่นี้?

วิธีการขยายIsNot<T>เป็นวิธีที่ดีในการขยายไวยากรณ์ เก็บไว้ในใจ

var container = child as IContainer;
if(container != null)
{
  // do something w/ contianer
}

ทำงานได้ดีกว่าทำสิ่งที่ชอบ

if(child is IContainer)
{
  var container = child as IContainer;
  // do something w/ container
}

ในกรณีของคุณมันไม่สำคัญว่าคุณจะกลับมาจากวิธีการ ในคำอื่น ๆ ระวังอย่าทำทั้งการตรวจสอบประเภทและการแปลงประเภททันทีหลังจาก

แม้ว่าสิ่งนี้จะไม่หลีกเลี่ยงปัญหาในวงเล็บ แต่เพื่อให้ผู้คนเดินทางมาที่นี่ผ่าน Google ได้ แต่ก็ควรจะกล่าวว่ามีไวยากรณ์ใหม่กว่า (เช่น C # 7) เพื่อทำให้โค้ดที่เหลือของคุณสะอาดขึ้นเล็กน้อย:

if (!(DocumentPart is IContainer container)) { return; }
foreach(DocumentPart child in container.Children) {
    ...

สิ่งนี้จะหลีกเลี่ยงการแสดงผลซ้ำซ้อน, การตรวจสอบค่าว่างและมีตัวแปรพร้อมใช้งานในขอบเขตที่สามารถเป็นค่าว่างได้

ในขณะที่ผู้ปฏิบัติงานตามปกตินั้นเป็นวิธีที่ดีที่สุด แต่ก็มีทางเลือกอื่นที่คุณสามารถใช้ได้ในบางสถานการณ์ คุณสามารถใช้เป็นโอเปอเรเตอร์และทดสอบหาค่าว่าง

MyClass mc = foo as MyClass;
if ( mc == null ) { }
else {}

C # 9 (ที่จะออกมาพร้อมกับ .NET 5) จะมีรูปแบบลอจิคัลand, orและnotซึ่งช่วยให้เราสามารถเขียนนี้มากขึ้นอย่างหรูหรา:

if (child is not IContainer) { ... }

ในทำนองเดียวกันรูปแบบนี้สามารถใช้ในการตรวจสอบโมฆะ:

if (child is not null) { ... }

คุณสามารถดูรายละเอียดเพิ่มเติมเกี่ยวกับปัญหา Github ที่ติดตามการเปลี่ยนแปลงนี้

if (child is IContainer ? false : true)