CODEWITHSUNDEEP
javaimagefor-looppixelmirror
Rasmus Hjorth Lüdeking
Rasmus Hjorth Lüdeking

Reputation: 780

Flipping an image in java

I'm trying to flip an image in java, but I ran into a problem that I just cannot figure out.

Using a double-for-loop, I'm running through all pixelvalues in the org image, and copying them to the new image in the pixel(width-x-1,y).

The following picture depicts, what happens when I run the method.

http://tinypic.com/r/nqzsiu/8

The new image, only seems to run through 1/10 of the image' height for some reason. Why is this and how do I fix it?

private Image img;
private Pixel pixel;
private Image newimg;
private Pixel newpixel;

public advancedFilters(Image img)
{
    this.img = img;
}

public Image Mirror(){
    newimg = new Image(img.getWidth(), img.getHeight(), "newimage");
    int curtone;
    for (int j=0; j<img.getHeight(); j++) {
        for (int i=0; i<img.getWidth(); i++) {
            pixel = img.getPixel(i,j);
            curtone = pixel.getValue();
            newpixel = newimg.getPixel(newimg.getWidth()-i-1, j);
            newpixel.setValue(curtone);

        }
    }
    return newimg;
}

Upvotes: 0

Views: 4045

Answers (2)

Yurippenet
Yurippenet

Reputation: 234

This is not containing the complete question. You just need to call newImg.pixelsUpdated(); before you return it. But there is NO way StackOverflow users can know this.

Upvotes: 0

Gnarlywhale
Gnarlywhale

Reputation: 4240

Why are you storing the temporary values private Pixel pixel and private Pixel newpixel outside of the scope where they're being used? This is unnecessary (from the code that you posted at least) and is asking for trouble. Try this implementation of your algorithm (which appears correct) instead:

private Image img;
private Pixel pixel;
private Image newimg;
private Pixel newpixel;

public advancedFilters(Image img)
{
    this.img = img;
}

public Image Mirror(){
    newimg = new Image(img.getWidth(), img.getHeight(), "newimage");
    for (int j=0; j<img.getHeight(); j++) {
        for (int i=0; i<img.getWidth(); i++) {
            Pixel oldPixel = img.getPixel(i,j);
            int oldPixelVal = oldPixel.getValue();
            Pixel mirrorPixel = newimg.getPixel((newimg.getWidth()-i-1), j);
            mirrorPixel.setValue(curtone);

        }
    }
    return newimg;
}

Upvotes: 1

Related Questions