GetLastError 함수 사용의 흔한 실수
2012.01.13 08:00 |
Programming
보통은 아래와 같이 사용한다.
HANDLE h = CreateFile(...);
if (h == INVALID_HANDLE_VALUE)
{
DWORD dw = GetLastError();
... Do something
}
if (h == INVALID_HANDLE_VALUE)
{
DWORD dw = GetLastError();
... Do something
}
경험이 많지 않거나 주의 깊지 않은 프로그래머들은 프로그램을 유지보수 하면서 이미 잘 만들어져있던 위와 같은 코드를 별 생각 없이 아래처럼 바꾸기도 한다.
HANDLE h = CreateFile(...);
if (h == INVALID_HANDLE_VALUE)
{
DoSomethingElse(); // 뭔가 예외를 처리하기 위해 추가적인 코드를 여기에 쑤셔넣는다. 아니, 왜 하필 여기에.
DWORD dw = GetLastError();
... Do something
}
처음에 말했듯이 DoSomethingElse()안에서 윈도 Api를 사용한다면 쓰레드 저장소에 있던 LastError 코드가 다른 값으로 바뀌어버릴 수 있다는 것을 예상할 수 있다. 항상 코드를 읽으면서 GetLastError를 호출하는 부분이 에러값을 확인하려고 했던 함수의 바로 아래에 붙어있지 않다면 섬뜩함을 느껴야 한다. 하지만 잘 모르고 있으면 보이지 않는 법.if (h == INVALID_HANDLE_VALUE)
{
DoSomethingElse(); // 뭔가 예외를 처리하기 위해 추가적인 코드를 여기에 쑤셔넣는다. 아니, 왜 하필 여기에.
DWORD dw = GetLastError();
... Do something
}
HANDLE h = CreateXXX(...);
DWORD dw = GetLastError();
if (dw == ERROR_SUCCESS)
{
... 핸들을 가지고 다른 무엇인가를 한다.
}
else
{
... 함수의 실패처리를 한다.
}
DWORD dw = GetLastError();
if (dw == ERROR_SUCCESS)
{
... 핸들을 가지고 다른 무엇인가를 한다.
}
else
{
... 함수의 실패처리를 한다.
}
이번에는 한 Api를 호출 한 뒤에 바로 GetLastError를 호출해서 에러값을 얻어왔다. 얼핏보면 맞는 것도 같지만 역시 틀린 코드이다. 함수의 성공 실패 여부는 함수의 스펙에 따라 리턴 값 등으로 확인해야지 GetLastError 값으로 확인해서는 안된다. 왜냐하면 Win32에서 제공되는 대부분의 Api들이 함수가 성공했을 때는 LastError 값을 건드리지 않기 때문이다. 위 코드에서는 함수가 성공할 때는 에러 값도 0(ERROR_SUCCESS)으로 셋팅시켜 줄 것이라 굳게 믿고 있다. 실상은 그렇지 않다. GetLastError는 오직 함수가 실패했을 때만(그 바로 직후에) 호출해야 한다.
대부분의 함수들은 그 성공 여부를 리턴값으로 가르쳐준다. 리턴 값으로 성공과 실패 여부를 호출자에게 전달해주기로 했다면 뭐하러 또 SetLastError(ERROR_SUCCESS) 와 같은 추가적인 코드를 호출하겠는가.
'Programming' 카테고리의 다른 글
| 왜 static 멤버 함수에는 const를 붙일 수 없을까? (0) | 2012.02.06 |
|---|---|
| SetFilePointer 보다는 SetFilePointerEx를 사용해야 한다 (1) | 2012.01.16 |
| GetLastError 함수 사용의 흔한 실수 (0) | 2012.01.13 |
| NTFS에서 Sparse 파일을 만들기 (3) | 2012.01.03 |
| 16TB 크기의 파일을 만들어내려면 얼마나 오래 걸릴까? (0) | 2012.01.01 |
| 구조체의 패킹에 대한 이야기 (4) | 2011.12.19 |
