Java RSS Java RSS

Applet question..

Reply

•
•
Join Date: Sep 2008
Posts: 38
Reputation: cproud21 has a little shameless behaviour in the past 
Solved Threads: 0
cproud21 cproud21 is offline Offline
Light Poster

Applet question..

 
0
  #1
Dec 2nd, 2008
I have the following code that produces an applet of two eyeballs. I have a mouseMotionListener that is set so that when the mouse is moved around the pair of eyes, the eyeballs follow the pointer. I have the eyeballs set up in four locations. To the left, right, north, and south in the eye. However, when I move the mouse onto the applet all four are displayed. Is there a simple way so that the correct set of eyes are displayed?

Thank you in advance, here is my code....

package project5;

import java.awt.event.MouseEvent;
import javax.swing.*;
import java.awt.*;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;

public class Main extends JApplet
{
    private int X;
    private int Y; 
    boolean mouseLeft= false;
    boolean mouseRight = false;
    boolean mouseUp = false;
    boolean mouseDown = false;
        
   public void init()
   {
      getContentPane().setBackground(Color.WHITE);  
      addMouseListener(new MyMouseListener());
      addMouseMotionListener(new MyMouseMotionListener());
   }
   
   public void paint(Graphics g)
   {
      super.paint(g);
     //Original
      g.setColor(Color.BLACK);
      g.drawOval(50,50,50,50);
      g.drawOval(200, 50, 50, 50);
      g.fillOval(65,65, 20, 20);
      g.fillOval(215, 65, 20, 20);
      
      
      //this is what is drawn when in zone- left
       if (mouseLeft)
     {
        g.fillOval(45,65, 20,20);
        g.fillOval(195, 65, 20, 20);
         
        g.setColor(Color.WHITE);
        g.fillOval(65,65, 20, 20);
        g.fillOval(215, 65, 20, 20);
       
        
     }
      //right 
      if (mouseRight)
       {
        g.setColor(Color.BLACK);
        g.fillOval(85,65, 20,20);
        g.fillOval(235, 65, 20, 20);
         
        g.setColor(Color.WHITE);
        g.fillOval(65,65, 20, 20);
        g.fillOval(215, 65, 20, 20);
       }
      //up
      if (mouseUp)
       {
        g.setColor(Color.BLACK);
        g.fillOval(65,45, 20,20);
        g.fillOval(215, 45, 20, 20);
         
        g.setColor(Color.WHITE);
        g.fillOval(65,65, 20, 20);
        g.fillOval(215, 65, 20, 20);
       }
      
      //down
      if (mouseDown)
       {
        g.setColor(Color.BLACK);
        g.fillOval(65,85, 20,20);
        g.fillOval(215, 85, 20, 20);
         
        g.setColor(Color.WHITE);
        g.fillOval(65,65, 20, 20);
        g.fillOval(215, 65, 20, 20);
       }
     else 
     {
        g.fillOval(65,65, 20, 20);
     }
      
      
      
      
      
      
   }

   
   private class MyMouseListener implements MouseListener
   {
        
       public void mousePressed(MouseEvent e)
       {
           
          
       }

        public void mouseClicked(MouseEvent e) {
            
        }

        public void mouseReleased(MouseEvent e) {
           
        }

        public void mouseEntered(MouseEvent e) {
           
        }

        public void mouseExited(MouseEvent e) {
            
        }
   }

   
   
   private class MyMouseMotionListener implements MouseMotionListener
   {
       public void mouseDragged(MouseEvent e)
       {
           
       }
       
       public void mouseMoved(MouseEvent e)
       {
             X = e.getX();
                Y = e.getY();
          
          mouseLeft = true;  
               
          if (X <= 50)
          {
            
               repaint();
                 
          }
          
          mouseRight = true;
          if (X >= 250)
          {
              repaint();
          }
          
          mouseUp = true;
          if(Y < 50)
          {
              repaint();
          }
          mouseDown = true;
          if(Y> 250)
          {
              repaint();
          }
          
   }
   }
}
Reply With Quote Quick reply to this message  
•
•
Join Date: Sep 2008
Posts: 1,566
Reputation: BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all 
Solved Threads: 196
BestJewSinceJC BestJewSinceJC is offline Offline
Posting Virtuoso

