At a dead end. A java student in need of help.

Question

I've hit a wall.
The program is supposed to allow the user to draw five different polygons using the polygon class and let the user pick the color for each polygon and move each polygon around the screen once it is filled in. The user clicks to add the points and then double clicks to fill in the polygon. Then the polygon is added to the array. Up to the dragging around the screen the program works fine. When I click on a polygon with the shift down nothing happens and I've spent several hours trying to figure out why.
I've placed several System.out.println's in the code and I think I've narrowed down the problem. (I hope) I think it might have something to do with the canvas repainting. Because the S.O.P's show that the program is going through and executing that code. It might also have something to the do with my polygon array, but to my knowledge that is setup fine.
Thanks a bunch if you try to help.
Brad
// Full Code Below
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class poly extends JFrame implements ActionListener
     /* Accessible Variables */
     private JPanel canvas;
     private JPanel mp;
     private JComboBox choice;
     private JButton reset;
     private Podly polyArray[] = new Podly[5];
     private Podly currentPoly = new Podly();
     private Podly movingPoly = new Podly();
     private int count;
     private Color currentColor;
     private Point firstPoint;
     private boolean newPoly = true;
     private boolean moving = false;
     private boolean overflowed = false;
     poly()
          setTitle("Polygon");
          setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
          count = 0;
          /* Interactive Buttons and Menus */
          JMenuBar tp = new JMenuBar();
          JMenu color = new JMenu("Color");
          JMenuItem MBlack = new JMenuItem("Black");
          color.add(MBlack);
          JMenuItem MRed = new JMenuItem("Red");
          color.add(MRed);
          JMenuItem MGreen = new JMenuItem("Green");
          color.add(MGreen);
          JMenuItem MYellow = new JMenuItem("Yellow");
          color.add(MYellow);
          MBlack.addActionListener(this);
          MRed.addActionListener(this);
          MGreen.addActionListener(this);
          MYellow.addActionListener(this);
          tp.add(color);
          canvas = new CanvasArt();
          canvas.setBackground(Color.white);
          canvas.addMouseListener(new Points());
          canvas.addMouseMotionListener(new Moved());
          choice = new JComboBox();
          choice.addItem("Black");
          choice.addItem("Red");
          choice.addItem("Green");
          choice.addItem("Yellow");
          choice.addActionListener(this);
          reset = new JButton("Reset");
          reset.addActionListener(this);
          JLabel chooseColor = new JLabel("Choose a color:");
          JLabel holdShift = new JLabel("Hold shift, click, and drag to move a polygon.");
          mp = new JPanel();
          mp.add(chooseColor);
          mp.add(choice);
          mp.add(reset);
          mp.add(holdShift);
          setJMenuBar(tp);
          getContentPane().add(canvas);
          getContentPane().add(mp, "South");          
     public static void main(String [] args)
          JFrame main = new poly();
          main.setSize(600, 600);
          main.setVisible(true);
     public void actionPerformed(ActionEvent e)
          Object test = e.getSource();
          Object selection;
          if (test instanceof JComboBox)
               JComboBox source = (JComboBox)e.getSource();
               selection = source.getSelectedItem();
               if (selection.equals("Black")) currentColor = Color.black;
               else if (selection.equals("Red")) currentColor = Color.red;
               else if (selection.equals("Green")) currentColor = Color.green;
               else if (selection.equals("Yellow")) currentColor = Color.yellow;
          else if (test instanceof JButton) // Repaints if Reset Button is Pressed
               repaint();
               currentPoly.reset();
               count = 0;
               overflowed = false;
          else
               JMenuItem source = (JMenuItem)e.getSource();
               selection = source.getText();
               if (selection.equals("Black")) currentColor = Color.black;
               else if (selection.equals("Red")) currentColor = Color.red;
               else if (selection.equals("Green")) currentColor = Color.green;
               else if (selection.equals("Yellow")) currentColor = Color.yellow;
     class Podly extends Polygon // Class adds Color Fuctionality to Polygon class
          Color polyColor;
          void setColor(Color y)
          {polyColor = y;}
          Color getColor()
          {return polyColor;}
     /* Canvas Painting Panel */
     class CanvasArt extends JPanel
          public void paintComponent(Graphics g)
               super.paintComponent(g);
               System.out.println("Canvas is called");
               System.out.println(polyArray);
               for (int i=0; i>count;i++)
                    System.out.println("Loop is going");
                    Podly y = polyArray;
                    g.setColor(y.getColor());
                    g.fillPolygon(y);
                    System.out.println(y);
                    System.out.println("Loop has filled in polygon"); // Test
               System.out.println("painting complete");
     class Points extends MouseAdapter
          public void mousePressed(MouseEvent e)
               Graphics g = canvas.getGraphics();
               if (overflowed) // Checks for overflow in program.
                    g.setColor(Color.RED);
                    Font font = new Font("SansSerif",Font.BOLD, 30);
                    g.setFont(font);
                    g.drawString("OVERFLOW", 10, 30);
                    Font font2 = new Font("SansSerif",Font.BOLD, 20);
                    g.setFont(font2);
                    g.setColor(Color.BLUE);
                    g.drawString("Double Click to Play Again", 10, 50);
                    if (e.getClickCount() == 2) // Allows user to play again.
                         repaint();
                         currentPoly.reset();
                         count = 0;
                         overflowed = false;
               else
                    int x = e.getX();
                    int y = e.getY();
                    if (newPoly)
                         firstPoint = new Point(x,y);
                         if (e.isShiftDown())
                              System.out.println("Gets before Check loop");
                              for (int r=count-1;r>=0;r--)
                                   System.out.println("Inside For Check Loop");
                                   System.out.println(polyArray[r]);
                                   if (!polyArray[r].contains(x,y))          
                                        System.out.println("Point is found");
                                        movingPoly = polyArray[r];
                                        System.out.println("MovingPoly Defined");
                                        moving = true;
                                        System.out.println("Moving is true"); // test
                                        break;
                         else
                              currentPoly.setColor(currentColor);
                              currentPoly.addPoint(x,y);
                              g.fillOval(x,y,1,1);
                              newPoly = false;
                    else
                         if (e.getClickCount() == 2)
                              g.setColor(currentPoly.getColor());
                              g.fillPolygon(currentPoly);
                              polyArray[count] = currentPoly;
                              currentPoly.reset();
                              count++;
                              if (count == polyArray.length)
                              {overflowed = true;}
                              newPoly = true;
                         else
                              g.setColor(currentPoly.getColor());
                              currentPoly.addPoint(x,y);
                              g.drawLine(firstPoint.x, firstPoint.y, x, y);
                              firstPoint.move(x,y);
          public void mouseReleased(MouseEvent e)
               if(e.isShiftDown())
               { moving = false; }
     class Moved extends MouseMotionAdapter
          public void mouseDragged(MouseEvent e)
               if (moving && e.isShiftDown())
                    int x = e.getX();
                    int y = e.getY();
                    System.out.println("Gets here");
                    movingPoly.translate((x-firstPoint.x),(y-firstPoint.y));
                    firstPoint.move(x,y);
                    canvas.repaint();

