The Pixel and Photograph classes

> hmm...

>

> how about using this link as a guide for helping me

> to answer my question?

>

> http://www.cs.umd.edu/class/summer2007/cmsc131/Project

> s/P3/proj3.html

How about asking a specific question instead of expecting us to study your homework assignment?

hunter9000a at 2007-7-12 23:00:52 >

I need to be able to:

Extract the data (width, height) of the original,

Extract data for each pixel (Red,blue,green, and position)

Translate it onto a new blank photograph

return the new full picture

From there I can do the manipulations necessary to construct the remaining scenarios.

Yin_Absola at 2007-7-12 23:00:52 >

> I need to be able to:

>

> Extract the data (width, height) of the original,

> Extract data for each pixel (Red,blue,green, and

> position)

> Translate it onto a new blank photograph

> return the new full picture

>

> From there I can do the manipulations necessary to

> construct the remaining scenarios.

Have you studied the Pixel and Photograph apis like the assignment says? If so, what specific problem are you having using them?

(btw, this teacher's supposed to be teaching you how to manipulate images, but he uses jpg for cartoons and screenshots? blech)

hunter9000a at 2007-7-12 23:00:52 >

Extract data for each pixel (Red,blue,green, and position)->

Translate it onto a new blank photograph

that's my main problem... I think the rest is just minor confusion.

I'll show you what I have so far... it''s going to be messy but

public static Photograph copy(Photograph photo) {

Photograph copyPhoto = new Photograph(photo.getHeight(), photo.getWidth());

int xPix, yPix;

int xPixMax=photo.getHeight();

int yPixMax=photo.getWidth();

int red,blue,green;

Pixel pixel=new Pixel (red,blue,green);

for (xPix = 1; xPix <= xPixMax; xPix++){

for (yPix = 1; yPix <= yPixMax; yPix++){

green= photo.getGreen(xPix,yPix);

red= photo.getRed(xPix,yPix);

blue= photo.getBlue(xPix,yPix);

copyPhoto.setPixel(xPix,yPix,pixel);

}

}

return copyPhoto;

Yin_Absola at 2007-7-12 23:00:52 >

A couple basic mistakes:

1. The API states: (Photohgraph's getPixel):

Pixel at position (x, y) in the photograph. Note that the origin (0, 0) is located at the upper left corner of the photo.

so x and y coordinates start with 0, not the 1 in your code.

2. This will not work:

Pixel pixel=new Pixel (red,blue,green);

//later:

green= photo.getGreen(xPix,yPix);

red= photo.getRed(xPix,yPix);

blue= photo.getBlue(xPix,yPix);

copyPhoto.setPixel(xPix,yPix,pixel);

assigning to int variables red, green and blue will not change the state of pixel. You need to create a new Pixel object for every tuple of values.

Also: Photograph does not have getRed/getGreen/getBlue methods. Those are Pixel methods, so apply them to a Pixel object, not a Photograph object.

Message was edited by:

BigDaddyLoveHandles

BigDaddyLoveHandlesa at 2007-7-12 23:00:52 >

public static Photograph copy(Photograph photo) {

Photograph copyPhoto = new Photograph(photo.getHeight(), photo.getWidth());

int xPix, yPix;

int xPixMax=photo.getHeight();

int yPixMax=photo.getWidth();

int red,blue,green;

Pixel pixel= new Pixel (red, blue, green); //this is the only remaining error right now.

for (xPix = 0; xPix <= xPixMax; xPix++){

for (yPix = 0; yPix <= yPixMax; yPix++){

photo.getPixel(xPix,yPix);

green=pixel.getGreen();

red= pixel.getRed();

blue= pixel.getBlue();

Pixel copyPix= new Pixel (red,blue,green);

copyPhoto.setPixel(xPix,yPix,copyPix);

}

}

return copyPhoto;

Yin_Absola at 2007-7-21 23:03:24 >

When you assign to a variable, that overwrites its previous value.

Then this:

Pixel pixel = new Pixel(red, green, blue); //assuming red, green and blue are initialized.

...

pixel= photo.getPixel(xPix,yPix);

Is the same as this:

Pixel pixel = null;

...

pixel= photo.getPixel(xPix,yPix);

Suggestion: define variable when you can give them a value:

Pixel pixel= photo.getPixel(xPix,yPix);

Do you see the difference?

BigDaddyLoveHandlesa at 2007-7-21 23:03:24 >