สลับหรือพจนานุกรมเมื่อกำหนดให้กับวัตถุใหม่

เมื่อเร็ว ๆ นี้ฉันชอบทำแผนที่ความสัมพันธ์ 1-1 โดยใช้คำสั่งDictionariesแทน Switchฉันคิดว่ามันจะเร็วขึ้นเล็กน้อยในการเขียนและง่ายต่อการประมวลผลทางจิตใจ น่าเสียดายที่เมื่อทำการแมปกับอินสแตนซ์ใหม่ของวัตถุฉันไม่ต้องการกำหนดเช่นนี้:

var fooDict = new Dictionary<int, IBigObject>()
{
    { 0, new Foo() }, // Creates an instance of Foo
    { 1, new Bar() }, // Creates an instance of Bar
    { 2, new Baz() }  // Creates an instance of Baz
}

var quux = fooDict[0]; // quux references Foo

จากการสร้างนั้นฉันได้สูญเสียรอบ CPU และหน่วยความจำในการสร้างวัตถุ 3 รายการทำสิ่งที่ตัวสร้างของพวกเขาอาจมีและสิ้นสุดลงด้วยการใช้หนึ่งในนั้น ฉันยังเชื่อว่าการทำแผนที่วัตถุอื่น ๆfooDict[0]ในกรณีนี้จะทำให้พวกเขาอ้างอิงในสิ่งเดียวกันแทนที่จะสร้างอินสแตนซ์ใหม่Fooตามที่ตั้งใจไว้ วิธีแก้ปัญหาคือใช้แลมบ์ดาแทน:

var fooDict = new Dictionary<int, Func<IBigObject>>()
{
    { 0, () => new Foo() }, // Returns a new instance of Foo when invoked
    { 1, () => new Bar() }, // Ditto Bar
    { 2, () => new Baz() }  // Ditto Baz
}

var quux = fooDict[0](); // equivalent to saying 'var quux = new Foo();'

นี่เป็นจุดที่ทำให้สับสนหรือไม่? มันง่ายที่จะพลาด()ในตอนท้าย หรือการทำแผนที่กับฟังก์ชั่น / การแสดงออกเป็นเรื่องธรรมดามาก? ทางเลือกอื่นคือใช้สวิตช์:

IBigObject quux;
switch(someInt)
{
    case 0: quux = new Foo(); break;
    case 1: quux = new Bar(); break;
    case 2: quux = new Baz(); break;
}

การอุทธรณ์ใดที่ยอมรับได้มากกว่า

• พจนานุกรมเพื่อการค้นหาที่รวดเร็วขึ้นและคำหลักน้อยลง (ตัวพิมพ์เล็กและใหญ่)
• สวิตช์: พบได้ทั่วไปในรหัสไม่ต้องการใช้วัตถุ Func <> สำหรับการอ้อม

นั่นคือใช้เวลาที่น่าสนใจในรูปแบบโรงงาน ฉันชอบการผสมผสานระหว่างพจนานุกรมและแลมบ์ดา มันทำให้ฉันดูที่คอนเทนเนอร์ในวิธีใหม่

ฉันไม่สนใจข้อกังวลของคุณเกี่ยวกับรอบการทำงานของ CPU ดังที่คุณได้กล่าวไว้ในความคิดเห็นว่าวิธีการที่ไม่ใช่แลมบ์ดาไม่ได้ให้สิ่งที่คุณต้องการ

ฉันคิดว่าวิธีการอย่างใดอย่างหนึ่ง (เปลี่ยนกับพจนานุกรม + แลมบ์ดา) เป็นไปได้ดี ข้อ จำกัด เพียงอย่างเดียวคือการใช้พจนานุกรมคุณกำลัง จำกัด ประเภทของอินพุตที่คุณสามารถรับเพื่อสร้างคลาสที่ส่งคืน

การใช้คำสั่ง switch จะทำให้คุณมีความยืดหยุ่นมากขึ้นในพารามิเตอร์อินพุต อย่างไรก็ตามหากนี่เป็นปัญหาคุณสามารถล้อมพจนานุกรมไว้ข้างในของวิธีการและมีผลลัพธ์ที่เหมือนกัน

หากเป็นเรื่องใหม่สำหรับทีมของคุณให้ใส่รหัสและอธิบายสิ่งที่เกิดขึ้น ขอให้มีการตรวจสอบรหัสทีมงานนำสิ่งที่ทำไปแล้วและทำให้พวกเขาตระหนักถึงมัน นอกเหนือจากนั้นมันดูดี

C # 4.0 ให้Lazy<T>คลาสซึ่งคล้ายกับโซลูชันที่สองของคุณ แต่ส่งเสียง "Lazy initialization" อย่างชัดเจนยิ่งขึ้น

var fooDict = new Dictionary<int, Lazy<IBigObject>>()
{
    { 0, new Lazy(() => new Foo()) }, // Returns a new instance of Foo when invoked
    { 1, new Lazy(() => new Bar()) }, // Ditto Bar
    { 2, new Lazy(() => new Baz()) }  // Ditto Baz
}

โวหารฉันคิดว่าการอ่านมีค่าเท่ากันระหว่างพวกเขา Dictionaryมันเป็นเรื่องง่ายที่จะทำฉีดพึ่งพากับ

อย่าลืมว่าคุณต้องตรวจสอบว่ามีคีย์อยู่Dictionaryหรือไม่และต้องระบุทางเลือกหากไม่มี

ฉันต้องการswitchคำสั่งสำหรับเส้นทางรหัสคงที่และDictionaryสำหรับเส้นทางรหัสแบบไดนามิก (ที่คุณอาจเพิ่มหรือลบรายการ) คอมไพเลอร์อาจจะสามารถที่จะดำเนินการเพิ่มประสิทธิภาพคงที่บางอย่างกับว่ามันไม่สามารถมีswitchDictionary

ที่น่าสนใจDictionaryรูปแบบนี้เป็นสิ่งที่ผู้คนทำใน Python บางครั้งเนื่องจาก Python ไม่มีswitchคำสั่ง มิฉะนั้นพวกเขาใช้โซ่ if-else

โดยทั่วไปแล้วฉันไม่ต้องการ

Func<int, IBigObject>สิ่งที่อยู่นานนี้ควรทำงานร่วมกับ จากนั้นแหล่งที่มาของการทำแผนที่ของคุณสามารถเป็นพจนานุกรมหรือวิธีการที่มีคำสั่งเปลี่ยนหรือโทรบริการบนเว็บหรือการค้นหาไฟล์ ... อะไรก็ตาม

สำหรับการนำไปใช้ฉันต้องการพจนานุกรมเนื่องจากง่ายกว่า refactored จาก 'พจนานุกรมรหัสยาก, คีย์การค้นหา, ส่งคืนผลลัพธ์' เป็น 'โหลดพจนานุกรมจากไฟล์, คีย์การค้นหา, ส่งคืนผลลัพธ์'