historia jednego wątku.. czyli co z zasobami :)

0

Cześć,

mam pytanie, czy w przedstawionym "schemacie" zasoby zostaną zwolnione czy nie:

public class startGui extends JFrame()
{
	protected void createGui() // funkcja generuje całą formatkę, brakuje w niej tylko polecenie setVisible(true)
private JFrame getMyself()
	{return this;}
public StartGui() {
	    SwingUtilities.invokeLater(new Runnable() {
	      @Override
	      public void run() {
	    	LoginWindow lw = new LoginWindow(new LoginToApp()); //loginToApp() klasa weryfikująca logowanie, mało ważne :) 
	    	lw.startLogin(getMyself());
	        createGui();
	      }
	    });
	  }
}

natomiast klasa LoginWindow:

public class LoginWindow
{
JFrame frameToOpen;
public void startLogin(JFrame frameToOpenT)
{
	frameToOpen = frameToOpenT;
	SwingUtilities.invokeLater(new Runnable() {
	  @Override
		public void run() {
		    createGui();
		 }
	 });
}
protected void createGui() 
{
(...)
JButton btnLogin = new JButton("Loguj");
	    btnLogin.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				login.setLoginAndPass(txtLogin.getText(),txtPass.getText()); //klasa która weryfikuje poprawność danych- nie istotna z perspektywy pytania
				if(login.login()) frameToOpen.setVisible(true); 
				else System.out.println("Login incomplit");
				
			}
		});
}
}

i teraz, chcę dodać guzik "Anuluj", ale chciałbym żeby kliknięcie go zwolniło wszystkie zasoby "po drodze", czyli te dla wątku z LoginWindow oraz StartGui. Można zmusić javę do zrobienia porządków za mnie?? czy właśnie wygenerowałem wspaniały wyciek zasobów?? :)

Proszę o rady
Pozdro, Piotrek

1

JFrame, jako specjalizację klasy Window, wyłączasz używając dispose. Powoduje to zwolnienie wszystkich zasobów. Który konkretnie obiekt wg ciebie ma szansę zostać w pamięci w postaci wycieku?

Jeżeli tracisz referencję do obiektu (czyli w uproszczeniu nie masz go w żadnej zmiennej), to Java automatycznie go usunie.

0

obawiam się o losy klasy startGui.

sterowanie programem chwilowo przerzucone jest do LoginWindow, i zwolnienie jej zasobów jest oczywiste. Teraz zastanawiam się czy muszę startGui() zakończyć oddzielnie, czy z zrobienie LoginWindow.disopse() to za mało.

Nie jest to skomplikowane, bo referencję już mam przekazaną w LoginWindow(), ale nie wiem czy jest to konieczne.

0

Ja ci nie pomogę. Coś tu jest namieszane. Nie ma takiej metody LoginWindow.dispose. Lepiej zapomnij o tych zasobach i martw się innymi rzeczami, np. żeby program działał :)

0

LoginWindow jest typu JFrame (zapomniałem o tym szczególe przepisując kod :P). Tak więc ma dispose :) TO dla pewności wszystko co się rusza będzie miało dispose :P nie ważne czy potrzeba :) Pytanie miało trochę charakter poznawczy" :P :D

Pozdrawiam
Piotrek

0
package guiZabawa;

import javax.swing.JFrame;
import javax.swing.SwingUtilities;


public class StartGui extends JFrame {
	protected void createGui() 
	{
		setDefaultCloseOperation(EXIT_ON_CLOSE);

	    pack();


	    setLocationRelativeTo(null);
	    setSize(800, 600);
	}
	private JFrame getMyself()
	    {return this;}
	
	public StartGui() {
		
	        SwingUtilities.invokeLater(new Runnable() {
	          @Override
	          public void run() {

	        	LgoinWindow lw = new LgoinWindow(); 
	            lw.startLogin(getMyself());
	            createGui();
	          }
	        });
	      }
	
	  public static void main(String[] args) {
		    new StartGui();
		  }
}

package guiZabawa;




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

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;

public class LgoinWindow extends JFrame {

	JFrame frameToOpen;
	
	public JFrame getMyself()
	{
		return this;
	}
	
