며칠전에 게시글 올린적 있는데 퍼블리싱 지적좀 많이 받았었음.
좀 개선했는데 피드백좀 해주라..
그리고 이걸로 중소, 스타트업 취업이 가능할까?
http://3.16.138.36:5000/
https://github.com/lexky82/link-developer
며칠전에 게시글 올린적 있는데 퍼블리싱 지적좀 많이 받았었음.
좀 개선했는데 피드백좀 해주라..
그리고 이걸로 중소, 스타트업 취업이 가능할까?
http://3.16.138.36:5000/
https://github.com/lexky82/link-developer
스터디 업로드 누르면 한 몇 초 동안 form 보여다가 signup으로 리디렉션해주네. 이거 회원가입 안 된 경우 form 안 보여주도록 막을 순 없는 거임?
아 그러니깐 그 문제 계속 인지하고있는데 쉽지않네..
페이지 로딩하고 비동기로 아이템 받는식 같은데 초기 로딩이랑 이후 아이템 렌더링되는 사이가 끊김. 로딩 중에 spinner나 아이템 인터페이스 보여주면 더 좋겠다
오.. 맞어 피드백 정말 고맙다..
솔직히 클라이언트 코드에 회사가 좋아하는게 별로 업음
아 그래? 회사에선 보통 어떤걸 원해??
웹은아니지만 앱이니까 앱신입뽑는관점으로 보면, 퍼블리싱 ui는 사실 기획디자인에서 주는대로 찍어내면 그만이야. 더 이뻐봐야 그냥 아 센스가있구나 정도지 개발자로서 큰 메리트가 없어. 니가 모델어떻게짰고 파싱어떻게했고 여기선 왜 이런 디자인패턴을 채용했고 왜 이런설계가 나왔는지 등 현업에선 항상 한줄이라고 근거있는 코드를 선호해 - dc App
생산성이 높고 센스있어봐야 다 손봐줘야하는 코드면 협업에선 항상 마이너스거든. 그래서 개발자도 문서가 중요하고 커뮤니케이션이 중요하단거야. 좀 포커스가 잘못잡힌거같아서그런데 ui때깔 잘나오는거보더 기본cs랑 문서화에 집중하는게 좋을거같다 - dc App
으따 어럽네 피드백 고마워~~!
신입포폴보면 항상 이쁘게만드는데 집착하는게 너무안타까움 그건 그냥 당연히하는거고 전혀 궁금해하지않음 개발자뽑을땨 - dc App
Cs는 알겠는데 문서화라는건 뭐야??
이쁘게만들기위해 어떻게 구현했는지 명확히 전달하는거지 - dc App
너가 배운대로 간단한 약식 uml이라도 있고, 모델은 어떻게짰고 컨트롤러에 어떻게넘겼는지 왜 여기선 mvvm이 필요하고 뷰모델을짰고 레포지토리를 구성했는지 등등 - dc App
아아.. 아예 모르던거네.. 고맙다!
클라이언트는 정말 단순하기도해서 기능구현만 보자면 어떻게든할수있지만, 규모있는곳일수록 다 이거만물어볼거야 꼭 공부해두렴 면접때 반드시물어볼거니까 - dc App
진짜 정성드려서 피드백 해줘서 고맙다~~!
react를 해도 ts를 더 쳐주고 api 클라이언트는 react-query나 swr로 꽤 넘어옴 redux + thunk 조합은 잘 안쓰고 쓰더라도 redux toolkit 이런걸로 구성하는 편인데 옛날방식으로 했음
아 그렇구나.. 다음 프로젝트를 ts 리액트쿼리로해야겠네 피드백 고마웍~~!
API만 해도 컴포넌트 안에 로직이 전부 들어가있는데 일단 이런것부터 좀 정리하는게 좋을 것 같음 훅도 잘못쓰고 있는 부분 꽤 있고
Api정리 참고할게!
API랑 데이터 바인딩이랑 컴포넌트랑 상태관리랑 다 섞여있는것도 좀 그런듯
어떤건 컴포넌트 안에 API를 만들고 어떤건 액션으로 하고 그랬는데 나눈 기준이 어떤거임
로그인 데이터는 다 액션으로 처리하고 나머지는 컴포넌트에..
님 방식이 더 편할수도 있는데 API는 나눠놔야 당장 한눈에 보기도 편하고 업데이트에 대처하거나 테스트 문서화 할때 유리함
오 참고할게!
서버도 컨트롤러에 로직이 다들어 있는 모양세라서 나눠야하는 곳이 좀 많음
진짜 나눌 수 있는건 다 나눠야하는군아.. 진짜 고맙다
서버측 package.json이 프로젝트 최상단에 있는것도 서버 안으로 넣고
아 오키!
프론트만 할거면 리액트 쓰는 회사는 좋아하는 기술들이 다 비슷함 typescript, redux + alpha(보통 saga 아니면 redux-toolkit), emotion이나 styled-components 같은 CSS in JS
타입스크립트 하는게 맞는군아.. Css는 어떤식으로 쓰든 취향차이라고 알고있었는데 아닌가보네..
CSS 따로써도 아무 상관 없음
hook은 그냥 문서 몇번 읽어보면 더 잘쓸 수 있고 컴포넌트랑 로직 분리하고 이런건 설계인데 관련된 포스팅은 여기저기 많으니까 몇개 읽어보고 적용하면 좋을것같음
뭐라고 검색해야 설계부분이 나올까?
react project architecture 대충 이렇게 치면 나올듯
오 정말 고맙다. 모르던거 너무 많이 알았다.
보고 따라하는게 아니라 이해하고 님방식으로 해석해서 안보고 만들줄 알면 댐
그게 진짜 어렵더라고 ㅠㅠ 암튼 너무 고맙다!
commit 메시지도 신경 쓰셈. 규격을 맞추면 좋겠지만, 그게 아니더라도 타이틀과 바디를 나눈다고 해야하나? 첫째줄에 요약하고, 두 번째 줄부터 세부내용 자세히쓰고
아! Conventional Commits 아시는구나! 진.짜.깔.끔.합.니.다 지금 바로 열람 ->
https://www.conventionalcommits.org/en/v1.0.0/
<-
아 커밋 메세지도 신경써야 하는군아 고맙습니다!
아 참고로 글작성자입니다!
혼자 하느거야 아무도 신경 안쓰지만, 포폴은 결국 보여주는거니까
폴더 구조를 좀 더 신경 써봐. src 바로 밑에 파일이 너무 많음.. 컴포넌트 분류도 더 해보고. 아토믹이든 component/container든 도입해보면 좋음 - dc App
또 App.test.js 같은 의미 없는 파일들 지우셈 - dc App
package.json에서 devDependency들 구분해서 놓고 - dc App
오 피드백 고마워~ 확실히 뭔가 다 뭉쳐져 있는 느낌은 여전한가보네