158-69-204

if (sock != null) { try { sock.close(); sock = null; } catch (IOException e) { // ignore } }
if (source != null) { try { source = FILE_UTILS.normalize(source.getAbsolutePath()); } catch (BuildException e) { // relative path } } return source;

Clone fragments detected by clone detection tool
File path: /apache-ant-1.7.0/src/tests/junit/org/apache/tools/mail/MailMessageTest.java |  | File path: /apache-ant-1.7.0/src/org/apache/tools/ant/util/LoaderUtils.java
Method name: `void disconnect()` |  | Method name: `File normalizeSource(File)`
Number of AST nodes: 4 |  | Number of AST nodes: 4

1

    if (sock != null) {↵

1

    if (source != null) {↵

2

                    try {↵

2

                try {↵

3

                        sock.close();↵

3

                    ↵

4

                        sock = null;↵

5

        ↵

4

    source = FILE_UTILS.normalize(source.getAbsolutePath());↵

6

                } catch (IOException e) {↵

5

                } catch (BuildException e) {↵

7

                        // ignore↵

6

                    ↵

8

        ↵

7

    // relative path↵

9

                }↵

8

                }↵

10

        ↵

9

            }↵

11

            }

10

            return source;

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.3
Clones location | Clones are in different classes
Number of node comparisons | 7

1. {Non-refactorable}

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

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

   10

       if (sock != null)

   10 | if (sock != null)
   1 | if (source != null)

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   sock | source | VARIABLE_NAME_MISMATCH
   java.net.Socket | java.io.File | VARIABLE_TYPE_MISMATCH

   Preondition Violations

   Type java.net.Socket of variable sock does not match with type java.io.File of variable source
   - Make classes java.net.Socket and java.io.File extend a common superclass

   1

       if (source != null)

   11

       try

   11 | try
   2 | try

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   java.io.IOException | org.apache.tools.ant.BuildException | SUBCLASS_TYPE_MISMATCH
   java.io.IOException | org.apache.tools.ant.BuildException | SUBCLASS_TYPE_MISMATCH

   2

       try

   3 | source = FILE_UTILS.normalize(source.getAbsolutePath());

   Preondition Violations
   Unmatched statement source=FILE_UTILS.normalize(source.getAbsolutePath()); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
   Unmatched statement source=FILE_UTILS.normalize(source.getAbsolutePath()); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted

   3

       source = FILE_UTILS.normalize(source.getAbsolutePath());

   12

       sock.close();

   12 | sock.close();

   Preondition Violations
   Unmatched statement sock.close(); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted

   13

       sock = null;

   4 | return source;

   Preondition Violations
   Unmatched return source;

   4

       return source;

   Precondition Violations (5)
   Row | Violation
   --- | ---
   1 | Type java.net.Socket of variable sock does not match with type java.io.File of variable source
   2 | Unmatched statement source=FILE_UTILS.normalize(source.getAbsolutePath()); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
   3 | Unmatched statement source=FILE_UTILS.normalize(source.getAbsolutePath()); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted
   4 | Unmatched statement sock.close(); cannot be moved before or after the extracted code, because it throws exception(s) that should be caught by a try block that will be extracted
   5 | Unmatched return source;