try {
IEventInfo model = (IEventInfo) store.get(activityId);
EditEventDialog dialog = new EditEventDialog(null, model);
if (dialog.success()) {
IEventInfo updatedModel = dialog.getModel();
// update store
store.modify(activityId, updatedModel);
}
} catch (StoreException e1) {
JOptionPane.showMessageDialog(FrameManager.getInstance()
.getActiveFrame(), e1.getMessage());
e1.printStackTrace();
}
try {
IEventInfo model = (IEventInfo) store.get(selected.getId());
EditEventDialog dialog = new EditEventDialog(null, model);
if (dialog.success()) {
IEventInfo updatedModel = dialog.getModel();
// update store
store.modify(selected.getId(), updatedModel);
}
} catch (StoreException e1) {
JOptionPane.showMessageDialog(this, e1.getMessage());
e1.printStackTrace();
}
Clone fragments detected by clone detection tool
File path: /columba-1.4-src/calendar/src/main/java/org/columba/calendar/facade/DialogFacade.java
|
|
File path: /columba-1.4-src/calendar/src/main/java/org/columba/calendar/ui/box/CalendarBox.java
|
Method name: void openEventEditorDialog(URI)
|
|
Method name: void openEditCalendarEventDialog(IEventInfo)
|
Number of AST nodes: 6
|
|
Number of AST nodes: 6
|
|
1 | try {
↵ | | 1 | try {↵
|
2 | IEventInfo model = (IEventInfo) store.get(activityId);↵ | | 2 | IEventInfo model = (IEventInfo) store.get(↵
|
3 |
↵ | | 3 | selected.getId());↵
|
|
4 | EditEventDialog dialog = new EditEventDialog(null, model);
↵ | | 4 | EditEventDialog dialog = new EditEventDialog(null, model);↵
|
5 | if (dialog.success()) {
↵ | | 5 | if (dialog.success()) {↵
|
6 | IEventInfo updatedModel = dialog.getModel();↵ | | 6 | IEventInfo updatedModel = dialog.getModel();↵
|
7 |
↵ | | |
|
|
8 | // update store
↵ | | 7 | // update store↵
|
9 | store.modify(activityId, updatedModel);
↵ | | 8 | store.modify(selected.getId(), updatedModel);↵
|
10 | }↵ | | 9 | }↵
|
11 |
↵ | | |
|
|
12 | } catch (StoreException e1) {
↵ | | 10 | } catch (StoreException e1) {↵
|
13 | JOptionPane.showMessageDialog(FrameManager.getInstance()↵ | | 11 | JOptionPane.showMessageDialog(↵
|
14 | .getActiveFrame(), e1.getMessage());
↵ | | 12 | this, e1.getMessage());↵
|
15 | e1.printStackTrace();
↵ | | 13 | e1.printStackTrace();↵
|
16 | } | | 14 | }
|
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.6 |
Clones location | Clones are in different classes |
Number of node comparisons | 18 |
-
{Non-refactorable}
Mapping Summary
Number of mapped statements | 6 |
Number of unmapped statements in the first code fragment | 0 |
Number of unmapped statements in the second code fragment | 0 |
Time elapsed for statement mapping (ms) | 19.3 |
Clone type | Type 2 |
Mapped Statements
ID |
Statement |
|
ID |
Statement |
9 | try | | 2 | try |
10 | IEventInfo model = (IEventInfo)store.get(activityId); | | 3 | IEventInfo model = (IEventInfo)store.get(selected.getId()); |
11 | EditEventDialog dialog = new EditEventDialog(null, model); | | 4 | EditEventDialog dialog = new EditEventDialog(null, model); |
12 | | | 5 | |
13 | IEventInfo updatedModel = dialog.getModel(); | | 6 | IEventInfo updatedModel = dialog.getModel(); |
14 | store.modify(activityId, updatedModel); | | 7 | store.modify(selected.getId(), updatedModel); |
Precondition Violations (2)
Row |
Violation |
1 | Expression selected.getId() is a method call throwing exception(s) that should be caught by a try block that will be extracted |
2 | Expression selected.getId() is a method call throwing exception(s) that should be caught by a try block that will be extracted |