인프런 커뮤니티 질문&답변

수업 내용 코드는

function checkPhone(myphone) {
  if (myphone.length < 10 || myphone.length > 11) {
    console.log("에러 발생!!! 핸드폰 번호를 제대로 입력해 주세요!!!");
    return false;
  } else {
    return true;
  }
}

function getToken() {
  let result = String(Math.floor(Math.random() * 100000)).padStart(6, "0");
  console.log(result);
  return result;
}

function sendTokenToSMS(myphone, result) {
  console.log(myphone + "번호로 인증번호" + result + "를 전송합니다.");
}

function createTokenOfPhone(myphone) {
  const isValid = checkPhone(myphone);
  if (isValid === false) return;
  const mytoken = getToken();
  sendTokenToSMS(myphone, mytoken);
}

createTokenOfPhone("01012345678"); // 01012345678: 인자(argument)

이렇게 되어있는데

createTokenOfPhone 함수에서
isValid 상수를 꼭 선언해야하나요??

function checkPhone(myphone) {
  if (myphone.length < 10 || myphone.length > 11) {
    console.log("에러 발생!!! 핸드폰 번호를 제대로 입력해 주세요!!!");
    return true;
  }
}

function getToken() {
  let result = String(Math.floor(Math.random() * 100000)).padStart(6, "0");
  console.log(result);
  return result;
}

function sendTokenToSMS(myphone, result) {
  console.log(myphone + "번호로 인증번호" + result + "를 전송합니다.");
}

function createTokenOfPhone(myphone) {
  if (checkPhone(myphone)) return;
  const mytoken = getToken();
  sendTokenToSMS(myphone, mytoken);
}

createTokenOfPhone("01012345678"); // 01012345678: 인자(argument)

이렇게 checkPhone()이 문제있을 때만 true를 반환하고,
checkPhone()이 true일 경우 return 처리하면 코드가 더 짧아지기는 하는데

수업에서 말씀하시는 협업의 가독성을 위해서 isValid라는 상수를 선언해줘야 하는지 아니면 제가 생각한 방식으로 코드를 줄여도 괜찮은건지 궁금합니다.