«code-reviews» 태그된 질문

소스 제어 저장소에서 검토 된 소스 코드를 관리하는 가장 좋은 방법은 무엇입니까? 체크인하기 전에 소스 코드가 검토 프로세스를 거쳐야합니까, 아니면 코드가 커밋 된 후 코드 검토가 수행되어야합니까? 코드가 저장소에 체크인 된 후 검토가 발생하면 어떻게 추적해야합니까?

12 version-control  code-reviews  svn 

내가 일하는 회사는 모든 코드를 커밋하기 전에 다른 개발자가 검토해야합니다. 다른 개발자가 너무 바빠서 리뷰를하기에 너무 바빠서, 특히 매우 긴 경우, 우리 팀원들은 종종 좌절합니다. 적시에 코드 검토를 수행하도록 다른 개발자에게 인센티브를 제공하는 방법은 무엇입니까? (우리는 git-svn을 사용하므로 검토를 기다리는 동안 코딩을 계속할 수 있습니다. 그러나 코드를 커밋하기 전에 오랜 …

12 code-reviews 

소프트웨어 개발에서 일반적인 시나리오는 다른 사람의 코드를 검토하는 코드입니다. 이를위한 일반적인 도구는 풀 요청을 여는 것입니다. 내 질문은 리뷰에서 문제가 발견되면 변경해야한다는 것입니다. 별도로 커밋 (새 커밋) 또는 기존 커밋을 수정해야합니다 (공유 브랜치에서 기록을 다시 쓰는 것이 나쁜 소식이기 때문에 아무도 이전 커밋에서 분기되지 않는다고 가정). 첫 번째 시나리오에서는 커밋 …

11 git  code-reviews  repository 

폐쇄되었습니다 . 이 질문은 더 집중되어야 합니다. 현재 답변을받지 않습니다. 이 질문을 개선하고 싶습니까? 이 게시물 을 편집 하여 한 가지 문제에만 집중할 수 있도록 질문을 업데이트하십시오 . 휴일 육년 전 . 우리는 두 가지 스프린트에 대해 각 커밋에 대해 필수 코드 검토를 시도하고 있습니다. 저자가 아닌 적어도 한 사람이 …

11 teamwork  code-reviews 

스키마 변경뿐만 아니라 모든 것을 의미합니다. 기본 키의 간단한 SELECT조차도 모든 개발자가 코드에서 추출하여 EXPLAIN 출력, 세부 정보로 제출 한 모든 명령문에 대한 DBA 검토없이 다른 개발자 (상황에 따라)에서 코드를 검토했지만 프로덕션에 들어갈 수 없습니다. 얼마나 자주 호출되는지 등 그러한 환경에 있었다면, 순이익이나 개발에 끌리는 것으로 나타 났습니까? 수년 동안 …

11 code-reviews  dba 

때때로 계산량이 많은 코드의 1 %가 가장 많은 종류의 저수준 최적화를 필요로합니다. 예를 들어 비디오 처리, 이미지 처리 및 모든 종류의 신호 처리가 있습니다. 목표는 코드를 유지 보수 할 수없고 새로운 개발자가 제거하기 쉽도록 최적화 기술을 문서화하고 가르치는 것 입니다. (*) (*) 미래의 일부 CPU에서는 특정 최적화가 완전히 쓸모 없을 …

11 documentation  code-reviews  optimization 

먼저 코드 검토 프로세스를 굳게 믿고 항상 다른 사람이 내 코드를 검토하기를 원합니다. 내 질문은 실제로 다른 사람을 위해 코드 검토를 더 잘 수행 할 수있는 방법에 중점을 둡니다. 코드 검토를 수행하려면 기존 코드의 작동 방식에 대한 지식과 현지 표준이 무엇인지 잘 알고 있어야합니다. 여전히 다른 사람들을 위해 코드를 충분히 …

11 code-reviews 

우리는 3 가지 방법 만 필요한 거대한 인터페이스를 제공하는 타사 서비스에 의존합니다. 또한 인터페이스가 자주 변경됩니다 ... 프로젝트의 클래스에서 인터페이스를 포장하기로 결정하고 필요한 메소드 만 공개하기로 결정했습니다. 그러나 반환 값을 어떻게 처리 해야하는지 잘 모르겠습니다 ... 인터페이스는 유형의 객체를 반환합니다 Storage. 우리는 내부적 StorageModel으로 우리의 내부 표현 인 타입 을 …

11 c#  design-patterns  programming-practices  code-reviews  wrapper 

현재 팀에 합류 한 주니어 개발자 코드 일부를 검토 중입니다.이 검토 결과를 어떻게 제공해야하는지 궁금합니다. 코드를 직접 수정해야합니까? 검토 프로세스에 대한 피드백을 제공하고 지침에 따라 수정을 수행해야합니까? 그렇다면 어떻게 피드백을 보내거나 특정 템플릿 문서를 작성하여 보내거나 나중에 코드 파일에서 문제를 확인하여 나중에 확인할 수있는 소프트웨어가 있습니까? (Visual Studio를 사용하고 있습니다). …

10 code-reviews  process 

아래 샘플을 포함시킨 끔찍한 코드를 물려 받았습니다. 이 특정 안티 패턴의 이름이 있습니까? 리팩토링에 대한 몇 가지 권장 사항은 무엇입니까? // 0=Need to log in / present username and password // 2=Already logged in // 3=Inactive User found // 4=Valid User found-establish their session // 5=Valid User found with password …

10 coding-style  code-quality  coding-standards  code-reviews  code-smell 

저는 현재 제가 일하는 회사에서 React를 소개하려고 노력하고 있습니다. 주제에 대한 과정을 한 유일한 사람은 누가 코드가 '모범 사례'를 따르고 올바르게 구현되었는지 검토를 요청할 수 있습니까?

10 code-reviews 

폐쇄되었습니다 . 이 질문은 의견 기반 입니다. 현재 답변을받지 않습니다. 이 질문을 개선하고 싶습니까? 이 게시물 을 편집 하여 사실과 인용으로 답변 할 수 있도록 질문을 업데이트하십시오 . 휴일 3 년 전 . 가설 적으로, 내 경험이 .NET에있을 때 새로운 PHP 개발자 입장에서 누군가와 인터뷰를한다면, 그들이 제공 한 코드 샘플이 …

10 code-quality  interview  code-reviews 

상용 개발에서 코드 검토가 크게 요구된다는 것을 알고 있습니다. 그러나 코드 검토는 오픈 소스 소프트웨어에서 사용됩니까, 아니면 신뢰를 기반으로합니까? 그렇다면 어떻게 수행됩니까? [지연된 커밋, "프리 커밋 환경", 패치를 다른 개발자에게 보낼 수있는 도구가 있습니까?] 코드 검토를 사용하는 프로젝트가 있습니까? 내가 이해 한 바에 따르면 리눅스 커널은 커미터에 대한 신뢰를 바탕으로합니다. …

10 open-source  code-reviews  source-code 

단위 테스트 전후에 코드 검토를 수행 할시기에 대해 동료와 논의 중입니다. 모범 사례는 무엇입니까? 우리가 고려해야 할 몇 가지 요소가 더있을 수 있습니다. 코드 변경 크기-큰 변경은 코드 검토에서 더 많은 변경이 발생 함을 의미합니다. 이러한 변경 사항이 UT보다 코드 검토 이전 인 경우보다 더 큰 경우 대부분의 UT를 다시 …

10 language-agnostic  code-reviews 

나는 여기에 몇 가지 아이디어를 찾고 있습니다. 코드 검토를 어떻게 수행 하고 코드를 검토해야합니까? 라는 기사를 읽었습니다 . 장점은 무엇입니까? 매우 유익하지만 아래 질문에 대해 더 명확해야합니다. 내 질문은 대상 개발자는 개발자가 코드를 검토하기 전에 통합 할 수있는 모범 사례를 제안 할 수 있습니다. 현재 저는 다음과 같은 방법을 연습합니다 …

10 c#  code-reviews