🚀 1단계 - 문자열 덧셈 계산기 #1069

Aimbe · 2024-11-25T10:48:59Z

안녕하세요 경록님 !�step1은 객체의 책임을 분리하려고 중점을 뒀습니다!
이번 미션 잘 부탁드립니다!

Rok93

정호님 안녕하세요.
로또 미션을 함께하게된 김경록입니다. 🤗

1단계 구현 잘해주셨습니다. 👍👍
몇몇 코멘트 남겨두었으니 확인해서 반영 부탁드릴게요. 🙏
미션 진행하시면서 어려운부분 생기면 언제든지 DM 주세요. 😁

Rok93 · 2024-11-26T00:35:13Z

README.md

+#### 기능 요구사항
+
+ - [ ] 기본 구분자를 이용한 문자열 분리 및 덧셈


기능 구현 목록표를 작성 👍💯

해당 기능이 구현되면 기능 구현 목록표도 같이 업데이트해서 커밋하면 조금 더 살아있는 문서가될 것 같아요. 🤗

앗 요구 명세를 업데이트 하는것을 깜빡했네요! ㅋㅋ 체크하겠습니다 감사합니다!

Rok93 · 2024-11-26T00:39:44Z

src/main/kotlin/calculator/DelimiterStrategy.kt

@@ -0,0 +1,6 @@
+package calculator
+
+interface DelimiterStrategy {


오.. 전략 패턴을 활용하여 기본 구분자(쉼표, 콜론)와 커스텀 구분자를 구현하셨군요? 👍💯
추후 새로운 구분자 정책이 생기더라도 Strategy를 하나 더 구현해주기만하면 쉽게 대응할 수 있겠군요? 😃

Rok93 · 2024-11-26T00:43:45Z

src/main/kotlin/calculator/CustomDelimiterStrategy.kt

+
+class CustomDelimiterStrategy : DelimiterStrategy {
+    override fun supports(text: String): Boolean {
+        return text.startsWith("//") && text.contains("\n")


음.. 이 부분은 의도와 달리 동작할 수 있는 여지가 있어보입니다. 🤔
이 케이스는 가능하면 정규식을 활용해보면 어떨까요? 😃

미션 소개 페이지에서는 커스텀 구분자를 아래와 같이 정의하고있어요.

커스텀 구분자는 문자열 앞부분의 “//”와 “\n” 사이에 위치하는 문자
1글자라고 명시한 것은 아니지만, 문자라거나 예제의 케이스를 생각했을 때, 한 글자가 아닌가 싶어요.

미션 소개 페이지의 입력 예제: //;\n1;2;3

오류가 날 수 있는 케이스1: '//;;\n1;2;3'

오류가 날 수 있는 케이스1: '//1;2;3\n'

저도 사실 구현하면서 예외적인 케이스에 동작을 안하겠는데..? 생각을 하다가
주요 구현 목표가 TDD라 잡고 신경을 덜썼던것 같습니다 ㅋㅋ 정규식 활용해서 한번 진행해보겠습니다!

Rok93 · 2024-11-26T00:45:54Z

src/main/kotlin/calculator/CustomDelimiterStrategy.kt

+        val delimiter = text.substring(2, 3)
+        return text.substring(4)


이 코드를 처음보는 사람이라면 2, 3, 4와 같은 숫자가 어떤 의미를 가지는지 이해하기가 어려울 것 같아요. 🤔
아래의 글을 참고해보시면 좋을 것 같습니다. 😃

의미가 불분명한 매직 넘버를 상수로 선언하라

넵 constant클래스 만들어서 �매직넘버 상수화 했습니다!

Rok93 · 2024-11-26T00:47:59Z

src/main/kotlin/calculator/DefaultDelimiterStrategy.kt

+    }
+
+    override fun parse(text: String): List<Int> {
+        return text.split("[,:]".toRegex())


Pattern과 같은 비싼 객체의 생성을 매번할 필요 없이 상수로 미리 정의해두고 재사용하면 어떨까요? 😃

객체의 생성과 파괴 - 불필요한 객체 생성을 피하라

오 감사합니다! ㅎㅎ 참고하겠습니다

Rok93 · 2024-11-26T00:52:01Z

src/main/kotlin/calculator/NumberParser.kt

+        DefaultDelimiterStrategy()
+    )
+
+    fun parse(text: String): List<Int> {


메서드 파라미터 이름을 조금 더 의미있게 변경해보면 어떨까요? 🤔
문자열 덧셈 계산기라는 도메인에서 입력 받는 값은 무엇인가요? 그 의미가 드러나는 네이밍이면 좋을 것 같아요.

Rok93 · 2024-11-26T00:53:10Z

src/main/kotlin/calculator/NumberParser.kt

+        DefaultDelimiterStrategy()
+    )
+
+    fun parse(text: String): List<Int> {


parse 메서드에 대한 테스트도 작성해보시면 어떨까요? 😃

Rok93 · 2024-11-26T00:53:51Z

src/main/kotlin/calculator/NumberParser.kt

+        return when {
+            text.length == 1 -> listOf(text.toInt())
+            else -> parseWithStrategy(text)
+        }


text의 길이로만 case를 구분하실거라면 아래와 같이 변경할 수도 있을 것 같네요. 😃

Suggested change

return when {

text.length == 1 -> listOf(text.toInt())

else -> parseWithStrategy(text)

}

return when (text.length) {

1 -> listOf(text.toInt())

else -> parseWithStrategy(text)

}

정규식으로 판단 로직 변경하면서 해당 로직들은 없어졌지만, 추후 저런 케이스 있으면 저런식으로 쓰는게 가독성 더 좋아보이네요 ㅎㅎ 감사합니다~

Rok93 · 2024-11-26T00:55:57Z

src/main/kotlin/calculator/NumberParser.kt

+
+    fun parse(text: String): List<Int> {
+        return when {
+            text.length == 1 -> listOf(text.toInt())


text.length == 1의 경우에만 하나의 숫자만 추출하는거군요? 🤔

만약 text가 "11"와 같이 두자리 숫자가 들어오게되면 어떻게 되는걸까요?

Rok93 · 2024-11-26T00:58:08Z

src/main/kotlin/calculator/NumberValidator.kt

@@ -0,0 +1,12 @@
+package calculator
+
+class NumberValidator {


지금처럼 문자열 덧셈 계산기라는 도메인에 필요한 Number 도메인 모델은 일반적인 Number랑 달리 양수여야하는 특징이 있는데요.
원시 값으로는 이런 값을 표현하는데에는 한계가 있는데요. 🥲

원시값을 포장함으로써 도메인에 특화된 도메인 모델을 직접 만들어보시면 어떨까요? 😃

넵 Numbers 도메인 모델에 대한 일급 컬렉션 생성 뒤 책임 분리했습니다!

Aimbe added 7 commits November 23, 2024 14:12

docs: 요구사항 정리

911b647

feat: StringCalculator 기본 구조 생성

1f10c44

feat: TDD로 요구사항 구현

05bef63

feat: single negative number일 경우 케이스 검증

edd2f65

feat: Delimiter 판단시 전략패턴 사용

814a1d9

feat: 객체의 책임 분리

3f08016

fix: StringCalculator 간소화

6db4230

Rok93 requested changes Nov 26, 2024

View reviewed changes

Aimbe added 8 commits November 28, 2024 12:17

fix(review): 커스텀 구분자 변경

ded488a

refactor(review): constants 클래스 생성

246de86

refactor(review): 네이밍 변경

0f168f2

refactor(review): 하나의 숫자로만 되는 문자열 정규식 사용

5a6eb31

test: NumberParser 테스트 추가

e64fbf2

test: 잘못된 커스텀 구분자 추가 테스트

74b77b7

refactor(review): Numbers 도메인 객체 생성

ae8836e

style: lint 적용

4ac13f8

Aimbe requested a review from Rok93 November 28, 2024 04:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 1단계 - 문자열 덧셈 계산기 #1069

🚀 1단계 - 문자열 덧셈 계산기 #1069

Aimbe commented Nov 25, 2024 •

edited

Loading

Rok93 left a comment

Rok93 Nov 26, 2024

Aimbe Nov 26, 2024

Rok93 Nov 26, 2024

Rok93 Nov 26, 2024

Aimbe Nov 26, 2024

Rok93 Nov 26, 2024

Aimbe Nov 28, 2024

Rok93 Nov 26, 2024

Aimbe Nov 26, 2024

Rok93 Nov 26, 2024

Rok93 Nov 26, 2024

Rok93 Nov 26, 2024

Aimbe Nov 28, 2024

Rok93 Nov 26, 2024

Rok93 Nov 26, 2024

Aimbe Nov 28, 2024

		#### 기능 요구사항

		- [ ] 기본 구분자를 이용한 문자열 분리 및 덧셈

		@@ -0,0 +1,6 @@
		package calculator

		interface DelimiterStrategy {

		val delimiter = text.substring(2, 3)
		return text.substring(4)

🚀 1단계 - 문자열 덧셈 계산기 #1069

Are you sure you want to change the base?

🚀 1단계 - 문자열 덧셈 계산기 #1069

Conversation

Aimbe commented Nov 25, 2024 • edited Loading

Rok93 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Aimbe commented Nov 25, 2024 •

edited

Loading