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

historia jednego wątku.. czyli co z zasobami :)
Piotr Chadamin
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 15
0

Cześć,

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

Kopiuj
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:

Kopiuj
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

jarekczek
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Siemianowice Śląskie
  • Postów: 500
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.

Piotr Chadamin
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 15
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.

jarekczek
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Siemianowice Śląskie
  • Postów: 500
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ł :)

Piotr Chadamin
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 15
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

Piotr Chadamin
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 15
0
Kopiuj
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();
		  }
}

Kopiuj
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);
	}
	
}
	

S9
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Warszawa
  • Postów: 3573
0

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

Piotr Chadamin
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 15
0

SE 1.8

jarekczek
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Siemianowice Śląskie
  • Postów: 500
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:

Kopiuj
    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.

Piotr Chadamin
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 15
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

jarekczek
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Siemianowice Śląskie
  • Postów: 500
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 :)

S9
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Warszawa
  • Postów: 3573
0

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

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.