generateData.js를 리팩토링하라

[yujonglee]

33 에서 말씀드린대로 generateData.js를 수정했습니다.

수정 전후 코드가 동일하게 동작함을 확인하기 위해 테스트 코드도 작성하였습니다. 수정 전과 코드의 동작에 차이가 없기 때문에 #34 는 해결되지 않았습니다.

테스트 코드는 generateData.test.js이고, 테스트 데이터는 testData폴더에 들어있습니다. 테스트는 jest를 사용했습니다.

[yujonglee]

이제 보니 8b2f011 커밋은 조금 더 나누어서 할 걸 그랬네요. 양해 부탁드리겠습니다.

[johngrib]

이제 보니 8b2f011 커밋은 조금 더 나누어서 할 걸 그랬네요. 양해 부탁드리겠습니다.

이 PR을 머지하게 될 경우 squash and merge를 할 예정입니다. 그래서 괜찮습니다. squash and merge를 하는 이유는 다음과 같습니다.

• 작성하신 커밋 메시지 스타일이 제가 이 리포지토리를 관리해오고 있는 규칙과 다릅니다.
• 이 리포지토리는 한국어 기반의 위키를 지향하고 있기 때문에 커밋 메시지도 한국어로 작성된 완성된 명령형 문장을 쓰고 있기 때문입니다.
• 예외: merge commit

[yujonglee]

.babelrc 파일을 삭제하고 테스트 코드와 테스트 데이터를 모두 삭제하였습니다. PR 타이틀도 수정했습니다.

[yujonglee]

궁금한 것이 있습니다.

1. 중괄호 없이 if문을 쓰는 것은(그것이 짧다해도) 일반적으로 안티패턴으로 여겨지는 것인가요 아니면 그렇게 하지 않는 것이 이 래포지토리의 코딩스타일인가요?
2. 코드에 동등연산자(==)가 사용된 것이 몇 군데 있는데 이것을 일치연산자(===)로 바꾸는 것이 좋을까요?

[johngrib]

궁금한 것이 있습니다.

1. 중괄호 없이 if문을 쓰는 것은(그것이 짧다해도) 일반적으로 안티패턴으로 여겨지는 것인가요 아니면 그렇게 하지 않는 것이 이 래포지토리의 코딩스타일인가요?
2. 코드에 동등연산자(==)가 사용된 것이 몇 군데 있는데 이것을 일치연산자(===)로 바꾸는 것이 좋을까요?

1. 네. 네. 이에 대해서는 특별한 부연 설명이 필요하지 않다고 생각해요. 이야기가 나온 김에 이와 관련된 유명하고 재미있는 버그 이야기가 있습니다. Apple의 SSL/TLS 프로토콜에서 발견된 끔찍한 실수입니다. https://embeddedgurus.com/barr-code/2014/03/apples-gotofail-ssl-security-bug-was-easily-preventable/
2. 그것도 괜찮겠죠.

[johngrib]

@yujong-lee 고생하셨습니다!