Ansicht Problem

[ramoser]

Hallo,

ich habe ein problem mit der sichtbarkeit des frames, d.h. wenn ich über einen button ein frame anzeigen, eine aktion ausführen und dann den frame wieder ausblenden, sehe ich einfach nichts im frame.

kennt jemand dieses prolem und kann mir helfen?

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

import javax.swing.JFrame;
import javax.swing.JProgressBar;


class Backup implements ActionListener {

	public Backup(){
	}

	public void actionPerformed(ActionEvent e) {
		JFrame backupFrame = new JFrame();
		backupFrame.setTitle("test");
		backupFrame.setLayout(null);
		backupFrame.setResizable(false);
		backupFrame.setAlwaysOnTop(true);
		backupFrame.setSize(300, 200);


		JProgressBar backupBar = new JProgressBar();
		backupBar.setStringPainted(true);
		backupBar.setIndeterminate(false);
		backupBar.setBounds(10,10,274,30);
		backupFrame.add(backupBar);


		backupFrame.setVisible(true);

		try {
			File backupDel1 = new File("c:/BACKUP/DBSTAM.FDB");
			if(!backupDel1.exists() || backupDel1.delete()){
				int i = 0;
				RandomAccessFile datei = new RandomAccessFile("c:/DB/DBSTAM.FDB","r");
				RandomAccessFile neudatei = new RandomAccessFile("c:/BACKUP/DBSTAM.FDB", "rw");
				backupBar.setMaximum((int) datei.length());
				backupBar.setValue(0);
				while (neudatei.length() < datei.length()) {
					neudatei.write(datei.read());
					backupBar.setValue(i);
					i++;
				}
				datei.close();
				neudatei.close(); 
			}
		} catch (Exception e1) {
			e1.printStackTrace();
		}


		backupFrame.setVisible(false);
	}
}

 

[MathiasBauer]

Ich behaupte mal, da fehlt ein backupFrame.repaint() oder ein backupFrame.validate().

 

[ramoser]

Du meinst in der While Schleife??

Habe ich gerade probiert, geht auch nicht, zeigt nur mein hindergrundbild

 

[ramoser]

Nach eineigem probieren und googlen hab ichs geschaft:

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

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JProgressBar;
import javax.swing.SwingUtilities;


class Backup implements ActionListener {

	static JProgressBar bar = new JProgressBar( 0, 1000000 ); 
	static JFrame f = new JFrame();
	static RandomAccessFile datei, neudatei;
	static int i = 0;

	public Backup(){
	}

	public void actionPerformed(ActionEvent e) {

		try {
			f.setLayout(null);
			f.setResizable(false);
			f.setSize(300, 200);

			JButton but = new JButton( "Starte Backup" ); 
			but.setBounds(47,70,200,60);
			but.addActionListener( new ActionListener() {
				public void actionPerformed( ActionEvent e ) {
					new Thread( new Runnable() 
					{
						public void run() 
						{
							try{
								File backupDel1 = new File("c:/BACKUP/DBSTAM.FDB");
								if(!backupDel1.exists() || backupDel1.delete()){
									i = 0;
									datei = new RandomAccessFile("c:/DB/DBSTAM.FDB","r");
									neudatei = new RandomAccessFile("c:/BACKUP/DBSTAM.FDB", "rw");
									bar.setMaximum((int) datei.length());
									bar.setValue(0);
									while (neudatei.length() < datei.length()) {
										final int j = i; 
										SwingUtilities.invokeLater( new Runnable() 
										{
											public void run() { 
												try {
													neudatei.write(datei.read());
													bar.setValue(j);
													i++; 
												} catch (IOException e2) {
													e2.printStackTrace();
												}
											}
										} );
									}
									datei.close();
									neudatei.close();
								} 
							}catch (Exception e1) {
								e1.printStackTrace();
							}
							f.setVisible(false);
						}
					}).start(); 
				}
			});

			bar.setBounds(10,10,274,30);
			bar.setStringPainted( true ); 

			f.add(bar); 
			f.add(but); 

			f.setVisible( true );
		} catch (Exception e1) {
			e1.printStackTrace();
		}
	}
}

 

[Marco13]

:shock: :autsch: In einer while Schleife Runnables in den Event-Dispatch-Thread legen, die in einer Datei rumfuhrwerken?! :autsch: Was auch immer dein ursprüngliches Problem war: Nur weil es jetzt nichtmehr auftritt, heißt das nicht, dass du die Lösung gefunden hat. "Nach einigem probieren"... sorry, aber so sieht das auch aus....

 

[ramoser]

Wenn du mir helfen kannst und es richtig stellen? Habe nichts dagegen!

 

[Marco13]

