Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4기] 3주차 과제 입니다. - 제이디 #21

Open
wants to merge 41 commits into
base: main
Choose a base branch
from

Conversation

jadie0319
Copy link

@jadie0319 jadie0319 commented Apr 2, 2023

안녕하세요!
꾸잉님과 함께 개발한 3주차 Wordle 과제 입니다.
페어프로그래밍을 처음 해봤는데 정말 유익하고 좋은 경험이었습니다 ㅎ
감사합니다 ^^

  • 페어 멤버: 꾸잉
  • 사용한 도구: Code With Me + 게더 화면공유
  • 페어 방식: 25분 코딩 5분 휴식 후 교대
  • 회고
    • 페어프로그래밍을 처음 해봤는데 제 생각보다 유익한 경험이었습니다.
    • 단순히 코딩을 함께 하는 것이 아닌 함께 개발, 설계를 진행할 수 있고 실시간으로 피드백을 주고 받을 수 있다는 점이 인상적이었습니다.

koola97620 and others added 30 commits March 28, 2023 22:12
authored-by: 꾸잉
authored-by: 꾸잉
authored-by: 꾸잉
authored-by: 꾸잉
- TODO: print, 파일에서 단어 찾기 로직, input 입력
 - print, input 로직 구현
 - 파일에서 단어 찾기 로직 구현 중
Co-authored-by: 꾸잉
Co-authored-by: 꾸잉
Copy link

@seung-00 seung-00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다 👍
테스트 코드와 도메인 코드들을 보면서 많이 배웠습니다 😄
리뷰들은 의견일 뿐이니 참고만 해주세요~

src/main/java/controller/WordleController.java Outdated Show resolved Hide resolved
src/main/java/controller/WordleController.java Outdated Show resolved Hide resolved
src/main/java/controller/WordleController.java Outdated Show resolved Hide resolved
src/main/java/controller/WordleController.java Outdated Show resolved Hide resolved
src/main/java/domain/Wordle.java Show resolved Hide resolved
src/main/java/domain/Wordles.java Outdated Show resolved Hide resolved
src/main/java/domain/Wordles.java Outdated Show resolved Hide resolved
src/main/java/domain/Wordles.java Outdated Show resolved Hide resolved
src/test/java/domain/WordlesTest.java Show resolved Hide resolved
@jadie0319
Copy link
Author

코드를 정말 꼼꼼하게 잘 봐주셨다는게 느껴지는 좋은 피드백이네요!!
덕분에 부족한 점을 많이 깨달을 수 있었습니다! 감사합니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants