क्या यह एक लूप लूप में एक कोशिश-पकड़ रखने का बुरा अभ्यास है?

Question

मैं एक कमांड लाइन उपयोगिता बना रहा हूं जो उप-निर्देशिका/फ़ाइलों को हटा देगा। यदि कोई फ़ाइल उपयोग में है, तो System.IO.IOException फेंक दिया गया है। मैं अपने लूप के अंदर एक कोशिश-पकड़ ब्लॉक का उपयोग करता हूं।

प्रश्न:

यह 1.Is खराब व्यवहार का पाश के लिए एक के भीतर एक कोशिश पकड़ के लिए?

2. यदि हाँ, बेहतर विकल्प क्या है?

मेरे कोड:

System.IO.DirectoryInfo di = new DirectoryInfo(path); 

    foreach (FileInfo file in di.GetFiles()) 
    { 
     try 
     { 
      file.Delete(); 
     } 
     catch(System.IO.IOException) 
     { 
      Console.WriteLine("Please Close the following File {0}", file.Name); 
     } 

    } 

Answer 1

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

System.IO.DirectoryInfo di = new DirectoryInfo(path); 

foreach (FileInfo file in di.GetFiles()) 
{ 
    try 
    { 
     file.Delete(); 
    } 
    catch(System.IO.IOException) 
    { 
     Console.WriteLine("Please Close the following File {0}", file.Name); 
     continue; 
    } 
    // 
    // Other Code 
    // 
} 

इस तरह से आप की समीक्षा करने के त्रुटि लॉग इन कर सकते बाद में, लेकिन अभी भी आप क्या कार्रवाई करने के लिए कोशिश कर रहे थे के बाकी की प्रक्रिया।