commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benedikt Ritter (JIRA)" <>
Subject [jira] [Created] (IMAGING-159) There should be a Parameters class
Date Sat, 10 Jan 2015 13:42:34 GMT
Benedikt Ritter created IMAGING-159:

             Summary: There should be a Parameters class
                 Key: IMAGING-159
             Project: Commons Imaging
          Issue Type: Improvement
          Components: imaging.*
            Reporter: Benedikt Ritter
             Fix For: Patch Needed

Currently options for image I/O are defined as Maps. The leads to the problem that our code
has to validate parameter types when they are used:

final Object value = params.get(PARAM_KEY_COMPRESSION);
if (value != null) {
  if (!(value instanceof Number)) {
    throw new ImageWriteException(
      "Invalid compression parameter, must be numeric: "
         + value);
  compression = ((Number) value).intValue();

This can be simplified if we define a Parameters class that provides additional methods like
{{public int getInt(String key)}}. The implementation could then look up the value from the
map through an exception if it is null or not a number.

This message was sent by Atlassian JIRA

View raw message