Giter VIP home page Giter VIP logo

projectssg's People

Contributors

jihun-24k avatar

Watchers

 avatar

projectssg's Issues

리뷰 남깁니다.

좋은점

  • post 에 equals와 hascode를 구현해주신점이 인상깊습니다. 해당 hashcode는 어떻게 응용되는 것인가요? post를 찾을 때 더 빠르게 찾기 위함인가요? 명언을 계속 listup 해야하니 굉장히 좋은 것 같습니다!!
  • TDD로 구현하신 부분이 또 인상깊고, 직접 달아주신 커밋메시지도 또 배워갑니다!!

아쉬운 점은 없습니다.! 감사합니다!! 고생하셨습니다!

리뷰 남겨요

좋은점

코드를 최대한 간결하게 작성하려고 하는게 느껴질정도로 깔끔하네요!
Util클래스에 내부 클래스(Pair)를 두어서 관리하는 부분이 좋아보여서 저도 적용해보려고합니다

아쉬운점

PostController에서 if else문 보다 if문에 return으로 던져주면 좀더 가동성이 좋은 코드가 될거 같습니다.
저도 우테코 코드 규칙을 보니깐 else예약어를 사용하지말라는 규칙을 본적이 있어서 찾아보니
else예약어를 사용안하면 객체지향적인 코드 작성으로 우리는 재사용성이 높고 유지 보수하기 좋은 코드를 가질 수 있다고하네요.
포스팅 한번 보시면 도움이 될거 같아서 밑에 링크 남겨놨습니다!!

잘봤습니다. 감사합니다. :)

링크 : https://tecoble.techcourse.co.kr/post/2020-07-29-dont-use-else/

리뷰 남깁니다!!!

좋은점

  1. 간결하고 깔끔한 코드가 너무 보기 좋았습니다!
  2. 각 계층마다 TDD를 진행하셔서 어떤 방향으로 코드를 짜셨는지에 대해 알게되어 좋았습니다!

아쉬운점

.

고생 많으셨습니다!!!

강민성_리뷰

각 계층 별로 단위 테스트 구현을 잘 하신 것 같습니다! 또한 작성하신 코드들 모두 깔끔하게 작성하셔서 코드리뷰하기 편했으며, wisesaying 대신 post라는 도메인으로 코드를 작성하셔서 좀 더 편하게 코드를 볼 수 있었습니다.

혹시, 데이터의 영속성을 위한 테스트 코드는 잘 작성하신 것 같은데, 실제로 프로그램을 실행시켰을 때도 영속성 처리가 잘 되었는지 궁금합니다. 제가 코드를 보았을 때는 영속성 처리가 하는 부분이 없었는데 혹시나 제가 놓친 부분이 있을까 해서 조심스럽게 여쭈어봅니다!

그리고 컨트롤러, 레포지토리, 도메인 별로 각 계층을 잘 나누셨는데, 서비스 계층도 추가하면 좀 더 결합도 낮은 구조를 만드실 수 있을 거 같습니다.

너무 수고많으셨습니다.

리뷰 남깁니다!

좋은점

리팩토링 하셔서 전체적인 코드가 간결해서 해석하기 좋았습니다!
참고해서 제 코드도 구조를 바꿔볼게요!

안좋은 점

없습니다.

지훈님의 리뷰 봤습니다 클래스명을 대문자로 하는 것 명심할게요!
항상 정성스러운 리뷰 감사합니다

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.