Problem in dragging an object.

[Neeba Rebbaca]

I have drawn a group of rectangles, i want to move and place it in a new place by draggging it. But it couldn't able to move it.

Paint method
protected void paintComponent(Graphics g)
{ 
            super.paintComponent(g);
            g.create();
            Graphics2D g2 = (Graphics2D)g;
            g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING,RenderingHints.VALUE_ANTIALIAS_ON);
            g2.setColor(getBackground());
            setBackground(Color.WHITE);
            drawPanel(g2);
            drawCalendar(g2);
            drawEquipment(g2);
}
// method for drawing rectangles
void drawEquipment(Graphics g)
{
            Graphics2D g2 = (Graphics2D)g;
            g2.setPaint(new Color(19, 146, 172));
            
            rectEquip = new  ArrayList<ArrayList<Rectangle>>();
            
            for(int i=0;i<equiment_rectList.size();i++)
            {
                int x = equiment_rectList.get(i).x+equiment_rectList.get(i).height+5;
                int y = equiment_rectList.get(i).y+equiment_rectList.get(i).height+5;
                
                g2.fill(equiment_rectList.get(i));
                g2.drawString(equipment_list.get(i),(equiment_rectList.get(i).x+equiment_rectList.get(i).width)+5,equiment_rectList.get(i).y+7);
                g2.drawLine(x-35, y, getWidth(),y);
                ArrayList><Rectangle> list  = new ArrayList<Rectangle>();
                for(int j=0;j<rect_list.get(i).size();j++)
                {
                   Rectangle rect = new Rectangle((Rectangle)rect_list.get(i).get(j));
                   g2.fillRect(rect.x+200,y-10,rect.width,rect.height);
                   list.add(new Rectangle(rect.x+200,y-10,rect.width,rect.height));
                }
                rectEquip.add(list);
            }
            
            repaint();
 }
public void mousePressed(MouseEvent e) 
{
            for(int i=0;i<rectEquip.size();i++)
            {
                for(int j=0;j<rectEquip.get(i).size();j++)
                {
                    if(((Rectangle)rectEquip.get(i).get(j)).contains(e.getPoint()))
                    {
                        index   = i;
                        x_Pos   = e.getX();
                        canDrag = true;
                        break;
                    }
                }
            }
}
public void mouseDragged(MouseEvent e) 
{
	int shiftedx = 0;
            	int arr_len  = rectEquip.get(index).size()-1;
            	if(canDrag)
            	{
	
		ArrayList><Rectangle> rect = new ArrayList<Rectangle>();
            
            for(int i=0;i<rectEquip.get(index).size();i++)
            {
                Point vp_p      = rectEquip.get(index).get(i).getLocation();
                shiftedx        =  vp_p.x + e.getX() - x_Pos;
                //rectEquip.set(index).get(i).x   =  e.getX();
                Rectangle tempRectangle = new Rectangle(e.getX(),rectEquip.get(index).get(i).y,rectEquip.get(index).get(i).width,rectEquip.get(index).get(i).height);
                rect.add(tempRectangle);
                //rectEquip.set(index, equiment_rectList)
                //rectEquip.get(index).get(0).x         = Math.max(rectEquip.get(index).get(0).x,200);
                //rectEquip.get(index).get(arr_len).x   = Math.min(rectEquip.get(index).get(arr_len).x,getWidth()-rectEquip.get(index).get(arr_len).width);
                //= Math.min(rectEquip.get(index)(arr_len).x,getWidth()-rectEquip.get(index).get(arr_len).width);
            }
            rectEquip.set(index, rect);
            repaint();
                	this.repaint();
            }
 }

this is my code. Kindly help in solving the problem.>

[Darryl Burke]

First off, I spot this in your code:

paintComponent(...) calls drawEquipment(...)

drawEquipment(...) calls repaint()

That's a strict no-no, as it's going to bog down the computer with painting events. Another no-no is inclusion of any business logic in painting methods, and I suspect you have that too. Then, I see two successive calls to repaint() at lines 81 and 82. Why?

edit: You hve a call to g.create() at line 6 but you don't assign the value to a variable. The result is creation of a Graphics object that is never dispose()d, wasting resources.

You also appear to be using parallel ArrayLists, and the code contains a lot of magic numbers. I strongly suggest you OOPify your code by creating a class that represents the parameters of an Equipment and knows how and where to paint itself.

You can get some coding ideas regarding the above, including dragging of shapes, here.

To get better help sooner, post a SSCCE (Short, Self Contained, Compilable and Executable) example that demonstrates your problem.