검토하고 더 나은 것을 작성하십시오.

이번 포스팅은 주로 픽스my old PR 에 대해 다룰 예정입니다. 이전 블로그 게시물에서 이전에 제출했지만 병합되지 않은 이전 PR을 다시 할 계획이라고 언급했습니다.

코딩을 시작한지 ​​2년 정도 되었습니다. 이 2년 동안 저는 솔루션을 제공하고 프로그램을 작동시키는 데 더 집중했습니다. 내 코드를 다시 보고 검토하는 경우는 거의 없습니다. 그러나 개발자가 되려면 깨끗하고 품질이 좋은 코드를 작성할 수 있는 능력이 중요합니다.

내 , 나는이 PR의 내 old commit가 중복되고 지저분하다고 언급했습니다. new commit에서 변경한 사항은 동일한 기능을 수행하지만 코드를 유지 관리하고 이해하기 쉽게 개선합니다. 예를 들어 'l', 'e', ​​'i' 문자를 사용하여 'large', 'medium' 및 'icon'의 이미지 크기를 나타내는 대신 배열 객체를 사용하여 이 3개의 값을 저장합니다. 이미지 크기. 문제 해결에 더 읽기 쉽고 좋습니다.

// store the value into an array of objects
const sizeOptions = {
  LARGE: { value: 0, name: 'Large', code: 'l' },
  MEDIUM: { value: 1, name: 'Medium', code: 'e' },
  ICON: {value: 2, name: 'Icon', code: 'i' }
};
// call the object.code value to replace the old letter 'l'
switch (size) {
  // case 'l': // <-- the old one
  case sizeOptions.LARGE.code: // <-- the improved one
  // ...
}

졸업 후에는 학교 과제 대신 실제 프로젝트를 수행할 것입니다. 학교 과제의 경우 일반적으로 제출 후 코드를 유지할 필요가 없습니다. 그러나 시스템이 작동하는 방식은 그렇지 않습니다. 실제 프로젝트를 수행할 때 클라이언트가 필요하지 않을 때까지 애플리케이션 작업을 유지해야 합니다. 이는 다른 문제에 직면하고 자주 수정해야 함을 의미합니다. 그렇기 때문에 코드를 유지 관리하기 쉽게 유지해야 합니다. 또한, 우리는 다른 개발자들과 협력할 것입니다. 우리는 특정 코딩 스타일과 규칙을 따라야 하며 다른 개발자가 우리의 코드 논리나 스타일을 이해하는 데 시간을 낭비하게 만들고 싶지 않습니다.


지난번과 비교해보면 repo 소유자가 많은 변경 요청을 하기 보다는 저의 새로운 변경 사항에 만족하는 것 같습니다. 나는 내 PR이 이번에 병합될 것이라고 믿습니다. 앞으로는 양질의 코드를 작성하고 내 코드를 검토하는 데 더 많은 시간을 할애할 것입니다.