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
- 6% duplication:
- 12,158 cleaned lines of cleaned code (without empty lines, comments, and frequently duplicated constructs such as imports)
- 828 duplicated lines
- 88 duplicates
Duplication per Extension
Duplication per Component (primary)
Longest Duplicates
The list of 20 longest duplicates.
| Size | # | Folders | Files | Lines | Code |
|---|---|---|---|---|---|
| 19 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...shard/storeops/recovery |  AddShardOperation.javaAttachShardOperation.java |
258:306 (20%)256:304 (21%) | view |
| 16 | x 2 | elastic-db-tools/src/mai...b/shard/storeops/mapperelastic-db-tools/src/mai...b/shard/storeops/mapper | AddMappingOperation.javaRemoveMappingOperation.java |
281:318 (17%)298:335 (16%) | view |
| 16 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...icdb/shard/storeops/map | AddShardOperation.javaRemoveShardOperation.java |
270:307 (17%)250:287 (19%) | view |
| 15 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...shard/storeops/recovery | RemoveShardOperation.javaAttachShardOperation.java |
250:286 (18%)268:304 (17%) | view |
| 13 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...shard/storeops/recovery | AddShardOperation.javaAttachShardOperation.java |
185:221 (14%)197:233 (14%) | view |
| 12 | x 2 | elastic-db-tools/src/mai...ure/elasticdb/shard/mapelastic-db-tools/src/mai...ure/elasticdb/shard/map | ShardMap.javaShardMap.java |
180:196 (5%)238:254 (5%) | view |
| 11 | x 2 | elastic-db-tools/src/mai...b/shard/storeops/mapperelastic-db-tools/src/mai...b/shard/storeops/mapper | AddMappingOperation.javaRemoveMappingOperation.java |
98:124 (11%)109:135 (11%) | view |
| 10 | x 2 | elastic-db-tools/src/mai...re/elasticdb/shard/baseelastic-db-tools/src/mai...re/elasticdb/shard/base | ValidationUtils.javaValidationUtils.java |
68:84 (12%)173:189 (12%) | view |
| 10 | x 2 | elastic-db-tools/src/mai...hard/storeops/mapmangerelastic-db-tools/src/mai...hard/storeops/mapmanger | AddShardMapGlobalOperation.javaRemoveShardMapGlobalOperation.java |
47:70 (38%)47:70 (30%) | view |
| 10 | x 2 | elastic-db-tools/src/mai...cdb/shard/storeops/baseelastic-db-tools/src/mai...cdb/shard/storeops/base | StoreOperationErrorHandler.javaStoreOperationErrorHandler.java |
82:91 (3%)178:187 (3%) | view |
| 10 | x 2 | elastic-db-tools/src/mai...shard/storeops/recoveryelastic-db-tools/src/mai...shard/storeops/recovery | CheckShardLocalOperation.javaGetShardsLocalOperation.java |
41:66 (55%)42:67 (52%) | view |
| 9 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...icdb/shard/storeops/map | AddShardOperation.javaRemoveShardOperation.java |
119:145 (9%)117:144 (10%) | view |
| 9 | x 2 | elastic-db-tools/src/mai...re/elasticdb/shard/baseelastic-db-tools/src/mai...re/elasticdb/shard/base | PointMappingCreationInfo.javaRangeMappingCreationInfo.java |
64:80 (31%)66:82 (28%) | view |
| 9 | x 2 | elastic-db-tools/src/mai...lasticdb/shard/recoveryelastic-db-tools/src/mai...lasticdb/shard/recovery | RecoveryManager.javaRecoveryManager.java |
254:267 (2%)291:304 (2%) | view |
| 9 | x 2 | elastic-db-tools/src/mai...re/elasticdb/shard/baseelastic-db-tools/src/mai...re/elasticdb/shard/base | PointMapping.javaRangeMapping.java |
207:230 (8%)216:239 (8%) | view |
| 9 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...icdb/shard/storeops/map | AddShardOperation.javaRemoveShardOperation.java |
197:221 (9%)185:209 (10%) | view |
| 9 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...shard/storeops/recovery | RemoveShardOperation.javaAttachShardOperation.java |
185:209 (10%)209:233 (10%) | view |
| 9 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...shard/storeops/recovery | AddShardOperation.javaAttachShardOperation.java |
97:123 (9%)100:126 (10%) | view |
| 9 | x 2 | elastic-db-tools/src/mai...reops/schemainformationelastic-db-tools/src/mai...reops/schemainformation | AddShardingSchemaInfoGlobalOperation....UpdateShardingSchemaInfoGlobalOperati... |
41:63 (42%)41:63 (42%) | view |
| 9 | x 2 | elastic-db-tools/src/mai...re/elasticdb/shard/baseelastic-db-tools/src/mai...re/elasticdb/shard/base | PointMapping.javaRangeMapping.java |
239:260 (8%)249:270 (8%) | view |
Duplicated Units
The list of top 11 duplicated units.
| Size | # | Folders | Files | Lines | Code |
|---|---|---|---|---|---|
| 16 | x 2 | elastic-db-tools/src/mai...re/elasticdb/shard/baseelastic-db-tools/src/mai...re/elasticdb/shard/base | PointMapping.javaRangeMapping.java |
257:277 267:287 | view |
| 14 | x 2 | elastic-db-tools/src/mai...re/elasticdb/shard/baseelastic-db-tools/src/mai...re/elasticdb/shard/base | PointMapping.javaRangeMapping.java |
228:245 237:255 | view |
| 7 | x 2 | elastic-db-tools/src/mai...cdb/shard/storeops/baseelastic-db-tools/src/mai...cdb/shard/storeops/base | StoreOperationRequestBuilder.javaStoreOperationRequestBuilder.java |
873:880 893:900 | view |
| 7 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...icdb/shard/storeops/map | AddShardOperation.javaRemoveShardOperation.java |
120:135 118:134 | view |
| 7 | x 4 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...shard/storeops/recovery | AddShardOperation.javaAddShardOperation.javaRemoveShardOperation.javaAttachShardOperation.java |
198:211 271:284 186:199 210:223 | view |
| 7 | x 2 | elastic-db-tools/src/mai...icdb/shard/storeops/mapelastic-db-tools/src/mai...icdb/shard/storeops/map | UpdateShardOperation.javaUpdateShardOperation.java |
200:213 267:280 | view |
| 7 | x 3 | elastic-db-tools/src/mai...b/shard/storeops/mapperelastic-db-tools/src/mai...b/shard/storeops/mapperelastic-db-tools/src/mai...b/shard/storeops/mapper | AddMappingOperation.javaAddMappingOperation.javaRemoveMappingOperation.java |
202:215 282:295 219:232 | view |
| 7 | x 2 | elastic-db-tools/src/mai...b/shard/storeops/mapperelastic-db-tools/src/mai...b/shard/storeops/mapper | ReplaceMappingsOperation.javaReplaceMappingsOperation.java |
225:238 325:338 | view |
| 7 | x 2 | elastic-db-tools/src/mai...b/shard/storeops/mapperelastic-db-tools/src/mai...b/shard/storeops/mapper | UpdateMappingOperation.javaUpdateMappingOperation.java |
325:338 453:466 | view |
| 6 | x 2 | elastic-db-tools/src/mai...hard/storeops/mapmangerelastic-db-tools/src/mai...hard/storeops/mapmanger | AddShardMapGlobalOperation.javaRemoveShardMapGlobalOperation.java |
46:52 46:52 | view |
| 6 | x 2 | elastic-db-tools/src/mai...b/shard/storeops/mapperelastic-db-tools/src/mai...b/shard/storeops/mapper | AddMappingOperation.javaRemoveMappingOperation.java |
107:113 118:124 | view |