Java-попытка с предупреждением о ресурсах от Eclipse

Question

Почему для Eclipse это должно управляться с помощью try-with arm?

BufferedReader reader = null; 
try { 
    if (condition) { 
    try { 
     reader = method1(); 
    } catch (Exception e) { 
     ... 
    } 
    } 
    if (reader == null) { 
    reader = method2(); 
    } 

    do things ... 
} catch(Exception e) { 
    ... 
} finally { 
    if (reader != null) { 
    reader.close(); 
    } 
} 

Существует лучший способ справиться с этим случаем? или просто предупреждение о нежелательном результате затмения?

Этот случай не является действительным:

try (BufferedReader reader = null) { 
    if (condition) { 
    reader = method1(); 
    } else { 
    reader = method2(); 
    } 

    do things ... 
} 

Answer 1

Try:

try (BufferedReader reader = createBufferedReader(condition)) { 
    do things ... 
} 

private BufferedReader createBufferedReader(boolean condition){ 
    if (condition) { 
    return method1(); 
    } else { 
    return method2(); 
    } 
} 

Answer 2

вы можете использовать функцию, которая имеет тип возвращаемого BufferedReader, в котором вы можете использовать свой условный вызов, эта функция может вызывается из основного только в том случае, если выполняется определенная конкретная ситуация, которую вы ищете.

private BufferedReader createReader(!XYZ){ 
if(a>b) 
return abc(); 
else 
return def(); 
} 

public static void main(String[] args){ 
createReader(!XYZ); 
} 

Answer 3

Как утверждает Спецификация языка Java в jls-14.20.3

Ресурс, объявленный в ResourceSpecification неявно объявляется окончательный (§4.12.4), если он явно не объявлен окончательным.

Таким образом, вы не можете изменить его в своем блоке try. Если вы хотите изменить его, используйте стандартный блок try-catch-finally. Другим вариантом будет выбор правильного ресурса, прежде чем использовать его с помощью try-with-resources.

Answer 4

Лучший способ имхо, ясно и сжато:

final String s = "abc"; 

try (BufferedReader reader = new Callable<BufferedReader>() { 
    @Override 
    public BufferedReader call() throws Exception { 
    if (condition) { 
     return method1(); 
    } 
    return method2(); 

    // different scope, s needs final keyword 
    } 
}.call()) { 

    do things ... 

} catch (Exception e) { 
    ... 
} 

Также проще в Java 8:

String s = "abc"; 

try (BufferedReader reader =() -> { 
    if (condition) { 
    return method1(); 
    } 
    return method2(); 

    // same scope, s does not need final keyword 
}) { 

    do things ... 

} catch (Exception e) { 
    ... 
} 

Answer 5

Вам не обязательно нужно отзывной или лямбда-выражение, как в ответ Seby в.

Учитывая, что проблема достаточно проста, вы можете просто использовать тернарный оператор, который работает во всех версиях java.

final String s = "abc"; 

try (BufferedReader reader = (condition) ? method1() : method2();) { 

    do things ... 

} catch (Exception e) { 
    ... 
}