«code-reviews» 태그된 질문

코드 검토를 수행하는 이상적인 방법을 찾지 못했지만 종종 고객이 요구합니다. 각 고객은 다른 방식으로 고객을 수행하는 것으로 보이며 고객 만족도를 느끼지 못했습니다. 코드 검토를 수행하는 가장 효과적인 방법은 무엇입니까? 예를 들면 다음과 같습니다. 한 사람이 품질의 문지기로 간주되어 코드를 검토합니까, 아니면 팀이 표준을 소유합니까? 프로젝터를 사용하여 팀 연습으로 코드를 검토합니까? …

71 code-quality  code-reviews 

나는 최근에 훨씬 더 오래된 개발자들 (약 50 세 이상)과 함께 일하기 시작했습니다. 그들은 시스템이 다운 될 수없는 항공을 다루는 중요한 애플리케이션에서 작업했습니다. 결과적으로 이전 프로그래머는 이런 식으로 코딩하는 경향이 있습니다. 그는 예외가 발생해야하는지 여부를 나타 내기 위해 객체에 부울을 넣는 경향이 있습니다. 예 public class AreaCalculator { AreaCalculator(bool shouldThrowExceptions) …

64 c#  design  object-oriented-design  exceptions  code-reviews 

내가 쓴 오래된 코드를 살펴 보았습니다. 작동하지만 훌륭한 코드는 아닙니다. 나는 당시보다 더 많은 것을 알고 있으므로 그것을 향상시킬 수 있습니다. 현재 프로젝트는 아니지만 현재 작동하는 프로덕션 코드입니다. 과거에 작성한 코드를 되돌아 가서 개선해야 할 책임이 있습니까? 아니면 "파손되지 않은 경우 수정하지 마십시오"라는 올바른 태도입니까? 우리 회사는 몇 년 전에 …

64 code-quality  code-reviews 

나는 보이 스카우트 규칙 에 대한 위대한 신자입니다 . 모듈을 체크 아웃 할 때보 다 항상 깨끗하게 점검하십시오. "원래 작성자가 누구든, 모듈을 개선하기 위해 항상 작은 노력을 기울이면 어떤 결과를 얻었습니까? 결과는 어떻습니까? 우리 모두는이 단순한 규칙을 따랐고, 우리는 소프트웨어 시스템의 끊임없는 열화의 종말을 보게 될 것입니다. 대신 우리의 시스템은 …

55 refactoring  code-reviews 

주니어 프로그래머 인 저의 팀원 중 한 명이 그의 경험 수준에 대해 인상적인 프로그래밍 기술을 가지고 있습니다. 그리고 코드 검토 중에 실수를 지적하지 않고 학습을 강조합니다. 그러나 상급 프로그래머가 상급 프로그래머를위한 코드 검토에 참여해야합니까? 또는 해당 경험이있는 프로그래머 만 코드 검토에 참석해야합니까?

55 teamwork  code-reviews  pair-programming 

코드 검토를위한 지침을 마련하고 있습니다. 아직 공식적인 프로세스가 하나도 없으며 공식화하려고합니다. 우리 팀은 지리적으로 분산되어 있습니다. 우리는 개발을 위해 Visual Studio 2008과 함께 소스 제어에 TFS 를 사용하고 있습니다 (태스크 / 버그 추적 / 프로젝트 관리에도 사용했지만 JIRA 로 마이그레이션했습니다 ). 코드 검토를 수행 할 때 무엇을 찾아야합니까? 이것들은 내가 …

55 code-reviews 

2 주 스프린트가 끝나면 작업에 코드 검토가 있습니다. 검토에서 우리는 작동하고 읽을 수있는 기능을 발견하지만 읽을 수는 있지만 꽤 길고 코드 냄새가 약간 있습니다. 쉬운 리팩터링 작업. 그렇지 않으면 작업이 완료의 정의에 적합합니다. 우리는 두 가지 선택이 있습니다. 이 스프린트에서 티켓이 닫히지 않도록 코드 검토에 실패하고 티켓을 전달할 수 없기 …

53 agile  code-reviews 

SO 에 대한 이 질문 에서는 OP가 생각하는 기능 부러움 코드를 수정하는 방법에 대해 설명합니다 . 이 멋진 문구가 인용 된 또 다른 예는 최근 프로그래머가 여기 에 주어진 대답에 있습니다 . 나는 그 대답에 대한 정보를 묻는 의견 을 들었지만 Q & A를 따르는 프로그래머가 feature-envy 라는 용어의 의미를 …

52 refactoring  code-reviews  clean-code  code-smell 

나는 꽤 오랫동안 오픈 소스 프로젝트를 꽤 많이 진행해 왔으며 그것을 발표하려는 시점에 가까워지고 있습니다. 그러나 나는 독학을 잘하며 내 프로젝트를 적절하게 검토 할 수있는 사람을 모른다. 몇 년 전, 나는 그것을 공개 한 포럼에서 (비판적인 의미로) 찢어 졌던 작은 코드를 공개했다. 코드가 작동했지만 비판은 정확했지만 잔인했습니다. 모든 것에 대한 …

51 open-source  self-improvement  code-reviews  personal-projects 

코드 검토 중에 코드 의 좋은 부분과 그것이 좋은 이유 를 지적하는 것이 중요 합니까? 긍정적 인 피드백은 검토중인 개발자 및 검토에 참여하는 다른 사람들에게 유용 할 수 있습니다. 우리는 온라인 도구를 사용하여 검토를 수행하므로 개발자는 커밋 된 코드에 대한 리뷰를 열 수 있고 다른 사람들은 지정된 기간 (예 : …

48 code-reviews 

고객을위한 새로운 솔루션을 구현하는 컨설턴트 팀의 일원입니다. 나는 클라이언트 측 코드베이스 (React 및 javascript)에 대한 대부분의 코드 검토를 담당합니다. 일부 팀원은 고유 한 코딩 패턴을 사용하여 무작위로 파일을 골라서 스타일을 가진 사람을 누가 알 수 있는지 알 수있었습니다. 예 1 (일회성 인라인 함수) React.createClass({ render: function () { var someFunc …

47 javascript  code-quality  code-reviews  reactjs 

정기적 인 git 및 github을 사용하면 마스터 브랜치로 작업중 인 기능 브랜치의 풀 요청을 간단히 작성하여 코드 검토를 수행 할 수 있습니다. git-flow로 코드 리뷰를 어떻게 수행합니까? "git flow feature finish"와 같은 워크 플로를 통해 코드 검토가 실제로 발생하는 위치와 git-flow 또는 git이 해당 검토를 용이하게하는 방법에 대해 혼란스러워합니다.

43 code-quality  git  code-reviews  github  gitflow 

나는 혼자서 일하는 것 외에는 선택의 여지 가 없으며 , 내 작업을 조사하고, 건강 상태를 확인하고, 아이디어를 브레인 스토밍하고, 모범 사례 등을 논의 할 수있는 적절한 솔루션을 찾을 수 없습니다. 나는 Jeff Atwood의 기사 : 프로그래밍에서 One Is The Loneliest Number을 통해 답을 얻을 것이라고 생각했는데 주제에서 찾을 수있는 가장 …

39 code-reviews  solo-development 

코드 리뷰와 관련하여 서로 다른 방법론을 가진 두 회사에서 근무했습니다. 첫 번째 회사에서는 팀 리더가 코드 검토를 수행했으며 모든 모듈을 완료 한 후에 필요했습니다. 그러나 두 번째 회사에서는 팀 리더가 코드 검토를 수행 할 필요가 없었으며 기능 및 디자인 문제를 확인했습니다. 그래서 혼란스러워합니다. 코드 검토 프로세스가 실제로 필요합니까? 그렇다면 왜 …

39 code-reviews  junior-programmer 

저는 최근에 주니어 개발자로 시작했습니다. 나는 팀에서 가장 경험이 적은 사람들 중 한 명일뿐만 아니라, 남성 지배적 인 환경에서 일하는 모든 종류의 도전과 함께 제공되는 여성이기도합니다. 요즘 내 작품에 대해 너무 많은 불합리한 비판을 받고있는 것처럼 느껴져서 문제가 생겼습니다. 최근에 일어난 일에 대한 예를 들어 보겠습니다. 팀장이 너무 바빠서 내가 …

39 code-quality  self-improvement  teamwork  code-reviews