Ikke modal jdialog viser ikke komponenter

Prøv noget ala dette:
myProgressbar.setText("Doing " + i + " af " + a)
myProgressbar.repaint();

(forudsætter at myProgressbar arver fra et eller andet relevant swing)

No go, der er først når den er færdig med loopet at den skriver den sidste tekst på jdialogen, myProgressbar arver fra JDialog, og den har kun en metode, setTest(String txt). Men som sagt er dialogen stadig tom.

Andre ideér er meget velkommen.

Venlig Hilsen
Jeppe

Må man se din myProgressbar klasse (jeg forstår nemlig ikke dit argument) ?

Ja da, men det virker fint hvis jeg kalder den med myProgressbar.setModal(true), men så køre koden hvor den bliver kaldet fra ikke videre.:

import java.awt.Dimension;
import javax.swing.JDialog;
import java.awt.BorderLayout;
import javax.swing.JPanel;
import java.awt.GridBagLayout;
import javax.swing.JButton;
import java.awt.GridBagConstraints;
import java.awt.Insets;
import javax.swing.JLabel;

public class MyProgressbar extends JDialog  {
    private BorderLayout borderLayout1 = new BorderLayout();
    private GridBagLayout gridBagLayout1 = new GridBagLayout();
    private JPanel jPanel1 = new JPanel();
    private JLabel jLabel1 = new JLabel("TEST 123");

    public MyProgressbar() {       
        try {
            jbInit();
        } catch(Exception e) {
            e.printStackTrace();
        }
    }

    private void jbInit() throws Exception {
        this.setSize(new Dimension(400, 300));       
        this.getContentPane().setLayout(borderLayout1);
        jPanel1.setLayout(gridBagLayout1);       
        jPanel1.add(jLabel1, new GridBagConstraints(0, 0, 1, 1, 0.0, 0.0, GridBagConstraints.CENTER, GridBagConstraints.NONE, new Insets(0, 0, 0, 0), 0, 0));
       
        this.getContentPane().add(jPanel1, BorderLayout.CENTER);
    }

    public void setText(String txt) {
        System.out.println("text set: " + txt);
        jLabel1.setText(txt);
    }
}

Hvad gør " >køre noget kode<" ?

Jeg har nemlig ikke problemer med din bar med følgende kode(a og b er afpasset mit system for at simulere en beregning):
    MyProgressbar bar = new MyProgressbar();
    bar.setVisible(true);
    int foo;
    for(int i=0;i<30;i++)
    {
        for(int a=0;a<2000000;a++)
        for(int b=0;b<20;b++)
        {
            foo = a+b;
        }
        bar.setText("Doing item: " + i);
    }
    bar.dispose();

Du får bare lige hele metoden, det den skal gøre er at lave nogle pdf filer og derefter printe dem. før lavede jeg progressbaren inde i denne metode, da det ikke virkede prøvede jeg at parametere overføre den.

Der er brugt egne klasser så du vil ikke kunne køre den uden lidt modifikation, men nu får du den "råt for usødet"

    public static void printReports(final ApplicationModule am, final String[] report, final JPanel panel, MyProgressbar progressbar) {       
               
        String localFilename;
        int tmp;
        String[] deleteArray = new String[report.length];
        if (System.getProperty("java.io.tmpdir") == null) {
            ValidationErrorHandler.handleError(new UtilException("Util-5075", "Error"), panel);
        } else {       
            try {
                //The URL of the reportsserver to use is fetched in the table ENV_PARAMETERS
                ViewObject envParamView = am.createViewObjectFromQueryStmt("EnvParamView", "select EnvParameters.VALUE as Value from SYSMODULE.ENV_PARAMETERS EnvParameters where EnvParameters.ENV_PARAMETER_NAME = 'REPORTS_SERVER'");
                Row envParam = envParamView.first();
                String repServer = envParam.getAttribute("VALUE").toString();       
                System.out.println("Antal i []: " + report.length);               
                for (int i=0; report.length > i; i++) {
                    System.out.println("Creating file " + (i+1) + " of " + report.length);
                    progressbar.setText("Creating file " + (i+1) + " of " + report.length);
                    progressbar.repaint();
                    //Putting the file in the users temp dir.
                    localFilename = System.getProperty("java.io.tmpdir") + "\\" + "temp" + System.currentTimeMillis() + ".pdf";
                    URL invoke = new URL(repServer + report[i]);
                    FileOutputStream output = new FileOutputStream(localFilename);
                    InputStream input = invoke.openStream();
               
                    tmp = 0;
                    while (tmp != -1) {
                      tmp = input.read();
                      output.write(tmp);
                    }

                    input.close();
                    output.close();
               
                    deleteArray[i] = localFilename;                   
                }
                System.out.println("Printing");
                for (int i=0;deleteArray.length > i;i++) {                   
                    //Runtime.getRuntime().exec("repPrint " + deleteArray[i]);
                }
                envParamView.remove();
                progressbar.dispose();
                System.out.println("disposing");
            } catch (IOException ex) {
                FailureErrorHandler.handleError(ex, new JPanel());
            }
            for (int i=0;deleteArray.length > i;i++) {
            //Deleting the files, not done, because
            //they are deleted before they are printed
    //            boolean val = new File(deleteArray[i]).delete();
            }
        }   
    }

