Làm cách nào để sử dụng lại mã trong các thủ tục lưu sẵn SQL?

Question

Chúng tôi sử dụng SQL Server 2005. Tất cả truy cập dữ liệu của chúng tôi được thực hiện thông qua các thủ tục được lưu trữ. Các thủ tục được lưu trữ lựa chọn của chúng tôi luôn trả về nhiều bộ kết quả.

Ví dụ:

CREATE PROCEDURE hd_invoice_select(@id INT) AS 
    SELECT * FROM Invoice WHERE InvoiceID = @id 
    SELECT * FROM InvoiceItem WHERE InvoiceID = @id 
    SELECT * FROM InvoiceComments WHERE InvoiceID = @id 
    RETURN 

lớp truy cập dữ liệu ứng dụng của chúng tôi của xây dựng một đồ thị đối tượng dựa trên kết quả (O/R Mapper phong cách).

Vấn đề tôi có là chúng tôi có nhiều lựa chọn lưu trữ hóa đơn khác nhau được lưu trữ. Tất cả đều trả về cùng một cấu trúc, chỉ cho các tiêu chí lựa chọn khác nhau. Ví dụ, tôi cũng có:

CREATE PROCEDURE hd_invoice_selectAllForCustomer(@customerID INT) AS 
    SELECT * FROM Invoice WHERE CustomerID = @customerID 
    SELECT * FROM InvoiceItem WHERE InvoiceID IN 
     (SELECT InvoiceID FROM Invoice WHERE CustomerID = @customerID) 
    SELECT * FROM InvoiceComments WHERE InvoiceID = @id 
     (SELECT InvoiceID FROM Invoice WHERE CustomerID = @customerID) 
    RETURN 

và tôi có nhiều người khác bao gồm:

hd_invoice_selectActive() 
hd_invoice_selectOverdue() 
hd_invoice_selectForMonth(@year INT, @month INT) 

và tôi có cùng một khuôn mẫu cho rất nhiều khái niệm (Khách hàng, nhân viên, vv)

Chúng tôi kết thúc sao chép rất nhiều mã và bảo trì thực sự là khó khăn. Khi "cấu trúc" của một khái niệm thay đổi, chúng ta phải đi và sửa chữa tất cả các procs và nó rất dễ bị lỗi.

Vì vậy, câu hỏi của tôi là: Cách tốt nhất để sử dụng lại mã trong kịch bản là gì?

Chúng tôi đã đưa ra giải pháp sử dụng bảng tạm thời. Nhưng nó không phải là rất thanh lịch. Tôi sẽ cho phép bạn chia sẻ ý tưởng của mình và nếu cần, tôi sẽ đăng chi tiết giải pháp của tôi trong bài đăng sắp tới để nhận xét của bạn về cách tiếp cận đó.

Cảm ơn

Answer 1

Đăng câu trả lời thứ hai này vì đây là một cách tiếp cận khác. Nếu bạn đang sử dụng SQL Server 2008:

CREATE TYPE InvoiceListTableType AS TABLE 
(
    InvoiceId INT 
); 
GO 

CREATE PROCEDURE hd_invoice_selectFromTempTable 
(
    @InvoiceList InvoiceListTableType READONLY 
) 
AS 
BEGIN 
    SELECT * FROM Invoice WHERE InvoiceID IN 
     (SELECT InvoiceId FROM @InvoiceList) 

    SELECT * FROM InvoiceItem WHERE InvoiceID IN 
     (SELECT InvoiceId FROM @InvoiceList) 

    SELECT * FROM InvoiceComments WHERE InvoiceID IN 
     (SELECT InvoiceId FROM @InvoiceList) 

    RETURN 
END 
GO 

CREATE PROCEDURE hd_invoice_select(@id INT) AS 
BEGIN 
    DECLARE @InvoiceList AS InvoiceListTableType; 

    SELECT id AS ID 
     INTO @InvoiceList 

    EXEC hd_invoice_selectFromTempTable(@InvoiceList) 
    RETURN 
END 
GO 

