[자바]겹치지 않은 여러개의 try-catch문 vs 코드 전부를 감싸는 하나의 try-catch문

Question

채팅 프로그램을 만들어보고 있는 고등학생입니다.run 메소드 부분에서 소켓이나 outputstream 생성 등에 발생할 수 있는 예외를 다 따로따로 잡고 있는데...이렇게 따로따로 try-catch문을 만드는 것과 하나의 큰 try-catch문에 코드를 전부 넣는것 둘 중 어느 쪽이 코딩 스타일과 성능면에서 더 좋을지 궁금합니다.워낙에 스파게티 코드라 run 메소드를 포함한 클래스를 모두 올려봅니다;; package com.intgames.JChat.runnables; import java.io.BufferedReader;import java.io.BufferedWriter;import java.io.IOException;import java.io.InputStreamReader;import java.io.OutputStreamWriter;import java.net.ServerSocket;import java.net.Socket; import com.intgames.JChat.Server;import com.intgames.JChat.GUI.MainGUI; public class ServerAccepter implements Runnable { private ServerSocket sock; private BufferedWriter bw; private Server svr; private MainGUI mg; public ServerAccepter(ServerSocket server, Server svr) { // TODO Auto-generated constructor stub this.sock &#61; server; this.svr &#61; svr; this.mg &#61; svr.mg; } &#64;Override public void run() { // TODO Auto-generated method stub while (true) { Socket sc &#61; null; BufferedReader br &#61; null; try { sc &#61; sock.accept(); } catch (IOException e) { // TODO Auto-generated catch block mg.error(&#34;클라이언트 연결 오류!&#34;, &#34;클라이언트와 연결하지 못했습니다.&#34;); continue; } try { bw &#61; new BufferedWriter(new OutputStreamWriter(sc.getOutputStream())); svr.putBufferedWriter(bw); } catch (IOException e) { // TODO Auto-generated catch block mg.error(&#34;클라이언트 연결 오류!&#34;, &#34;클라이언트의 OutputStream을 가져오는 데 실패했습니다.&#34;); continue; } try { br &#61; new BufferedReader(new InputStreamReader(sc.getInputStream())); } catch (IOException e) { // TODO Auto-generated catch block mg.error(&#34;클라이언트 연결 오류!&#34;, &#34;클라이언트의 InputStream을 가져오는 데 실패했습니다.&#34;); continue; } new Thread(new MessageGetter(br, this.svr)).start(); } } }

이설란 · Answer

try-catch를 굳이 나눠서 사용해야할 필요는 없죠!!어짜피 그안에서 에러나면 다 catch문으로 떨어질텐데...특별한 목적이 있지 않는한 그냥 한곳에서 사용하는게 좋다고 생각합니다.(코드 낭비에요) 저걸가지고 성능을 따지기도 참 애매하네요!! 솔직히 대한민국에서 성능은하드웨어가 거의다 커버해주니까요!!저정도는 성능과 아무런 관련은 없지만 첨언을 드리자면 하나라도 덜 실행시키는게개미눈물만큼이라도 좋아지겠죠?

redeye0922 · Answer

코드의 의도를 제대로 파악한지는 모르겠습니다만 일단 코드를봤을때무한루프를 돌리면서 3개의 try catch문 사이에서 error가 안난다면 스레드를 시작하게 되어있네요특별히 catch에서 처리하는거라곤 에러 표기정도? 하고 계속 진행시키고있습니다단순하게 질문으로 봤을때는 전체 try잡는것보단 각각각도는게 맞아보이긴하는데요일단 기능들이 다 다르니 구조적으로 설계를 다시 해보시는게 어떨지요?목적이나 용도에 맞게 try catch 문안에 try catch문이 들어갈수도있지만....구조적으로 바꿔보는게 더 좋아보이네요

사과맛오렌지 · Answer

두 분 모두 답변 감사합니다!!저는 처음에 각각의 코드마다 Execption이 난 이유가 달라서 하나하나에서 이유를 각각 출력해주는 게 디버깅할때도 더 나을 거라고 생각해서 저렇게 만들었습니다.run 메소드에서는 계속 무한루프를 돌리며 3개의 try catch문 사이에서 error가 나면 무한루프로 continue 해서 다시 소켓을 받아오도록 설계했습니다.답변 들어보니 제 걱정이 기우였나 하는 생각도 듭니다.두 분 모두 좋은 답변 다시 감사드리며 이외에 혹시 코드 구조에 대해 조언이 더 있으시다면 가르쳐주시면 감사드리겠습니다.

제르맹 · Answer

예외발생이유는 e안에 정보가 있으니 한번의 catch에서 e의 정보를 가지고 로그를 남겨보세요

[자바]겹치지 않은 여러개의 try-catch문 vs 코드 전부를 감싸는 하나의 try-catch문

4개의 답변

Q&A 목록