코드에 대한 간단한 질문

코딩 습관과도 관련된 이야기입니다만,

함수를 만들 때 실행되지 않아야 할 조건 등을 미리 걸러서 예외 처리를 하는 것이 좋은 습관으로 믿고 있습니다.

int foo(void)
{
if(unexpected_condition)
{
return -1; /*함수를 시행하지 말아야할 조건*/
}
else /*질문1: 여기 else를 쓰는 것과 안쓰는 것이 이론적이 아닌 실용적으로 차이가 있습니까? */
{
/*아래의 함수들이 매 실행될 때마다 컴퓨터는 if(unexpected_condition) 조건을 체크합니까? */
do_something1();
do_something2();
do_something3();
return 0;
}
}

위의 것을 아래와 같이 쓰는 습관은 좋지 않은 것 같습니다. 함수가 짧으면 상관 없는데 함수가 길어지면
괄호가 점점 깊어지니까요.

int foo_2(void)
{
if(!unexpected_condition)
{
do_something1();
do_something2();
do_something3();
return 0;
}
return -1;
}