提问者:小点点

javax. swing.JFrame被覆盖


我正在Windows 7上使用NetBeans 7.3.1上的JavaSE进行开发。

我javamain方法有以下调用

static Vector<Point2D> acceptedByFilter, coords;

// Some code to read the coords from a file

// Some code to filter coords and produce a subset called acceptedByFilter

DisplayInputPoints();
DisplayPointsAcceptedByFilter();

这些方法定义如下

static protected void DisplayPointsAcceptedByFilter(){
    Panel panel=new Panel();
    panel.DisplayInputPoints(acceptedByFilter, xMin, xMax, yMin, yMax, true, "Points                  accepted by filter");
 }

static void DisplayInputPoints(){
    Panel panel=new Panel();
    panel.DisplayInputPoints(coords, xMin, xMax, yMin, yMax, true, "Original Points");
}

Panel. DisplayInputPoint定义如下

import javax.swing.JFrame;
import javax.swing.JPanel;

public 
class Panel extends JPanel  
{
    public static void DisplayInputPoints(Vector<Point2D> coords, double xMin, double xMax, double yMin, 
        double yMax, boolean invert, String label){

        JFrame frame = new JFrame(label);
        Panel panel = new Panel();
        panel.setPreferredSize(new Dimension((int)Math.round(xMax-xMin) + 10, 
            (int)Math.round(yMax-yMin) + 10));
        panel.loadPoints(coords);
        frame.setContentPane(panel);
        frame.pack();
        frame.setVisible(true);
        frame.repaint();
    }
}

当我呼唤

DisplayInputPoints();

出现第一帧并显示和弦向量中的点。当我调用

DisplayPointsAcceptedByFilter();

我得到另一个帧,接受的ByFilter中的点出现在两个帧中。也就是说,第一帧被第二帧中的显示覆盖。

什么是最好的方法来阻止第一帧被应该只在第二帧覆盖?


共2个答案

匿名用户

您的代码结构似乎大错特错,这可能是您的主要问题。例如,

  • 你不应该为一件事使用静态方法或字段。唯一的静态方法应该是main和关联的方法以及实用程序方法。唯一的静态字段应该是类字段,例如常量。
  • 您的DisplayPointsAcctedByFilter创建了一个Panel对象,该对象导致显示第二个JFrame。从Panel类中获取显示代码,它不属于那里并且导致了您的问题。
  • 不要将类命名为与Java核心类同名的类,例如Panel。
  • 你的Panel类甚至不是一个真正的OOP类,而是一个静态方法的持有者。这违背了所有OOP规则。
  • 如果这是我的代码,我会从一开始就使用OOP原则废弃它并重写它。

要获得更详细和更好的帮助,请考虑在您的问题中提供更多信息,并可能创建和发布一个sscce。

编辑
考虑创建:

  • 从文件中读取坐标的类。然后它可以将坐标放入ArrayList。它应该没有静态方法或字段。
  • 用于过滤坐标的类。它也不应该有静态方法或字段,也不应该有GUI代码。
  • 扩展用于显示数据的JPanel的GUI类。
  • GUI类接受坐标数组列表,然后以某种方式绘制坐标。
  • GUI类使用setter方法接受新坐标,然后重新绘制新坐标。
  • 一个带有main方法的类,该方法为要显示的GUI类创建一个JFrame,并协调所有其他类,启动它们。

匿名用户

你有没有想过让Panel成为一个抽象类,也许为了两个目的专门改变它?我的意思是,我觉得你应该展示更多的代码来正确理解问题

相关问题