CREATE PROCEDURE hd_invoice_selectAllForCustomer(@customerID INT) AS 
BEGIN 
    DECLARE @InvoiceList AS InvoiceListTableType; 

    SELECT invoiceID as ID 
     INTO @InvoiceList 
     FROM Invoice WHERE CustomerID = @customerID 

    EXEC hd_invoice_selectFromTempTable(@InvoiceList) 
    RETURN 
END 
GO 

CREATE PROCEDURE hd_invoice_selectAllActive AS 
BEGIN 
    DECLARE @InvoiceList AS InvoiceListTableType; 

    SELECT invoiceID as ID 
     INTO @InvoiceList 
     FROM Invoice WHERE Status = 10002 

    EXEC hd_invoice_selectFromTempTable(@InvoiceList) 
    RETURN 
END 
GO 

Answer 2

Đây là một trong những vấn đề chính với thủ tục được lưu trữ và lý do mọi người không thích chúng.

Tôi chưa bao giờ tìm thấy hoặc thấy một cách nào đó xung quanh nó.

Answer 3

Tôi đã bắt đầu sử dụng các thủ tục được lưu trữ được tạo bởi Trình tạo mã cho CRUD cơ bản của tôi. Tôi sử dụng procs được lưu trữ cho các báo cáo hoặc công việc SQL phức tạp.

Tôi cũng có đề xuất không liên quan đến câu hỏi của bạn - thay vì sử dụng mệnh đề IN, hãy sử dụng mệnh đề EXISTS trong câu lệnh SQL của bạn.

Answer 4

Cách "tốt nhất" cho trường hợp cụ thể này sẽ là sử dụng một số loại tạo mã. Hãy đến với một số loại quy ước và cắm nó vào một bộ tạo mã.

Answer 5

Tôi đôi khi làm điều đó theo hai bước:

Tôi tìm ra danh sách InvoiceID. Sau đó, tôi gọi thủ tục lưu trữ của tôi với danh sách này dưới dạng tham số.

Vào 2005, chúng tôi không có bảng giá trị tham số, vì vậy tôi đóng gói danh sách các ID trong một BLOB nhị phân và trình SQL Server, như mô tả ở đây: Arrays and Lists in SQL Server 2005

Bạn cũng có thể gửi một danh sách các ID dưới dạng danh sách được phân tách bằng dấu phẩy (hơi chậm) hoặc dưới dạng kết nối của các biểu diễn chuỗi có chiều rộng cố định (nhanh hơn nhiều).

Answer 6

Tôi đã kế thừa một ứng dụng sử dụng cách tiếp cận bảng tạm thời trước và tôi đồng ý rằng nó rất lộn xộn.Trong dự án đó, chúng tôi có thể loại bỏ rất nhiều bảng tạm thời bằng cách thay thế chúng bằng Chế độ xem có chứa 'đối tượng' mong muốn, sau đó chúng tôi đã cập nhật các thủ tục được lưu trữ của chúng tôi để truy vấn các chế độ xem đó.

Có lẽ điều đó cũng có thể hoạt động trong tình huống của bạn.

Answer 7

Bạn đã thử đặt nhiều loại thông số truy vấn trong danh sách tham số cho proc chính của mình chưa? Tôi chỉ viết các proc để trang trải bảng Invoice, bạn sẽ cần phải mở rộng nó cho các bảng bổ sung của bạn.

CREATE PROCEDURE hd_invoice_select 
(
    @id INT = NULL 
    , @customerId INT = NULL 
) AS 
BEGIN 
    SELECT * 
     FROM Invoice 
     WHERE 
      (
       @id IS NULL 
       OR InvoiceID = @id 
      ) 
      AND (
       @customerId IS NULL 
       OR CustomerID = @customerId 
      ) 
    RETURN 
END 

proc này có thể được gọi là mở rộng bằng cách gửi @ id và @customerId như NULLs, đối với một InvoiceID cụ thể dựa trên @ id với @customerId như NULL (hoặc chỉ để lại nó đi tất cả cùng nhau), hoặc cho một khách hàng cụ thể dựa trên @customerId để lại @id là NULL hoặc loại trừ nó khỏi truy vấn.

