if (dialog.getResult()) {
try {
// modify card properties in folder
folder.modify(uids[0], dialog.getDestModel());
} catch (Exception e1) {
if (Logging.DEBUG)
e1.printStackTrace();
ErrorDialog.createDialog(e1.getMessage(), e1);
}
if (folder instanceof GroupFolder)
// re-select folder
mediator.getTree().setSelectedFolder(
(AbstractFolder) folder);
}
if (dialog.getResult()) {
try {
// modify card properties in folder
folder.modify(contactId, dialog.getDestModel());
} catch (Exception e1) {
if (Logging.DEBUG)
e1.printStackTrace();
ErrorDialog.createDialog(e1.getMessage(), e1);
}
}
Clone fragments detected by clone detection tool
File path: /columba-1.4-src/contact/src/main/java/org/columba/addressbook/gui/action/EditPropertiesAction.java
|
|
File path: /columba-1.4-src/contact/src/main/java/org/columba/addressbook/facade/DialogFacade.java
|
Method name: void actionPerformed(ActionEvent)
|
|
Method name: void openContactDialog(URI)
|
Number of AST nodes: 5
|
|
Number of AST nodes: 3
|
|
1 | if (dialog.getResult()) {↵ | | 1 | if (dialog.getResult()) {↵
|
|
2 | try {↵ | | 2 | try {↵
|
3 | // modify card properties in folder↵ | | 3 | // modify card properties in folder↵
|
4 | folder.modify(uids[0], dialog.getDestModel());↵ | | 4 | folder.modify(contactId, dialog.getDestModel());↵
|
5 | } catch (Exception e1) {↵ | | 5 | } catch (Exception e1) {↵
|
6 | if (Logging.DEBUG)↵ | | 6 | if (Logging.DEBUG)↵
|
7 | e1.printStackTrace();↵ | | 7 | e1.printStackTrace();↵
|
|
8 | ErrorDialog.createDialog(e1.getMessage(), e1);↵ | | 8 | ErrorDialog.createDialog(e1.getMessage(), e1);↵
|
9 | }↵ | | 9 | }↵
|
|
10 | if (folder instanceof GroupFolder)↵ | | |
|
11 | // re-select folder↵ | | |
|
12 | mediator.getTree().setSelectedFolder(↵ | | |
|
13 | (AbstractFolder) folder);↵ | | |
|
|
14 | } | | 10 | }
|
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.4 |
Clones location | Clones are in different classes |
Number of node comparisons | 6 |
-
{Non-refactorable}
Mapping Summary
Number of mapped statements | 2 |
Number of unmapped statements in the first code fragment | 1 |
Number of unmapped statements in the second code fragment | 1 |
Time elapsed for statement mapping (ms) | 0.7 |
Clone type | Type 2 |
Mapped Statements
ID |
Statement |
|
ID |
Statement |
16 | if (dialog.getResult()) | | 14 | if (dialog.getResult()) |
17 | | | 15 | |
| | | 16 | folder.modify(contactId, dialog.getDestModel()); |
18 | folder.modify(uids[0], dialog.getDestModel()); | | | |
Precondition Violations (2)
Row |
Violation |
1 | Unmatched statement folder.modify(contactId,dialog.getDestModel()); 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 folder.modify(uids[0],dialog.getDestModel()); 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 |