Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
taek2222 opened a pull request on woowacourse/java-lotto
[2단계 - 로또 리팩토링] 비타(오연택) 미션 제출합니다.
시간내어 리뷰해 주셔서 정말 감사합니다. 1차 리뷰 과정에서 남겨주신 코멘트와 공통 피드백 기반으로 학습하고, 다른 테스트를 추가했습니다! 🙇🏻 --- ### QnA Q. 그동안 가독성 향상을 위해 메서드 분리를 실천하고 있지만, 어디까지의 분리가 적절한 지 궁금합니다. 현재 `Lotto` 클래스도 `Step1` 코드에서 개선해 메서드 분...taek2222 created a review comment on a pull request on woowacourse/java-lotto
### Q. 다시 설계를 뒤엎는 빈도가 늘어났고, 동시에 작업 시간도 늘어났습니다. 이를 개선하기 위한 조언이 있을까요? > A. 이 부분은 앞으로 우테코에서 여러 프로젝트를 만들어보면서 직접 경험하며 배워보면 좋을 것 같아요. 유닛테스트, 리팩터링 등등... 이런 개념들을 잘 활용하다보면 노하우가 생길거예요 :) 답변 정말 감사합니다. 말씀...
taek2222 created a review comment on a pull request on woowacourse/java-lotto
사실 클래스에 final 명시는 페어인 '시소'의 드라이버 역할 수행 중 삽입된 거 같습니다. 같이 지켜보면서 해당 키워드를 인지하지 못해 사용의 의도를 파악하지 못했습니다. 다음부터는 페어의 행동에 집중을 더 해야겠습니다. 😢 > **final 클래스는 어떤 특징이 있을까요?** 의 답변은 현재의 지식으로는 "상속을 할 수 없어 불...
taek2222 created a review comment on a pull request on woowacourse/java-lotto
현재 저는 프로그래밍 하면서 가장 중요하게 생각하고, 앞으로 더 발전하고 싶은 부분이 `네이밍`입니다. 이번 피드백으로 또 새로운 지식을 알게 되어서 정말 감사합니다! ☺️ 다시 고민해보며 올바른 컨벤션을 유지한 네이밍을 해보겠습니다! 🔥
taek2222 created a review comment on a pull request on woowacourse/java-lotto
맞습니다!.. 이 부분도 컨벤션에 적용했는데 위에 알려주신 변경 파일 정리랑 응용해서 설정하겠습니다! (지금까지.. 수동으로 정렬해 이렇게 빠진 부분이 존재한 거 같아요! 😢)
taek2222 created a review comment on a pull request on woowacourse/java-lotto
답변을 듣고 생각했을 때 > 현재는 하나의 작은 미션이지만, 점점 커지면 거대해질 거 같다 라고 육성으로 나올정도로 공감되는 답변이 되었습니다. 😲 `Global` 보다는 중복으로 사용되는 도메인에 직접적으로 적용하는 게 좋을 거 같다고 생각했습니다. 감사합니다. ☺️
taek2222 created a review comment on a pull request on woowacourse/java-lotto
아하! 저도 다시 한 번 코드를 짚어볼 때, 1. "'[' 이게 'NUMBER_OPEN_BRACE'" 로 상수 이름으로 관리하는 구나" 2. " 'NUMBER_OPEN_BRACE' 이 상수 이름이 의미하는 바가 뭐지?" 라고 두 번 생각하게 되는 거 같아요! 좋은 답변 감사합니다! ☺️
taek2222 created a review comment on a pull request on woowacourse/java-lotto
감사합니다! 최대한 여러 값에 따라 다른 결과값이 나오면 파라미터 테스트가 우선적으로 생각이 났습니다! 🙇🏻
taek2222 created a review comment on a pull request on woowacourse/java-lotto
정말 좋은 꿀팁 감사합니다. 바로 적용해 추후 Step 2에서는 적용된 모습으로 올리겠습니다. ☺️
taek2222 created a review comment on a pull request on woowacourse/java-lotto
'Model과 View의 분리'를 목표로 했습니다. 서로 완전 독립된 형태로 보관하고, 영향을 최소화 하려고 했습니다! 🤔 프리코스 과정속에서 깨달은 점이지만, DTO 사용하면 전달되어야 하는 값이 정해져서 편리한 점도 있었습니다!
taek2222 created a review comment on a pull request on woowacourse/java-lotto
감사합니다! 추후 요구 사항을 더욱 깔끔하게 수정할 예정인데, 현재 문서를 참고해 주시면 감사하겠습니다!
taek2222 closed an issue on BUTTERFLY-CODE-NEAR/Butterfly_BE
[refactor] 상품 설명 이미지 리팩토링
## 📄 설명 현재 한개로 고정되어 있는 상품 설명 이미지 등록 개수를 여러개로 변경 ## ✅ TO-DO - [ ] image 저장 테이블 생성 - [ ] 이미지를 여러개 받을 수 있도록 MultipartFile을 List로 변경 ## 🌱 예상기간 2월 06일 ~ 2월 10일taek2222 pushed 5 commits to main BUTTERFLY-CODE-NEAR/Butterfly_BE
- feat(descriptionImage): 상품 설명 이미지 테이블 생성 2d1b7fc
- refactor(descriptionImage): 상품설명 이미지 여러개로 변경 a2d0acf
- refactor(admin): 상품설명 이미지 여러개로 변경 f221fa9
- Merge branch 'main' of https://github.com/BUTTERFLY-CODE-NEAR/Butterfly_BE into 413-refactor-상품-설명-이미지-리팩토링 ee18ecd
- Merge pull request #414 from BUTTERFLY-CODE-NEAR/413-refactor-상품-설명-이미지-리팩토링 413 refactor 상품 설명 이미지 리팩토링 029bb8d