	public void startLogin(JFrame frameToOpenT)
	{
	    frameToOpen = frameToOpenT;
	    SwingUtilities.invokeLater(new Runnable() {
	      @Override
	        public void run() {
	            createGui();
	         }
	     });
	}
	protected void createGui() 
	{
		
		setTitle("Podaj login i hasło");


	    JPanel outsideLayout = new JPanel();
	    BoxLayout boxlayout = new BoxLayout(outsideLayout, BoxLayout.Y_AXIS);
	    outsideLayout.setLayout(boxlayout);

	JButton btnLogin = new JButton("Loguj");
	btnLogin.addActionListener(new ActionListener() {
		
		@Override
		public void actionPerformed(ActionEvent e) {
			// TODO Auto-generated method stub

			getMyself().dispose();
			frameToOpen.setVisible(true); 
			
			
		}
	});

	
	JButton btnCancel = new JButton("Dispose");
	btnCancel.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
        	getMyself().dispose();
        }
    });
    
    outsideLayout.add(btnCancel);
    outsideLayout.add(btnLogin);
    
    setContentPane(outsideLayout);
    
    setDefaultCloseOperation(EXIT_ON_CLOSE);

    pack();

    setLocationRelativeTo(null);
    setSize(800, 600);
    setVisible(true);
	}
	
}
	

0

@Piotr Chadamin: w której wersji Javy piszesz ten kod?

0

SE 1.8

1

Dzięki, teraz to można sobie całość uruchomić. No i jest problem, bo java.exe się nie kończy. To dlatego, że - jak przypuszczałeś - obiekt startGui ciągle żyje. A czemu żyje? Bo nie wykonałeś dla niego dispose.

Kończy się prawidłowo np. po takiej przeróbce:

    JButton btnCancel = new JButton("Dispose");
    btnCancel.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
            getMyself().dispose();
            frameToOpen.dispose(); // <-------------------------
        }
    });

Ja bym nie powoływał ramki startGui do życia, dopóki użytkownik nie zechce jej wyświetlić. A jak zamknie - wtedy dispose. Tu mamy nienormalną sytuację, że JFrame jest utworzony, ale nie wyświetlony. Nie ma potrzeby tak komplikować. Może jak dokończysz program, to stanie się jasne, dlaczego startGui jest typu JFrame. Na razie to niepotrzebne.

0

StartGui na chwile obecną jest typu JFrame z powodu mojego lenistwa ;) łatwiej buduje mi się GUI w ten sposób, poza tym łatwiej mi przekazać referencje do niego do innych klas- nie muszę budować interfejsów, bawić się w refleksje itd. Można zrobić to prawdopodobnie na wiele sposobów, ale chyba mój pomysł nie jest "niepoprawny" albo mniej optymalny od innych, prawda?

a tak jeszcze kończąc wywód.. czy można zniszczyć całą apkę/wszystkie wątki apki na raz przy pomocy standardowych funkcji? Nie zastanawiać się, czy gdzieś jakiś wątek jeszcze gdzieś coś mieli, tylko wszystko skończyć na raz?

co do powołania ramki z StartGUI do życia zbyt wcześnie, pewnie masz rację... na początku nie było logowania, teraz logowanie "dorabiam" i dla tego wyszła taka poczwara :) Zmienię to. Klasą startującą będzie ciągle StartGui, ale przed budowaniem ramki wywołam LoginWindow-> i dopiero LoginWindow wywoła funkcję createGui() z klasy StartGui która ramkę buduje i wyświetla.

Dziękuję za pomoc!
Pozdrawiam
Piotrek

1

czy można zniszczyć całą apkę/wszystkie wątki apki na raz przy pomocy standardowych funkcji?

System.exit(), przeczytaj JFrame.dispose() vs System.exit(). Jest jeszcze flaga JFrame.EXIT_ON_CLOSE, stackoverflow: System.exit(0) vs JFrame.EXIT_ON_CLOSE.

No i zaakceptuj odpowiedź i oznacz wartościowe posty (ikonka kciuka). To najlepsze podziękowanie :)

0

@Piotr Chadamin: masz Jave 8 a z Lambd nie korzystasz?

1 użytkowników online, w tym zalogowanych: 0, gości: 1