[문자열 덧셈 계산기] 이홍진 미션 제출합니다. #126

hongmyeoun · 2024-10-21T13:22:56Z

기능 목록

입력

출력

결과 : 구분자를 기준으로 분리된 각 숫자의 합

오류처리

입력이 잘못될 경우
- 문자열이 입력되지 않은 경우
- 숫자가 양수가 아닌 경우
- case 1) 커스텀 구분자가 없을 경우
  - 입력된 문자열에 숫자와 쉼표(,), 콜론(:)을 제외한 다른 문자가 있을 경우
- case 2) 커스텀 구분자가 있을 경우
  - 커스텀 구분자와 숫자, 쉼표, 콜론을 제외한 다른 문자가 있을 경우

테스트

오류처리들에 관한 내용
- 음수입력
- 구분자 이외의 양수가 아닌 입력
예상되는 결과
- 커스텀 구분자와 기본 구분자(콤마, 콜론)의 혼용

1. Functional Requirements 2. Functional Lists & Checklist

1. 기능 요구 사항에 실행 결과 예시 추가

1. 입력 메세지 "덧셈할 문자열을 입력해 주세요" 추가 2. 쉼표 또는 콜론을 구분자로 갖는 문자열에 대한 덧셈 연산기능 추가 3. 구분자를 기준으로 각 숫자의 합을 출력 todo 1. 커스텀 구분자 확인 2. 오류처리 3. 테스트 4. model, view, controller로 구분

1. 입력 메세지, 구분자 확인, 쉼표, 콜론 2. 출력 결과 todo 1. 커스텀 구분자 확인 2. 오류처리 3. 테스트 4. model, view, controller로 구분

1. 커스텀 구분자를 subStringAfter, subStringBefore로 구분, 이는 정규식을 사용하는 것보다 직관적이라고 생각하여 정함 todo 1. 오류처리 2. 테스트 3. model, view, controller로 구분

1. 입력된 문자열에 숫자가 양수가 아닌경우 "양수로 구성된 문자열을 입력해 주세요."라는 오류메세지와 함께 어플을 종료시킨다 todo 1. 오류처리 2. 테스트 3. model, view, controller로 구분

1. case1과 case2에 경우 둘다 정규식을 이용해 숫자와 구분자가 아닌 경우를 모두 찾아낸 다음 IllegalArgumentException를 발생시켜 어플을 종료시킴 2. 문자열이 입력되지 않은 경우는 없다. readeLine은 String! 타입이기 때문, 빈값이면 0을 반환한다. todo 1. 테스트 2. model, view, controller로 구분

1. 커스텀 구분자 사용시 콤마와 콜론이 같이 혼용될 수 있는지 확인하는 테스트 추가 2. 예외 테스트 음수포함에 에러메세지 확인 3. 예외 테스트 입력오류로 구분자가 아닌 값이 들어왔을때 해당 문자열을 반환하는 에러메세지 확인 4. 커스텀 구분자 사용시 예외 테스트 입력오류로 구분자가 아닌 값이 들어왔을때 해당 문자열을 반환하는 에러메세지 확인 5. BUILD SUCCESSFUL in 0s 확인 todo 1. model, view, controller로 구분

1. 오류처리, 테스트 진행완료 todo 1. model, view, controller로 구분

1. 커스텀 구분자와 콤마, 쉼표를 혼용할때 커스텀 구분자가 가장 뒤, 콤마가 가장 앞에 사용될 경우 앱이 종료되던 버그 수정, 문자열을 분할한 후 또다시 확인하는 방식으로 변경 2. 구분자가 아닌 문자열 찾기 오류 수정, 구분자를 오류를 확인할때 음수를 나타내는 "-"도 정규식에 포함 todo 1. model, view, controller로 구분

1. Application main 함수에서 실행했던 모든 함수를 MVC 패턴의 아키텍처로 변경 2. Model에서는 구분자로 구분된 숫자리스트를 만드는 함수와 그를 계산하는 함수, 오류메세지를 띄우는 함수로 구분 3. View에서는 입력 메세지와 출력 메세지를 보여줌 4. Controller에서는 View의 input을 받아 Model에서 result값을 View로 보내줘 UI를 갱신함

