Flickering screen and unable to create smooth moving

Question

hey, i have been trying to do a player moving in a frame according to cordinations(spl) i'll recieve from the mouse by clicking. well
the plan was to repaint() the screen everytime he moves and then see it moving along the screen.
well, all i saw in the input that that the X,Y are changing correctly but the picture of the "player" is stuck and only repaints only when its in the requested location.
Moreover, there is a weird flickering which is very very annoying :(
ok
enjoy the code
import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
public class GameFrame extends JFrame {
     private ImageIcon battleground= new ImageIcon ("Grass.jpg");
     private int prefferedWidth=1280;
     private int prefferedHeight=920;
     private Player p=new Player();
     public GameFrame() {
          super();
          setLayout(new GridBagLayout());
          setBackground(Color.blue);
          setVisible(true);
          setSize(prefferedHeight, prefferedWidth);
          setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
addMouseListener(new MouseAdapter() { public void mouseClicked(MouseEvent e1) {
               Point point=new Point(e1.getPoint());
               //point.setLocation( p.getX()+(p.getPic().getIconWidth()/2), (p.getY()+(p.getPic().getIconHeight()/2)));
               while ((point.equals(p.getLocation())==false)) {
               //System.out.println(p.getLocation());
if (point.getX()>p.getX()) {
     p.setX(p.getX()+p.getSpeed());
                                                                                                                   try{Thread.sleep(2);} catch (Exception e) {};
System.out.println("1:"+p.getLocation()+ point+" "+(point.equals(p.getLocation())));
else {
if (point.getX()<p.getX()) {
     p.setX(p.getX()-p.getSpeed());
try{Thread.sleep(2); } catch (Exception e) {};
System.out.println("2:"+p.getLocation()+point+" "+(point.equals(p.getLocation())));
if (point.getY()>p.getY()) {
     p.setY(p.getY()+p.getSpeed());
     try{Thread.sleep(2);} catch (Exception e) {};
System.out.println("4:"+p.getLocation()+point+" "+(point.equals(p.getLocation())));
else{          if (point.getY()<p.getY()) {
     p.setY(p.getY()-p.getSpeed());
               try{Thread.sleep(2);} catch (Exception e) {};
               System.out.println("3:"+p.getLocation()+point+(point.equals(p.getLocation())));
     public void paint (Graphics g) {
          g.setColor(Color.white);
          g.fillRect(0,0,prefferedWidth,prefferedHeight);
          int H=battleground.getIconHeight();
          int W=battleground.getIconWidth();
     for (int i=0; i<=1280/*prefferedHeight*/; i=i+W) {
          for (int y=0;y<= 960 /*prefferedWidth*/;y=y+H) {
               battleground.paintIcon(this,g,i,y);
     p.getPic().paintIcon(this,g, (int) p.getX(), (int) p.getY());
     }

Answer 1

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
//attributes
public class GameFrame extends JFrame {
private ImageIcon battleground= new ImageIcon ("Grass.jpg");
private int prefferedWidth=1280;
private int prefferedHeight=920;
private Player p=new Player(); //Player has own picutre, and location basically
public GameFrame() {
super();
setLayout(new GridBagLayout());
setBackground(Color.blue);
setVisible(true);
setSize(prefferedHeight, prefferedWidth);
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); //have no idea what that does
// using a mouseclick listener to run those following commands:
addMouseListener(new MouseAdapter() { public void mouseClicked(MouseEvent e1) {
//getting the location of click point
Point point=new Point(e1.getPoint());
//aslong as the location of the player and the location of the requested cordinations are different move the player towards its point. Also Player.speed==1.
while ((point.equals(p.getLocation())==false)) {
//System.out.println(p.getLocation());
if (point.getX()>p.getX()) {
p.setX(p.getX()+p.getSpeed());
try{Thread.sleep(2);} catch (Exception e) {};
System.out.println("1:"+p.getLocation()+ point+" "+(point.equals(p.getLocation())));
else {
if (point.getX()<p.getX()) {
p.setX(p.getX()-p.getSpeed());
try{Thread.sleep(2); } catch (Exception e) {};
System.out.println("2:"+p.getLocation()+point+" "+(point.equals(p.getLocation())));
if (point.getY()>p.getY()) {
p.setY(p.getY()+p.getSpeed());
try{Thread.sleep(2);} catch (Exception e) {};
System.out.println("4:"+p.getLocation()+point+" "+(point.equals(p.getLocation())));
else{ if (point.getY()<p.getY()) {
p.setY(p.getY()-p.getSpeed());
try{Thread.sleep(2);} catch (Exception e) {};
System.out.println("3:"+p.getLocation()+point+(point.equals(p.getLocation())));
//the painiting stuff
public void paint (Graphics g) {
g.setColor(Color.white);
g.fillRect(0,0,prefferedWidth,prefferedHeight);
int H=battleground.getIconHeight();
int W=battleground.getIconWidth();
for (int i=0; i><=1280/*prefferedHeight*/; i=i+W) {
for (int y=0;y<= 960 /*prefferedWidth*/;y=y+H) {
battleground.paintIcon(this,g,i,y);
p.getPic().paintIcon(this,g, (int) p.getX(), (int) p.getY());
}