Places in code with 6 or more lines that are exactly the same.
Intro
- For duplication, we look at places in code where there are 6 or more lines of code that are exactly the same.
- Before duplication is calculated, the code is cleaned to remove empty lines, comments, and frequently duplicated constructs such as imports.
- You should aim at having as little as possible (<5%) of duplicated code as high-level of duplication can lead to maintenance difficulties, poor factoring, and logical contradictions.
- To learn more about duplications and techniques for eliminating duplication, Sokrates recommends the following resources:
- Avoid Repetition, MartinFlower.com (IEEE Software article)
- Refactoring duplicated code, sourcemaking.com
- Beck Design Rules, MartinFowler.com
- DRY (Don't Repeat Yourself) Principle, Wikipedia
- The Cut-and-Paste Programming Software Development Anti-Pattern, sourcemaking.com
- Code Smells / Duplicated Code, Jeff Atwood, codinghorror.com:
- "Duplicated code is the bane of software development. Stamp out duplication whenever possible. You should always be on the lookout for more subtle cases of near-duplication, too. Don't Repeat Yourself!"
- Reducing Coupling, MartinFlower.com (IEEE Software article):
- "There are several ways to describe coupling, but it boils down to this: If changing one module in a program requires changing another module, then coupling exists. ... Duplication always implies coupling, because changing one piece of duplicate code implies changing the other."
Duplication Overall
- 33% duplication:
- 41,981 cleaned lines of cleaned code (without empty lines, comments, and frequently duplicated constructs such as imports)
- 14,087 duplicated lines
- 7,016 duplicates
Duplication per Extension
Duplication per Component (primary)
Duplication Between Components (50+ lines)
Open 3D force graph...
Show more details on duplication between components...
From Component
--> To Component
Duplicated
Lines
File Pairs
Details
javatests/com/google/gerrit/plugins/codeowners/acceptance (6%)
--> javatests/com/google/gerrit/plugins/codeowners/backend (6%)
1828
61 file pairs
details...
javatests/com/google/gerrit/plugins/codeowners/acceptance (<1%)
--> javatests/com/google/gerrit/plugins/codeowners/restapi (14%)
198
9 file pairs
details...
javatests/com/google/gerrit/plugins/codeowners/backend (<1%)
--> javatests/com/google/gerrit/plugins/codeowners/restapi (11%)
154
1 file pair
details...
| From Component --> To Component |
Duplicated Lines |
File Pairs | Details |
|---|---|---|---|
| javatests/com/google/gerrit/plugins/codeowners/acceptance (6%) --> javatests/com/google/gerrit/plugins/codeowners/backend (6%) |
1828 | 61 file pairs | details... |
| javatests/com/google/gerrit/plugins/codeowners/acceptance (<1%) --> javatests/com/google/gerrit/plugins/codeowners/restapi (14%) |
198 | 9 file pairs | details... |
| javatests/com/google/gerrit/plugins/codeowners/backend (<1%) --> javatests/com/google/gerrit/plugins/codeowners/restapi (11%) |
154 | 1 file pair | details... |
Longest Duplicates
The list of 20 longest duplicates.
| Size | # | Folders | Files | Lines | Code |
|---|---|---|---|---|---|
| 54 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api |  AbstractGetCodeOwnersForPathIT.javaAbstractGetCodeOwnersForPathIT.java |
694:757 (4%)813:876 (4%) | view |
| 54 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | GetCodeOwnerBranchConfigIT.javaGetCodeOwnerProjectConfigIT.java |
246:322 (25%)274:350 (23%) | view |
| 51 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | GetOwnedPathsIT.javaGetOwnedPathsIT.java |
527:590 (8%)643:706 (8%) | view |
| 51 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | GetOwnedPathsIT.javaGetOwnedPathsIT.java |
207:270 (8%)643:706 (8%) | view |
| 51 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | GetOwnedPathsIT.javaGetOwnedPathsIT.java |
207:270 (8%)527:590 (8%) | view |
| 36 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | CodeOwnersPluginProjectConfigSnapshot...CodeOwnersPluginProjectConfigSnapshot... |
212:249 (2%)283:320 (2%) | view |
| 35 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | CodeOwnerConfigValidatorIT.javaCodeOwnerConfigValidatorIT.java |
1900:1937 (1%)1996:2033 (1%) | view |
| 34 | x 2 | javatests/com/google/ger...gins/codeowners/backendjavatests/com/google/ger...gins/codeowners/backend | CodeOwnerApprovalCheckTest.javaCodeOwnerApprovalCheckTest.java |
1243:1284 (2%)1295:1336 (2%) | view |
| 34 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | AbstractGetCodeOwnersForPathIT.javaAbstractGetCodeOwnersForPathIT.java |
548:590 (2%)691:733 (2%) | view |
| 34 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | GetCodeOwnersForPathInBranchIT.javaGetCodeOwnersForPathInChangeIT.java |
86:128 (15%)109:150 (5%) | view |
| 33 | x 2 | javatests/com/google/ger...gins/codeowners/backendjavatests/com/google/ger...gins/codeowners/backend | PathCodeOwnersTest.javaPathCodeOwnersTest.java |
1145:1188 (1%)1836:1879 (1%) | view |
| 33 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | OnCodeOwnerApprovalIT.javaOnCodeOwnerApprovalIT.java |
129:166 (3%)179:216 (3%) | view |
| 33 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | AbstractGetCodeOwnersForPathIT.javaAbstractGetCodeOwnersForPathIT.java |
551:590 (2%)813:852 (2%) | view |
| 32 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | CodeOwnerConfigValidatorIT.javaCodeOwnerConfigValidatorIT.java |
1288:1323 (1%)1353:1388 (1%) | view |
| 32 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | CodeOwnersPluginProjectConfigSnapshot...CodeOwnersPluginProjectConfigSnapshot... |
466:499 (1%)531:564 (1%) | view |
| 29 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | GetCodeOwnerStatusIT.javaGetCodeOwnerStatusIT.java |
227:263 (6%)378:414 (6%) | view |
| 29 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | AbstractGetCodeOwnersForPathIT.javaAbstractGetCodeOwnersForPathIT.java |
1002:1038 (2%)1071:1107 (2%) | view |
| 29 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | GetCodeOwnerStatusIT.javaGetCodeOwnerStatusIT.java |
88:124 (6%)227:263 (6%) | view |
| 29 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | GetCodeOwnerStatusIT.javaGetCodeOwnerStatusIT.java |
88:124 (6%)378:414 (6%) | view |
| 28 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | GetCodeOwnerConfigFilesIT.javaGetCodeOwnerConfigFilesIT.java |
67:97 (8%)366:396 (8%) | view |
Duplicated Units
The list of top 20 duplicated units.
| Size | # | Folders | Files | Lines | Code |
|---|---|---|---|---|---|
| 43 | x 2 | javatests/com/google/ger...gins/codeowners/backendjavatests/com/google/ger...gins/codeowners/backend | PathCodeOwnersTest.javaPathCodeOwnersTest.java |
2202:2256 2259:2313 | view |
| 34 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | CodeOwnerConfigValidatorIT.javaCodeOwnerConfigValidatorIT.java |
1237:1275 1352:1390 | view |
| 17 | x 2 | javatests/com/google/ger...gins/codeowners/backendjavatests/com/google/ger...gins/codeowners/backend | CodeOwnerApprovalCheckWithSelfApprova...CodeOwnerApprovalCheckWithSelfApprova... |
209:232 261:284 | view |
| 14 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | CodeOwnersPluginProjectConfigSnapshot...CodeOwnersPluginProjectConfigSnapshot... |
1200:1214 1218:1232 | view |
| 13 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | GeneralConfigTest.javaGeneralConfigTest.java |
461:474 477:490 | view |
| 13 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | GeneralConfigTest.javaGeneralConfigTest.java |
695:708 724:737 | view |
| 12 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | GeneralConfigTest.javaGeneralConfigTest.java |
401:413 416:428 | view |
| 12 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | GeneralConfigTest.javaGeneralConfigTest.java |
638:650 653:665 | view |
| 12 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | GeneralConfigTest.javaGeneralConfigTest.java |
913:925 982:994 | view |
| 12 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | CodeOwnerConfigValidatorIT.javaCodeOwnerConfigValidatorIT.java |
2637:2649 2651:2663 | view |
| 12 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | CheckCodeOwnerConfigFilesIT.javaCheckCodeOwnerConfigFilesIT.java |
276:289 292:305 | view |
| 11 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | StatusConfigTest.javaStatusConfigTest.java |
151:162 165:176 | view |
| 11 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | CodeOwnerConfigValidatorIT.javaCodeOwnerConfigValidatorIT.java |
144:157 161:174 | view |
| 11 | x 2 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | CodeOwnerConfigValidatorIT.javaCodeOwnerConfigValidatorIT.java |
2597:2608 2610:2621 | view |
| 10 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | GeneralConfigTest.javaGeneralConfigTest.java |
940:950 953:963 | view |
| 10 | x 3 | javatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/apijavatests/com/google/ger...deowners/acceptance/api | CodeOwnerConfigValidatorOnSubmitIT.javaCodeOwnerConfigValidatorIT.javaCheckCodeOwnerConfigFilesInRevisionIT... |
366:377 2559:2570 514:525 | view |
| 10 | x 2 | java/com/google/gerrit/plugins/codeowners/api/impljava/com/google/gerrit/plugins/codeowners/api/impl | CodeOwnerConfigsFactory.javaCodeOwnersFactory.java |
81:91 106:116 | view |
| 9 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | GeneralConfigTest.javaGeneralConfigTest.java |
889:898 901:910 | view |
| 9 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | GeneralConfigTest.javaGeneralConfigTest.java |
1160:1169 1184:1193 | view |
| 8 | x 2 | javatests/com/google/ger...deowners/backend/configjavatests/com/google/ger...deowners/backend/config | GeneralConfigTest.javaGeneralConfigTest.java |
1115:1123 1126:1134 | view |