更新 jlabel

Question

我在更新方法中的 jlabel 时遇到问题。这是我的代码：

    JLabel curStatus = new JLabel("");
JButton jbtnSubmit;

public static void main(String[] args) {
    test gui = new test();
    gui.startGUI();
    // gui.setCurStatus("testing!"); << seems to work here, 
    //but when i call it from another class, it doesn't want to run.
}

// Set up the GUI end for the user
public void startGUI() {
    // These are all essential GUI pieces
    new JTextArea("");
    final JFrame jfrm = new JFrame("my program");
    jfrm.setLayout(new FlowLayout());
    jfrm.setSize(300, 300);
    jfrm.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    jbtnSubmit = new JButton("Submit");

    jfrm.add(jbtnSubmit);
    jfrm.add(curStatus);
    jfrm.setVisible(true);
}

public void setCurStatus(String inCurStatus) {
    curStatus.setText(inCurStatus);
    curStatus.setVisible(true);
}

正在发生的事情是标签 curStatus 没有出现。例如，这是一个调用：

gui1.setCurStatus("Now running diagnostics... Please wait!");

Answer 1

您的问题似乎是错误的引用之一。

以下是创建 GUI 的方法：

  public static void main(String[] args) {
     test gui = new test();
     gui.startGUI();
     // gui.setCurStatus("testing!"); << seems to work here,
     // but when i call it from another class, it doesn't want to run.
  }

您在 main 方法中创建您的“测试”对象（为了符合 Java 命名约定，应该将其命名为“测试”）。由于它是在 main 内部声明的，因此该变量仅在 main 内部具有作用域，并且在其他任何地方都不可见。

然后你告诉我们你正在调用这样的方法：

gui1.setCurStatus("Now running diagnostics... Please wait!");

gui1 变量引用了一个测试类对象，但它可能引用了一个 与正在显示的测试对象不同的对象，因为原始显示的测试对象仅由本地变量引用主要方法。

要解决这个问题，您必须确保在当前显示的测试对象上调用 setCurStatus。如何执行此操作取决于您的其余代码，尽管我们要求您这样做，但您拒绝向我们展示这些内容。

编辑：根据您发布的最新代码（由于缺少方法，我仍然无法编译，createTasksFile()，我的假设是正确的，您正在调用@987654329 @ 在不是显示的 gui 对象上：

  public static String[] runDiagnostics() throws IOException {

     gui gui1 = new gui(); // (A)
     gui1.setCurStatus("Now running diagnostics... Please wait!");

在 (A) 行，您创建一个新的 gui 对象并在其上调用 setCurStatus，但它不是正在显示的 GUI 对象，而是一个完全不同且不相关的对象。唯一的关系是它是与正在显示的对象属于同一类的对象，仅此而已。解决方案是获取对显示的 GUI 的引用并在该对象上调用此方法，并且仅针对该对象。

另外，Robin 的假设是正确的，即使你解决了这个问题，你也会遇到 Swing 并发问题。 JLabel 不会更新，因为 Swing 线程正在尝试打开文件：

  public static String[] runDiagnostics() throws IOException {

     gui gui1 = new gui();
     gui1.setCurStatus("Now running diagnostics... Please wait!");

     int i = 0;
     int errorsI = 0;
     File f = new File("tasks.txt");
     String[] errors = { "", "", "", "", "" };

     // try to create the file three times
     do {
        f.createNewFile();
        i++;
     } while (!f.exists() && i < 3);

所以我们都是对的。解决方案是在后台线程上打开文件，SwingWorker 在这里可以很好地工作。

编辑 2
因此，要解决引用问题，请使用 gui 参数将 gui 的引用传递给 runDiagnostics 方法。然后在这个参数上调用 setCurStatus 方法。例如：

  public static String[] runDiagnostics(gui gui1) throws IOException {

     //!! gui gui1 = new gui(); // !! no longer needed
     gui1.setCurStatus("Now running diagnostics... Please wait!");

调用方法时必须传入 GUI：

        //!! results = taskBckg.runDiagnostics();
        results = taskBckg.runDiagnostics(gui);

另外，请编辑您的所有代码，使其符合 Java 命名约定。所有类名都应以大写字母开头。这让其他人更容易理解您的代码在做什么。

Answer 2

我也会根据您尝试显示的消息进行猜测，因为该问题缺少一些基本信息。基于

“正在运行诊断...请稍候！”

消息，我假设您正在运行诊断程序并尝试在同一线程上更新 UI。您发布的代码不包含任何明显的错误，这可以解释您的调用原因

gui1.setCurStatus("Now running diagnostics... Please wait!");

不会更新标签内容。

你要做的都在Swing concurrency tutorial里解释了。要点是您在Event dispatch thread 上更新 UI，并且您永远不会在该线程上执行大量计算，因为这会阻塞 UI，从而导致糟糕的用户体验。繁重的计算应该在工作线程上完成，例如使用SwingWorker 类，并且只有 UI 的更新（例如用于报告进度）应该在 EDT 上进行。

有了这些信息和链接，您应该能够找到所有相关信息。同样在这个网站上，您会发现多个关于如何使用 SwingWorker 执行后台计算和更新 UI 的示例，例如 my answer on a previous question