«code-reviews» 태그된 질문

StackOverflow에서 다시 게시를 요청했습니다. 개발 시간이 매우 짧은 소규모 개발 시간에 일하고 있습니다. 우리는 업무 결과에 중요하지만 매일 사용하지 않는 도구를 개발합니다. 나는 프로그래머로서의 배경을 가진 팀의 유일한 사람입니다. 내 문제는 1 년 이상 동안 트렁크로 병합하기 전에 코드 검토를 추진하고 있다는 것입니다. 모두가 이것에 동의했지만 여전히 검토 된 것은 …

10 code-quality  collaboration  code-reviews 

닫은. 이 질문은 주제에 맞지 않습니다 . 현재 답변을받지 않습니다. 이 질문을 개선하고 싶습니까? Software Engineering Stack Exchange에 대한 주제가 되도록 질문을 업데이트하십시오 . 휴일 오년 전에 . 익숙하지 않은 코드를 수동으로 검사 (검토 또는 수정) 할 때 세 가지 옵션이있는 것 같습니다. 하향식 읽기 파일 이름이 보인다 방법을 기초로 …

9 tools  code-reviews  source-code  maintenance 

정적 코드 분석과 코드 검토의 차이점이 무엇인지 알고 싶었습니다. 이 두 가지는 각각 어떻게 이루어 집니까? 보다 구체적으로, 오늘날 PHP의 코드 검토 / 정적 분석을 위해 사용할 수있는 도구는 무엇입니까? 또한 모든 언어의 코드 검토를위한 유용한 도구에 대해 알고 싶습니다.

9 code-quality  terminology  code-reviews  quality 

클래스를 만들고 객체를 반환하는 대신 여러 항목을 반환하려는 경우 공용 메서드에서 해시 테이블을 반환 할 때의 디자인 문제는 무엇입니까? 문제가 있다면 어떤 상황에서 그렇게하는 것이 합리적입니까? 이 질문에 대한 답은 언어가 동적인지 아닌지에 따라 어떻게 변합니까? 편집 : 이것은 키가 일정하고 데이터가 아닌 코드의 일부임을 분명히하기 위해입니다. 우리가 일반적으로 클래스를 …

9 design  object-oriented  code-reviews 

CSS, JS 및 HTML을 검토하기위한 지침을 작성하라는 요청을 받았습니다. JS에 대한 코딩 지침이 있지만 HTML 및 CSS에 대해서는 잘 모릅니다. JS를 검토하기 위해 나는 반드시 그 지침을 따르고 언급 할 것입니다. 그러나 CSS와 HTML은 어떻습니까? 논리적 오류 및 들여 쓰기 문제 외에도 마크 업 및 CSS를 검토 할 때 확인해야 …

9 javascript  html  code-reviews  css 

오늘 직장에서 동료 중 한 명이 내 코드를 검토하고 set-only 속성을 제거하고 대신 메서드를 사용하도록 제안했습니다. 우리 둘 다 다른 일로 바빴 기 때문에 그는 Property Design"프레임 워크 디자인 지침"책 의 섹션 을 보라고 말했습니다 . 이 책에서 작가는 다음과 같이 피했다. 게터보다 접근성이 더 넓은 세터가있는 속성 그리고 왜 …

9 design  code-quality  code-reviews  code-smell