[ashishdonvir]

What is the to get RGB value of each pixel of a Buffered Image?

Right now i am getting RGb value using two for loops as shown in code below but it took too much long time to get rgb values as loop runs 479999 times for my image and it can even rise when i will use 16 bit image.

SO i need some faster way to get pixel values

here is my code what i tried

 BufferedImage bi=ImageIO.read(new File("C:\\images\\Sunset.jpg"));    

      {
     int countloop=0;  

           for (int x = 0; x <bi.getWidth(); x++) {

                    for (int y = 0; y < bi.getHeight(); y++) {

                         Color c = new Color(bi.getRGB(x, y));

                          System.out.println("red=="+c.getRed()+" green=="+c.getGreen()+"    blue=="+c.getBlue()+"  countloop="+countloop++);                                                                                                                              
        }
    }

[macosxnerd101]

You can use the getRGB() method that returns an int[], but it won't change the number of pixels you have to evaluate.

[CasiOo]

I believe it is actually the System.out.println's that are taking most of your time

[xclite]

I was thinking about suggesting building up a StringBuilder object and then calling a print on it once you've finished. I've been out of Java for so long I kind of forget if that works. Definitely worth a try since it's an easy change.

[pbl]

You are calling getWidth() and getHeight() so many times for nothing just call them once and store their values in a local variable

Color c = new Color()

no need to create A new Color object for every pixel, if your only purpose is to display the value simply save getRGB() in a int and print out of it. For each pixel, you create a new Color object andf you access 3 of its method

You can always use that method

http://docs.oracle.c...html#getRGB(int, int, int, int, int[], int, int)

[pbl]

Yup xclite

the execution of a stament like:
System.out.println("red==" + c.getRed()+ " green==" + c.getGreen() + " blue==" + c.getBlue()+" countloop=" + countloop++);

Implies the creation of a new StringBuilder() (ONE for each processed pixel) and the creation of a String object (when doing stringBuilder.toString() to pass to println())