File path: /sql12/app/src/net/sourceforge/squirrel_sql/client/gui/mainframe/WidgetUtils.java | File path: /sql12/app/src/net/sourceforge/squirrel_sql/client/gui/mainframe/WidgetUtils.java | |||
Method name: IWidget[] getOpenToolWindows(IWidget[])
|
Method name: IWidget[] getOpenNonToolWindows(IWidget[])
|
|||
Number of AST nodes: 8 | Number of AST nodes: 7 | |||
1 | if (frames == null)↵ | 1 | if (frames == null)↵ | |
2 | {↵ | 2 | {↵ | |
3 | throw new IllegalArgumentException("null JInternalFrame[] passed");↵ | 3 | throw new IllegalArgumentException("null JInternalFrame[] passed");↵ | |
4 | }↵ | 4 | }↵ | |
5 | List<IWidget> framesList = new ArrayList<IWidget>();↵ | 5 | List<IWidget> framesList = new ArrayList<IWidget>();↵ | |
6 | for (int i = 0; i < frames.length; ++i)↵ | 6 | for (int i = 0; i < frames.length; ++i)↵ | |
7 | {↵ | 7 | {↵ | |
8 | IWidget fr = frames[i];↵ | 8 | ↵ | |
9 | if (fr.isToolWindow() && !fr.isClosed())↵ | 9 | if (!frames[i].isToolWindow() && !frames[i].isClosed())↵ | |
10 | {↵ | 10 | {↵ | |
11 | framesList.add(frames[i]);↵ | 11 | framesList.add(frames[i]);↵ | |
12 | }↵ | 12 | }↵ | |
13 | }↵ | 13 | }↵ | |
14 | return framesList.toArray(new IWidget[framesList.size()]); | 14 |
| |
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.4 |
Clones location | Clones are declared in the same class |
Number of node comparisons | 12 |
Number of mapped statements | 7 |
Number of unmapped statements in the first code fragment | 1 |
Number of unmapped statements in the second code fragment | 0 |
Time elapsed for statement mapping (ms) | 3.7 |
Clone type | Type 3 |
ID | Statement | ID | Statement | ||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1 | if (frames == null) | 1 | if (frames == null) | ||||||||||||||||||
2 | throw new IllegalArgumentException("null JInternalFrame[] passed"); | 2 | throw new IllegalArgumentException("null JInternalFrame[] passed"); | ||||||||||||||||||
3 | List<IWidget> framesList = new ArrayList<IWidget>(); | 3 | List<IWidget> framesList = new ArrayList<IWidget>(); | ||||||||||||||||||
4 | for (int i = 0; i < frames.length; ++i) | 4 | for (int i = 0; i < frames.length; ++i) | ||||||||||||||||||
5 | IWidget fr = frames[i]; |
| | ||||||||||||||||||
6 | if (fr.isToolWindow() && !fr.isClosed()) |
| 5 | if (!frames[i].isToolWindow() && !frames[i].isClosed()) | |||||||||||||||||
7 | framesList.add(frames[i]); | 6 | framesList.add(frames[i]); | ||||||||||||||||||
8 | return framesList.toArray(new IWidget[framesList.size()]); | 7 | return framesList.toArray(new IWidget[framesList.size()]); |
Row | Violation |
---|---|
1 | Unmatched statement IWidget fr=frames[i]; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
2 | Expression fr.isToolWindow() cannot be parameterized, because it has dependencies to/from statements that will be extracted |
3 | Expression !frames[i].isToolWindow() cannot be parameterized, because it has dependencies to/from statements that will be extracted |
4 | Expression fr cannot be parameterized, because it has dependencies to/from statements that will be extracted |
5 | Expression frames[i] cannot be parameterized, because it has dependencies to/from statements that will be extracted |
6 | Clone fragment #1 returns variables i , while Clone fragment #2 returns variables |