एसक्यूएल संग्रहीत प्रक्रियाओं में कोड का पुन: उपयोग कैसे करें?

Question

हम SQL सर्वर 2005 का उपयोग करते हैं। हमारे सभी डेटा एक्सेस संग्रहीत प्रक्रियाओं के माध्यम से किया जाता है। हमारी चयन संग्रहित प्रक्रियाएं हमेशा कई परिणाम सेट लौटाती हैं।

उदाहरण के लिए:

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 

हमारे आवेदन के डेटा का उपयोग स्तर एक ऑब्जेक्ट परिणाम (ओ/आर मैपर शैली) पर आधारित ग्राफ बनाता है।

मेरी समस्या यह है कि हमारे पास कई अलग-अलग चालान चयन संग्रहित प्रोसेस हैं। वे सभी अलग-अलग चयन मानदंडों के लिए, एक ही संरचना को वापस करते हैं। उदाहरण के लिए, मैं भी है:

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 

और मैं सहित कई अन्य लोगों के होते हैं:

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

और मैं अवधारणाओं का एक बहुत के लिए एक ही पैटर्न (ग्राहकों, कर्मचारियों, आदि)

है हम बहुत सारे कोड की प्रतिलिपि बनाते हैं और रखरखाव वास्तव में कठिन है। जब एक अवधारणा की "संरचना" बदल जाती है, तो हमें सभी प्रोसेस को ठीक करना और ठीक करना पड़ता है और यह बहुत ही त्रुटि प्रवण होता है।

तो मेरा सवाल है: परिदृश्य में कोड का पुन: उपयोग करने का सबसे अच्छा तरीका क्या है?

हम एक समाधान के साथ आए जो temp तालिकाओं का उपयोग करता है। लेकिन यह बहुत ही सुरुचिपूर्ण नहीं है। मैं आपको अपने विचार साझा करने दूंगा और यदि आवश्यक हो तो मैं उस दृष्टिकोण पर आपकी टिप्पणियां प्राप्त करने के लिए आगामी समाधान में अपने समाधान का विवरण पोस्ट करूंगा।

धन्यवाद

Answer 1

इसे दूसरे उत्तर के रूप में पोस्ट करना क्योंकि यह एक अलग दृष्टिकोण है। यदि आप 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

इस संग्रहित प्रक्रियाओं और क्यों लोग उन्हें पसंद नहीं है के साथ मुख्य समस्याओं में से एक है।

मुझे कभी भी इसके आसपास कोई रास्ता नहीं मिला या देखा नहीं है।

Answer 3

मैंने अपने मूल सीआरयूडी के लिए कोड जेनरेटर द्वारा जेनरेट की गई संग्रहित प्रक्रियाओं का उपयोग करना शुरू कर दिया है। मैं रिपोर्ट या जटिल एसक्यूएल काम के लिए संग्रहीत प्रोसेस का उपयोग करता हूं।

मेरे पास आपके प्रश्न से संबंधित कोई सुझाव नहीं है - IN खंड का उपयोग करने के बजाय, अपने SQL कथन में EXISTS क्लॉज का उपयोग करें।

Answer 4

इस विशिष्ट परिदृश्य के लिए "सर्वश्रेष्ठ" तरीका कुछ प्रकार की कोड जनरेशन का उपयोग करना होगा। कुछ प्रकार के सम्मेलन के साथ आओ और इसे कोड जनरेटर में प्लग करें।

Answer 5

मैं कभी कभी दो चरणों में कर:

मैं InvoiceID की एक सूची के साथ आने के। फिर मैं अपनी संग्रहित प्रक्रिया को इस सूची के साथ पैरामीटर के रूप में कॉल करता हूं।

2005 को हम तालिका महत्वपूर्ण पैरामीटर नहीं है, तो मैं एक बाइनरी ब्लॉब में आईडी की मेरी सूची में पैक और यहां बताए अनुसार, एसक्यूएल सर्वर में जमा करें: Arrays and Lists in SQL Server 2005

तुम भी आईडी की सूची प्रस्तुत कर सकते हैं एक अल्पविराम से अलग सूची (कुछ हद तक धीमी) या निश्चित चौड़ाई स्ट्रिंग प्रस्तुतियों (बहुत तेज़) के एक संयोजन के रूप में।

Answer 6