Naja, dass da ein Thread drumgewickelt ist, ist richtig, und dass das setValue mit den SwingUtilities gemacht wird, auch. Warum da nochmal ein weiterer ActionListener drumgewickelt ist, obwohl das ganze schon in der ActionPerformed steht, ist mir aber nicht klar. Soll der Zweite Frame nur so eine Art Fortschritts-Dialog sein? However, das eigentliche Kopieren sollte eben nicht im Event-Dispatch-Thread gemacht werden. Im Moment könnte es (wenn ich das richtig sehe) passieren, dass u.U. hunderte von Swingworkern in den EDT gelegt, bevor der erste dazu kommt, ein byte zu kopieren ... und dann vielleicht merkt, dass die Datei schon am ende ist ???:L Wenn es schnell gehen soll, sollte man übrigens auch nicht einzelne bytes kopieren. Dann müßte man mit den SwingUtilities wirklich nur den Progress so setzen, dass er wirklich dem Fortschritt entspricht, aber ... das macht er bei dir ja jetzt auch nicht (oder ist die Datei immer genau 1000000 bytes groß?). Ggf. solltest du dir auch mal die SwingWorker-Klasse ansehen, die ist nämlich genau für solche Aufgaben da ... mit modelen Dialogen und allem drum und dran... damit geht das auch einfacher als... so

// Für [url]http://www.java-forum.org/de/viewtopic.php?t=69382&highlight=[/url]

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


class BackupTest extends JFrame
{
    public static void main(String args[])
    {
        new BackupTest();
    }

    public BackupTest()
    {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        JButton b = new JButton("Copy");
        b.addActionListener(new Backup());
        getContentPane().add(b);
        pack();
        setVisible(true);
    }

}

class Backup implements ActionListener
{
   private JProgressBar backupBar = new JProgressBar();
   private JFrame backupFrame = new JFrame();

   public Backup()
   {
   }

   public void actionPerformed(ActionEvent e)
   {
      backupFrame.setTitle("test");
      backupFrame.setLayout(null);
      backupFrame.setResizable(false);
      backupFrame.setAlwaysOnTop(true);
      backupFrame.setSize(300, 200);

      backupBar.setStringPainted(true);
      backupBar.setIndeterminate(false);
      backupBar.setBounds(10,10,274,30);
      backupFrame.add(backupBar);
      backupFrame.setVisible(true);

      SwingUtilities.invokeLater(new Runnable()
      {
          public void run()
          {
              doCopy("input.txt", "output.txt");
          }
      });
   }

   private void doCopy(final String inputFileName, final String outputFileName)
   {
      Thread t = new Thread(new Runnable()
      {
          public void run()
          {
              try
              {
                  File inputFile = new File(inputFileName);
                  long len = inputFile.length();
                  FileInputStream fis = new FileInputStream(inputFile);
                  FileOutputStream fos = new FileOutputStream(new File(outputFileName));
                  byte buffer[] = new byte[8192];
                  int read = -1;
                  long totalRead = 0;
                  while ((read = fis.read(buffer)) != -1)
                  {
                      fos.write(buffer, 0, read);
                      totalRead += read;

                      int progress = (int)(((double)totalRead/len) * 100);
                      setProgress(progress);
                  }
                  fis.close();
                  fos.close();
              }
              catch (IOException e)
              {
                  e.printStackTrace();
              }

              SwingUtilities.invokeLater(new Runnable()
              {
                  public void run()
                  {
                      backupFrame.setVisible(false);
                  }
              });

          }
      });
      t.start();
  }




   private void setProgress(final int progress)
   {
       SwingUtilities.invokeLater(new Runnable()
       {
           public void run()
           {
               backupBar.setValue(progress);
           }
       });
   }

}

 

[Marco13]

Ach ja, wenn du es möglichst wenig ändern willst: Der Haupkritikpunkt war ja die innere Schleife... und diesen Punkt könnte man auch so beheben, dass man

                          while (neudatei.length() < datei.length()) {
                              neudatei.write(datei.read()); //-----------------!!!!!!!!!!!!!!!-- <<< diese Zeile hier einfügt
                              final int j = i;
                              SwingUtilities.invokeLater( new Runnable()
                              {
                                 public void run() {
                                    try {
                                       //neudatei.write(datei.read()); //-----------------!!!!!!!!!!!!!!!-- hier wegnimmt
                                       bar.setValue(j);
                                       i++;
                                    } catch (IOException e2) {
                                       e2.printStackTrace();
                                    }
                                 }
                              } );
                           }

aus damit dem EDT rauszieht. (Ja, sorry, sooo dramatisch wie ich in der ersten Antwort dachte war das ganze jetzt doch nicht.... :roll: ... d.h. es sah auf den ersten Blick einfach abstruser aus, als es auf den zweiten Blick dann war... )

 

[ramoser]

Danke für deine Hilfe, klappt jetzt perfekt!! :lol: :lol:

P.S. kein problem!! :toll: