Update README.md #2
Conversation
|
고생하셨습니다 |
| return Promise.reject(error); | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
주석을 상세하게 달아두셔서 코드를 이해하기 쉽습니다. 클래스를 따로 생성하셔서 run()함수를 간결하게 작성하신 것이 인상 깊습니다:)
| if (numbers.startsWith('//')) { // startsWith : 문자열이 //로 시작하는지 확인 | ||
| const linebreakIndex = numbers.indexOf('\\n'); // indexOf : 찾은 문자 위치 반환, 찾지 못하면 -1 반환 | ||
| if (linebreakIndex !== -1) { | ||
| const customDelimiter = numbers.slice(2, linebreakIndex); // 구분자가 두 번 연속으로 들어있는 경우 처리 |
There was a problem hiding this comment.
구분자가 두 번 연속으로 들어있는 경우에 어떻게 처리가 되는 지 설명해주실 수 있나요?
| } | ||
|
|
||
| export default App; | ||
| export default App; |
There was a problem hiding this comment.
App 클래스만 export 하셨는데 StringCalculator 클래스를 따로 export 하지 않아도 test파일에서 정상적으로 작동하나요?? 작동한다면 왜 그런지...?(순수하게 몰라서 하는 질문입니닷...)
|
|
||
| static extractCustomDelimiter(numbers) { | ||
| if (numbers.startsWith('//')) { // startsWith : 문자열이 //로 시작하는지 확인 | ||
| const linebreakIndex = numbers.indexOf('\\n'); // indexOf : 찾은 문자 위치 반환, 찾지 못하면 -1 반환 |
There was a problem hiding this comment.
저는 input으로 들어오는 \n을 \n으로 replace하는 방식을 사용했는데
이런 식으로 하면 더 간단하게 사용할 수 있겠네요
참고하겠습니다
| if (numbers.startsWith('//')) { // startsWith : 문자열이 //로 시작하는지 확인 | ||
| const linebreakIndex = numbers.indexOf('\\n'); // indexOf : 찾은 문자 위치 반환, 찾지 못하면 -1 반환 | ||
| if (linebreakIndex !== -1) { | ||
| const customDelimiter = numbers.slice(2, linebreakIndex); // 구분자가 두 번 연속으로 들어있는 경우 처리 |
There was a problem hiding this comment.
혹시 이 부분이 어떤 방식으로 구분자가 두 번 연속으로 들어있는 경우를 처리하는지 알려주실 수 있나요?
코드대로라면 //이후부터 \n직전까지의 모든 부분을 slice하는 것 같은데
그럼 만약 //***\n이면 ***이 모두 포함되는 거 같아서요
| class StringCalculator { | ||
| static add(numbers) { | ||
| const { customDelimiter, input } = this.extractCustomDelimiter(numbers); | ||
| const delimiters = [',', ':', customDelimiter].filter(Boolean); // customDelimiter = null이면 delimiters에 포함되지 않음 |
There was a problem hiding this comment.
filter(Boolen)을 사용하는 방식도 있네요!
이렇게 작성하면 delimiters에 대입하는 코드 작성 없이
간결하게 쓸 수 있겠네요
| Console.print(`결과 : ${result}`); | ||
| } catch(error) { | ||
| Console.print(error.message); | ||
| return Promise.reject(error); |
There was a problem hiding this comment.
async 함수 내에서 예외가 발생하면 Promise객체는 자동으로 reject상태로 변경되기 때문에
여기서 51행이 불필요한 걸루 알고 있어요!
근데 저도 아직 공부가 부족해서 구체적으로는... 설명하기가 어렵네요
이 부분은 제가 더 공부해서 다시 리뷰 작성하러 오겠습니다
No description provided.