Venlig Hilsen
Jeppe

Bliver disse:
for (int i=0; report.length > i; i++) {
  System.out.println("Creating file " + (i+1) + " of " + report.length);

Printet i konsollen ?

Ja, og den i progressbaren udskriver også:

her er konsollen printout

Antal i []: 3
Creating file 1 of 3
text set: Creating file 1 of 3
Creating file 2 of 3
text set: Creating file 2 of 3
Creating file 3 of 3
text set: Creating file 3 of 3
Printing
disposing
Tid at udskrive 3 Ordre: 82359

Venlig Hilsen
Jeppe

Hvis du har noget gui der skal opdateres i takt med at noget andet kode afvikles, er det en god idé at afvikle det andet kode i en ny tråd. Ellers vil du løbe ind i det problem, at gui'en først opdaterer når koden er færdigkørt.

Så det bliver noget a la:

ProgressBar bar = new ProgressBar();

new Thread()
{
  // afvikel din kode...
  // opdatér progressbar - bar.setString ("bla bla");
}.start();


På den måde låser du ikke gui'en.

Et lille hurtigt eksempel på noget der ikke virker:

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

public class Test extends JFrame
{   
    private JLabel fakeBar = new JLabel();
    private JButton start = new JButton("Kør");

    public Test() throws Exception
    {
        this.setSize(300, 100);
        this.getContentPane().add(fakeBar, BorderLayout.CENTER);
        this.getContentPane().add(start, BorderLayout.SOUTH);
       
        start.addActionListener(new ActionListener()
        {
            public void actionPerformed (ActionEvent e)
            {
                start();   
            }
        });
    }
   
    public void start()
    {
        for (int i = 0; i<20; i++)
        {
            fakeBar.setText("Test " + i);
            try
            {
                Thread.sleep(300);
            }catch (Exception ex) {}
        }
    }
   
    public static void main(String[] args) throws Exception
    {       
        new Test().setVisible(true);
    }
}

Men hvis start() metoden laves om til:

public void start()
{
    new Thread()
    {
        public void run()
        {
            for (int i = 0; i<20; i++)
            {
                fakeBar.setText("Test " + i);
                try
                {
                    Thread.sleep(300);
                }catch (Exception ex) {}
            }
        }
    }.start();
}


Kører det efter hensigten.

Det der forvirre dig er nok at hver gang du har starter noget Swing eller GUI op i jave bliver der tilføjet (som minimum) en extra tråd til dit program - the event-dispatching thread der står for håndteringen af at "tegne på skærmen" og har til opgave at varetage events.

Hvis vi gennemgår din kode....

myProgressbar myProgressbar = new myProgressbar();
myProgressbar.setVisible(true);

for (int i=0;i<a;i++) {
>køre noget kode<
myProgressbar.setText("Doing " + i + " af " + a)
(Her vil din din jlabell.setText(txt) blive kaldt og det kald bliver sat i den kø der "tegner på skærmen" MEN din kode fortsætter)
}
myProgressbar.dispose();

Nå du når til dette punkt så har du disposed din myprogressbar og det er ikke sikkert at det kald der står i køen er blevet udført!

Ok, lyder logisk, nok og synes måske også der er noget der dæmre omkring dette. Men har aldrig brugt tråde, så kan I forklare mig hvordan jeg lige laver denne om så den bruger tråden, som I forklare det:

public static void printReports(ApplicationModule am, String[] report, JPanel panel) {       

        BI2Progressbar bI2Progressbar = new BI2Progressbar();       
        bI2Progressbar.setVisible(true);               
        String localFilename;
        int tmp;
        String[] deleteArray = new String[report.length];
        if (System.getProperty("java.io.tmpdir") == null) {
            ValidationErrorHandler.handleError(new UtilException("Util-5075", "Error"), panel);
        } else {       
            try {
                //The URL of the reportsserver to use is fetched in the table ENV_PARAMETERS
                ViewObject envParamView = am.createViewObjectFromQueryStmt("EnvParamView", "select EnvParameters.VALUE as Value from SYSMODULE.ENV_PARAMETERS EnvParameters where EnvParameters.ENV_PARAMETER_NAME = 'REPORTS_SERVER'");
                Row envParam = envParamView.first();
                String repServer = envParam.getAttribute("VALUE").toString();       
                System.out.println("Antal i []: " + report.length);               
                for (int i=0; report.length > i; i++) {
                    System.out.println("Creating file " + (i+1) + " of " + report.length);
                    progressbar.setText("Creating file " + (i+1) + " of " + report.length);
                    progressbar.repaint();
                    //Putting the file in the users temp dir.
                    localFilename = System.getProperty("java.io.tmpdir") + "\\" + "temp" + System.currentTimeMillis() + ".pdf";
                    URL invoke = new URL(repServer + report[i]);
                    FileOutputStream output = new FileOutputStream(localFilename);
                    InputStream input = invoke.openStream();
               
                    tmp = 0;
                    while (tmp != -1) {
                      tmp = input.read();
                      output.write(tmp);
                    }

                    input.close();
                    output.close();
               
                    deleteArray[i] = localFilename;                   
                }
                System.out.println("Printing");
                for (int i=0;deleteArray.length > i;i++) {                   
                    //Runtime.getRuntime().exec("repPrint " + deleteArray[i]);
                }
                envParamView.remove();
                progressbar.dispose();
                System.out.println("disposing");
            } catch (IOException ex) {
                FailureErrorHandler.handleError(ex, new JPanel());
            }
        }   
    }

Venlig Hilsen
Jeppe

Det må blive noget a la:

public static void printReports(ApplicationModule am, String[] report, JPanel panel)
{       
    BI2Progressbar bI2Progressbar = new BI2Progressbar();       
    bI2Progressbar.setVisible(true);               
   
    new Thread()
    {
        public void run()
        {
            String localFilename;
            int tmp;
            String[] deleteArray = new String[report.length];
            if (System.getProperty("java.io.tmpdir") == null) {
                ValidationErrorHandler.handleError(new UtilException("Util-5075", "Error"), panel);
            } else {       
                try {
                    //The URL of the reportsserver to use is fetched in the table ENV_PARAMETERS
                    ViewObject envParamView = am.createViewObjectFromQueryStmt("EnvParamView", "select EnvParameters.VALUE as Value from SYSMODULE.ENV_PARAMETERS EnvParameters where EnvParameters.ENV_PARAMETER_NAME = 'REPORTS_SERVER'");
                    Row envParam = envParamView.first();
                    String repServer = envParam.getAttribute("VALUE").toString();       
                    System.out.println("Antal i []: " + report.length);               
                    for (int i=0; report.length > i; i++) {
                        System.out.println("Creating file " + (i+1) + " of " + report.length);
                        progressbar.setText("Creating file " + (i+1) + " of " + report.length);
                        progressbar.repaint();
                        //Putting the file in the users temp dir.
                        localFilename = System.getProperty("java.io.tmpdir") + "\\" + "temp" + System.currentTimeMillis() + ".pdf";
                        URL invoke = new URL(repServer + report[i]);
                        FileOutputStream output = new FileOutputStream(localFilename);
                        InputStream input = invoke.openStream();
                   
                        tmp = 0;
                        while (tmp != -1) {
                          tmp = input.read();
                          output.write(tmp);
                        }
       
                        input.close();
                        output.close();
                   
                        deleteArray[i] = localFilename;                   
                    }
                    System.out.println("Printing");
                    for (int i=0;deleteArray.length > i;i++) {                   
                        //Runtime.getRuntime().exec("repPrint " + deleteArray[i]);
                    }
                    envParamView.remove();
                    progressbar.dispose();
                    System.out.println("disposing");
                } catch (IOException ex) {
                    FailureErrorHandler.handleError(ex, new JPanel());
                }
            } 
        }
    }.start(); 
}

Nogle af dine variabler skal muligvis erklæres final:

public static void printReports(final ApplicationModule am, final String[] report, final JPanel panel)
{       
    final BI2Progressbar bI2Progressbar = new BI2Progressbar();       
    bI2Progressbar.setVisible(true); 

    .....
}

Tag et kig på http://java.sun.com/docs/books/tutorial/uiswing/misc/threads.html
nederst på siden er der også et link til "How to monitor Progress". Så for du også en dyber forståelse af hvad der forgår i din egen kode ;0)

Jojo der er jo alt hvad man skal bruge.

Har det jeg skal bruge tror jeg.

Nogen der vil smide et svar :)

Tjoh...

Værsgo og tak