Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
S2uJeong closed a pull request on DoItAlgorithm/algorithm
[FEAT] #2 최수정
### 어려웠던 점 생각보다 구현이 까다로웠다. 그 이유는 조건이 서로 연관 되어 있었기 때문이다. 단어의 첫 글자에서 단축키가 안 나오면 뒤의 글자에서 단축키를 찾아야 하는 로직이 그 예시이다. ### 개선점 - IF 문을 안 쓰고 최대한 함수형 프로그래밍으로 풀어보려 했다. : 점점 코드가 꼬여 결국은 IF를 사용함 - dict...S2uJeong pushed 2 commits to main S2uJeong/python-algorithm
S2uJeong opened an issue on team-abcdedu/abcdedu-backend
[FIX] post의 내용 글자 수 증가
## 📝 개요 현재 post의 내용이 1,000자로 스키마 설정 되어 있어 1,000자 이상 기입 시 오류가 발생함. 이에 따른 문의가 접수되어 글자 제한을 2,000자로 늘린다. ## ✅ TODO - [v] DB 스키마 변경 ## 💬 기타S2uJeong created a review comment on a pull request on team-abcdedu/abcdedu-backend
네 생각해보니 workbook을 만드는 것은 굳이 인터페이스에 만들기보다 각 구현체에서 필요할 때 private 함수로 두는게 나을 것 같아 삭제 했습니다. 리뷰 감사합니다!
S2uJeong created a review comment on a pull request on team-abcdedu/abcdedu-backend
의존성 주입에 대해서 **응집도**에 대한것만 중점으로 생각하다 보니, 유틸에 관련한 것을 의존성에 넣는게 맞나 하는 생각이 들어서 의존관계를 설정해주지 않았습니다. 다만 **구현체를 직접 생성하면 전략패턴을 사용한 이유가 없어지는 점** 에 대해서 생각을 못한것 같아 좀 더 생각해서 고쳐보겠습니다!
S2uJeong created a review comment on a pull request on team-abcdedu/abcdedu-backend
인터페이스 오버라이드 메서드이고, 인터페이스 메서드의 기본 접근제어자가 public 이여서 변경이 따로 불가한 것으로 알고 있습니다.. 혹시 다른 내용 있다면 알려주시면 반영하겠습니다!
S2uJeong pushed 1 commit to feat/homework-206 team-abcdedu/abcdedu-backend
- [refactor] #206 response에 전달하는 string 값들 값 객체 안의 필드로 변경 후 private final 추가 54256a0
S2uJeong pushed 1 commit to feat/homework-206 team-abcdedu/abcdedu-backend
- [refactor] #206 애노테이션으로 변환 가능한 코드 애노테이션으로 변환 f759b54