Homework help please

[chick]

hello all,
I am new to java and am having problems finding information on how to do this problem for my class here is the problem, and then I have also including the code I have so far. any help is appreciated thanks!

FRAME
Create a new method:
public Picture frame(int frameSize, Color frameColor)
in Picture.java. This method will create and return a new picture object. The new picture object
will contain the original picture inside a frame. The thickness of the frame is given by the parameter
frameSize. The color of the frame is given by the parameter frameColor. The new picture object
will be larger than the original because of the frame.

MIRROR
Create a new method:
public Picture mirror()
in Picture.java. This method will create and return a new picture object. The new picture object
will contain a mirrored version of the original picture. The mirroring will be done across a vertical axis,
so things that were on the left will now be on the right and things that were on the right will now be on
the left.

MIRRORFRAME
Create a new method:
public Picture mirrorFrame(int frameSize, Color frameColor)
in Picture.java. This method will create and return a new picture object. The new picture object
will contain a mirrored version of the original picture inside a frame. The thickness of the frame is
given by the parameter frameSize. The color of the frame is given by the parameter frameColor.
The mirroring will be the same as in the mirror method.


HERE IS THE CODE I HAVE WRITTEN SO FAR:
however when compiling i do receive an error that says:

1 error found:
File: C:\Users\Matt\Desktop\bookClasses\Picture.java [line: 92]
Error: C:\Users\Matt\Desktop\bookClasses\Picture.java:92: reached end of file while parsing



public Picture frame(int frameSize, Color frameColor)
{



public Picture mirror()
{
int width= this.getWidth();
int mirrorPoint = width / 2;
Pixel leftPixel = null;
Pixel rightPixel = null;
for (int y = 0; y < getHeight(); y++)
{
for (int x = 0; x < mirrorPoint; x++)
{
leftPixel = getPixel(x, y);
rightPixel = getPixel(width - 1 - x, y);
rightPixel.setColor(leftPixel.getColor());
}
}
}



THANKS IN ADVANCE =)

[Fubarable]

Can you post your entire Picture class using code tags? For details on how to do this, please see my signature below.

Much luck!

[chick]

import java.awt.*;
import java.awt.font.*;
import java.awt.geom.*;
import java.awt.image.BufferedImage;
import java.text.*;
import java.util.*;
import java.util.List; // resolves problem with java.awt.List and java.util.List