UiHyeon-Kim

MVC 패턴을 적용한 코드 잘 봤습니다 고생하셨습니다~

UiHyeon-Kim · 2024-10-21T16:06:23Z

src/main/kotlin/calculator/Application.kt

+import calculator.controller.CalculatorController
+import calculator.model.CalculatorModel
+import calculator.view.CalculatorView
+
 fun main() {


MVC 패턴이 어떤 식으로 구현되는지 궁금했는데 각각 잘 나누어 구분해주신 덕에 이해하기 쉬웠습니다!

감사합니다!

UiHyeon-Kim · 2024-10-21T16:13:15Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+    private fun customSeparator(input: String, separator: String): List<String> {
+        val numList = mutableListOf<String>()
+
+        for (n in input.split(separator)) {


이 for문 안에 중첩 for문과 if문이 너무 많은데 조금 더 간단하게 할 수 있지 않을까 아쉬움이 남네요

네...그냥 정규식을 이용해서 간단히 구분하는게 훨씬 간단하고 편한거 같네요

단순 코드 라인 수를 줄이기 위해서라면 numList와for loop대신 map을 사용해보는 것도 좋은 방법일 것 같아요!

for문 안에 input.split같은 연산 처리보다 상위에서 처리하여 할당한 변수를 적는 것이 가독성이 좋을 것 같아요.

jhseo12

아키텍처 패턴에 대해 전혀 몰랐는데, 이번 기회에 MVC 패턴을 알 수 있어서 좋았습니다.

jhseo12 · 2024-10-21T16:51:42Z

src/main/kotlin/calculator/view/CalculatorView.kt

+
+class CalculatorView {
+    fun getInput(): String {
+        println("덧셈할 문자열을 입력해 주세요.")


출력문을 상수화하면 더 보기 좋을거같아요!

m6z1

1주 차 너무 고생많으셨습니다!!
이번 주차도 같이 파이팅해욥 ㅎㅎ🙌

m6z1 · 2024-10-22T03:14:13Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+    }
+
+    fun sumWithCustomSeparator(input: String): Int {
+        val newSeparator = input.substringAfter("//").substringBefore("\\n")


오 이렇게도 나눌 수가 있군요!! 배워갑니다 👍

subStringAfter와 substsringBefore가 있었군요!
새로운 지식 배워갑니다 😃

호오 저도 배워갑니다!🙇🏻‍♂️

오! 저런 메서드가 있군요! 저도 배워갑니다!

저도 배워갑니다!!

m6z1 · 2024-10-22T03:15:26Z

src/main/kotlin/calculator/controller/CalculatorController.kt

+) {
+    fun runCalculator() {
+        val input = view.getInput()
+        val result = if (input.contains("//")) model.sumWithCustomSeparator(input) else model.sumWithDefaultSeparator(input)


여기서 //로 분기를 나누신 의도가 궁금합니다!! 👀
커스텀 구분자와 기본 구분자를 나누기 위함이라면 \n은 포함시키지 않으신 이유가 어떤 것일까요!!?

커스텀 구분자에만 '//'이 사용되기 때문에 넣었습니다!

m6z1 · 2024-10-22T03:17:06Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+        return numList
+    }
+
+    private fun commaSeparator(s: String, numList: MutableList<String>) {


매개변수 네이밍을 s가 아닌 다른 이름으로 짓는 것이 어떨까요!? 어떤 매개변수인지 알기 어려울 것 같습니다!

넵 참고하겠습니다.
단축키로 함수를 빼다보니 매개변수에 대해 수정을 안했나보네요ㅠ

hwannow

1주차 수고하셨습니다!!!! 2주차도 같이 파이팅해요!!

hwannow · 2024-10-22T03:40:23Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+            errorNotPositiveNumber(s)
+            numList.add(s)
+        }
+    }


colon 이랑 comma separator를 따로 구분하신 이유가 있나요?

반복되어서 많이 사용될까봐 함수로 만들어 봤었던것 같습니다.

moondev03

1주차 미션 고생하셨습니다!!

모델의 함수명이 명사형으로 되어있는 것 같습니다.
동사 + 명사 형태로 수정하는게 가독성 면에서 더 좋을 것 같아요!

moondev03 · 2024-10-22T03:26:28Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+    }
+
+    fun sumWithCustomSeparator(input: String): Int {
+        val newSeparator = input.substringAfter("//").substringBefore("\\n")


subStringAfter와 substsringBefore가 있었군요!
새로운 지식 배워갑니다 😃

moondev03 · 2024-10-22T03:29:59Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+    private fun customSeparator(input: String, separator: String): List<String> {
+        val numList = mutableListOf<String>()
+
+        for (n in input.split(separator)) {


단순 코드 라인 수를 줄이기 위해서라면 numList와for loop대신 map을 사용해보는 것도 좋은 방법일 것 같아요!

minseok0416

1주차 진행하시느라 고생하셨습니다!!
리뷰하면서 MVC패턴 적용을 느껴볼 수 있어 도움이 많이 되었습니다.

다른 분들 리뷰처럼 정규식과 매개변수 네이밍이 조금 아쉽네요.
이번주도 화이팅 하세요!!!!

minseok0416 · 2024-10-22T03:39:04Z

README.md

MVC 패턴을 적용시켜서 일까요? 설계를 잘하셨다는 느낌이 드네요.

minseok0416 · 2024-10-22T03:51:49Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+        return numList.sumOf { it.toInt() }
+    }
+
+    private fun defaultSeparator(input: String): List<String> {


함수명이 직관적이지 않아보입니다. 기능에 맞게 splitByDefaultSeparator 등으로 네이밍하면 어떨까요?

네이밍에 신경더 쓰겠습니다. 감사합니다!

minseok0416 · 2024-10-22T03:58:48Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+    fun sumWithDefaultSeparator(input: String): Int {
+        errorNotPositiveNumber(input)
+        val numList = defaultSeparator(input)
+        return numList.sumOf { it.toInt() }
+    }


제가 놓쳤을 수도 있지만, Defalut케이스에서는 반환하기 전에 입력이 숫자가 아닌지 체크하는 부분이 없는 듯 합니다.

그렇네요. 정확한 지적 감사합니다!!

ijh1298

1주차 고생 많으셨습니다! 코드 리뷰 감사합니다. 화이팅!😋

ijh1298 · 2024-10-22T04:45:06Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+            throw IllegalArgumentException("${invalidChar}는 구분자가 아닙니다.")
+        }
+    }
+}


Separator 마다 따로 메소드를 정의해놓았는데, 중복을 줄일 수 있지 않을까요?

넵 저도 아쉬운점이라 생각하고 있습니다. 복기하면서 최대한 줄여보겠습니다! 감사합니다

ijh1298 · 2024-10-22T04:45:58Z

src/test/kotlin/calculator/ApplicationTest.kt

+    fun `예외 테스트 입력오류2`() {
+        assertSimpleTest {
+            assertThrows<IllegalArgumentException> { runException("//;\\nb1,2:a3;4c") }
+            assertThat(output().contains("bac는 구분자가 아닙니다."))
        }
    }



JUnit 단위 테스트 하는 방법을 배워갈 수 있었습니다😊

wjdrjs00

1주차 고생하셨습니다~~!
남은 주차도 같이 힘내요!!!😤

wjdrjs00 · 2024-10-22T05:17:37Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+    }
+
+    fun sumWithCustomSeparator(input: String): Int {
+        val newSeparator = input.substringAfter("//").substringBefore("\\n")


호오 저도 배워갑니다!🙇🏻‍♂️

wjdrjs00 · 2024-10-22T05:26:06Z

src/main/kotlin/calculator/model/CalculatorModel.kt

제 개인적인 생각이긴 한데 지금 구현하신 CalculatorModel에서 모든 도메인 로직을 수행하고 있다는 생각이 듭니다.
좀더 명확성을 위해 계산기는 계산에 대한 책임만 수행하고, 구분자와 피연산자에 대해서는 분리해서 각각의 책임을 가지는 객체를 선언함으로써 책임을 분산시키는 방법도 가능하지 않을까? 하는 생각을 남겨봅니다!

오오 좋은 의견 감사합니다!
모델에서도 각각 책임을 나눠 객체를 만드는 방법 배워갑니다!

SANDY-9 · 2024-10-22T02:42:34Z

src/main/kotlin/calculator/controller/CalculatorController.kt

+) {
+    fun runCalculator() {
+        val input = view.getInput()
+        val result = if (input.contains("//")) model.sumWithCustomSeparator(input) else model.sumWithDefaultSeparator(input)


"input.contains("//")"라는 조건문을 사용하셨는데
컨트롤러 영역에서 이 조건문의 의미가 커스텀 구분자가 존재할 수 있는지 확인하는 조건문이라는 것을 계산기의 요구사항에 대한 배경지식이 없다면 처음 코드를 읽는 자가 이해하기가 어려울 수 있다고 생각합니다.

if (input.contains("//")) model.sumWithCustomSeparator(input) else model.sumWithDefaultSeparator(input) 이 부분에 대한 기능을 하나의 함수로 만들거나,

val containCustomPrefix = input.contains("//") 이런식으로 조건문을 변수로 만들어 조건문에 활용하면

가독성이 더 좋아질 것 같습니다.

좋은 지적 감사합니다!
이해하기 쉽게 가독성을 높이는 법도 중요하단 것 배워갑니다.

jbrunoo

1 주차 고생하셨습니다~~
리뷰가 이미 많아서 리뷰 없는 부분들 위주로 봤어요 !ㅎㅎ

jbrunoo · 2024-10-22T07:21:32Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+    private fun customSeparator(input: String, separator: String): List<String> {
+        val numList = mutableListOf<String>()
+
+        for (n in input.split(separator)) {


for문 안에 input.split같은 연산 처리보다 상위에서 처리하여 할당한 변수를 적는 것이 가독성이 좋을 것 같아요.

jbrunoo · 2024-10-22T07:23:00Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+        for (n in input.split(separator)) {
+            if (n.contains(":") || n.contains(",")) {
+                for (s in n.split(":")) {
+                    if (s.contains(",")) {


split안에 구분자들을 여러개 넣어 한 번에 처리하면 효율적이고 코드가 개선될 것 같아요.
또는 구분자들을 관리하는 배열이나 리스트를 활용하면 좋을 것 같아요.

맞아요 split 구분자를 여러개 사용할 수 있더라고요.. 다음에 활용해 보겠습니다!

jbrunoo · 2024-10-22T07:34:50Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+
+    private fun errorNotPositiveNumber(num: String) {
+        if (num.toInt() < 1) {
+            throw IllegalArgumentException("양수로 구성된 문자열을 입력해 주세요.")


예외 처리 기능과 테스트 코드 등을 읽어보았으나 0 값을 처리하는 부분을 따로 확인을 못한 것 같아서 질문합니다.
입력 값이 ""가 0으로 처리되는 부분이 어디인가요?
"0" 입력 시 예외가 발생하는것인지 궁금합니다.

그렇네요
코드 구조를 변경하다보니 함수 리펙토링을 진행했고, 그 과정에서 빈값이 들어왔을 때 0값 처리를 해주던 연산을 고려하지 않았네요
테스트 케이스에도 넣지 않아서 알아차리기 어려웠습니다!

Model에 sumWithDefaultSeparator() 함수 첫줄에

if (input.isEmpty()) return 0

를 추가해 해결했습니다.
감사합니다

CHUIWWWW · 2024-10-22T12:12:01Z

1주차동안 고생 많으셨습니다.
덕분에 MVC 패턴에 대해 새롭게 배우게 되었습니다.
model, view, controller로 나누어 각 클래스별로 어떤 역할을 하는지 명확하게 나타낼 수 있어 좋은 방법인 것 같습니다.
다음 미션도 파이팅입니다!!!

nuyhhyun

아이고 제가 리뷰를 남겨놓고 펜딩 상태로 뒀네요 죄송합니다 지금서 알아차려서 늦게나마 올려둡니다!
3주차도 화이팅입니다!

nuyhhyun · 2024-10-22T15:00:24Z

src/main/kotlin/calculator/Application.kt

+    if (input.isNotEmpty()) {
+        for (n in input.split(",")) {
+            if (n.contains(":")) {
+                for (s in n.split(":")) {


임시 변수로 보이긴 하지만 n이나 s보다는 좀 더 명확한 변수명을 사용해보시는 건 어떨까 싶습니다!

nuyhhyun · 2024-10-22T15:07:41Z

src/main/kotlin/calculator/Application.kt

-    if (invalidChar.isNotEmpty()) {
-        throw IllegalArgumentException("${invalidChar}는 구분자가 아닙니다.")
-    }
+    controller.runCalculator()


app에서 Model이랑 View 불러와서 Controller로 주는 건 생각 못했는데 배워갑니다!

nuyhhyun · 2024-10-22T15:10:54Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+
+    fun sumWithCustomSeparator(input: String): Int {
+        val newSeparator = input.substringAfter("//").substringBefore("\\n")
+        val newInput = input.substringAfter("\\n")


전 냅다 인덱스로 구별했는데 이런 방법이 있었네요!! bb

nuyhhyun · 2024-10-22T15:12:34Z

src/main/kotlin/calculator/model/CalculatorModel.kt

+    }
+
+    fun sumWithCustomSeparator(input: String): Int {
+        val newSeparator = input.substringAfter("//").substringBefore("\\n")


제가 못찾은 걸 수도 있지만 이렇게 앞뒤를 확인해서 잘랐을 때 커스텀 구분자가 여러 개일 경우에도 체크하는 게 있을까요?

해당 경우는 생각하지 못했네요!
subStringAfter와 subStringBefore는 문자열의 앞에서부터 작동하기 때문에, //;\n//?\n과 같이 커스텀 구분자가 두 개가 있다면 ; 값만 체크하고 뒤에 값은 입력오류가 나겠네요.
while과 같은 중복문으로 커스텀 구분자를 여러번 체크하는 방법도 있겠네요!

hongmyeoun added 11 commits October 17, 2024 18:23

docs: README.md draft

7ff2b70

1. Functional Requirements 2. Functional Lists & Checklist

docs: README 실행 결과 예시 추가

33317a0

1. 기능 요구 사항에 실행 결과 예시 추가

docs: 완료작업 체크리스트 정리

9262b77

1. 입력 메세지, 구분자 확인, 쉼표, 콜론 2. 출력 결과 todo 1. 커스텀 구분자 확인 2. 오류처리 3. 테스트 4. model, view, controller로 구분

feat: 커스텀 구분자 확인

f68b282

1. 커스텀 구분자를 subStringAfter, subStringBefore로 구분, 이는 정규식을 사용하는 것보다 직관적이라고 생각하여 정함 todo 1. 오류처리 2. 테스트 3. model, view, controller로 구분

docs: 오류처리 및 테스트 진행완료 체크

3b25d38

1. 오류처리, 테스트 진행완료 todo 1. model, view, controller로 구분

UiHyeon-Kim reviewed Oct 21, 2024

View reviewed changes

jhseo12 reviewed Oct 21, 2024

View reviewed changes

m6z1 reviewed Oct 22, 2024

View reviewed changes

hwannow reviewed Oct 22, 2024

View reviewed changes

moondev03 reviewed Oct 22, 2024

View reviewed changes

minseok0416 reviewed Oct 22, 2024

View reviewed changes

ijh1298 reviewed Oct 22, 2024

View reviewed changes

wjdrjs00 reviewed Oct 22, 2024

View reviewed changes

SANDY-9 reviewed Oct 22, 2024

View reviewed changes

jbrunoo reviewed Oct 22, 2024

View reviewed changes

nuyhhyun reviewed Oct 29, 2024

View reviewed changes

[문자열 덧셈 계산기] 이홍진 미션 제출합니다. #126

Are you sure you want to change the base?

[문자열 덧셈 계산기] 이홍진 미션 제출합니다. #126

Conversation

hongmyeoun commented Oct 21, 2024

기능 목록

입력

출력

오류처리

테스트

UiHyeon-Kim 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

jhseo12 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

m6z1 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

TH-97 Oct 25, 2024 • edited Loading

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

hwannow left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

moondev03 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

minseok0416 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

ijh1298 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

wjdrjs00 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

jbrunoo 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

CHUIWWWW commented Oct 22, 2024

nuyhhyun 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

TH-97 Oct 25, 2024 •

edited

Loading