Bạn cũng nên xem các chế độ xem và các hàm do người dùng định nghĩa do người dùng định nghĩa. Bạn có thể đặt chúng trong procs của bạn để bọc một số logic ra khỏi procs để họ có thể được chia sẻ và duy trì ở một nơi duy nhất. Việc có một số logic trong chế độ xem/chức năng cũng cho phép bạn xử lý dữ liệu trong cửa sổ truy vấn như thể nó là một bảng.

Answer 8

Trong một số trường hợp, tôi sử dụng VIEWS để sử dụng lại "mã". Trong các trường hợp như bộ lọc, các mục đang hoạt động, những thứ lỗi thời, v.v ...

Answer 9

Có thể bạn nên học cách sử dụng các phép nối. Bạn có thể đặt tham gia cơ bản của ba bảng trong một khung nhìn và chỉ cần truy vấn rằng với sp trao các tham số khác nhau. Ngoài ra, bạn không nên sử dụng chung chọn * bao giờ trong mã sản xuất. Chỉ trả lại một số cột bạn thực sự cần trong hoàn cảnh và toàn bộ hệ thống của bạn sẽ hoạt động tốt hơn. Ngoài ra, bạn sẽ không có kết quả ngoài ý muốn khi mọi người thay đổi cấu trúc của bạn.

Answer 10

Tôi là người đã đặt câu hỏi này ngay từ đầu. Tôi đang trả lời câu hỏi của riêng tôi ở đây để cho bạn biết giải pháp sử dụng lại mã mà tôi sử dụng và nhận được nhận xét của bạn về cách tiếp cận đó. Nếu câu trả lời này nhận được rất nhiều phiếu bầu, tôi sẽ chọn nó làm câu trả lời cuối cùng.

Cách tiếp cận này hoạt động và rất dễ sử dụng. Tôi không biết liệu nó có tác động hiệu quả hay không vì nó phụ thuộc rất nhiều vào các bảng tạm thời.

Đối với mỗi khái niệm trong ứng dụng của tôi, tôi có một storec proc như thế này:

CREATE PROCEDURE hd_invoice_selectFromTempTable AS 

    /* Get the IDs from an existing #TempInvoiceIDs temporary table */ 

    SELECT * FROM Invoice WHERE InvoiceID IN 
     (SELECT ID FROM #TempInvoiceIDs) 

    SELECT * FROM InvoiceItem WHERE InvoiceID IN 
     (SELECT ID FROM #TempInvoiceIDs) 

    SELECT * FROM InvoiceComments WHERE InvoiceID IN 
     (SELECT ID FROM #TempInvoiceIDs) 

    RETURN 

Sau đó, tôi tạo bao nhiêu lựa chọn lưu trữ proc như tôi cần:

CREATE PROCEDURE hd_invoice_select(@id INT) AS 

    /* Fill #TempInvoiceIDs with matching IDs */ 
    SELECT id AS ID INTO #TempInvoiceIDs 

    EXEC hd_invoice_selectFromTempTable 
    RETURN 

CREATE PROCEDURE hd_invoice_selectAllForCustomer(@customerID INT) AS 

    /* Fill #TempInvoiceIDs with matching IDs */ 
    SELECT invoiceID as ID 
    INTO #TempInvoiceIDs 
    FROM Invoice WHERE CustomerID = @customerID 

    EXEC hd_invoice_selectFromTempTable 
    RETURN 

CREATE PROCEDURE hd_invoice_selectAllActive AS 

    /* Fill #TempInvoiceIDs with matching IDs */ 
    SELECT invoiceID as ID 
    INTO #TempInvoiceIDs 
    FROM Invoice WHERE Status = 10002 

    EXEC hd_invoice_selectFromTempTable 
    RETURN 

Bạn nghĩ gì về cách tiếp cận này? Nó là hơi tương tự như câu trả lời của AlexKuznetsov nhưng tôi sử dụng bảng tạm thời thay vì một tham số BLOB.