중복없는 dice 구현

[curry-ing]

6 중복없는 dice구현을 위한 작업입니다.

눈팅만 하다 영원히 참여 못할 것 같아, WIP라벨 시작도 끊어 볼 겸 일단 한 줄이라도 작성 해봤습니다. (이렇게 하면 되나요;;)

추가 구현되는대로 커밋 예정입니다.

[jwChung]

제목 [WIP] 머릿말은 제거하셔도 되지 않을까요?

[myeesan]

현재 n과 d 이상의 커맨드가 확정 되지 않은 상태이고, n의 필요성은 명확한 상태입니다. 두 요구사항을 구현 한 상태에서 코드를 이해하기는 case문을 활용하는 것이 맞다고 생각됩니다.

의견을 교환하기 위한 글을 쓰거나 찾아보도록 하겠습니다. :)

[jwChung]

@masunghoon 저도 @myeesan님 말씀에 동의합니다. 제가 괜히 일을 복잡게 만든 것 같습니다. 현재 의도하시는 방향으로 구현하시는 것이 좋다는 생각입니다.

[curry-ing]

몇 가지 테스트케이스 추가 후 WIP 떼도록 하겠습니다~

[jwChung]

질문 및 의견 몇 가지 남겼습니다. @masunghoon 님 의견을 듣기 위해 -1 로 표시해두었습니다.

[jwChung]

@masunghoon 위 제 의견을 다시 읽어 보니 좀 강한 어조가 느껴지는군요. 불필요한 인삿말 (eg. thanks) 군더더기를 제거하고 제 생각을 담백한게 적은 것 뿐입니다. 모든 문장 끝에 ^^에 있다고 보시면 됩니다 ㅎ

[myeesan]

저는 작동에 결함이 없는 만큼 merge 하는게 좋다고 생각됩니다.

1. 현재 이 브랜치에 의존성을 가진 이슈들이 몇 개 있습니다.
2. @jwChung 님께서 제기해 주신 의문은 충분히 가치 있다고 생각합니다. 그래서, 현재 쓰레드에서 계속 진행 되는 것 보다, merge 후에 새로운 이슈를 생성해서 논의를 이어나간 다면 좀 더 집중 할 수 있지 않을까 합니다.

[hkjlee109]

Great work! @masunghoon Hope you sleep well tonight. I am oooooofff!

[curry-ing]

@jwChung 어조는 전혀 문제되지 않습니다! 저기 사이사이에 ^^있으면 더 이상할 듯... ㅎㅎ

[jwChung]

@myeesan

저는 작동에 결함이 없는 만큼 merge 하는게 좋다고 생각됩니다.

저도 같은 생각입니다. 그러나 위에서 제가 드린 의견이 반영되든 아니든 종결된 뒤에 -1 이모지를 제거하겠습니다.

1. 현재 이 브랜치에 의존성을 가진 이슈들이 몇 개 있습니다.
2. @jwChung 님께서 제기해 주신 의문은 충분히 가치 있다고 생각합니다. 그래서, 현재 쓰레드에서 계속 진행 되는 것 보다, merge 후에 새로운 이슈를 생성해서 논의를 이어나간 다면 좀 더 집중 할 수 있지 않을까 합니다.

네 다른 이슈를 통해서 처리되는 것도 방법이겠습니다. 다만 이런 사항에 대해 동의가 있어야 되고 가능하다면 이슈까지 만들어 놓고 병합되는 것이 바람직하다고 생각합니다.