/**
* A class that represents a picture. This class inherits from
* SimplePicture and allows the student to add functionality to
* the Picture class.
*
* Copyright Georgia Institute of Technology 2004-2005
* @author Barbara Ericson ericson@cc.gatech.edu
*/
public class Picture extends SimplePicture
{
///////////////////// constructors //////////////////////////////////

/**
* Constructor that takes no arguments
*/
public Picture ()
{
/* not needed but use it to show students the implicit call to super()
* child constructors always call a parent constructor
*/
super();
}

/**
* Constructor that takes a file name and creates the picture
* @param fileName the name of the file to create the picture from
*/
public Picture(String fileName)
{
// let the parent class handle this fileName
super(fileName);
}

/**
* Constructor that takes the width and height
* @param width the width of the desired picture
* @param height the height of the desired picture
*/
public Picture(int width, int height)
{
// let the parent class handle this width and height
super(width,height);
}

/**
* Constructor that takes a picture and creates a
* copy of that picture
*/
public Picture(Picture copyPicture)
{
// let the parent class do the copy
super(copyPicture);
}

/**
* Constructor that takes a buffered image
* @param image the buffered image to use
*/
public Picture(BufferedImage image)
{
super(image);
}

////////////////////// methods ///////////////////////////////////////

public Picture frame(int frameSize, Color frameColor)
{



public Picture mirror()
{
int width= this.getWidth();
int mirrorPoint = width / 2;
Pixel leftPixel = null;
Pixel rightPixel = null;
for (int y = 0; y < getHeight(); y++)
{
for (int x = 0; x < mirrorPoint; x++)
{
leftPixel = getPixel(x, y);
rightPixel = getPixel(width - 1 - x, y);
rightPixel.setColor(leftPixel.getColor());
}
}
}





















// this is the end of class Picture, put all new methods before this


before the methods line, I have left untouched. hope this is what you were looking for.

[Fubarable]

Please edit the post above to add code tags (as suggested). Much luck.

edit: also, have you matched all your curly braces, including the main one for the class itself? Are you missing a closing brace "}"?

[chick]

import java.awt.*;
import java.awt.font.*;
import java.awt.geom.*;
import java.awt.image.BufferedImage;
import java.text.*;
import java.util.*;
import java.util.List; // resolves problem with java.awt.List and java.util.List

/**
 * A class that represents a picture.  This class inherits from 
 * SimplePicture and allows the student to add functionality to
 * the Picture class.  
 * 
 * Copyright Georgia Institute of Technology 2004-2005
 * @author Barbara Ericson [email]ericson@cc.gatech.edu[/email]
 */
public class Picture extends SimplePicture 
{
  ///////////////////// constructors //////////////////////////////////
  
  /**
   * Constructor that takes no arguments 
   */
  public Picture ()
  {
    /* not needed but use it to show students the implicit call to super()
     * child constructors always call a parent constructor 
     */
    super();  
  }
  
  /**
   * Constructor that takes a file name and creates the picture 
   * @param fileName the name of the file to create the picture from
   */
  public Picture(String fileName)
  {
    // let the parent class handle this fileName
    super(fileName);
  }
  
  /**
   * Constructor that takes the width and height
   * @param width the width of the desired picture
   * @param height the height of the desired picture
   */
  public Picture(int width, int height)
  {
    // let the parent class handle this width and height
    super(width,height);
  }
  
  /**
   * Constructor that takes a picture and creates a 
   * copy of that picture
   */
  public Picture(Picture copyPicture)
  {
    // let the parent class do the copy
    super(copyPicture);
  }
  
  /**
   * Constructor that takes a buffered image
   * @param image the buffered image to use
   */
  public Picture(BufferedImage image)
  {
    super(image);
  }
  
  ////////////////////// methods ///////////////////////////////////////
  
public Picture frame(int frameSize, Color frameColor)
{
  
  
  
public Picture mirror()
  {
    int width= this.getWidth();
    int mirrorPoint = width / 2;
    Pixel leftPixel = null;
    Pixel rightPixel = null;
    for (int y = 0; y < getHeight(); y++)
    {
      for (int x = 0; x < mirrorPoint; x++)
      {
        leftPixel = getPixel(x, y);
        rightPixel = getPixel(width - 1 - x, y);
        rightPixel.setColor(leftPixel.getColor());
      }
    }
}

// this is the end of class Picture, put all new methods before this


before the methods line, I have left untouched. hope this is what you were looking for.[/QUOTE]

[gcalvin]

You pretty much have the right idea with your mirror() method, but when you take a pixel from the left and put it on the right, you need to also take the pixel that was on the right and put it on the left. This means you're going to need a tempPixel (tempColor?) variable to store it in.

    Color tempColor = rightPixel.getColor(); // not sure about the Color type here -- check that
    rightPixel.setColor(leftPixel.getColor());
    leftPixel.setColor(tempColor);

Hope that helps. Your frame() method should be easy enough, and of course mirrorFrame() will simply call mirror() then frame().

-Gary-

[chick]

public Picture mirror()
{
  int width = this.getWidth();
  int mirrorPoint = width / 2;
  Pixel leftPixel = null;
  Pixel rightPixel = null;
  for (int y = 0; y < getHeight(); y++)
  {
    for (int x = 0; x < mirrorPoint; x++)
    {
      leftPixel = getPixel(x,y);
      rightPixel = getPixel(width - 1 - x, y);
    }
  }
}}

im getting a missing return statement error
or when that doesn't happen I am getting an incorrect parsing error. I checked all my brackets and they do coorelate, and I am having trouble starting the frame method, do I scaleUp the picture first or is there a way to create a frame using java a different iway. this is the method I am having the most trouble with. can anyone help describe how the loops might go, I can't figure out the way to create a frame.

[gcalvin]

Well, in the last code you posted, you had this:

  
public Picture frame(int frameSize, Color frameColor)
{
  
  
  
public Picture mirror()
  {
    int width= this.getWidth();
...

So, yeah, you were missing a closing brace on your frame() method. Unless you have fixed that, it will definitely cause you problems.

-Gary-

[chick]

i fixed that, but can you help me start the frame method, do I need to scale up. and now I am getting an error, that says
1 error found:
File: C:\Users\Matt\Desktop\bookClasses\Picture.java [line: 132]
Error: C:\Users\Matt\Desktop\bookClasses\Picture.java:132 : reached end of file while parsing


this is the code I have now :

 
public Picture frame()
  {
    {
    Picture targetPicture =
     new Picture(this.getWidth() * 2,
                 this.getHeight() * 2);
    Pixel sourcePixel = null;
    Pixel targetPixel = null;
    int targetX = 0;
    int targetY = 0;

    for (int sourceX = 0;
         sourceX < this.getWidth();
         sourceX++)
    {
      for (int sourceY = 0;
           sourceY < this.getHeight();
           sourceY++)
      {
        sourcePixel = this.getPixel (sourceX, sourceY);
        for (int indexY = 0; indexY < 2; indexY++)
        {
          for (int indexX = 0; indexX < 2; indexX++)
          {
            targetX = sourceX * 2 + indexX;
            targetY = sourceY * 2 + indexY;
            targetPixel = targetPicture.getPixel(targetX,targetY);
            targetPixel.setColor(sourcePixel.getColor());

          }
        }
      }
    }
    return targetPicture;
  }
}

  
  
public Picture mirror()
{
  int width = this.getWidth();
  int mirrorPoint = width / 2;
  Pixel leftPixel = null;
  Pixel rightPixel = null;
  for (int y = 0; y < getHeight(); y++)
  {
    for (int x = 0; x < mirrorPoint; x++)
    {
      leftPixel = getPixel(x,y);
      rightPixel = getPixel(width - 1 - x, y);
      Color tempColor = rightPixel.getColor();
      rightPixel.setColor(leftPixel.getColor());
      leftPixel.setColor(tempColor);
      
    }
  }
 return targetPicture; 
}

any advice on the Frame method, or how to get rid of my error.

[gcalvin]

What's on line 132? We don't have line numbers here, and your code has omitted the imports and whatever else may have been at the top.

-Gary-

[JosAH]

What's on line 132? We don't have line numbers here, and your code has omitted the imports and whatever else may have been at the top.

The OP should carefully count all curly braces; they should all pair up. Such a compilation error may be tedious to find by hand but most text editors are quite good at finding the missing brace.

kind regards,

Jos

[chick]

import java.awt.*;
import java.awt.font.*;
import java.awt.geom.*;
import java.awt.image.BufferedImage;
import java.text.*;
import java.util.*;
import java.util.List; // resolves problem with java.awt.List and java.util.List

/**
 * A class that represents a picture.  This class inherits from 
 * SimplePicture and allows the student to add functionality to
 * the Picture class.  
 * 
 * Copyright Georgia Institute of Technology 2004-2005
 * @author Barbara Ericson ericson@cc.gatech.edu
 */
public class Picture extends SimplePicture 
{
  ///////////////////// constructors //////////////////////////////////
  
  /**
   * Constructor that takes no arguments 
   */
  public Picture ()
  {
    /* not needed but use it to show students the implicit call to super()
     * child constructors always call a parent constructor 
     */
    super();  
  }
  
  /**
   * Constructor that takes a file name and creates the picture 
   * @param fileName the name of the file to create the picture from
   */
  public Picture(String fileName)
  {
    // let the parent class handle this fileName
    super(fileName);
  }
  
  /**
   * Constructor that takes the width and height
   * @param width the width of the desired picture
   * @param height the height of the desired picture
   */
  public Picture(int width, int height)
  {
    // let the parent class handle this width and height
    super(width,height);
  }
  
  /**
   * Constructor that takes a picture and creates a 
   * copy of that picture
   */
  public Picture(Picture copyPicture)
  {
    // let the parent class do the copy
    super(copyPicture);
  }
  
  /**
   * Constructor that takes a buffered image
   * @param image the buffered image to use
   */
  public Picture(BufferedImage image)
  {
    super(image);
  }
  
  ////////////////////// methods ///////////////////////////////////////

public Picture frame()
  {
  {
    Picture targetPicture =
     new Picture(this.getWidth() * 2,
                 this.getHeight() * 2);
    Pixel sourcePixel = null;
    Pixel targetPixel = null;
    int targetX = 0;
    int targetY = 0;

    for (int sourceX = 0;
         sourceX < this.getWidth();
         sourceX++)
    {
      for (int sourceY = 0;
           sourceY < this.getHeight();
           sourceY++)
      {
        sourcePixel = this.getPixel (sourceX, sourceY);
        for (int indexY = 0; indexY < 2; indexY++)
        {
          for (int indexX = 0; indexX < 2; indexX++)
          {
            targetX = sourceX * 2 + indexX;
            targetY = sourceY * 2 + indexY;
            targetPixel = targetPicture.getPixel(targetX,targetY);
            targetPixel.setColor(sourcePixel.getColor());

          }
        }
      }
    }
    return targetPicture;
  }
}

  
  
public Picture mirror()
{
  int width = this.getWidth();
  int mirrorPoint = width / 2;
  Pixel leftPixel = null;
  Pixel rightPixel = null;
  Pixel targetPixel = null;
  for (int y = 0; y < getHeight(); y++)
  {
    for (int x = 0; x < mirrorPoint; x++)
    {
      leftPixel = getPixel(x,y);
      rightPixel = getPixel(width - 1 - x, y);
      Color tempColor = rightPixel.getColor();
      rightPixel.setColor(leftPixel.getColor());
      leftPixel.setColor(tempColor);
      
    }
  }
return targetPicture; 
}
}









 
  
  
  
  
  
  
  
  
  
// this is the end of class Picture, put all new methods before this

with the error :

1 error found:
File: C:\Users\Matt\Desktop\bookClasses\Picture.java [line: 132]
Error: C:\Users\Matt\Desktop\bookClasses\Picture.java:132 : cannot find symbol
symbol : variable targetPicture
location: class Picture

i am sorry i dont know how to do line numbers. this is really frustrating I keep getting different errors and i can't run it to see if I have mirror or frame

[gcalvin]

First of all, I don't know the Picture or SimplePicture classes you're working with, so I'm just going based on the code and instructions you've shown us so far.

For your frame() method, you're given frameSize in pixels as a parameter. You'll create a new Picture, and its width will be 2 * frameSize + oldWidth, and its height will be 2 * frameSize + oldHeight. Now, I don't know if your Picture class provides any methods for slapping a whole image into a specified place on a new Picture object -- if it does provide them, use them. What you've shown us so far is getPixel(), setPixel(), getColor() and setColor(). If that's all you've got, then you need to go something like this:

    for frameSize rows
        for 0 to newWidth
            set pixel x, y to frameColor

That gets you to the bottom of the top frame. Now:

    for oldHeight rows
        for 0 to frameSize
            set pixel x, frameSize + y to frameColor
        for 0 to oldWidth
            get oldPixel x, y from oldPicture
            get oldColor from oldPixel
            set pixel frameSize + x, frameSize + y to oldColor
        for 0 to frameSize
            set pixel frameSize + oldWidth + x, frameSize + y to frameColor

Now you just have to do the bottom part of the frame, which is pretty much like the top:

    for frameSize rows
        for 0 to newWidth
            set pixel x, frameSize + oldHeight + y to frameColor

Again, this is doing things a pixel at a time. It will work, but if you have methods available that allow you to do things in blocks, you should do that.

-Gary-

[gcalvin]

By the way, you don't want to scale the Picture -- you just want to place it in the center of the new Picture.

-Gary-

[JosAH]

i am sorry i dont know how to do line numbers. this is really frustrating I keep getting different errors and i can't run it to see if I have mirror or frame

Does your mirror() method define a variable 'targetPicture'? If not, why are you trying to return one?

kind regards,

Jos

[gcalvin]

Jos makes an excellent point, and since you're generating a new Picture for the mirror() method, you don't need to do the swapping I described earlier. You just go all the way across each row of the old Picture, pulling pixels from the left and putting them on the right of the new Picture. Write your method so that you have an optional offset to x and y -- that way you can use it in mirrorFrame() as well. You don't have to do your frame() row by row like I described above. You can do the top, the left, the right, the bottom, and the Picture independently (and it's probably better -- simpler and more efficient -- than the row-by-row design I described above).

-Gary-

[chick]

Thanks guys its helping a lot. I am down to the last stretch and am getting unknown errors and am missing the last part of the frame. is anyone on now to help?? I have another hour and forty minutes. here is my code with error.

import java.awt.*;
import java.awt.font.*;
import java.awt.geom.*;
import java.awt.image.BufferedImage;
import java.text.*;
import java.util.*;
import java.util.List; // resolves problem with java.awt.List and java.util.List
/**
 * A class that represents a picture.  This class inherits from 
 * SimplePicture and allows the student to add functionality to
 * the Picture class.  
 * 
 * Copyright Georgia Institute of Technology 2004-2005
 * @author Barbara Ericson ericson@cc.gatech.edu
 */
public class Picture extends SimplePicture 
{
  ///////////////////// constructors //////////////////////////////////
  
  /**
   * Constructor that takes no arguments 
   */
  public Picture ()
  {
    /* not needed but use it to show students the implicit call to super()
     * child constructors always call a parent constructor 
     */
    super();  
  }
  
  /**
   * Constructor that takes a file name and creates the picture 
   * @param fileName the name of the file to create the picture from
   */
  public Picture(String fileName)
  {
    // let the parent class handle this fileName
    super(fileName);
  }
  
  /**
   * Constructor that takes the width and height
   * @param width the width of the desired picture
   * @param height the height of the desired picture
   */
  public Picture(int width, int height)
  {
    // let the parent class handle this width and height
    super(width,height);
  }
  
  /**
   * Constructor that takes a picture and creates a 
   * copy of that picture
   */
  public Picture(Picture copyPicture)
  {
    // let the parent class do the copy
    super(copyPicture);
  }
  
  /**
   * Constructor that takes a buffered image
   * @param image the buffered image to use
   */
  public Picture(BufferedImage image)
  {
    super(image);
  }
  

  ////////////////////// methods ///////////////////////////////////////

public Picture frame(int frameSize, Color frameColor)
  {
    {
    Picture targetPicture =
     new Picture(this.getWidth() + frameSize *2,
                 this.getHeight() + frameSize *2);
        
    Pixel sourcePixel = null;
    Pixel targetPixel = null;
    int targetX = 0;
    int targetY = 0;

      Pixel[]pixelArray = framepic.getPixels();
      for (int sourceX = 0;
         sourceX < this.getWidth();
         sourceX++)
    {
      for (int sourceY = 0;
           sourceY < this.getHeight();
           sourceY++)
      {
        sourcePixel = this.getPixel (sourceX, sourceY);
        for (int indexY = 0; indexY < 2; indexY++)
        {
          for (int indexX = 0; indexX < 2; indexX++)
          {
            targetX = sourceX * 2 + indexX;
            targetY = sourceY * 2 + indexY;
            targetPixel = targetPicture.getPixel(targetX,targetY);
            targetPixel.setColor(sourcePixel.getColor());
            

          }
        }
      }
    }
    return framePic;
  }


  
  

[COLOR="Red"]public Picture mirror()[/COLOR][/COLOR]
{
  int width = this.getWidth();
  int mirrorPoint = width / 2;
  Pixel leftPixel = null;
  Pixel rightPixel = null;
  Pixel targetPixel = null;
  for (int y = 0; y < getHeight(); y++)
  {
    for (int x = 0; x < mirrorPoint; x++)
    {
      leftPixel = getPixel(x,y);
      rightPixel = getPixel(width - 1 - x, y);
      Color tempPixel = rightPixel.getColor();
      rightPixel.setColor(leftPixel.getColor());
      leftPixel.setColor(tempPixel);
      
    }
  }
}
return mirrorPic;
}
}

[COLOR="red"]public Picture mirrorFrame(int frameSize, Color frameColor)[/COLOR]
  {
    frame(int frameSize, Color frameColor);
    [COLOR="red"]mirror();[/COLOR]
[COLOR="red"]  }[/COLOR]

// this is the end of class Picture, put all new methods before this

5 errors found:
File: C:\Users\Matt\Desktop\bookClasses\Picture.java [line: 117]
Error: C:\Users\Matt\Desktop\bookClasses\Picture.java:117 : illegal start of expression
File: C:\Users\Matt\Desktop\bookClasses\Picture.java [line: 117]
Error: C:\Users\Matt\Desktop\bookClasses\Picture.java:117 : ';' expected
File: C:\Users\Matt\Desktop\bookClasses\Picture.java [line: 141]
Error: C:\Users\Matt\Desktop\bookClasses\Picture.java:141 : class, interface, or enum expected
File: C:\Users\Matt\Desktop\bookClasses\Picture.java [line: 144]
Error: C:\Users\Matt\Desktop\bookClasses\Picture.java:144 : class, interface, or enum expected
File: C:\Users\Matt\Desktop\bookClasses\Picture.java [line: 145]
Error: C:\Users\Matt\Desktop\bookClasses\Picture.java:145 : class, interface, or enum expected



all the error points are in red. anything will help. I cant hand it in unless it atleast compiles i have been doing this for days

[chick]

all i am stuck on now is what the return statement for the frame is

[iluxa]

Extra curly bracket before "return mirrorPic;"

[iluxa]

and for frame(): try "return mirror ();"