2016-12-27 2 views
0

소프트웨어 시스템에 대한 코드 검토를하고 있습니다. 나는 어떤 사용자의 이야기와 심지어 하위 태스크가 구현 될 때주의를 기울였다. 그들은 대용량의 코드 커밋을 초래하고 대개 수십 개의 소스 코드 파일 (자바 클래스, 자바 스크립트 파일, HTML 등)을 변경하게됩니다.이것은 높은 커플 링의 표시입니까

누군가가 하위 작업을 구현하거나 단일 사용자 스토리에서만 몇 가지 구성 요소가 변경되었습니다. 때때로 커밋은 최대 60 개의 파일을 변경할 수 있습니다.

당신이 수정 확실히 알고있는 경우에 경우를 enter image description here

답변

1

을 설명하기 위해 스크린 샷을 첨부 특정 클래스/메소드, 다음의 결과로, 당신은 다른 모든 장소, 다음 예, 편집했다 코드가 밀접하게 결합되어 있습니다.

here을 참조하십시오.

+0

코드를 검토 중입니다. 변경 한 사람이 아닙니다. 그러나 모든 변경 사항은 하나의 사용자 스토리에 연결됩니다. – Ubaidah

3

코드가 고립되어 변경된 파일의 수를보고 밀접하게 결합되어 있다는 결론을 내릴 수는 없습니다. 사용자 스토리의 컨텍스트에서 이 실제로 변경된 것을 고려하지 마십시오..

클래스와 메소드를 연결하는 것을 고려하면 사용자 스토리를 터치해야하는 클래스 수 사이에 상관 관계가없는 일반 이 있습니다. 이는 실제로 스토리 자체에 달려 있습니다.

변경된 파일 목록을 보면 스토리를 구현하는 동안 특정 클래스에 특정 자동화 된 IDE 리팩터링 (예 : 메소드 이름 바꾸기)을 적용하는 것과 같이 특정 변경 사항이 발생하면 실제로 긴밀한 결합을 암시하게됩니다.

+1

@ Alpar의 의견을 확장하십시오 - High Coupling은 객체 지향 설계와 관련된 문제이며 사용자 스토리와 직결 될 수 있습니다. 즉, 코드 디자인이 반드시 사용자 설 명과 동일한 디자인을 공유하지는 않습니다. 우리 팀에서는 많은 레거시 시스템을 사용하고 있으며 때로는 가장 단순한 이야기 ​​(코드를 10 자리에서 11 자리로 변경)도 여러 가지 다른 프로세스의 Java 변경 사항을 포함하여 여러 가지 다른 부분에 영향을줍니다. –

관련 문제