डीएओ विधियों और सिंक्रनाइज़

Question

निम्नलिखित विधियों में वर्तमान में एक सार डीएओ कक्षा में हैं। यदि समवर्ती कॉल हैं, तो क्या वे सुरक्षित हैं क्योंकि सिंक्रनाइज़ेशन का उपयोग किया जाना चाहिए या नहीं? मुझे पता है कि अगर किसी विधि के दायरे से बाहर एक विशेषता का संदर्भ है तो सिंक्रनाइज़ेशन का उपयोग किया जाना चाहिए, लेकिन यह स्पष्ट नहीं है कि चीजों को बाहरी संसाधन के साथ कैसे संभाला जाना चाहिए।

public Connection getConnection() { 
    // Call to singleton handling JDBC stuff 
    return Database.getInstance().getCon(); 
} 

public boolean isConnectionAvailable(){  
    if(getConnection() != null){ 
     return true; 
    } 

    return false; 
} 

public PreparedStatement getPreparedStatement(String sqlStatement){ 
    Connection connection = getConnection(); 
    PreparedStatement pS = null; 

    if(connection != null){ 
     try { 
      pS = connection.prepareStatement(sqlStatement); 
     } catch (SQLException e) { 
      return null; 
     } 
    } 

    return pS; 
} 

संपादित करें: मैं इस प्रश्न reformulate सकता है के रूप में यह यहाँ क्या महत्वपूर्ण है DAOs लेखन के बारे में जानकारी शामिल करने के लिए।

Answer 1

मैं इस कार्यान्वयन से सहमत नहीं हूं।

सबसे पहले, डीएओ को उन सेवाओं द्वारा उनकी कनेक्शन जानकारी दी जानी चाहिए जो काम और लेनदेन की इकाइयां हैं।

दूसरा, मुझे एक इंटरफ़ेस दिखाई नहीं देता है।

तीसरा, मुझे मॉडल या डोमेन ऑब्जेक्ट्स नहीं दिखते हैं।

चौथा, तैयार बयान केवल आंतरिक कार्यान्वयन का हिस्सा होना चाहिए। अगर वे आपके डीएओ से बाहर निकल रहे हैं, तो आप इसे गलत कर रहे हैं।

पांचवें, वस्तु से बाहर एक तैयार बयान पारित करने से इसे बंद करने और बहुत कम स्पष्ट सफाई की जिम्मेदारी मिलती है। आपका डीएओ किसी भी समय संसाधन रिसाव की मौत मर जाएगा।

यहां एक सामान्य डीएओ के लिए एक इंटरफेस है। आप देखेंगे कि यह सभी सीआरयूडी संचालन है, जिसमें कनेक्शन से कोई संदर्भ नहीं है या जावा से कोई इंटरफेस नहीं है।एसक्यूएल पैकेज:

package persistence; 

import java.io.Serializable; 
import java.util.List; 

public interface GenericDao<T, K extends Serializable> 
{ 
    T find(K id); 
    List<T> find(); 
    List<T> find(T example); 
    List<T> find(String queryName, String [] paramNames, Object [] bindValues); 

    K save(T instance); 
    void update(T instance); 
    void delete(T instance); 
} 

आप इसके साथ एक लंबा सफर तय कर सकते हैं। यह एक बेहतर अमूर्त है। T आपका व्यावसायिक ऑब्जेक्ट प्रकार है, और K प्राथमिक कुंजी है।

Answer 2

getCon() एक नया Connection हर बार यह कहा जाता है देता है, या एक ThreadLocal कनेक्शन देता है, तो आप सुरक्षित हैं और synchronized

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

और सामान्य डिजाइन सिद्धांतों पर कुछ नोट्स। डीएओ एक अलग परत बनाते हैं। प्रत्येक परत एक कारण के लिए मौजूद है, अच्छे नाम रखने के लिए झटके नहीं। डीएओ परत अमूर्त, या दूसरे शब्दों में मौजूद है - डीएओ ऑब्जेक्ट्स का उपयोग करने वाली सेवाओं से डेटाबेस एक्सेस को छुपाएं। इसे और अधिक स्पष्ट रूप से कल्पना करने के लिए - डीएओ को इस तरह से लिखा जाना चाहिए कि अगर कल आप आरडीबीएमडी स्टोरेज (जेडीबीसी के माध्यम से) एक्सएमएल स्टोरेज में स्विच करने का फैसला करते हैं, तो आपको को बदलकर डीएओ ऑब्जेक्ट्स बदलकर ऐसा करने में सक्षम होना चाहिए और कुछ नहीं।

Answer 3

जेडीबीसी कनेक्शन वर्ग को थ्रेड सुरक्षित होने की गारंटी नहीं है। यदि आपका डेटाबेस .getInstance()। GetCon() विधि हमेशा एक ही कनेक्शन को वापस कर रही है, तो आप समस्याओं में भाग लेंगे। यदि, हालांकि, यह एक पूल का उपयोग कर रहा है जैसे कि प्रत्येक कॉल प्राप्त करने के लिए कॉल()। GetCon() एक अलग कनेक्शन देता है जो आप ठीक होंगे।

यह कहा गया है कि, यदि आप प्रत्येक कॉल के साथ एक अलग कनेक्शन लौट रहे हैं तो कॉन() प्राप्त करें, तो getPreparedStatement() काम नहीं करेगा यदि आप दो तैयार स्टेटमेंट कॉल को उसी कनेक्शन (और उसी लेनदेन) का उपयोग करने के लिए चाहते हैं।

मुझे अपने डीएओ कक्षाओं के आधार के रूप में वसंत की जेडीबीसीटीप्लेट कक्षा पसंद है।

Answer 4

मुझे ठीक लग रहा है। फ़ंक्शन सुरक्षित हैं।

Answer 5

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

आपका शेष कोड सुरक्षित लगता है।

Answer 6

देखें कि वसंत कैसे करता है, वे पहले से ही इन सभी चीजों को समझ चुके हैं और इसे फिर से आविष्कार करने की आवश्यकता नहीं है। स्प्रिंग के पूर्ण वितरण के साथ बंडल किए गए पेटक्लिनिक नमूना कोड को देखें, या (गैर-वसंत दृष्टिकोण के लिए) बाउर/किंग हाइबरनेट पुस्तक के डीएओ अध्याय को पढ़ें।

निश्चित रूप से डीएओ डेटाबेस कनेक्शन प्राप्त करने का प्रभारी नहीं होना चाहिए, क्योंकि आप एक ही लेनदेन में एकाधिक डीएओ कॉल समूहित करना चाहते हैं। जिस तरह से स्प्रिंग करता है, वहां एक सेवा परत होती है जो एक लेन-देन के तरीकों को एक इंटरसेप्टर में लपेटती है जो डेटासोर्स से कनेक्शन खींचती है और इसे थ्रेडलोकल वैरिएबल में रखती है जहां डीएओ इसे पा सकते हैं।

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