File path: /jEdit-4.2/src/org/gjt/sp/jedit/GUIUtilities.java | File path: /jEdit-4.2/src/org/gjt/sp/jedit/menu/EnhancedMenu.java | |||
Method name: JPopupMenu loadPopupMenu(ActionContext, String)
|
Method name: void EnhancedMenu(String, String, ActionContext)
|
|||
Number of AST nodes: 6 | Number of AST nodes: 6 | |||
1 | StringTokenizer st = new StringTokenizer(menuItems);↵ | 1 | StringTokenizer st = new StringTokenizer(menuItems);↵ | |
2 | while(st.hasMoreTokens())↵ | 2 | while(st.hasMoreTokens())↵ | |
3 | {↵ | 3 | {↵ | |
4 | String menuItemName = st.nextToken();↵ | 4 | String menuItemName = st.nextToken();↵ | |
5 | if(menuItemName.equals("-"))↵ | 5 | if(menuItemName.equals("-"))↵ | |
6 | menu.addSeparator();↵ | 6 | addSeparator();↵ | |
7 | else↵ | 7 | else↵ | |
8 | menu.add(loadMenuItem(context,menuItemName,false));↵ | 8 | add(GUIUtilities.loadMenuItem(context,menuItemName,true));↵ | |
9 | } | 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.3 |
Clones location | Clones are in different classes |
Number of node comparisons | 15 |
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) | 1.9 |
Clone type | Type 2 |
ID | Statement | ID | Statement | |||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
4 | StringTokenizer st = new StringTokenizer(menuItems); | 15 | StringTokenizer st = new StringTokenizer(menuItems); | |||||||||||||||||||||
5 | while (st.hasMoreTokens()) | 16 | while (st.hasMoreTokens()) | |||||||||||||||||||||
6 | String menuItemName = st.nextToken(); | 17 | String menuItemName = st.nextToken(); | |||||||||||||||||||||
7 | if (menuItemName.equals("-")) | 18 | if (menuItemName.equals("-")) | |||||||||||||||||||||
8 | menu.addSeparator(); |
| 19 | addSeparator(); | ||||||||||||||||||||
else | else | |||||||||||||||||||||||
9 | menu.add(loadMenuItem(context, menuItemName, false)); |
| 20 | add(GUIUtilities.loadMenuItem(context, menuItemName, true)); |
Row | Violation |
---|---|
1 | Expression menu.addSeparator() is a void method call, and thus it cannot be parameterized |
2 | Expression addSeparator() is a void method call, and thus it cannot be parameterized |
3 | Expression loadMenuItem(context,menuItemName,false) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
4 | Expression GUIUtilities.loadMenuItem(context,menuItemName,true) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
5 | Expression menu.add(loadMenuItem(context,menuItemName,false)) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
6 | Expression add(GUIUtilities.loadMenuItem(context,menuItemName,true)) cannot be parameterized, because it has dependencies to/from statements that will be extracted |