[
https://issues.apache.org/jira/browse/IMAGING-157?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272431#comment-14272431
]
Benedikt Ritter commented on IMAGING-157:
-----------------------------------------
What is {{Color.Hsl}}? Is it an enum? That would duplicate the class hierarchy
wie already have (every class would need an equivalent enum value). So I'd vote
for the second solution. The problem here is, that we need to restrict the
range of allowed classes, hence we need a common interface or class, that all
color classes share. We can not use an interface, because it is not possible to
restrict clients from implementing it, so we'll need a base class. I propose
this:
{code:java}
public abstract class Color {
Color() {
// package private constructor so that only we can create sub classes
}
public T convertTo(Class<T extends Color> targetClass) {
// implement logic here?
// make this abstract and implement logic in individual color classes?
// delegate to ColorConversions?
}
}
{code}
> Add method to color classes to convert colors to other color spaces
> -------------------------------------------------------------------
>
> Key: IMAGING-157
> URL: https://issues.apache.org/jira/browse/IMAGING-157
> Project: Commons Imaging
> Issue Type: Improvement
> Reporter: Stephan Köninger
> Fix For: Discussion
>
>
> Instead of having a static util class it would be nice to have a method
> available to all Color types which allows one to convert a color to another
> color space. See following example for clearance:
> {code:java}
> ColorCmyk cymk = new ColorCmyk(100, 0, 100, 0);
> ColorHsl colorHsl = cymk.convertTo(Color.Hsl);
> // or
> ColorHsl colorHsl = cymk.convertTo(ColorHsl.class);
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
