Skip to content

Commit d0cb7f5

Browse files
committed
PDFBOX-5660: refactor, as suggested by Valery Bokov; closes #252
git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1928508 13f79535-47bb-0310-9956-ffa450edef68
1 parent 5370a0b commit d0cb7f5

File tree

6 files changed

+11
-56
lines changed

6 files changed

+11
-56
lines changed

pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotation.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -478,9 +478,14 @@ public final void setPage(int page)
478478
* @return The annotation color, or null if there is none.
479479
*/
480480
public Color getColor()
481+
{
482+
return getColor(COSName.C);
483+
}
484+
485+
final Color getColor(COSName colorName)
481486
{
482487
Color retval = null;
483-
COSArray array = annot.getCOSArray(COSName.C);
488+
COSArray array = annot.getCOSArray(colorName);
484489
if (array != null)
485490
{
486491
float[] rgb = array.toFloatArray();

pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationCircle.java

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -111,17 +111,7 @@ public final void setInteriorColor(Color color)
111111
*/
112112
public Color getInteriorColor()
113113
{
114-
Color retval = null;
115-
COSArray array = annot.getCOSArray(COSName.IC);
116-
if (array != null)
117-
{
118-
float[] rgb = array.toFloatArray();
119-
if (rgb.length >= 3)
120-
{
121-
retval = new Color(rgb[0], rgb[1], rgb[2]);
122-
}
123-
}
124-
return retval;
114+
return getColor(COSName.IC);
125115
}
126116

127117
/**

pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationLine.java

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -257,17 +257,7 @@ public final void setInteriorColor(Color color)
257257
*/
258258
public Color getInteriorColor()
259259
{
260-
Color retval = null;
261-
COSArray array = annot.getCOSArray(COSName.IC);
262-
if (array != null)
263-
{
264-
float[] rgb = array.toFloatArray();
265-
if (rgb.length >= 3)
266-
{
267-
retval = new Color(rgb[0], rgb[1], rgb[2]);
268-
}
269-
}
270-
return retval;
260+
return getColor(COSName.IC);
271261
}
272262

273263
/**

pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolygon.java

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -147,16 +147,6 @@ public final void setInteriorColor(Color color)
147147
*/
148148
public Color getInteriorColor()
149149
{
150-
Color retval = null;
151-
COSArray array = annot.getCOSArray(COSName.IC);
152-
if (array != null)
153-
{
154-
float[] rgb = array.toFloatArray();
155-
if (rgb.length >= 3)
156-
{
157-
retval = new Color(rgb[0], rgb[1], rgb[2]);
158-
}
159-
}
160-
return retval;
150+
return getColor(COSName.IC);
161151
}
162152
}

pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationPolyline.java

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -230,16 +230,6 @@ public void setInteriorColor(Color color)
230230
*/
231231
public Color getInteriorColor()
232232
{
233-
Color retval = null;
234-
COSArray array = annot.getCOSArray(COSName.IC);
235-
if (array != null)
236-
{
237-
float[] rgb = array.toFloatArray();
238-
if (rgb.length >= 3)
239-
{
240-
retval = new Color(rgb[0], rgb[1], rgb[2]);
241-
}
242-
}
243-
return retval;
233+
return getColor(COSName.IC);
244234
}
245235
}

pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFAnnotationSquare.java

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -112,17 +112,7 @@ public final void setInteriorColor(Color color)
112112
*/
113113
public Color getInteriorColor()
114114
{
115-
Color retval = null;
116-
COSArray array = annot.getCOSArray(COSName.IC);
117-
if (array != null)
118-
{
119-
float[] rgb = array.toFloatArray();
120-
if (rgb.length >= 3)
121-
{
122-
retval = new Color(rgb[0], rgb[1], rgb[2]);
123-
}
124-
}
125-
return retval;
115+
return getColor(COSName.IC);
126116
}
127117

128118
/**

0 commit comments

Comments
 (0)