วัตถุธุรกิจภายใน Data Access Layer

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

วิธีการภายใน Data Access Layer ของฉัน (สั้นสำหรับ DAL) จะค่อนข้างง่าย พวกเขาเป็นไปตามขั้นตอนการจัดเก็บในฐานข้อมูล (ไม่มีวิธีอื่นในการโทรเข้าเพื่อรักษาความสะอาด) และพวกเขามีพารามิเตอร์เดียวกันกับที่ทำ พวกเขาเพียงแค่เชื่อมต่อกับฐานข้อมูลและส่งคืนผลลัพธ์แบบสอบถาม นี่คือตัวอย่างหนึ่ง:

public int DeleteRecord(int recordId)
{
    recordId.RequireThat("recordId").NotZeroOrLess();

    List<SqlParameter> parameters = new List<SqlParameter>();
    parameters.Add(new SqlParameter { ParameterName = "@RecordId", SqlDbType = SqlDbType.Int, Direction = ParameterDirection.Input, Value = recordId});

    return this.ExecuteNonQuery("DeleteRecord", parameters.ToArray());
}

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

อย่างไรก็ตามพูดในวิธีอื่น DAL ฉันต้องการโหลดบันทึก ขั้นตอนการโหลดของฉันกำลังจะถูกดำเนินการselectsกับพวงของตารางและกลับมาได้DataSetแต่ฉันกำลังต่อสู้กับว่า DAL ฉันควรสร้าง Business Objects ภายในวิธีใช้DataSet, หรือถ้าธุรกิจของฉันวัตถุตัวเองก็ควรจะมีLoad()วิธีการที่ได้รับDataSetจาก DAL และจากนั้นจะเติมตัวเองโดยทั่วไป

การทำผ่าน DAL จะส่งผลให้เกิดตรรกะน้อยลงใน Business Objects (แม้ว่านี่จะเป็นเพียงการเลือกตรรกะมันยังคงเป็นตรรกะ) แต่จะทำให้ DAL ดูเล็กน้อยและทำให้รู้สึกว่ามันกำลังทำสิ่งที่ควรทำ ' ไม่ต้องทำ

พวกคุณคิดอย่างไร

DAL ของคุณควรส่งคืนวัตถุข้อมูลของคุณ

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

นอกจากนี้คุณมักไม่ต้องการให้วัตถุทางธุรกิจของคุณสามารถสร้างตัวเองได้ สิ่งนี้อาจทำให้เกิดช่องโหว่ด้านความปลอดภัยซึ่งใครบางคนสามารถใช้ไลบรารีของคุณและสร้างอินสแตนซ์ใหม่ของวัตถุของคุณโดยการเรียก.Load(someId)ใช้และรวมสองชั้นเข้าด้วยกันซึ่งควรแยกจากกันอย่างสมบูรณ์

ฉันยังไม่แนะนำให้ใช้.Load(DataSet ds)วิธีการเพราะหากคำจำกัดความชุดข้อมูลมีการเปลี่ยนแปลงคุณจะต้องค้นหาวัตถุข้อมูลที่ใช้ชุดข้อมูลนั้นและเปลี่ยนพวกเขา ง่ายกว่าที่จะเก็บรหัสการเข้าถึงข้อมูลทั้งหมดไว้ในที่เดียวดังนั้นหากคุณเปลี่ยนแบบสอบถามการเข้าถึงข้อมูลคุณควรเปลี่ยน DAL layer ของคุณเท่านั้น

วิธีการของฉันแม้กระทั่งก่อน LINQ-To-SQL และ Entity Framework ก็คือการมีส่วนต่อประสานและไลบรารีระดับนามธรรมที่ให้ "สัญญาที่เป็นลายลักษณ์อักษร" สำหรับการสื่อสารระหว่างเลเยอร์ต่างๆของแอป บางครั้งสิ่งนี้เรียกว่าontologyซึ่งเป็นคำนิยามสำหรับโดเมนงาน สิ่งใดก็ตามที่ผ่านไปมาระหว่างเลเยอร์ใช้ 'สัญญา' นี้

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

โค้ดตัวอย่างที่คุณมีคล้ายกับรหัสที่ฉันเคยมีใน LINQ ฉันมีคลาสฟังก์ชัน DB ทั่วไปที่ฉันใช้ภายในวัตถุ DAL ของฉัน คลาส DAL จะอ่านข้อมูลและใส่ลงในวัตถุ 'สัญญา' ผลลัพธ์แบบสเกลาร์เช่นตัวอย่างการลบของคุณจะคืนค่ามักเป็นบูลีน

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

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