Keine richtige Codeabarbeitung?!

[philipp65]

Hallo!

Ich bin dabei, ein Java Programm zu schreiben, dass mir unter anderem ein paar Dateien auf einem FTP uploaded. Irgendwie wird das Programm aber nicht sequentiell abgearbeitet...

hier mal der wichtige Auszug aus dem Code:

		if (e.getSource() == uploadB)
		{
			
			c.remove(list);
			c.add(txt,BorderLayout.CENTER);

			
			txt.setText("Bildergalerie wird hochgeladen, dies kann bis zu einige Minuten dauern...\n\n");
			System.out.println("Text hier");
			
			uploadB.setEnabled(false);		
			
			

			try 
			{
				TestFTP ftp = new TestFTP();
				String serverName = "xxxxxxxxxxxxxxxx";
				if (ftp.connectAndLogin(serverName, "xxxxx", "xxxxx")) 
				{
					txt.append("Connected to " + serverName + "\n\n");
					try 
					{
						ftp.changeWorkingDirectory("www");
.
.
.

Was passiert:
Ich klicke den Uploadbutton und die obige Methode wird im ActionPerformed aufgerufen.
Nun wichtig!! Die Zeilen vor dem try werden bis auf den System.out NICHT beachtet und erst nach dem try catch ausgeführt.
Weiters wird txt.append nicht ausgeführt innerhalb des try/catch.

Das Problem hatte ich schon mehrmals in verschiedenen Zusammenhängen. Vielleicht steh ich auch nur auf der Leitung.

Was meint ihr?


MfG
Philipp

 

[Schandro]

Du musst Veränderungen von GUI-Objecten im EDT ausführen.
Einfach um alle Anweisungen oder Anweisungsblöcke welche die GUI-Objecte verändern diesen Code drumbauen:

SwingUtilities.invokeLater(new Runnable() {
	public void run() {
		// hier nur GUI-veränderenden Code ausführen
	}
});

Bin mir grad nicht ganz sicher, ob es dein Problem beheben wird, man sollte es aber so oder so machen, deswegen wirds nix schaden

 

[Leroy42]

Was passiert:
Ich klicke den Uploadbutton und die obige Methode wird im ActionPerformed aufgerufen.
Nun wichtig!! Die Zeilen vor dem try werden bis auf den System.out NICHT beachtet und erst nach dem try catch ausgeführt.

Falsch! Die Zeilen (Methodenaufrufe) werden der Reihe nach abgearbeitet.

Erst die Visualisierung (durch den Event-Dispatch-Thread) wird ausgeführt wenn die gesamte Methode beendet ist.

 

[philipp65]

Falsch! Die Zeilen (Methodenaufrufe) werden der Reihe nach abgearbeitet.

Erst die Visualisierung (durch den Event-Dispatch-Thread) wird ausgeführt wenn die gesamte Methode beendet ist.

man lernt nie aus..danke


@schandro
werde es gleich probieren..ebenfalls danke

 

[philipp65]

Hmm...also ich hab das jetzt probiert und es ändert zwar die reihenfolge, jedoch noch immer nicht so wie gewünscht. Codemäßig siehts jetzt so aus:

		if (e.getSource() == uploadB)
		{
			SwingUtilities.invokeLater(new Runnable() 
			{
			    public void run() 
			    {
			    	c.remove(list);
					c.add(txt,BorderLayout.CENTER);
		
					
					txt.setText("Bildergalerie wird hochgeladen, dies kann bis zu einige Minuten dauern...\n\n");
					
					
					uploadB.setEnabled(false);		

			    }
			});	
			
			System.out.println("Text hier");

			try 
			{
				TestFTP ftp = new TestFTP();
				String serverName = "xxxxxxxxxx";
				if (ftp.connectAndLogin(serverName, "xxxx", "xxxx")) 
				{
					txt.append("Connected to " + serverName + "\n\n");
					try 
					{
						ftp.changeWorkingDirectory("www");
.
.
.

Habe ich was falsch verstanden? Benötigt ihr noch mehr Code?

 

[Beni]

Da dieser Code von einem ActionListener (bzw. einem Button) aufgerufen wird, wird er im EDT ausgeführt. Und solange du irgendetwas im EDT machst, kann die GUI nicht gezeichnet werden.

In diesem Fall müsstest du alle Arbeit im try{...} in einen eigenen Thread auslagern um den EDT freizukriegen. Das "invokeLater" ist dann nur notwendig, wenn du von diesem neuen Thread aus die GUI ändern möchtest.

 

[philipp65]

Da dieser Code von einem ActionListener (bzw. einem Button) aufgerufen wird, wird er im EDT ausgeführt. Und solange du irgendetwas im EDT machst, kann die GUI nicht gezeichnet werden.

In diesem Fall müsstest du alle Arbeit im try{...} in einen eigenen Thread auslagern um den EDT freizukriegen. Das "invokeLater" ist dann nur notwendig, wenn du von diesem neuen Thread aus die GUI ändern möchtest.

Hallo,
Das müsste aber auch heißen, dass GUI Änderungen nur dann nicht möglich sind, wenn in dieser actionPerformed Methode AUCH Threads vorkommen. Sofern nur GUI Änderungen in der actionPerformed vorkommen, habe ich kein Problem.

Siehe:

	public void actionPerformed(ActionEvent e) 
	{
		if (e.getSource() == pfadB)
		{
			if (fc.showOpenDialog(this) == JFileChooser.APPROVE_OPTION)
			{
				c.remove(txt);
				c.add(new JScrollPane(list),BorderLayout.CENTER);
				c.repaint();
				c.validate();
				repaint();
				validate();
				
				lm.removeAllElements();
				helf = fc.getSelectedFile();
				pfadTxt.setText(String.valueOf(helf));
				files = helf.listFiles();
		

				for (int i=0; i<files.length; i++)
					lm.addElement(files[i]);
				
				uploadB.setEnabled(true);
			}
		}
		if (e.getSource() == uploadB)
		{
	    	c.remove(list);
			c.add(txt,BorderLayout.CENTER);

			txt.setText("Bildergalerie wird hochgeladen, dies kann bis zu einige Minuten dauern...\n\n");
			
			uploadB.setEnabled(false);					

	    	try 
			{
				TestFTP ftp = new TestFTP();
.
.
.

Man sieht hier, dass beim ersten Button ebenfalls bzw ausschließlich GUI Änderungen vorgenommen werden und dies ohne Probleme. Probleme kommen nur, wenn ich zusätzlich FTP Aktionen durchführe.


Könntest du / Könntet ihr mir einen Tipp geben, wie ich den Tread auslagere. Ich finde dazu keine Literatur und nichts im Forum.
Alles, was ich eigentlich will, ist dem Anwender ein paar Feedbacks über den Verlauf des FTP Uploads zu geben.

MfG
Philipp

 

[JohannisderKaeufer]

	public void actionPerformed(ActionEvent e) 
	{
//Jetzt schauen wir mal ob es sich um den EDT Handelt?
System.out.println("EDT"+SwingUtilities.isEventDispatchThread());
		if (e.getSource() == pfadB)
		{
			if (fc.showOpenDialog(this) == JFileChooser.APPROVE_OPTION)
			{
				c.remove(txt);
				c.add(new JScrollPane(list),BorderLayout.CENTER);
				c.repaint();
				c.validate();
				repaint();
				validate();
				
				lm.removeAllElements();
				helf = fc.getSelectedFile();
				pfadTxt.setText(String.valueOf(helf));
				files = helf.listFiles();
		

				for (int i=0; i<files.length; i++)
					lm.addElement(files[i]);
				
				uploadB.setEnabled(true);
			}
		}
		if (e.getSource() == uploadB)
		{
	    	c.remove(list);
			c.add(txt,BorderLayout.CENTER);

			txt.setText("Bildergalerie wird hochgeladen, dies kann bis zu einige Minuten dauern...\n\n");
			
			uploadB.setEnabled(false);					

	    	try 
			{
new Thread(new Runnable(){
public void run(){
//Das sollte jetzt nicht der EDT sein
System.out.println("EDT an Stelle 2:"+SwingUtilities.isEventDispatchThread());
				TestFTP ftp = new TestFTP();
//GUI UPdates
SwingUtilities.invokeLater(new Runnable(){
public void run(){
//GUI Updates können dann hier von statten gehen.
...
}
});
ftp= new TestFTP();
}
}).start();
.
.
.

Das ganze sollte in etwa so aussehen;

 

[philipp65]

Das ganze sollte in etwa so aussehen;

Du bist mein Held!
Nach ein wenig rumprobieren läuft es jetzt einwandfrei!!
Jedoch nur in Eclipse...sobald ich das Ganze als JAR exportiere und starte, passiert nachdem ich den uploadB gedrückt habe gar nichts


MfG

Philipp

 

[philipp65]

So...hier noch für alle mit dem selbigen Problem, das ich hatte der finale Code mit Komentaren versehen, wie ich es mir vorstelle:

	public void actionPerformed(ActionEvent e) 
	{
		//EDT, wird aufgerufen, sobald der obere Button gedrückt wird
		if (e.getSource() == pfadB)
		{
			//Sobald man den Ordner bestätigt
			if (fc.showOpenDialog(this) == JFileChooser.APPROVE_OPTION)
			{
				//Textfeld wird entfernt, Liste mit Inhalt hinzugefügt, neuzeichnen des Layouts
				c.remove(txt);
				c.add(new JScrollPane(list),BorderLayout.CENTER);
				c.repaint();
				c.validate();
				repaint();
				validate();
				
				//alle Elemente der Liste werden Gelöscht, sofern welche vorhanden sind
				lm.removeAllElements();
				
				//Die Liste wird nun mit den Ordnerinhaltsverzeichnissen gefüllt
				helf = fc.getSelectedFile();
				pfadTxt.setText(String.valueOf(helf));
				files = helf.listFiles();
				for (int i=0; i<files.length; i++)
					lm.addElement(files[i]);
				
				//Der Uploadbutton wird nun klickbar
				uploadB.setEnabled(true);
			}
		}
		
		//Wird aufgerufen, sobald der untere Button geklickt wird
		if (e.getSource() == uploadB)
		{
		    try 
            {
		    	//Es wird ein eigener Thread erzeugt, um Swingänderungen synchron durchzuführen
				new Thread(new Runnable()
				{
					public void run()
					{
						//nicht der EDT, GUI Updates
			            TestFTP ftp;
			
			            SwingUtilities.invokeLater(new Runnable()
			            {
			            	public void run()
			            	{
			            		//GUI Updates können dann hier von statten gehen.
						    	c.remove(list);
								c.add(txt,BorderLayout.CENTER);

								txt.setText("\nBildergalerie wird hochgeladen, dies kann einige Minuten dauern...\n\n");
								
								uploadB.setEnabled(false);
			            	}
			            });
			            
			            ftp= new TestFTP();
				    	try 
						{
							//Der Servername wird festgelegt
							String serverName = "xxxxxxxxxxxx";
							
							//Verbindungsversuch
							if (ftp.connectAndLogin(serverName, "xxxxxx", "xxxxxx")) 
							{
								//Wenn erfolgreich, dann visuelle Rückmeldung
								txt.append("Connected to " + serverName + "\n\n");
								try 
								{
									//Galeriename und -pfad werden für spätere Zwecke gespeichert
									String galeryname = files[list.getSelectedIndex()].getName();
									String galerypath = list.getSelectedValue().toString();
									

									ftp.setPassiveMode(true);
									ftp.ascii();
									
									//Wechsel in richtigen FTP Pfad + visuelle Rückmeldung
									txt.append("Current Directory: " + ftp.printWorkingDirectory() + "\n");
									ftp.changeWorkingDirectory("www");
									txt.append("Current Directory: " + ftp.printWorkingDirectory() + "\n");
									ftp.changeWorkingDirectory("home");
									txt.append("Current Directory: " + ftp.printWorkingDirectory() + "\n");
									ftp.changeWorkingDirectory("photography");
									txt.append("Current Directory: " + ftp.printWorkingDirectory() + "\n");
									ftp.changeWorkingDirectory("home");
									txt.append("Current Directory: " + ftp.printWorkingDirectory() + "\n");
									ftp.changeWorkingDirectory("_galery");
									txt.append("Current Directory: " + ftp.printWorkingDirectory() + "\n");
									
									
									//Die Bildergalerie wird nun hochgeladen
									ftp.makeDirectory(galeryname);
									ftp.changeWorkingDirectory(galeryname);
									txt.append("Current Directory: " + ftp.printWorkingDirectory() + "\n");
									
.
.
.

Lg und vielen Dank für die Hilfe

Thread kann geschlossen werden

Philipp