Skip to content

Latest commit

 

History

History
 
 

item77

Folders and files

NameName
Last commit message
Last commit date

parent directory

..
 
 

예외를 무시하지 말라.


// catch 블록을 비워두면 예외가 무시된다. 아주 의심스러운 코드다!
try {
    ...
} catch (SomeException e) {
    //아무 일도 하지 않음.
}

요약

catch 블록을 비워두면 예외가 존재할 이유가 없어진다. 예외를 무시하기로 했다면 catch 블록안에 그렇게 결정한 이유를 주석으로 남기고 예외 변수의 이름도 ignored로 바꿔 놓도록 하자.

[추가의견]

 이전 소스에 어떠한 log와 exception도 없이 catch 블록을 비워두어서 문제를 찾아 내는데 하루를 보낸 경험이 있다.
catch 블록이 비었을 경우, 문제를 찾기는 정말 정말 어렵다.
되도록이면 catch문을 비워두지 않고, 로그나 에러처리를 하는것이 좋다고 생각한다. 
굳이 비워둘 이유는 없다고 본다.