File path: /sql12/fw/src/net/sourceforge/squirrel_sql/fw/gui/FontChooser.java | File path: /sql12/fw/src/net/sourceforge/squirrel_sql/fw/gui/FontChooser.java | |||
Method name: void addNotify()
|
Method name: void removeNotify()
|
|||
Number of AST nodes: 7 | Number of AST nodes: 7 | |||
1 | super.addNotify();↵ | 1 | super.removeNotify();↵ | |
2 | if (_previewUpdater == null)↵ | 2 | if (_previewUpdater != null)↵ | |
3 | {↵ | 3 | {↵ | |
4 | _previewUpdater = new PreviewLabelUpdater();↵ | 4 | ↵ | |
5 | _fontNamesCmb.addActionListener(_previewUpdater);↵ | 5 | _fontNamesCmb.removeActionListener(_previewUpdater);↵ | |
6 | _fontSizesCmb.addActionListener(_previewUpdater);↵ | 6 | _fontSizesCmb.removeActionListener(_previewUpdater);↵ | |
7 | _boldChk.addActionListener(_previewUpdater);↵ | 7 | _boldChk.removeActionListener(_previewUpdater);↵ | |
8 | _italicChk.addActionListener(_previewUpdater);↵ | 8 | _italicChk.removeActionListener(_previewUpdater);↵ | |
9 | ↵ | 9 | _previewUpdater = null;↵ | |
10 | } | 10 | } | |
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 declared in the same class |
Number of node comparisons | 2 |
Number of mapped statements | 1 |
Number of unmapped statements in the first code fragment | 6 |
Number of unmapped statements in the second code fragment | 6 |
Time elapsed for statement mapping (ms) | 1.6 |
Clone type | Type 3 |
ID | Statement | ID | Statement | |||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1 | super.addNotify(); |
| 1 | super.removeNotify(); | ||||||||||||||
| 2 | if (_previewUpdater != null) | ||||||||||||||||
| 3 | _fontNamesCmb.removeActionListener(_previewUpdater); | ||||||||||||||||
| 4 | _fontSizesCmb.removeActionListener(_previewUpdater); | ||||||||||||||||
| 5 | _boldChk.removeActionListener(_previewUpdater); | ||||||||||||||||
| 6 | _italicChk.removeActionListener(_previewUpdater); | ||||||||||||||||
|
| 7 | _previewUpdater = null; | |||||||||||||||
2 | if (_previewUpdater == null) | | ||||||||||||||||
3 | _previewUpdater = new PreviewLabelUpdater(); |
| | |||||||||||||||
4 | _fontNamesCmb.addActionListener(_previewUpdater); | | ||||||||||||||||
5 | _fontSizesCmb.addActionListener(_previewUpdater); | | ||||||||||||||||
6 | _boldChk.addActionListener(_previewUpdater); | | ||||||||||||||||
7 | _italicChk.addActionListener(_previewUpdater); | |
Row | Violation |
---|---|
1 | Expression super.addNotify() is a void method call, and thus it cannot be parameterized |
2 | Expression super.removeNotify() is a void method call, and thus it cannot be parameterized |
3 | Super method call super.addNotify(); cannot be extracted from method |
4 | Super method call super.removeNotify(); cannot be extracted from method |
5 | Unmatched statement _previewUpdater=null; cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted |
6 | Unmatched statement _previewUpdater=new PreviewLabelUpdater(); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted |