try块采用try-with-resource结构

Signed-off-by: hecy7 <hecy7@asiainfo.com>
pull/2758/head
hecy7 1 year ago
parent bd2bb36205
commit a8f4900d08

@ -42,21 +42,11 @@ public class FileUtils {
* @since IO 2.1 * @since IO 2.1
*/ */
public static void writeByteArrayToFile(File file, byte[] data, boolean append) throws IOException { public static void writeByteArrayToFile(File file, byte[] data, boolean append) throws IOException {
OutputStream out = null; try (OutputStream out = openOutputStream(file, append)) {
try {
out = openOutputStream(file, append);
out.write(data); out.write(data);
out.close(); // don't swallow close Exception if copy completes normally
} finally {
try {
if (out != null) {
out.close();
} }
} catch (IOException ioe) {
// ignore // ignore
} }
}
}
/** /**
* Opens a {@link FileOutputStream} for the specified file, checking and * Opens a {@link FileOutputStream} for the specified file, checking and
@ -111,10 +101,8 @@ public class FileUtils {
* @param file the file to save the history * @param file the file to save the history
*/ */
public static void saveCommandHistory(List<int[]> history, File file) { public static void saveCommandHistory(List<int[]> history, File file) {
OutputStream out = null; try (OutputStream out = new BufferedOutputStream(openOutputStream(file, false))) {
try { for (int[] command : history) {
out = new BufferedOutputStream(openOutputStream(file, false));
for (int[] command: history) {
String commandStr = Helper.fromCodePoints(command); String commandStr = Helper.fromCodePoints(command);
if (isAuthCommand(commandStr)) { if (isAuthCommand(commandStr)) {
command = AUTH_CODEPOINTS; command = AUTH_CODEPOINTS;
@ -127,16 +115,9 @@ public class FileUtils {
} }
} catch (IOException e) { } catch (IOException e) {
// ignore // ignore
} finally {
try {
if (out != null) {
out.close();
} }
} catch (IOException ioe) {
// ignore // ignore
} }
}
}
public static List<int[]> loadCommandHistory(File file) { public static List<int[]> loadCommandHistory(File file) {
BufferedReader br = null; BufferedReader br = null;
@ -167,10 +148,8 @@ public class FileUtils {
* @param file the file to save the history * @param file the file to save the history
*/ */
public static void saveCommandHistoryString(List<String> history, File file) { public static void saveCommandHistoryString(List<String> history, File file) {
OutputStream out = null; try (OutputStream out = new BufferedOutputStream(openOutputStream(file, false))) {
try { for (String command : history) {
out = new BufferedOutputStream(openOutputStream(file, false));
for (String command: history) {
if (!StringUtils.isBlank(command)) { if (!StringUtils.isBlank(command)) {
if (isAuthCommand(command)) { if (isAuthCommand(command)) {
command = ArthasConstants.AUTH; command = ArthasConstants.AUTH;
@ -181,16 +160,9 @@ public class FileUtils {
} }
} catch (IOException e) { } catch (IOException e) {
// ignore // ignore
} finally {
try {
if (out != null) {
out.close();
} }
} catch (IOException ioe) {
// ignore // ignore
} }
}
}
public static List<String> loadCommandHistoryString(File file) { public static List<String> loadCommandHistoryString(File file) {
BufferedReader br = null; BufferedReader br = null;
@ -218,8 +190,7 @@ public class FileUtils {
} }
public static String readFileToString(File file, Charset encoding) throws IOException { public static String readFileToString(File file, Charset encoding) throws IOException {
FileInputStream stream = new FileInputStream(file); try (FileInputStream stream = new FileInputStream(file)) {
try {
Reader reader = new BufferedReader(new InputStreamReader(stream, encoding)); Reader reader = new BufferedReader(new InputStreamReader(stream, encoding));
StringBuilder builder = new StringBuilder(); StringBuilder builder = new StringBuilder();
char[] buffer = new char[8192]; char[] buffer = new char[8192];
@ -228,8 +199,6 @@ public class FileUtils {
builder.append(buffer, 0, read); builder.append(buffer, 0, read);
} }
return builder.toString(); return builder.toString();
} finally {
stream.close();
} }
} }

Loading…
Cancel
Save