मुझे एक ऐसा एप्लिकेशन मिला है जिसने पहले temp तालिका दृष्टिकोण का उपयोग किया था और मैं मानता हूं कि यह बहुत गन्दा है।

उस प्रोजेक्ट पर हम उन अस्थायी 'ऑब्जेक्ट्स' वाले दृश्यों के साथ उन्हें बदलकर कई अस्थायी तालिकाओं को हटाने में सक्षम थे, फिर हमने उन दृश्यों से पूछने के लिए हमारी संग्रहीत प्रक्रियाओं को अपडेट किया।

शायद यह आपकी स्थिति में भी काम कर सकता है।

Answer 7

क्या आपने अपनी मुख्य प्रो के पैरामीटर की सूची में 1 से अधिक क्वेरी पैरामीटर प्रकार डालने का प्रयास किया है? मैंने केवल चालान तालिका को कवर करने के लिए proc लिखा था, आपको इसे अपनी अतिरिक्त तालिकाओं के लिए विस्तारित करने की आवश्यकता होगी।

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 एक विशिष्ट शून्य के रूप में @customerId साथ @id के आधार पर InvoiceID के लिए, NULLs रूप @id और @customerId भेजकर खुली कहा जा सकता है (या बस इसे बंद कर छोड़ सब एक साथ), या एक के लिए विशिष्ट ग्राहक @customerId पर आधारित @id को NULL के रूप में छोड़कर या क्वेरी से बाहर कर दें।

आपको विचारों और तालिका-मूल्यवान उपयोगकर्ता-परिभाषित कार्यों को भी देखना चाहिए। आप इन प्रोसेस में प्रोसेस से कुछ तर्क को लपेटने के लिए डाल सकते हैं ताकि उन्हें एक ही स्थान पर साझा और रखरखाव किया जा सके। विचारों/कार्यों में कुछ तर्क होने से आपको एक क्वेरी विंडो में डेटा से निपटने की अनुमति मिलती है जैसे कि यह एक टेबल थी।

Answer 8

कुछ मामलों में मैं "कोड" का पुन: उपयोग करने के लिए दृश्यों का उपयोग करता हूं। फिल्टर, सक्रिय आइटम, पुरानी चीजें, और इसी तरह के मामलों में ...

Answer 9

शायद आपको जुड़ने का उपयोग करना सीखना चाहिए। आप तीन तालिकाओं के मूलभूत दृश्य को एक दृश्य में डाल सकते हैं और बस पूछ सकते हैं कि एसपी के साथ विभिन्न मानकों को सौंपना। साथ ही, आपको सामान्य रूप से कभी भी उत्पादन कोड में चयन * का उपयोग नहीं करना चाहिए। केवल परिस्थितियों में आपको आवश्यक कुछ कॉलम लौटाएं और आपकी पूरी प्रणाली बेहतर प्रदर्शन करेगी। इसके अलावा जब आपके पास लोग संरचना बदलते हैं तो आपके पास अनपेक्षित परिणाम नहीं होंगे।

Answer 10

मैं वह व्यक्ति हूं जिसने इस प्रश्न से पहले स्थान पर पूछा था। मैं यहां आपके स्वयं के प्रश्न का उत्तर दे रहा हूं ताकि आप मुझे उपयोग किए जाने वाले कोड पुन: उपयोग समाधान और उस दृष्टिकोण पर अपनी टिप्पणियां प्राप्त कर सकें। अगर इस उत्तर को बहुत सारे वोट मिलते हैं, तो मैं इसे अंतिम उत्तर के रूप में चुनूंगा।

यह दृष्टिकोण काम करता है और उपयोग करने में आसान है। मुझे नहीं पता कि इसका प्रदर्शन प्रभाव पड़ता है क्योंकि यह अस्थायी तालिकाओं पर भारी निर्भर करता है।

अपने आवेदन में एक अवधारणा के लिए, मैं इस तरह एक storec proc है:

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 

तो मैं के रूप में कई चयन संग्रहीत proc बनाने के रूप में मैं की जरूरत है:

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 

आप को क्या समझते हो यह दृष्टिकोण? यह कुछ हद तक AlexKuznetsov के उत्तर के समान है लेकिन मैं एक BLOB पैरामीटर के बजाय temp टेबल का उपयोग करता हूं।