4

形状タスクのサイズ変更を完了しようとしていますが、マウスがサイズ変更カーソルに変わった後にのみドラッグすると、最初に同じタイプの別の形状が描画され、この2番目に描画された形状のサイズが変更されます。そのような写真になります:

ここに画像の説明を入力

これは私の関連コードです:

  @Override
public void mouseMoved(MouseEvent e) {

    if (e.isControlDown()) {
        updateShapeUnderMouse(e.getX(), e.getY());
    } // deals with drawing shape if control button held

    //deals with identifying shape to resize
        int x = e.getX();
         int y = e.getY();

    for (int i = myShapes.size() - 1; i >= 0; i--) {
        Shape s = (Shape) myShapes.get(i);
            if (s.isedgePoint(x, y)) {
            ShapetoResize = s;
            setCursor(crnw);
            prevDragX = x;
            prevDragY = y;
            return;
        }
    }

@Override
public void mouseDragged(MouseEvent event) {

         if (event.isControlDown()) {
            if (shapeUnderMouse != null) {
            shapeUnderMouse.setXPos(event.getX());
            shapeUnderMouse.setYPos(event.getY());
            repaint();
        }
    } // deals with moving the shape

  //deals with identifying and resizing shape
    int x = event.getX();
    int y = event.getY();

     if (ShapetoResize != null) {
         if (ShapetoResize instanceof Square) {
            ShapetoResize.resizeSE(x - prevDragX, y - prevDragY);
        } else if (ShapetoResize instanceof Rectangle) { // SAME CODE FOR EACH SHAPE
   }

  repaint();
    }
}

何が起こっているのでしょうか?

public ArrayList<Shape> myShapes = new ArrayList();


@Override
public void paintComponent(Graphics g) {

    super.paintComponent(g);
    int length = myShapes.size();

    for (int i = 0; i < length; i++) {

        myShapes.get(i).paint(g);
    }

}
4

2 に答える 2

3

ShapetoResize.resizeSE(int x, int y);コードのこの部分を見ていなくても、それは単なる推測ですが、メソッドで新しい形状を作成していると思います。つまり、2つの形状を作成していると思います。

またはインターフェース(2つの組み込み形状)でresizeSE使用できるものがないため、これは独自の方法だと思います。もちろん、インターフェイスにはメソッドがないため、おそらくカスタムインターフェイスも使用しています(このShapeインターフェイスのコードを提供しなかったため混乱します)。これらのメソッドのコードを投稿すると、確認できます。RectangleShapepaintShape

これは、コードを機能する方法(およびSSCCE)にまとめる例です。私はこれをRectanglesのみに制限し(Rectanglesが唯一の組み込みであるためShape)、サイズ変更に関係のない多くのものを取り除きました。それでも問題が解決しない場合は、次の例を使用して問題を再現してみてください。

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.util.ArrayList;

public class ShapeResizer extends Box{
    Dimension preferredSize = new Dimension(400,300);
    public ArrayList<Shape> myShapes = new ArrayList();
    //Shape that's targeted for editing
    Shape currentShape;

    public ShapeResizer(){
        super(BoxLayout.X_AXIS);

        //Shapes (because I don't want to make write code for the user to make shapes)
        myShapes.add(new Rectangle(100, 100, 20, 20));
        myShapes.add(new Rectangle(200, 200, 30, 30));

        addMouseMotionListener(new MouseMotionListener() {
            @Override
            public void mouseMoved(MouseEvent e) {

                //deals with identifying shape to resize
                int x = e.getX();
                int y = e.getY();

                boolean foundShape = false;
                for (int i = myShapes.size() - 1; i >= 0; i--) {
                    Shape s = (Shape) myShapes.get(i);
                    if (s.contains(e.getPoint())) {
                        //We found a shape to target
                        currentShape = s;
                        foundShape = true;
                    }
                }

                if(!foundShape){
                    //Reset the shape and cursor only if needed
                    if(currentShape != null){
                        currentShape = null;
                        setCursor(Cursor.getPredefinedCursor(Cursor.DEFAULT_CURSOR));
                    }
                } else {
                    setCursor(Cursor.getPredefinedCursor(Cursor.SE_RESIZE_CURSOR));
                }

            }

            @Override
            public void mouseDragged(MouseEvent event) {
                if (currentShape != null) {
                    resizeShape(currentShape, event.getPoint());
                }
                repaint();
            }
        });
    }

    public void resizeShape(Shape s, Point p){
        if(s instanceof Rectangle){
            Rectangle r = (Rectangle)s;
            r.setSize(p.x - r.x, p.y - r.y);
        }
    }

    public void drawShape(Graphics g, Shape s){
        if(s instanceof Rectangle){
            Rectangle r = (Rectangle)s;
            g.drawRect(r.x, r.y, r.width, r.height);
        }
    }

    @Override
    public void paintComponent(Graphics g) {

        super.paintComponent(g);
        int length = myShapes.size();

        for (int i = 0; i < length; i++) {
            drawShape(g, myShapes.get(i));
        }

    }

    public Dimension getPreferredSize(){
        return preferredSize;
    }   

    public static void main(String[] args) {
        JFrame frame = new JFrame();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setContentPane(new ShapeResizer());
        frame.validate();
        frame.pack();
        frame.setVisible(true);
    }
}
于 2012-12-19T18:29:15.640 に答える
1

形状を追加する背後にあるロジックは何ですか?描画キャンバスを左クリックして図形を追加しますか?もしそうなら、あなたの問題はおそらくクリックでシェイプが追加されており、それがサイズ変更されていることです。オブジェクトをドラッグした後に実行を中断することで、これをデバッグできます。リストには、1つではなく2つの形状が含まれます。

于 2012-12-19T14:30:08.037 に答える