Assistance with image conversion program.

Question

hello,
I am relatively new to java so have already requested some help within the "new to java" section of the forum but wondered if it might be more appropriate to post my question in here? anyway...
I am trying to write a program which will convert a color image into a greyscale image. I have managed to get my code to compile but the output i receive is not what i expected. When running the conversion it produces a black image. The idea is to create an array, use pixelgrabber to place each pixel into this array, get the red, green, and blue values, convert into greyscale, add these coverted pixels to an array and make a new image from these. My code is below:
class SaveGreyMenuListnr implements ActionListener {
private ImagePanel panel = null;
SaveGreyMenuListnr (ImagePanel pn) {
panel = pn;
public void actionPerformed (ActionEvent ae) {
Image displayed = panel.getDisplayImage();
int width = displayed.getWidth(panel);
int height = displayed.getHeight(panel);
int pixels[] = new int[width*height]
PixelGrabber pg = new PixelGrabber(displayed, 0, height, width, height, pixels, 0, width);
try{
pg.grabPixels();
} catch (InterruptedException e) {
System.err.println("interrupted waiting for pixels");
for (int i = 0; i < (width*height); i++){
Color myColor = new Color(pixels);
int Red = myColor.getRed();
int Green = myColor.getGreen();
int Blue = myColor.getBlue();
double temp = 0.56 * (double)Green + 0.33 *(double) Red + 0.11 *(double) Blue;
myColor = new Color((int)temp,(int)temp,(int)temp);
int compressedInt = myColor.getRGB();
pixels[i] = compressedInt;
Image image = panel.getToolkit().createImage(new MemoryImageSource(width,height,pixels,0,width));
panel.setDisplayImage(image);
panel.repaint();
I think an issue is arising when using the getRed getGreen and getBlue methods... for example If i request System.out.println(Red); then a stream of 0's are shown on the command line.
Not 100% sure if this is the issue or if it is further back in my code. Any help would be greatly appreciated
cheers

Answer 1

Nope, don't crosspost. Please stay in your [original thread|http://forums.sun.com/thread.jspa?threadID=5350452&messageID=10520330#10520330]. I'm locking this one.