1. Facts(사실, 객관)
- 작업 commit 반영하기
- 재렌탈 프로세스 목록조회 개선
- AS통계화면 코드 정리
2. Feelings(느낌, 주관)
재렌탈 프로세스 목록 조회를 개선했습니다. 재렌탈로 할인 설정을 할 때, 이전 주문기록을 검색할 수 있는데, 다른 사람의 할인정보를 가져다 쓰지 못하도록 무조건 계약자 정보를 이용해야만 하도록 했습니다. 만약에 계약자 정보를 미리 입력하지 않으면 계약자 정보를 입력하라는 메세지를 날립니다.
또한 작업을 commit했는데, 충돌나는 곳은 서로 코드를 비교해서 update를 받은 후, 모두 내쪽에서 합이 맞춰졌을 때, commit을 쳐서 해결합니다. 언제든지 충돌나는 경우들이 있습니다. 같은 파일 내의 충돌이 다른곳이라면 괜찮지만, 같은 곳이라면 골치가 아픕니다. 이번에 같은 쪽에서 코드변경이 일어났고, 단순히 해결하기 어려워서 먼저 다른 선임의 코드를 override and update로 한 이후에 다시 작업을 해서 commit했습니다.
AS 통계화면 코드 정리의 경우, 불필요한 코드들을 삭제하고 깔끔하게 정리했습니다. 또한 주석을 활용해 어떤 함수이고, 매개변수와 리턴 데이터는 무엇인지 명시합니다. 물론, 매개변수와 리턴 데이터는 확인하면 바로 나오는 정보들이므로 DATASET에 직접 description을 적어서 다른 사람들이 해당 변수명이 어떤 것인지 잘 확인할 수 있도록 했습니다. 어떻게하면 다른 사람들이 잘 코드를 보고 이해할지 신경썼씁니다.
3. Findings (배운 점)
4. Action (구체적 계획)
commit 이전에 update를 꼭 하고, 저장 한 다음 commit을 한다.
'회고' 카테고리의 다른 글
TIL_220210 (0) | 2022.02.11 |
---|---|
TIL_210209 (0) | 2022.02.11 |
TIL_200207 (0) | 2022.02.07 |
20220131~20220206 구체적 계획 (0) | 2022.02.07 |
TIL_220206 (0) | 2022.02.07 |