Progressbar aktualisierung nach file übergabe

[java66]

hallo zusammen.

habe eine frage bezüglich einer progress bar.

Ich moechte lese dateien ein in mein programm und übergebe diese an eine funktion.
Nun moechte ich das nach jeder Datei übergabe meine Progressbar aktualisiert wird.

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

functionDOit(flist[i].getAbsolutePath());
 //an dieser stelle soll nun die Progressbar um bspw.10 erhöht und aktualisiert werden
}

wollte es so loesen

Thread t1=new Thread(new Runnable(){
public void run()
{
//....
SwingUtilities.invokeLater(new Runnable(){
public void run()
{
progressbar.setValue(progressbar.getValue()+10);
}

});
}
});

leider hab ich es nicht hin bekommen.
Einfach gesagt nach jeder übergabe eines Pfades soll meine progressbar um 10 erhöht werden und aktualisiert im mom sehe ich nur am ende des Programms die progressbar auf einem wert von z.b 40

 

[SlaterB]

um einen Baum zu fällen kauft der Förster eine Motorsäge und fragt dann das Forum, warum der Baum nicht fällt..

ein Thread ist schön und gut, aber man kann noch tausend andere Dinge nicht oder falsch machen,
nur mit einem Komplettprogramm kann man alle Fehler erkennen

 

[Verjigorm]

So in etwa:

for(int i=0;i<flist.length;i++)
{
   functionDOit(flist[i].getAbsolutePath());
   value += 10;
}

Thread(...
   progressbar.setValue(value);
...

t1.start(); nicht vergessen, das sehe ich bei dir nirgendwo

 

[java66]

sorry hast recht

also

testfilesbtn(ActionEvent e)
{

 //in einer anderen fkt flist=filechooser.getSelectedFiles()

Thread t1=new Thread(new Runnable(){
public void run()
{

SwingUtilities.invokeLater(new Runnable(){
public void run()
{
for(int i=0;i<flist.length;i++)
  {
   functionDOit(flist[i].getabsolutePath());
   progressbar.setValue(progressbar.getValue()+10); 

          }
}
});


}

});


}

Habs jetzt von hand tippen müssen hoffe sind keine fehler drin

 

[SlaterB]

ich will ja nicht störisch sein, aber ein Programm besteht aus Klassen,
fängt mit main-Methode an usw.,

der Thread wird hier nicht gestartet, das ist ein Punkt,
die for-Schleife gehört gewiss nicht in das SwingUtilities.invokeLater, denn wenn alles in invokeLater ausgeführt wird, dann ja vom AWT-Thread, also ist die GUI solange blockiert

 

[java66]

Nochmals sorry Slater,
hast recht

Da mein Eigentliches Programm zu groß ist und alles mit einander verbunden hab ich ma kurz ein bsp programmiert das funzt aber leider nicht so wie ich es will.

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;

import javax.swing.*;




public class probar extends JFrame
{
JPanel pan1=new JPanel();
JButton btn1=new JButton("open");
JProgressBar jb1=new JProgressBar();
JFileChooser choos=new JFileChooser();
File[] flist;
	
public probar()
{
this.setSize(200,200);
this.setDefaultCloseOperation(EXIT_ON_CLOSE);
pan1.add(btn1);
pan1.add(jb1);
this.add(pan1);
this.pack();
choos.setMultiSelectionEnabled(true);
btn1.addActionListener(new ActionListener(){
	public void actionPerformed(ActionEvent e)
	{
		
    choos.showOpenDialog(btn1);
	flist=choos.getSelectedFiles();
    getfiles();
	}	
});


}
	public void fileread(String path)
	{
			System.out.println("The File:"+path+"\nis Readed");	
				return;
	}
	
	public void getfiles()
	{
		
			for(int i=0;i<flist.length;i++)
			{
				Thread t1=new Thread(new Runnable(){
					public void run()
					{
						try{
							Thread.sleep(1000);
							
						}catch(Exception e){}
					
					SwingUtilities.invokeLater(new Runnable(){
						public void run()
						{
							jb1.setValue(jb1.getValue()+10);
							
						}
						
					});
					}
				});
				fileread(flist[i].getAbsolutePath());	 
			t1.start();
			}
		
		
	}
	
	
	public static void main(String[] args)
	{
		
			probar pro1=new probar();
			pro1.setVisible(true);
	}



}

 

[SlaterB]

nun ist es leicht:

public class Test
{
    public static void main(String[] args)
        throws Exception
    {
        probar pro1 = new probar();
        pro1.setVisible(true);
    }
}

class probar
    extends JFrame
{
    JPanel pan1 = new JPanel();
    JButton btn1 = new JButton("open");
    JProgressBar jb1 = new JProgressBar();

    public probar()
    {
        this.setSize(200, 200);
        this.setDefaultCloseOperation(EXIT_ON_CLOSE);
        pan1.add(btn1);
        pan1.add(jb1);
        this.add(pan1);
        this.pack();
        btn1.addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent e)
                {
                    getfiles();
                }
            });
    }

    public void getfiles()
    {
        Thread t1 = new Thread(new Runnable()
            {
                public void run()
                {
                    for (int i = 0; i < 10; i++)
                    {
                        try
                        {
                            Thread.sleep(1000);
                        }
                        catch (Exception e)
                        {
                        }

                        SwingUtilities.invokeLater(new Runnable()
                            {
                                public void run()
                                {
                                    jb1.setValue(jb1.getValue() + 10);
                                }
                            });
                    }
                }
            });
        t1.start();
    }
}

Klassen groß schreiben!

> Da mein Eigentliches Programm zu groß ist und alles mit einander verbunden hab ich ma kurz ein bsp programmiert

so ist es genau richtig, große Programme mit unwichtigen Bestandteilen will niemand lesen müssen

 

[java66]

Danke Slater das du so geduldig warst
Klappt. bis auf ein paar Exceptions .

Nochmal danke