ฉันใช้ Java 7 try-with-resources อย่างถูกต้องหรือไม่

ฉันคาดหวังว่าโปรแกรมอ่านบัฟเฟอร์และโปรแกรมอ่านไฟล์จะปิดและทรัพยากรที่ปล่อยออกมาหากมีการโยนข้อยกเว้น

public static Object[] fromFile(String filePath) throws FileNotFoundException, IOException
{
    try (BufferedReader br = new BufferedReader(new FileReader(filePath)))
    {
        return read(br);
    } 
}

อย่างไรก็ตามจำเป็นต้องมีcatchข้อสำหรับการปิดสำเร็จหรือไม่?

แก้ไข:

โดยพื้นฐานแล้วคือโค้ดด้านบนใน Java 7 เทียบเท่ากับด้านล่างสำหรับ Java 6:

public static Object[] fromFile(String filePath) throws FileNotFoundException, IOException
{

    BufferedReader br = null;

    try
    {
        br = new BufferedReader(new FileReader(filePath));

        return read(br);
    }
    catch (Exception ex)
    {
        throw ex;
    }
    finally
    {
        try
        {
            if (br != null) br.close();
        }
        catch(Exception ex)
        {
        }
    }

    return null;
}

ถูกต้องและไม่มีข้อกำหนดสำหรับcatchอนุประโยค Oracle java 7 doc กล่าวว่าทรัพยากรจะถูกปิดไม่ว่าจะมีข้อยกเว้นเกิดขึ้นจริงหรือไม่ก็ตาม

คุณควรใช้catchอนุประโยคเฉพาะเมื่อคุณต้องการตอบสนองต่อข้อยกเว้น catchข้อจะถูกดำเนินการหลังจากที่ทรัพยากรถูกปิด

นี่คือตัวอย่างจากบทช่วยสอนของ Oracle :

ตัวอย่างต่อไปนี้อ่านบรรทัดแรกจากไฟล์ ใช้อินสแตนซ์ของ BufferedReader เพื่ออ่านข้อมูลจากไฟล์ BufferedReader เป็นทรัพยากรที่ต้องปิดหลังจากโปรแกรมเสร็จสิ้น:

static String readFirstLineFromFile(String path) throws IOException {
    try (BufferedReader br =
                   new BufferedReader(new FileReader(path))) {
        return br.readLine();
    }
} // In this example, the resource declared in the try-with-resources statement is a BufferedReader.

... เนื่องจากอินสแตนซ์ BufferedReader ถูกประกาศในคำสั่ง try-with-resource มันจะถูกปิดโดยไม่คำนึงว่าคำสั่ง try จะเสร็จสมบูรณ์ตามปกติหรือในทันที (อันเป็นผลมาจากวิธี BufferedReader.readLine ที่ขว้าง IOException)

แก้ไข

เกี่ยวกับคำถามที่แก้ไขใหม่:

โค้ดใน Java 6 เรียกใช้งานcatchและหลังจากนั้นfinallyบล็อก ซึ่งทำให้ทรัพยากรยังคงเปิดอยู่ในcatchบล็อก

ใน Java 7 ไวยากรณ์ทรัพยากรถูกปิดก่อนcatchบล็อกเพื่อให้ทรัพยากรที่ปิดไปแล้วในระหว่างcatchการดำเนินการบล็อก นี่คือเอกสารในลิงค์ด้านบน:

ในคำสั่ง try-with-resources การดักจับหรือการบล็อกในที่สุดจะถูกเรียกใช้หลังจากปิดทรัพยากรที่ประกาศแล้ว

การใช้ทรัพยากรแบบลองใช้กับทรัพยากรของคุณจะทำงานได้ดีในกรณีนี้โดยเฉพาะ แต่ก็ไม่ถูกต้องโดยทั่วไป คุณไม่ควรเชื่อมโยงทรัพยากรเช่นนั้นเพราะอาจนำไปสู่ความประหลาดใจที่ไม่พึงประสงค์ สมมติว่าคุณมีขนาดบัฟเฟอร์ตัวแปร:

public static Object[] fromFile(String filePath) throws FileNotFoundException, IOException
{
    int sz = /* get buffer size somehow */
    try (BufferedReader br = new BufferedReader(new FileReader(filePath), sz))
    {
        return read(br);
    } 
}

สมมติว่ามีบางอย่างผิดพลาดและคุณszถูกมองในแง่ลบ ในกรณีนี้ทรัพยากรไฟล์ของคุณ (สร้างผ่านnew FileReader(filePath)) จะไม่ถูกปิด

เพื่อหลีกเลี่ยงปัญหานี้คุณควรระบุแต่ละทรัพยากรแยกกันดังนี้:

public static Object[] fromFile(String filePath) throws FileNotFoundException, IOException
{
    int sz = /* get buffer size somehow */
    try (FileReader file = new FileReader(filePath);
         BufferedReader br = new BufferedReader(file, sz))
    {
        return read(br);
    } 
}

ในกรณีนี้แม้ว่าการเริ่มต้นbrล้มเหลวfileจะยังคงถูกปิด คุณสามารถค้นหารายละเอียดเพิ่มเติมได้ที่นี่และที่นี่