173-139-209

if (defaultSaveOptions == null) { defaultSaveOptions = new HashMap<Object, Object>(); } return defaultSaveOptions;
if (result == null) { result = new ArrayList<EStructuralFeature>(); } result.add(eStructuralFeature); break;

Clone fragments detected by clone detection tool
File path: /emf-2.4.1/src/org/eclipse/emf/ecore/xmi/impl/XMLResourceImpl.java |  | File path: /emf-2.4.1/src/org/eclipse/emf/ecore/util/BasicExtendedMetaData.java
Method name: `Map getDefaultSaveOptions()` |  | Method name: `List getElements(EClass)`
Number of AST nodes: 3 |  | Number of AST nodes: 4

1

    if (defaultSaveOptions == null)↵

1

    if (result == null)↵

2

        {↵

2

              {↵

3

          defaultSaveOptions = new HashMap<Object, Object>();↵

3

                result = new ArrayList<EStructuralFeature>();↵

4

        }↵

4

              }↵

5

        return defaultSaveOptions↵

5

              result.add(eStructuralFeature);↵

6

    ;

6

              break;

See real code fragment |  | See real code fragment

Summary
Number of common nesting structure subtrees | 1
Number of refactorable cases | 0
Number of non-refactorable cases | 1
Time elapsed for finding largest common nesting structure subtrees (ms) | 0.2
Clones location | Clones are in different classes
Number of node comparisons | 6

1. {Non-refactorable}

   Mapping Summary
   Number of mapped statements | 1
   Number of unmapped statements in the first code fragment | 2
   Number of unmapped statements in the second code fragment | 3
   Time elapsed for statement mapping (ms) | 36.2
   Clone type | Type 3

   Mapped Statements
   ID | Statement |  | ID | Statement
   --- | --- | --- | --- | ---

   1

       if (defaultSaveOptions == null)

   1 | if (defaultSaveOptions == null)
   9 | if (result == null)

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   defaultSaveOptions | result | VARIABLE_NAME_MISMATCH
   java.util.Map | java.util.List | VARIABLE_TYPE_MISMATCH

   Preondition Violations

   Type java.util.Map<java.lang.Object,java.lang.Object> of variable defaultSaveOptions does not match with type java.util.List<org.eclipse.emf.ecore.EStructuralFeature> of variable result
   - Make classes java.util.Map and java.util.List extend a common superclass

   9

       if (result == null)

   2

       defaultSaveOptions = new HashMap<Object, Object>();

   2 | defaultSaveOptions = new HashMap<Object, Object>();

   Preondition Violations
   Unmatched statement defaultSaveOptions=new HashMap<Object,Object>(); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted

   10 | result = new ArrayList<EStructuralFeature>();

   Preondition Violations
   Unmatched statement result=new ArrayList<EStructuralFeature>(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted

   10

       result = new ArrayList<EStructuralFeature>();

   3

       return defaultSaveOptions;

   3 | return defaultSaveOptions;

   Preondition Violations
   Unmatched return defaultSaveOptions;

   11 | result.add(eStructuralFeature);

   Preondition Violations
   Unmatched statement result.add(eStructuralFeature); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted

   11

       result.add(eStructuralFeature);

   12 | break;

   Preondition Violations
   Unmatched break;

   12

       break;

   Precondition Violations (7)
   Row | Violation
   --- | ---
   1 | Type java.util.Map<java.lang.Object,java.lang.Object> of variable defaultSaveOptions does not match with type java.util.List<org.eclipse.emf.ecore.EStructuralFeature> of variable result
   2 | Unmatched statement defaultSaveOptions=new HashMap<Object,Object>(); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted
   3 | Unmatched statement result=new ArrayList<EStructuralFeature>(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
   4 | Unmatched return defaultSaveOptions;
   5 | Unmatched statement result.add(eStructuralFeature); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted
   6 | Unmatched break;
   7 | The refactoring of the clones is infeasible, because the number of macthed statements is equal to zero