178-4-6

if (this.stroke != null) { return this.stroke; } // otherwise look up the paint table Stroke result = getSeriesStroke(series); if (result == null && this.autoPopulateSeriesStroke) { DrawingSupplier supplier = getDrawingSupplier(); if (supplier != null) { result = supplier.getNextStroke(); setSeriesStroke(series, result, false); } } if (result == null) { result = this.baseStroke; } return result;
if (this.shape != null) { return this.shape; } // otherwise look up the shape list Shape result = getSeriesShape(series); if (result == null && this.autoPopulateSeriesShape) { DrawingSupplier supplier = getDrawingSupplier(); if (supplier != null) { result = supplier.getNextShape(); setSeriesShape(series, result, false); } } if (result == null) { result = this.baseShape; } return result;

Clone fragments detected by clone detection tool
File path: /jfreechart-1.0.10/src/org/jfree/chart/renderer/AbstractRenderer.java |  | File path: /jfreechart-1.0.10/src/org/jfree/chart/renderer/AbstractRenderer.java
Method name: `Stroke lookupSeriesStroke(int)` |  | Method name: `Shape lookupSeriesShape(int)`
Number of AST nodes: 11 |  | Number of AST nodes: 11

1

    if (this.stroke != null) {↵

1

    if (this.shape != null) {↵

2

                return this.stroke;↵

2

                return this.shape;↵

3

            }↵

3

            }↵

4

            // otherwise look up the paint table↵

4

            // otherwise look up the shape list↵

5

            Stroke result = getSeriesStroke(series);↵

5

            Shape result = getSeriesShape(series);↵

6

            if (result == null && this.autoPopulateSeriesStroke) {↵

6

            if (result == null && this.autoPopulateSeriesShape) {↵

7

                DrawingSupplier supplier = getDrawingSupplier();↵

7

                DrawingSupplier supplier = getDrawingSupplier();↵

8

                if (supplier != null) {↵

8

                if (supplier != null) {↵

9

                    result = supplier.getNextStroke();↵

9

                    result = supplier.getNextShape();↵

10

                    setSeriesStroke(series, result, false);↵

10

                    setSeriesShape(series, result, false);↵

11

                }↵

11

                }↵

12

            }↵

12

            }↵

13

            if (result == null) {↵

13

            if (result == null) {↵

14

                result = this.baseStroke;↵

14

                result = this.baseShape;↵

15

            }↵

15

            }↵

16

            return result;

16

            return result;

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 declared in the same class
Number of node comparisons | 30

1. {Non-refactorable}

   Mapping Summary
   Number of mapped statements | 6
   Number of unmapped statements in the first code fragment | 5
   Number of unmapped statements in the second code fragment | 5
   Time elapsed for statement mapping (ms) | 2.7
   Clone type | Type 2

   Mapped Statements
   ID | Statement |  | ID | Statement
   --- | --- | --- | --- | ---

   1

       if (this.stroke != null)

   1 | if (this.stroke != null)
   1 | if (this.shape != null)

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   stroke | shape | VARIABLE_NAME_MISMATCH
   java.awt.Stroke | java.awt.Shape | VARIABLE_TYPE_MISMATCH

   Preondition Violations

   Type java.awt.Stroke of variable this.stroke does not match with type java.awt.Shape of variable this.shape
   - Make classes java.awt.Stroke and java.awt.Shape extend a common superclass

   1

       if (this.shape != null)

   2 | return this.shape;

   Preondition Violations
   Unmatched return this.shape;

   2

       return this.shape;

   2

       return this.stroke;

   2 | return this.stroke;

   Preondition Violations
   Unmatched return this.stroke;

   3

       Shape result = getSeriesShape(series);

   3

       Stroke result = getSeriesStroke(series);

   4

       if (result == null && this.autoPopulateSeriesStroke)

   4 | if (result == null && this.autoPopulateSeriesStroke)
   4 | if (result == null && this.autoPopulateSeriesShape)

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   java.awt.Stroke | java.awt.Shape | VARIABLE_TYPE_MISMATCH
   autoPopulateSeriesStroke | autoPopulateSeriesShape | VARIABLE_NAME_MISMATCH

   Preondition Violations

   Type java.awt.Stroke of variable result does not match with type java.awt.Shape of variable result
   - Make classes java.awt.Stroke and java.awt.Shape extend a common superclass

   4

       if (result == null && this.autoPopulateSeriesShape)

   5

       DrawingSupplier supplier = getDrawingSupplier();

   5

       DrawingSupplier supplier = getDrawingSupplier();

   6

       if (supplier != null)

   6

       if (supplier != null)

   7 | result = supplier.getNextShape();

   Preondition Violations
   Unmatched statement result=supplier.getNextShape(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted

   7

       result = supplier.getNextShape();

   7

       result = supplier.getNextStroke();

   7 | result = supplier.getNextStroke();

   Preondition Violations
   Unmatched statement result=supplier.getNextStroke(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted

   8

       setSeriesStroke(series, result, false);

   8 | setSeriesStroke(series, result, false);
   8 | setSeriesShape(series, result, false);

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   setSeriesStroke | setSeriesShape | METHOD_INVOCATION_NAME_MISMATCH
   java.awt.Stroke | java.awt.Shape | VARIABLE_TYPE_MISMATCH

   Preondition Violations
   Expression setSeriesStroke(series,result,false) is a void method call, and thus it cannot be parameterized
   Expression setSeriesShape(series,result,false) is a void method call, and thus it cannot be parameterized

   Type java.awt.Stroke of variable result does not match with type java.awt.Shape of variable result
   - Make classes java.awt.Stroke and java.awt.Shape extend a common superclass

   8

       setSeriesShape(series, result, false);

   9

       if (result == null)

   9 | if (result == null)
   9 | if (result == null)

   Differences
   Expression1 | Expression2 | Difference
   --- | --- | ---
   java.awt.Stroke | java.awt.Shape | VARIABLE_TYPE_MISMATCH

   Preondition Violations

   Type java.awt.Stroke of variable result does not match with type java.awt.Shape of variable result
   - Make classes java.awt.Stroke and java.awt.Shape extend a common superclass

   9

       if (result == null)

   10 | result = this.baseShape;

   Preondition Violations
   Unmatched statement result=this.baseShape; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted

   10

       result = this.baseShape;

   10

       result = this.baseStroke;

   10 | result = this.baseStroke;

   Preondition Violations
   Unmatched statement result=this.baseStroke; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted

   11 | return result;

   Preondition Violations
   Unmatched return result;

   11

       return result;

   11

       return result;

   11 | return result;

   Preondition Violations
   Unmatched return result;

   Precondition Violations (14)
   Row | Violation
   --- | ---
   1 | Type java.awt.Stroke of variable this.stroke does not match with type java.awt.Shape of variable this.shape
   2 | Unmatched return this.shape;
   3 | Unmatched return this.stroke;
   4 | Type java.awt.Stroke of variable result does not match with type java.awt.Shape of variable result
   5 | Unmatched statement result=supplier.getNextShape(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
   6 | Unmatched statement result=supplier.getNextStroke(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
   7 | Expression setSeriesStroke(series,result,false) is a void method call, and thus it cannot be parameterized
   8 | Expression setSeriesShape(series,result,false) is a void method call, and thus it cannot be parameterized
   9 | Type java.awt.Stroke of variable result does not match with type java.awt.Shape of variable result
   10 | Type java.awt.Stroke of variable result does not match with type java.awt.Shape of variable result
   11 | Unmatched statement result=this.baseShape; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
   12 | Unmatched statement result=this.baseStroke; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted
   13 | Unmatched return result;
   14 | Unmatched return result;