시저 암호

문제 내용

풀이🙎🏻‍♂️

function solution(s, n) {
  const result = [];

  for (let string of s) {
    let convertedString = string === " " ? " " : string.charCodeAt();
    const notblank = !(convertedString === " ");

    if (notblank && convertedString <= 90) {
      convertedString += n;
      if (convertedString > 90) {
        convertedString -= 26;
      }
    }

    if (notblank && convertedString >= 97) {
      convertedString += n;
      if (convertedString > 122) {
        convertedString -= 26;
      }
    }

    notblank
      ? result.push(String.fromCharCode(convertedString))
      : result.push(" ");
  }
  return result.join("");
}

/*
(풀이)
테스트 케이스에서 한번 막혔었는데 대문자 Z도 A로 넘어가야하는지 몰랐다.🥲
for of 문으로 문자열s를 넣고 변수 string을 아스키 코드로 변환. 공백일 경우는 공백유지
문자열이 공백인지 확인하는 코드는 반복되므로 변수에 넣어주고
if문으로 공백이 아니고 변환한 아스키코드 값이 90 이하일 경우와
공백이 아니고 97보다 큰 경우로 나누어 소문자와 대문자를 분리한 후
내부에서 아스키코드값이 z를 넘어가면 -26 으로 a부터 시작하게 해준다.
마지막으로 공백일 경우와 아닌경우를 나눠서 result 배열에 push해주고 join해서 제출.
*/

• 아스키코드 후.. 문제도 잘못읽고 생각보다 골치아팠다.
• 너무 좋은 피드백을 받았기에 정리 필수👍

 

피드백 풀이👨‍💻

function solution(s, n) {
  // map을 이용해 새로운 배열을 반환함으로써 문제 접근
  const result = s.split("").map((string) => {
    let convertedString = string.charCodeAt();

    // " " 곧바로 리턴
    if (string === " ") return " ";

    if (getIsUpperCase(convertedString)) {
      convertedString += n;
      convertedString = diminishAscii(convertedString, 90);
    } else {
      convertedString += n;
      convertedString = diminishAscii(convertedString, 122);
    }

    return String.fromCharCode(convertedString);
  });

  return result.join("");
}

function diminishAscii(ascii, validRange) {
  if (ascii > validRange) {
    return ascii - 26;
  }
  return ascii;
}

function getIsUpperCase(ascii) {
  if (ascii >= 65 && ascii < 91) return true;
  return false;
}

• 대소문자를 판별하는 getIsUpperCase 함수와 z를 넘는 아스키코드를 a부터 시작으로 돌려주는 diminishAscii 함수를 만들어서 변경 하셨다.
• 함수를 분리한 이유는 SOLID원칙에서 S 단일 책임 원칙 ( 함수는 오직 하나의 책임을 가져야 한다. )을 적용.
• 결국 제출 함수인 solution은 더 깔끔해졌고 한개의 함수는 단 한가지의 역할을 맡게됐다👍

- SOLID원칙은 객체지향 프로그래밍의 원칙이지만 js, ts에서도 적용해볼 수 있다! 알아보기 필수 (공부 후 링크 추가예정)