«code-reviews» 태그된 질문

이 태그는 코드 검토 및 코드 연습에 대한 질문입니다. 기존의 작동 코드에 대한 리뷰는 http://codereview.stackexchange.com을 참조하십시오.

4
풀 요청에서 TODO를 처리하는 방법은 무엇입니까?
이 질문은 Software Engineering Stack Exchange에서 답변을 얻을 수 있기 때문에 Software Quality Assurance & Testing Stack Exchange에서 마이그레이션 되었습니다. 작년에 이주 . 풀 요청의 변경 사항을 검토 할 때 종종 "TODO"메모가 포함 된 주석을 우연히 발견 할 수 있습니다. 문제를 해결하는 데 사용되는 솔루션은 개선 될 수 있지만 훨씬 …
10
응용 프로그램이 잘못된 코드베이스에 구축되어 있음을 증명하는 방법은 무엇입니까?
나는 현재 내 직장에서 일했던 일부 개발자가 만든 시스템을 검토하고 있습니다. 이 시스템은 사용자의 관점에서 잘 작동하지만 코드 검토를 할 때는 완전히 엉망입니다. 응용 프로그램이 구축되는 방식은 사용량이 크게 증가하지는 않지만 향후 업데이트를 보류하지 않을 것이라고 확신합니다. 문제는 그것이 얼마나 나쁜지 알지만 상사에게는 그렇지 않다는 것입니다. 관리자에게 실제로 문제를 확인하고 …
4
고독한 프로그래머를위한 응용 프로그램 / 코드 검토?
'합리적인 가격으로'적용 할 수있는 좋은 기술적 조언을 제공하고 제공하는 서비스가 있습니까? 많은 프로젝트에서 나는 일반적으로 유일한 개발자이고 때로는 효율성, 더 나은 MVC 상호 작용 등을 위해 일부 작업을 개선해야한다고 생각합니다. 실제로 할 수있는 전문적인 서비스가 있다면 좋을 것입니다 그런 리뷰를 할 것입니다
6
최소 시간 검토를 위해 코드를 어떻게 문서화합니까? [닫은]
휴무 . 이 질문은 더 집중되어야 합니다. 현재 답변을받지 않습니다. 이 질문을 개선하고 싶습니까? 이 게시물 을 편집 하여 한 가지 문제에만 집중할 수 있도록 질문을 업데이트하십시오 . 작년에 문을 닫았 습니다 . 몇 달 후에 코드를 다시 읽고 탐색해야 할 필요성이 최소화되도록 코드를 문서화하고 싶습니다. 소스 코드와 외부, 시퀀스 …
3
기능 브랜치에서 마스터로 병합하기 전에 코드 검토 전략
나와 팀은 기능 분기 (git와 함께)를 사용합니다. 마스터로 병합하기 전에 코드 검토에 가장 적합한 전략이 무엇인지 궁금합니다. 마스터에서 새 브랜치를 체크 아웃하고 fb_ # 1이라고 부를 수 있습니다. 몇 번 커밋하고 다시 마스터로 병합하고 싶습니다. 내가 병합하기 전에 누군가가 코드를 검토해야합니다 이제 두 가지 가능성이 있습니다. 1 일 나는 병합 …
3
Github가없는 Github 유사 "풀 요청"
저는 데이터 감도로 인해 클라우드에 데이터를 저장하지 않는 금융 기관의 분석가로 일하고 있습니다. 그러나 팀에서 코드 관리에 Git을 사용하는 데 성공했습니다. 자체 서버에서 Github와 같은 풀 요청을 구현할 수있는 방법이 있는지 궁금했습니다. 내가 관심있는 특정 기능은 실제로 지정된 브랜치로 병합하지 않고 의견에 대한 변경 세트를 제출하는 기능 입니다. 나는 (1) …
20 git  code-reviews 
17
기본 규칙을 따르도록 프로그래머를 설득하는 방법
이 질문은 Software Engineering Stack Exchange에서 응답 할 수 있기 때문에 Stack Overflow에서 마이그레이션 되었습니다. 8 년 전에 이주했습니다 . 프로그래머가 자주 따라야한다고 요구하는 몇 가지 규칙이 있습니다. 그들은 코드를 작성하고 작동하면 작업이 완료됩니다. 가장 기본적인 규칙은 다음과 같습니다. 커밋 변경 View 또는 Controllers에서 모델 문제를 작성하지 않음 하드 코딩 …
9
코드 검토가 좋은 것으로 보스를 설득하는 팁 [닫기]
휴무 . 이 질문은 더 집중되어야 합니다. 현재 답변을받지 않습니다. 이 질문을 개선하고 싶습니까? 이 게시물 을 편집 하여 한 가지 문제에만 집중할 수 있도록 질문을 업데이트하십시오 . 휴일 3 년 전 . 프로젝트에서 거의 협력하지 않는 개발자가 여러 명인 가상의 회사에서 근무하고 있으며 보스가 코드 검토가 시간과 비용이 가치가 …
4
경험이 적은 프로그래머의 코드를 수정하려면 어떻게해야합니까?
약간의 배경 지식 : 저는 10 명의 직원으로 구성된 두 명의 프로그래머 중 한 명입니다 (나머지는 예술가와 경영진입니다). 우리 둘은 흐름을 잘 만들고 필요한 프로젝트를 개발하는 데 필요한 모든 코딩 작업을 수행합니다. 나는 약 4 년 동안 프로그래밍을 해왔으며, 이것이 그의 첫 번째 "실제"직업이다. 우리는 일반적으로 언제든지 다른 프로젝트를 진행하고 …
19 code-reviews  bug 
5
표준 코드 검토에는 무엇이 포함됩니까?
우리 회사에서는 코드를 작성하는 사람이 어떤 기능을 구현하고 어떤 종류의 버그를 수정했는지에 대한 이메일 토론입니다. 그리고 우편을받는 검토자는 코드를 검토하고 품질 및 코드 편집 방법에 대해 의견을 나눕니다. 표준 코드 검토에는 무엇이 포함됩니까?
5
코드가 "나쁜"상태 일 때 검토 자로 무엇을해야합니까?
몇 달 전에 필수 코드 검토가 도입 된 잘못 설계되고 과도하게 엔지니어링 된 프로젝트에서 일하고 있습니다. 새로운 기능을 구현하는 실질적인 코드를 검토하라는 요청을 받았습니다. 다른 코드베이스와 같은 결함이 있습니다. 나는 그 결함이 예를 들어 새로운 코드에 들어간다는 것을 이해합니다. 호환성을 유지하면서 잘못 설계된 클래스에서 상속하거나 잘못 설계된 인터페이스를 구현해야하므로 코드베이스의 …
3
제대로 작동하더라도 노인 및 상사와 함께 프로그램을 검토하는 것이 좋습니까?
내 회사에서 프로젝트를 진행하기 전에 상사는 내 선배에게 나와 다른 팀원이 작성한 프로그램을 검토하도록 요청하거나 때로는 상사도 검토를 위해 우리와 함께 앉아 있습니다. 지식을 얻는 좋은 방법이라고 생각하지만 때로는 프로그램이 제대로 작동하면 검토 후에도 작동하지 않으므로 프로그램을 다시 살펴 봐야합니다. 그들은 검토가 프로그램과 쿼리 실행을 최적화하는 데 도움이되지만 실제 프로그램 …
12
모든 코드를 검토해야합니까?
현재 개발 프로세스를 수정 중이며 커밋 피어의 100 %를 검토 해야하는지 궁금합니다. 코드 검토에 대한 경험은 무엇입니까? 당신은 그들에게 "많은"시간을 보내는 경향이 있습니까 (예 : 하루에 1/2 시간), 아니면 최대 5/10 분 동안 탈지합니까? 하루 / 주 / 스프린트 / 프로젝트 당 고정 된 시간이 있습니까? 가장 중요한 것은 코드의 …
2
수은 코드 검토를위한 권장 프로세스
우리는 일반적으로 Perforce와 SmartBear의 Code Collaborator를 사용 Big Corp했으며 이제 특정 프로젝트에 Mercurial을 사용할 것입니다. Code Collaborator는 Mercurial (버전 5를 사용 중)을 지원하며 최선의 시간 (커밋 / 푸시 투 서버 중) 프로세스가 코드 검토에 가장 적합한 시간을 결정하려고합니다. 감사
5
릴리스 열 동안 효율적인 코드 검토를 수행하는 방법은 무엇입니까?
릴리스 마감일은 내일이며, 동료가 마침내이 릴리스에 중요한 작업을 완료했으며, 프로젝트 관리자가 어깨 너머에 서서 최종적으로 빌드하도록 압력을 가하며 검토하는 동안 동료 코드의 결함을 발견했습니다. 중요한 것은 아니지만 내일 출시되지 않으면 놓치지 않을 것입니다. 그리고 상황을 악화시키기 위해서는 최대한 빨리 끝내야 할 일이 있습니다. 그래서, 당신은 무엇을합니까? 압박에도 불구하고 이의를 제기합니까? …
당사 사이트를 사용함과 동시에 당사의 쿠키 정책과 개인정보 보호정책을 읽고 이해하였음을 인정하는 것으로 간주합니다.
Licensed under cc by-sa 3.0 with attribution required.