क्या आखिर में कोशिश करने का प्रयास करना ठीक है?

Question

मैं एक buffered लेखक और मेरा कोड का उपयोग कर रहा हूँ, अंत में ब्लॉक में लेखक बंद कर देता है। मेरा कोड इस तरह है।

........... 
    BufferedWriter theBufferedWriter = null; 
    try{ 
    theBufferedWriter =..... 
    .... 
    ...... 
    ..... 
    } catch (IOException anException) { 
    .... 
    } finally { 
     try { 
      theBufferedWriter.close();    
     } catch (IOException anException) { 
      anException.printStackTrace(); 
      } 
    } 

मैं अंत में में साफ ऊपर कोड के अंदर कोशिश पकड़ उपयोग करने के लिए के रूप में theBufferedWriter भी एक IOException फेंक सकती है। मैं इस अपवाद को कॉलिंग मेथोस पर फेंकना नहीं चाहता हूं। आखिर में कोशिश करने का प्रयास करने के लिए यह एक अच्छा अभ्यास है? यदि विकल्प नहीं है तो क्या विकल्प है? कृपया सुझाव दे।

सादर, हीरल

Answer 1

इस अपाचे commons-io से IOUtils.closeQuiety उपयोग करने के लिए है करने के लिए एक कुछ हद तक अच्छे तरह से कर सकते हैं। यह आपके कोड को साफ रखता है और जावा में निहित बॉयलरप्लेट में से कुछ को हटा देता है।

आप कोड तो हो जाता है:

BufferedWriter theBufferedWriter = null; 
try{ 
    theBufferedWriter = ... 
    ... 
} catch (IOException anException) { 
    ... 
} finally { 
    IOUtils.closeQuietly(theBufferedWriter); 
} 

बहुत अच्छे और अधिक अर्थपूर्ण।

Answer 2

पूर्व जावा 7 में, मैं कहना चाहता हूँ कि तुम क्या लिखा है सबसे अच्छा समाधान है।

जावा 7 में और इसके बाद आपके पास Automatic Resource Management इन चीजों को सरल बनाने के लिए है। इस सुविधा के साथ, आप

BufferedWriter theBufferedWriter = null; 
try (BufferedWriter theBufferedWriter = ...) { 
.... 
...... 
..... 
} catch (IOException anException) { 
.... 
} 

Answer 3

यही वह है जिसे हमें जावा 7 और ARM Blocks तक जीना होगा।

Answer 4

या आप Lombok और @Cleanup एनोटेशन का उपयोग कर सकते हैं और आप कभी भी अंत में फिर से प्रयास करने की कोशिश नहीं करेंगे।

इस तरह आप सामान्य रूप से लिखते थे यह है (ध्यान दें throws IOException):

//Vanilly Java 

import java.io.*; 

public class CleanupExample { 
    public static void main(String[] args) throws IOException { 
    InputStream in = new FileInputStream(args[0]); 
    try { 
     OutputStream out = new FileOutputStream(args[1]); 
     try { 
     byte[] b = new byte[10000]; 
     while (true) { 
      int r = in.read(b); 
      if (r == -1) break; 
      out.write(b, 0, r); 
     } 
     } finally { 
     out.close(); 
     } 
    } finally { 
     in.close(); 
    } 
    } 
} 

अब लंबोक के साथ आप सिर्फ @Cleanup धाराओं पर लिखने

import lombok.Cleanup; 
import java.io.*; 

public class CleanupExample { 
    public static void main(String[] args) throws IOException { 
    @Cleanup InputStream in = new FileInputStream(args[0]); 
    @Cleanup OutputStream out = new FileOutputStream(args[1]); 
    byte[] b = new byte[10000]; 
    while (true) { 
     int r = in.read(b); 
     if (r == -1) break; 
     out.write(b, 0, r); 
    } 
    } 
} 

Answer 5

यह ठीक है, लेकिन आप अगर theBufferedWriter का परीक्षण करना चाहिए इसे बंद करने से पहले शून्य नहीं है।
तुम भी कर सकता है:

BufferedWriter theBufferedWriter; 
try { 
    theBufferedWriter = new ... 
    try { 
     ... 
    } finally { 
     try { 
      theBufferedWriter.close(); 
     } catch (IOException closeException) { 
      closeException.printStackTrace(); 
     } 
    } 
} catch (IOException anException) { 
    ... 
} 

या:

BufferedWriter theBufferedWriter; 
try { 
    theBufferedWriter = new ... 
} catch (IOException createException) { 
    // do something with createException 
    return; // assuming we are in a method returning void 
} 

try { 
    ... 
} catch (IOException anException) { 
    ... 
    // assuming we don't return here 
} 

try { 
    theBufferedWriter.close(); 
} catch (IOException closeException) { 
    closeException.printStackTrace(); 
} 

लेकिन ज्यादातर मैं ऐसे कार्य कर एक समर्पित विधि में (उदाहरण के लिए एक फ़ाइल लेखन) और इसलिए फोन करने वाले/एक अपवाद फेंकने के लिए पसंद करते हैं

void someMethod(...) throws IOException { 
    BufferedWriter theBufferedWriter = new ... 

    try { 
     ... 
    } catch (IOExcepption anException) { 
     try { 
      theBufferedWriter.close(); 
     } catch (IOException closeException) { 
      closeException.printStackTrace(); 
      // closeException is not thrown, anException represents the main/first problem 
     } 
     throw anException; 
    } 

    theBufferedWriter.close(); // throws the Exception, if any 
} 

_{कृपया ध्यान दें:: अंग्रेजी मेरी एफआईआर नहीं है, यह (उदाहरण के लिए किसी अन्य फ़ाइल के लिए पूछ, आवेदन रोक, ...) संभाल कर सकते हैं टी और न ही मेरी दूसरी भाषा, किसी भी मदद की सराहना की जाएगी}

Answer 6

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

try { 
    BufferedWriter writer = ..... 
    try { 
     ..... 
    } finally { 
     writer.close(); 
    } 
} catch (IOException e) { 
    .... 
}