Re: Applet question..

 
0
  #2
Dec 3rd, 2008
You should only be setting mouseLeft, mouseRight, etc to true if, in fact, the mouse was moved in that sector. When your mouseMoved method gets called, you are setting ALL of them to true, then calling repaint. The way you're currently doing it, if mouseLeft were to be set true, it would always remain true, which is one problem. Also, currently, if mouseDown were true, it would call repaint. However, the mouseMoved method just set mouseUp, mouseDown, mouseLeft, and mouseRight ALL to true - so all of them will be repainted.

hope that helps.
Last edited by BestJewSinceJC; Dec 3rd, 2008 at 1:20 am.
Reply With Quote Quick reply to this message  
•
•
Join Date: Sep 2008
Posts: 1,566
Reputation: BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all BestJewSinceJC is a name known to all 
Solved Threads: 196
BestJewSinceJC BestJewSinceJC is offline Offline
Posting Virtuoso

Re: Applet question..

 
1
  #3
Dec 3rd, 2008
To demonstrate the second point I made above, consider what happens when you move the mouse. Lets say you move the mouse down, but you do not move it in any other direction. The mouseMoved method is called, and mouseLeft is set to true. Now, lets say you eventually make it down to mouseDown = true. Then, repaint is called where you have if (Y > 250). Now, you enter the repaint method. Since mouseLeft is true, and mouseRight is true... both of those things will be repainted.

Now, another thing will happen: since you never reset mouseLeft, mouseRight, mouseUp, and mouseDown to "false"... every time you enter the repaint method, these things might be repainted. Consider using "else if" statements inside of mouseMoved to call repaint, so that repaint is never called when you don't want it to be. Also, make sure that at the end of mouseMoved, you reset all of the mouseUp, down, left, and right variables to false. Also, do not set these variables to "true" until you have determined that they should be true. In other words, setting them to true should be done inside the if statement, not outside of it.

Also, I'm not exactly sure how your assignment works, but you should consider that any time you color something, you are never un-coloring it. So lets say you color something because of mouseDown. What happens when the thing you colored isn't in the "mouseDown" sector anymore? Shouldn't you un-color it?


•
•
•
•
Originally Posted by cproud21 View Post

Java Syntax (Toggle Plain Text) 
  1.  
  2.        public void mouseMoved(MouseEvent e)
  3.        {
  4.              X = e.getX();
  5.                 Y = e.getY();
  6.  
  7.           mouseLeft = true;  
  8.  
  9.           if (X <= 50)
  10.           {
  11.  
  12.                repaint();
  13.  
  14.           }
  15.  
  16.           mouseRight = true;
  17.           if (X >= 250)
  18.           {
  19.               repaint();
  20.           }
  21.  
  22.           mouseUp = true;
  23.           if(Y < 50)
  24.           {
  25.               repaint();
  26.           }
  27.           mouseDown = true;
  28.           if(Y> 250)
  29.           {
  30.               repaint();
  31.           }
  32.  
  33.    }
  34.    }
  35. }
Last edited by BestJewSinceJC; Dec 3rd, 2008 at 1:23 am.
Reply With Quote Quick reply to this message  
Reply

Quick Reply to Thread
This thread is more than three months old.
Perhaps start a new thread instead?
•
•
•
•
•
•
•
•
Message:
•
•
•
•



Similar Threads
Other Threads in the Java Forum
Thread Tools Search this Thread



911 actionlistener addressbook android api append applet application array arrays automation binary blackberry block bluetooth character chat class client code component consumer csv database desktop developmenthelp eclipse error fractal ftp game givemetehcodez graphics gui html ide image integer j2me j2seprojects japplet java javaarraylist javac javaee javaprojects jni jpanel julia lego linked linux list loops mac map method methods mobile netbeans newbie number objects online oriented panel printf problem program programming project projects properties recursion replaydirector reporting researchinmotion rotatetext rsa scanner se server set singleton sms sort sql string swing test textfields threads time title tree tutorial-sample ubuntu update windows working
About Us | Contact Us | Advertise | DaniWeb | Acceptable Use Policy | RSS Feed

©2003 - 2009 DaniWeb® LLC