Answer 1

Here is the updated code still with the color error I talked about above.
:) Thanks for all the help everyone.
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class poly extends JFrame implements ActionListener
     /* Accessible Variables */
     private JPanel canvas;
     private JPanel mp;
     private JComboBox choice;
     Podly [] polyArray;
     private Podly currentPoly;
     private Podly movingPoly;
     private int count;
     private Color currentColor;
     private Point firstPoint;
     private Color polyColor;
     private boolean newPoly = true;
     private boolean moving = false;
     private boolean overflowed = false;
     poly()
          setTitle("Polygon");
          setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);  // Exits program on window close.
          polyArray = new Podly[5];
          count = 0;                                      // Set count to zero.
          /* Interactive Buttons and Menus */
          JMenuBar tp = new JMenuBar();
          JMenu color = new JMenu("Color");
          JMenuItem MBlack = new JMenuItem("Black");
          color.add(MBlack);
          JMenuItem MRed = new JMenuItem("Red");
          color.add(MRed);
          JMenuItem MGreen = new JMenuItem("Green");
          color.add(MGreen);
          JMenuItem MYellow = new JMenuItem("Yellow");
          color.add(MYellow);
          MBlack.addActionListener(this);
          MRed.addActionListener(this);
          MGreen.addActionListener(this);
          MYellow.addActionListener(this);
          tp.add(color);
          canvas = new CanvasArt();
          canvas.setBackground(Color.white);
          canvas.addMouseListener(new Points());
          canvas.addMouseMotionListener(new Moved());
          choice = new JComboBox();
          choice.addItem("Black");
          choice.addItem("Red");
          choice.addItem("Green");
          choice.addItem("Yellow");
          choice.addActionListener(this);
          JLabel chooseColor = new JLabel("Choose a color:");
          JLabel holdShift = new JLabel("Hold shift, click, and drag to move a polygon.");
          mp = new JPanel();
          mp.add(chooseColor);
          mp.add(choice);
          mp.add(holdShift);
          setJMenuBar(tp);
          getContentPane().add(canvas);
          getContentPane().add(mp, "South");          
     /* Button Listeners */
     public void actionPerformed(ActionEvent e)
          Object test = e.getSource();
          Object selection;
          if (test instanceof JComboBox)
               JComboBox source = (JComboBox)e.getSource();
               selection = source.getSelectedItem();
               if (selection.equals("Black")) currentColor = Color.black;
               else if (selection.equals("Red")) currentColor = Color.red;
               else if (selection.equals("Green")) currentColor = Color.green;
               else if (selection.equals("Yellow")) currentColor = Color.yellow;
          else
               JMenuItem source = (JMenuItem)e.getSource();
               selection = source.getText();
               if (selection.equals("Black")) currentColor = Color.black;
               else if (selection.equals("Red")) currentColor = Color.red;
               else if (selection.equals("Green")) currentColor = Color.green;
               else if (selection.equals("Yellow")) currentColor = Color.yellow;
     class Podly extends Polygon  // Class adds Color Fuctionality to Polygon class
          void setColor(Color y)  // sets the Polygon's color.
          {polyColor = y;}
          Color getColor()   // returns the Polygon's color.
          {return polyColor;}
     /* Canvas Painting Panel */
     class CanvasArt extends JPanel
          public void paintComponent(Graphics g)
               super.paintComponent(g);
                    for (int i=0; i<count;i++)
                         g.setColor(polyArray.getColor());
                         g.fillPolygon(polyArray[i]);
     class Points extends MouseAdapter
          /* Listens for mouse press, constructs polygons, stores them in array. */
          public void mousePressed(MouseEvent e)
               Graphics g = canvas.getGraphics();
               if (overflowed) // Checks for overflow in program.
                    g.setColor(Color.RED);
                    Font font = new Font("SansSerif",Font.BOLD, 30);
                    g.setFont(font);
                    g.drawString("OVERFLOW", 10, 30);
                    Font font2 = new Font("SansSerif",Font.BOLD, 20);
                    g.setFont(font2);
                    g.setColor(Color.BLUE);
                    g.drawString("Double Click to Play Again", 10, 50);
                    if (e.getClickCount() == 2) // allows user to play again by resetting.
                         repaint();
                         count = 0;
                         overflowed = false;
               else
                    int x = e.getX();
                    int y = e.getY();
                    if (newPoly)
                         firstPoint = new Point(x,y);
                         if (e.isShiftDown())
                              for (int r=count-1;r>=0;r--)
                                   if (polyArray[r].contains(x,y))          
                                        movingPoly = polyArray[r];
                                        moving = true;
                                        break; // exits for loop after Polygon with point is found.
                         else
                              currentPoly = new Podly();
                              currentPoly.addPoint(x,y); // Adds the first point.
                              currentPoly.setColor(currentColor); // Sets the Polygon Color
                              g.fillOval(x,y,1,1);
                              newPoly = false;
                    else
                         /* Close the current Polygon at double click,
                         * then moves on to the next. */
                         if (e.getClickCount() == 2)
                              g.setColor(currentPoly.getColor());
                              g.fillPolygon(currentPoly);
                              polyArray[count] = currentPoly;
                              count++;
                              if (count == polyArray.length)
                              {overflowed = true; canvas.repaint();}
                              newPoly = true;
                         else
                              g.setColor(currentPoly.getColor());
                              currentPoly.addPoint(x,y);
                              g.drawLine(firstPoint.x, firstPoint.y, x, y);
                              firstPoint.move(x,y);
          /* Listens for mouse release */
          public void mouseReleased(MouseEvent e)
               if(e.isShiftDown())
               { moving = false; }
     /* Moves selected Polygon around */
     class Moved extends MouseMotionAdapter
          public void mouseDragged(MouseEvent e)
               if (moving && e.isShiftDown())
                    int x = e.getX();
                    int y = e.getY();
                    movingPoly.translate((x-firstPoint.x),(y-firstPoint.y));
                    firstPoint.move(x,y);
                    canvas.repaint();
     public static void main(String [] args)
          JFrame poly = new poly();
          poly.setSize(600, 600);
          poly.setVisible(true);