현재 버전

method return string null if

method 작성 시 코딩 형태 비교


1. 

생략...

string sReturn = string.Empty;
string requestURL = GetConfig("HRSYNC_WS_URL");

if(!String.IsNullOrEmpty(requestURL))
{

  sReturn = getJson(requestURL, parameters, pSearchKey);
}
else
{
  sReturn = "FAILED|호출에 필요한 URL 정보가 존재하지 않습니다";
}

return sReturn;


2. 

생략...

string returnData = string.Empty;
string requestURL = GetConfig("HRSYNC_WS_URL");

if(String.IsNullOrEmpty(requestURL))
{
  returnData = "FAILED|호출이 필요한 URL 정보가 존재하지 않습니다";
}
else
{
  returnData = getJson(requestURL, parameters, pSearchKey);
}

return returnData;



사소한 부분인 거 같은데 메서드 작성 시에 신경이 쓰여 문의 드립니다.

보통 어떤식으로 작성하는 게 더 깔끔하고 좋은 코드 일까요?

Q1. 반환을 위한 변수 선언 시 sReturn 과 같이 앞에 string 표시를 표기하여 변수 선언하는 게 나은지
, 혹은 returnValue, returnData 와 같이 앞의 타입표기문자를 지우고 선언하시는 지

Q2. NULL체크 시, 실제 작업 수행 부분이 if 문 상단에 위치하도록 if 문에 not equal 처리 하는게 나은지
아니면 그냥 if equal 처리하고 else 문에 실제 작업 수행 부문 작성하시는 지


수정 이력

2017-11-16 10:48:04 에 아래 내용에서 변경 됨 #4

1. 

생략...

string sReturn = string.Empty;
string requestURL = GetConfig("HRSYNC_WS_URL");

if(String.IsNullOrEmpty(requestURL))
{

  sReturn = getJson(requestURL, parameters, pSearchKey);
}
else
{
  sReturn = "FAILED|호출에 필요한 URL 정보가 존재하지 않습니다";
}

return sReturn;


2. 

생략...

string returnData = string.Empty;
string requestURL = GetConfig("HRSYNC_WS_URL");

if(String.IsNullOrEmpty(requestURL))
{
  returnData = "FAILED|호출이 필요한 URL 정보가 존재하지 않습니다";
}
else
{
  returnData = getJson(requestURL, parameters, pSearchKey);
}

return returnData;



사소한 부분인 거 같은데 메서드 작성 시에 신경이 쓰여 문의 드립니다.

보통 어떤식으로 작성하는 게 더 깔끔하고 좋은 코드 일까요?

Q1. 반환을 위한 변수 선언 시 sReturn 과 같이 앞에 string 표시를 표기하여 변수 선언하는 게 나은지
, 혹은 returnValue, returnData 와 같이 앞의 타입표기문자를 지우고 선언하시는 지

Q2. NULL체크 시, 실제 작업 수행 부분이 if 문 상단에 위치하도록 if 문에 not equal 처리 하는게 나은지
아니면 그냥 if equal 처리하고 else 문에 실제 작업 수행 부문 작성하시는 지

2017-11-16 10:44:17 에 아래 제목에서 변경 됨 #3

둘 중 어느 형태로 코딩하는게 바람직 한가요?

2017-11-16 10:31:57 에 아래 내용에서 변경 됨 #2

1. 

생략...

string sReturn = string.Empty;
string requestURL = GetConfig("HRSYNC_WS_URL");

if(String.IsNullOrEmpty(requestURL))
{

  sReturn = getJson(requestURL, parameters, pSearchKey);
}
else
{
  sReturn = "FAILED|호출에 필요한 URL 정보가 존재하지 않습니다";
}

return sReturn;


2. 

생략...

string sReturn = string.Empty;
string requestURL = GetConfig("HRSYNC_WS_URL");

if(String.IsNullOrEmpty(requestURL))
{
  sReturn = "FAILED|호출이 필요한 URL 정보가 존재하지 않습니다";
}
else
{
  sReturn = getJson(requestURL, parameters, pSearchKey);
}

return sReturn;



사소한 부분인 거 같은데 메서드 작성 시에 신경이 쓰여 문의 드립니다.

보통 어떤식으로 작성하는 게 더 깔끔하고 좋은 코드 일까요?

Q1. 반환을 위한 변수 선언 시 sReturn 과 같이 앞에 string 표시를 표기하여 변수 선언하는 게 나은지
, 혹은 returnValue, returnData 와 같이 앞의 타입표기문자를 지우고 선언하시는 지

Q2. NULL체크 시, 실제 작업 수행 부분이 if 문 상단에 위치하도록 if 문에 not equal 처리 하는게 나은지
아니면 그냥 if equal 처리하고 else 문에 실제 작업 수행 부문 작성하시는 지

2017-11-16 10:31:09 에 아래 내용에서 변경 됨 #1

1. 

생략...

string sReturn = string.Empty;
string requestURL = GetConfig("HRSYNC_WS_URL");

if(String.IsNullOrEmpty(requestURL))
{

  sReturn = getJson(requestURL, parameters, pSearchKey);
}
else
{
  sReturn = "FAILED|호출에 필요한 URL 정보가 존재하지 않습니다";
}

return sReturn;

2. 

생략...

string sReturn = string.Empty;
string requestURL = GetConfig("HRSYNC_WS_URL");

if(String.IsNullOrEmpty(requestURL))
{
  sReturn = "FAILED|호출이 필요한 URL 정보가 존재하지 않습니다";
}
else
{
  sReturn = getJson(requestURL, parameters, pSearchKey);
}

return sReturn;


사소한 부분인 거 같은데 메서드 작성 시에 신경이 쓰여 문의 드립니다.

보통 어떤식으로 작성하는 게 더 깔끔하고 좋은 코드 일까요?

Q1. 반환을 위한 변수 선언 시 sReturn 과 같이 앞에 string 표시를 표기하여 변수 선언하는 게 나은지
, 혹은 returnValue, returnData 와 같이 앞의 타입표기문자를 지우고 선언하시는 지

Q2. NULL체크 시, 실제 작업 수행 부분이 if 문 상단에 위치하도록 if 문에 not equal 처리 하는게 나은지
아니면 그냥 if equal 처리하고 else 문에 실제 작업 수행 부문 작성하시는 지