File path: /apache-ant-1.7.0/src/org/apache/tools/ant/taskdefs/Manifest.java | File path: /apache-ant-1.7.0/src/tests/junit/org/apache/tools/mail/MailMessageTest.java | |||
Method name: void write(PrintWriter)
|
Method name: void disconnect()
|
|||
Number of AST nodes: 4 | Number of AST nodes: 5 | |||
1 | if (signatureVersion != null) {↵ | 1 | if (out != null) {↵ | |
2 | try {↵ | 2 | try {↵ | |
3 | Attribute svAttr = new Attribute(ATTRIBUTE_SIGNATURE_VERSION,↵ | 3 | out.flush();↵ | |
4 | signatureVersion);↵ | 4 | out.close();↵ | |
5 | mainSection.addConfiguredAttribute(svAttr);↵ | 5 | ↵ | |
6 | out = null;↵ | |||
6 | } catch (ManifestException e) {↵ | 7 | } catch (IOException e) {↵ | |
7 | // shouldn't happen - ignore↵ | 8 | // ignore↵ | |
8 | }↵ | 9 | ↵ | |
10 | }↵ | |||
9 | } | 11 | } | |
See real code fragment | See real code fragment |
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 | 13 |
Number of mapped statements | 2 |
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) | 1.0 |
Clone type | Type 3 |
ID | Statement | ID | Statement | |||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
7 | if (signatureVersion != null) |
| 1 | if (out != null) | ||||||||||||||
8 | try |
| 2 | try | ||||||||||||||
|
| 3 | out.flush(); | |||||||||||||||
|
| 4 | out.close(); | |||||||||||||||
| 5 | out = null; | ||||||||||||||||
9 | Attribute svAttr = new Attribute(ATTRIBUTE_SIGNATURE_VERSION, signatureVersion); | | ||||||||||||||||
10 | mainSection.addConfiguredAttribute(svAttr); |
| |
Row | Violation |
---|---|
1 | Type java.lang.String of variable signatureVersion does not match with type java.io.BufferedWriter of variable out |
2 | Unmatched statement out.flush(); 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 | Unmatched statement out.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 |
4 | Unmatched statement mainSection.addConfiguredAttribute(svAttr); 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 |