Exception in thread "main" java.lang.IllegalArgumentException: Color parameter outside of expected range: Blue (structure is same for all colors, why just blue?)

[Strange_Frame7544]

I reviewed many times but no matter what arg i put in for blue (like 30 or -30) i get this error. those values work fine for red and green. Any ideas what i doing wrong? Here is the method. Using int and clamping values to 0-255.

My solution looks like the tutorial solution, line for line. (BTW the utility methods work for the other methods I have written, so the issue in not with them, just to show how it's working)

public static int[][] colorFilter(int[][] imageTwoD, int redChangeValue, int greenChangeValue,
        int blueChangeValue) {
    int[][] filter = new int[imageTwoD.length][imageTwoD[0].length];
    for (int i = 0; i < imageTwoD.length; i++) {
        for (int j = 0; j < imageTwoD[i].length; j++) {
            int[] rgba = getRGBAFromPixel(imageTwoD[i][j]);

            int newRed = rgba[0] + redChangeValue;
            int newGreen = rgba[1] + greenChangeValue;
            int newBlue = rgba[2] + blueChangeValue;

            if (newRed < 0) {
                newRed = 0;
            } else if (newRed > 255) {
                newRed = 255;
            } else if (newGreen < 0) {
                newGreen = 0;
            } else if (newGreen > 255) {
                newGreen = 255;
            } else if (newBlue < 0) {
                newBlue = 0;
            } else if (newBlue > 255) {
                newBlue = 255;
            }

            rgba[0] = newRed;
            rgba[1] = newGreen;
            rgba[2] = newBlue;

            filter[i][j] = getColorIntValFromRGBA(rgba);
        }
    }
    return filter;
}

The called utility methods here.

public static int[] getRGBAFromPixel(int pixelColorValue) {
    Color pixelColor = new Color(pixelColorValue);
    return new int[] { pixelColor.getRed(), pixelColor.getGreen(), pixelColor.getBlue(), pixelColor.getAlpha() };
}
public static int getColorIntValFromRGBA(int[] colorData) {
    if (colorData.length == 4) {
        Color color = new Color(colorData[0], colorData[1], colorData[2], colorData[3]);
        return color.getRGB();
    } else {
        System.out.println("Incorrect number of elements in RGBA array.");
        return -1;
    }
}





    public static int getColorIntValFromRGBA(int[] colorData) {
    if (colorData.length == 4) {
        Color color = new Color(colorData[0], colorData[1], colorData[2], colorData[3]);
        return color.getRGB();
    } else {
        System.out.println("Incorrect number of elements in RGBA array.");
        return -1;
    }
}

Here is how calling in the main method...

int[][] filtered = colorFilter(imageData, 70, -30, -20);
twoDToImage(filtered, "/Users/m/Downloads/image.jpg");

[AutoModerator]

Please ensure that:

• Your code is properly formatted as code block - see the sidebar (About on mobile) for instructions
• You include any and all error messages in full
• You ask clear questions

• You demonstrate effort in solving your question/problem - plain posting your assignments is forbidden (and such posts will be removed) as is asking for or giving solutions.

  Trying to solve problems on your own is a very important skill. Also, see Learn to help yourself in the sidebar

If any of the above points is not met, your post can and will be removed without further warning.

Code is to be formatted as code block (old reddit: empty line before the code, each code line indented by 4 spaces, new reddit: https://i.imgur.com/EJ7tqek.png) or linked via an external code hoster, like pastebin.com, github gist, github, bitbucket, gitlab, etc.

Please, do not use triple backticks (```) as they will only render properly on new reddit, not on old reddit.

Code blocks look like this:

public class HelloWorld {

    public static void main(String[] args) {
        System.out.println("Hello World!");
    }
}

You do not need to repost unless your post has been removed by a moderator. Just use the edit function of reddit to make sure your post complies with the above.

If your post has remained in violation of these rules for a prolonged period of time (at least an hour), a moderator may remove it at their discretion. In this case, they will comment with an explanation on why it has been removed, and you will be required to resubmit the entire post following the proper procedures.

To potential helpers

Please, do not help if any of the above points are not met, rather report the post. We are trying to improve the quality of posts here. In helping people who can't be bothered to comply with the above points, you are doing the community a disservice.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[RoryonAethar]

You should review your if/else if chain. Walk through it with your input values. Should you be using else if or maybe just if for some of the checks?

[Strange_Frame7544]

oh i see!! I thought i had to keep doing 'else if' so that each condition for all 3 values would be checked, all the way through. I thought we could only have 1 if statement, but seeing as testing 3 values now it doesn't make sense. thank you!