File path: /apache-ant-1.7.0/src/org/apache/tools/ant/taskdefs/optional/PropertyFile.java | File path: /apache-ant-1.7.0/src/tests/junit/org/apache/tools/mail/MailMessageTest.java | |||
Method name: void executeDate(String)
|
Method name: void disconnect()
|
|||
Number of AST nodes: 2 | Number of AST nodes: 5 | |||
1 | try {↵ | |||
2 | currentValue.setTime(fmt.parse(currentStringValue))↵ | 1 | if (out != null) {↵ | |
2 | try {↵ | |||
3 | out.flush();↵ | |||
4 | out.close();↵ | |||
3 | ;↵ | 5 | out = null;↵ | |
4 | } catch (ParseException pe) {↵ | 6 | } catch (IOException e) {↵ | |
5 | // swallow↵ | 7 | // ignore↵ | |
6 | } | 8 | }↵ | |
9 |
| |||
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.1 |
Clones location | Clones are in different classes |
Number of node comparisons | 8 |
Number of mapped statements | 1 |
Number of unmapped statements in the first code fragment | 1 |
Number of unmapped statements in the second code fragment | 3 |
Time elapsed for statement mapping (ms) | 0.6 |
Clone type | Type 3 |
ID | Statement | ID | Statement | |||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
10 | try |
| 2 | try | ||||||||||||||||
|
| 3 | out.flush(); | |||||||||||||||||
|
| 4 | out.close(); | |||||||||||||||||
| 5 | out = null; | ||||||||||||||||||
11 | currentValue.setTime(fmt.parse(currentStringValue)); |
| |
Row | Violation |
---|---|
1 | 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 |
2 | 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 |
3 | Unmatched statement currentValue.setTime(fmt.parse(currentStringValue)); 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 |