ควรให้บริการมีข้อยกเว้นหรือส่งคืนเมื่อไม่มีรายการที่ระบุไว้สำหรับการลบ

ฉันมีโค้ดบางส่วนที่สามารถแสดงเป็น:

public class ItemService {

    public void DeleteItems(IEnumerable<Item> items)
    {
        // Save us from possible NullReferenceException below.
        if(items == null)
            return;

        foreach(var item in items)
        {
            // For the purpose of this example, lets say I have to iterate over them.
            // Go to database and delete them.
        }
    }
}

ตอนนี้ฉันสงสัยว่านี่เป็นวิธีการที่เหมาะสมหรือฉันควรจะยกเว้น ฉันสามารถหลีกเลี่ยงข้อยกเว้นได้เนื่องจากการส่งคืนจะเหมือนกับการวนซ้ำคอลเลกชันที่ว่างเปล่าความหมายไม่มีรหัสที่สำคัญใด ๆ ถูกเรียกใช้งาน แต่ในทางกลับกันฉันอาจซ่อนปัญหาในรหัสเพราะทำไมทุกคนต้องการโทรDeleteItemsมีnullพารามิเตอร์หรือไม่ นี่อาจบ่งบอกว่ามีปัญหาที่อื่นในรหัส

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

นี่เป็นคำถามสองข้อที่แตกต่างกัน

คุณควรยอมรับnullหรือไม่ ขึ้นอยู่กับนโยบายทั่วไปของคุณเกี่ยวกับnullในฐานรหัส ในความคิดของฉันห้ามnullทุกที่ยกเว้นเอกสารที่ระบุไว้อย่างชัดเจนเป็นวิธีปฏิบัติที่ดีมาก

คุณควรยอมรับคอลเล็กชันที่ว่างเปล่าหรือไม่? ในความคิดของฉัน: ใช่แน่นอน มันเป็นความพยายามมากขึ้นในการ จำกัด ผู้โทรทั้งหมดไปยังคอลเล็กชั่นที่ไม่ว่างเปล่ากว่าจะทำในสิ่งที่ถูกต้องทางคณิตศาสตร์แม้ว่ามันจะทำให้นักพัฒนาบางคนประหลาดใจกับแนวคิดของศูนย์

ค่า Null

ดังที่ @KilianFoth ได้กล่าวไปแล้วโปรดทำตามนโยบายทั่วไปของคุณ หากนั่นคือการถือว่าnullเป็น "ชวเลข" สำหรับรายการว่างทำอย่างนั้น

หากคุณไม่มีนโยบายที่สอดคล้องกับnullค่าฉันขอแนะนำนโยบายต่อไปนี้:

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

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

รายการว่างเปล่า

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

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

โยนข้อยกเว้นและจัดการ nulls ในรหัสโทร

ตามกฎการออกแบบพยายามหลีกเลี่ยงค่า null เป็นค่าพารามิเตอร์ มันจะลด NullPointerExceptions โดยทั่วไปเนื่องจากค่า Null จะเป็นข้อยกเว้นจริงๆ

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

โดยทั่วไปการพูดข้อยกเว้นการขว้างปาควรถูกสงวนไว้สำหรับสถานการณ์พิเศษเช่นหากรหัสนั้นไม่มีวิธีการดำเนินการที่สมเหตุสมผลในบริบทปัจจุบัน

คุณสามารถใช้กระบวนการคิดนั้นกับสถานการณ์นี้ ระบุว่านี่เป็นคลาสสาธารณะที่มีวิธีการสาธารณะคุณมี API สาธารณะและในทางทฤษฎีไม่สามารถควบคุมสิ่งที่ผ่านไปได้

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

คำถามนี้ดูเหมือนจะไม่เกี่ยวกับข้อยกเว้น แต่nullเป็นอาร์กิวเมนต์ที่ถูกต้อง

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

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

พื้นที่ของการต่อสู้ก็คือไม่ว่าจะเป็นฟังก์ชั่นห้องสมุดควรจะเป็นจริงๆเข้มงวดเกี่ยวกับสิ่งนั้นหรือเป็นผ่อนปรนที่เป็นไปได้ตราบใดที่มันไม่ได้พยายามที่จะ "แก้ไข" พารามิเตอร์ที่ผิดพลาด เปรียบเทียบสิ่งนี้กับโลกของโปรโตคอลเครือข่ายการถ่ายโอนจดหมาย ฯลฯ (ซึ่งเป็นสัญญาส่วนต่อประสานเหมือนกับวิธีในการเขียนโปรแกรม) โดยปกติแล้วนโยบายคือผู้ส่งควรปฏิบัติตามอย่างเคร่งครัดที่สุดเท่าที่จะเป็นไปได้สำหรับโปรโตคอลในขณะที่ผู้รับควรออกนอกเส้นทางของพวกเขาเพื่อให้สามารถทำงานกับสิ่งที่กำลังจะเกิดขึ้น

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

ฉันอาจทำผิดด้านการอนุญาตnullค่ากำหนดและจัดทำเอกสารความหมายของพวกเขา (เช่นnull = empty arrayในกรณีนี้) และไม่ส่งข้อยกเว้นเว้นแต่ 99% ของรหัสที่คล้ายกันอื่น ๆ ของคุณ (สไตล์ "ไลบรารี่") ทำแบบอื่น 'รอบ.