सोनार

Question

को लॉगिंग और पुनर्विचार के बारे में शिकायत करने के बारे में शिकायत कर रहा है मेरे पास मेरे प्रोग्राम में कोड का निम्न भाग है और मैं इसे मैवेन के साथ एकीकृत करने के बाद कोड गुणवत्ता जांच के लिए सोनारक्यूब 5 चला रहा हूं।

हालांकि, सोनार शिकायत कर रहा है कि मुझे या तो इस अपवाद को लॉग या पुनर्स्थापित करना होना चाहिए।

मुझे यहां क्या याद आ रही है? क्या मैं पहले ही अपवाद लॉग नहीं कर रहा हूं?

private boolean authenticate(User user) { 
     boolean validUser = false; 
     int validUserCount = 0; 
     try { 
      DataSource dataSource = (DataSource) getServletContext().getAttribute("dataSource"); 
      validUserCount = new MasterDao(dataSource).getValidUserCount(user); 
     } catch (SQLException sqle) { 
      LOG.error("Exception while validating user credentials for user with username: " + user.getUsername() + " and pwd:" + user.getPwd()); 
      LOG.error(sqle.getMessage()); 
     } 
     if (validUserCount == 1) { 
      validUser = true; 
     } 
     return validUser; 
    } 

Answer 1

आप इसे इस तरह से करना चाहिए:

try { 
    DataSource dataSource = (DataSource) getServletContext().getAttribute("dataSource"); 
    validUserCount = new MasterDao(dataSource).getValidUserCount(user); 
} catch (SQLException sqle) { 
    LOG.error("Exception while validating user credentials for user with username: " + 
      user.getUsername() + " and pwd:" + user.getPwd(), sqle); 
} 

सोनार आप अब और परेशान नहीं करना चाहिए

Answer 2

आपको लगता है कि SQLException सुरक्षित रूप से अनदेखा किया जा सकता है, तो आप यह सूची में जोड़ सकते स्क्विड के लिए अपवादों का: S1166 नियम।

1. नियम-> खोज स्क्विड: S1166 पर जाएं।
2. गुणवत्ता प्रोफ़ाइल में अपवाद संपादित करें।
3. सूची में SQLException जोड़ें।

Answer 3

मैं एक ही मुद्दे पर ठोकर खाई। मैं 100% निश्चित नहीं हूं कि अगर मैं इस बिंदु पर पूरी तरह से सही हूं, लेकिन मूल रूप से आपको पूर्ण अपवाद को पुनर्स्थापित या लॉग करना चाहिए। जबकि e.getMessage() सिर्फ आपको विस्तृत संदेश देता है लेकिन निष्पादन स्टैक का स्नैपशॉट नहीं।

Oracle docs (Throwable) से:

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

इसका मतलब है अबाउट कार्यों द्वारा प्रदान किया गया समाधान, क्योंकि संपूर्ण अपवाद ऑब्जेक्ट (स्क्ले) लॉगर को पास किया जा रहा है।

उम्मीद है कि यह मदद करता है। चीयर्स।

Answer 4

क्या सोनार आपको करने के लिए कह रहा है, पूरे अपवाद वस्तु को जारी रखना है। आप की तरह कुछ का उपयोग कर सकते हैं:

try { 
     ...   
    } catch (Exception e) { 
     logger.error("Error", e); 
    }