क्लोजिंग BufferedReader और इनपुटस्ट्रीम रीडर

Question

कोड का यह टुकड़ा BufferedReader और InputStreamReader के कारण स्मृति रिसाव के मुद्दों को बना रहा है जो मुझे लगता है कि कुछ अपवादों का कारण हो रहा है। मुझे इसे कैसे बदलना चाहिए?

try{ 
    URL url = new URL(sMyUrl); 
    BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream())); 
    while ((str = in.readLine()) != null) { 
     jsonString += str; 
    } 
    in.close(); 
}catch(Exception e){ 

} 

Answer 1

यह एक try..finally ब्लॉक का उपयोग अपनी स्ट्रीम बंद करने के लिए सुरक्षित होगा। आप StringBuilder का भी उपयोग कर सकते हैं क्योंकि इसे तारों को संयोजित करने के लिए डिज़ाइन किया गया है। आपको Exception को पकड़ने और इसके साथ कुछ भी करने से बचने चाहिए। साथ ही, आपका कोड किसी लाइन-ब्रेक के बिना लाइनों को जोड़ रहा है। यह अच्छी तरह से नहीं आप क्या चाहते हैं, जिस स्थिति append("\n") में आप में प्रत्येक पंक्ति को पढ़ने के जब हो सकता है

यहाँ उन संशोधनों के साथ एक संस्करण है:।

StringBuilder json = new StringBuilder(); 
try { 
    URL url = new URL(sMyUrl); 
    BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream())); 
    try { 
     String str; 
     while ((str = in.readLine()) != null) { 
      json.append(str).append("\n"); 
     } 
    } finally { 
     in.close(); 
    } 
} catch (Exception e) { 
    throw new RuntimeException("Failed to read JSON from stream", e); 
} 

Answer 2

कोड सुंदर नहीं है लेकिन मेमोरी रिसाव नहीं बनायेगा। मेरा सुझाव है कि आप यह निर्धारित करने के लिए मेमोरी प्रोफाइलर का उपयोग करें कि आपकी मेमोरी का उपयोग कहां किया जा रहा है। नहीं तो आप सिर्फ अनुमान लगा रहे हैं, भले ही आप जावा में दस वर्ष का अनुभव + प्रदर्शन ट्यूनिंग है;)

एक बेहतर विकल्प जावा 7

URL url = new URL(sMyUrl); 
try(BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream()))) { 
    while ((str = in.readLine()) != null) { 
    jsonString.append(str).append("\n"); 
    } 
} 

आप जावा 6 या उससे अधिक आप उपयोग कर सकते हैं, तो उपयोग करने के लिए है।

BufferedReader in = new BufferedReader(new InputStreamReader(url.openStream()))) { 
try { 
    while ((str = in.readLine()) != null) { 
    jsonString.append(str).append("\n"); 
    } 
} finally { 
    in.close(); 
}