Volání metody v jiné třídě binárního stromu – Java – Fórum – Programujte.com

Dobrý den,

potřeboval bych poradit ohledně mého projektu na binární strom. Mám vytvořenou třídu Start, ve které je naprogramováno textové rozhraní a třídu Strom, která přidává prvky do stromu, odebírá, hledá hodnotu a vypisuje celý strom. Když chci spustit metodu (např. na vypsání stromu) ze třídy Start musím napsat: 

Strom str = new Strom();

System.out.println("Strom obsahuje:");

str.vypisStromu();

Bohužel se asi vytvoří zcela nový prázdný strom a nic nevypíše. Pokud už je strom hodnotami naplnění potřeboval bych, aby se metoda z jiné třídy volala a použila data z již vytvořeného stromu.

Předem děkuji za odpověď.

Marek Musil

#1 Marek Musil
Však ten strom je prázdný. Co by se mělo vypisovat?

Pro výpis stromu bych použil jednodušší zápis:

System.out.println(str);

A samozřejmě doporučuji proměnnou "str" přejmenovat na "strom".

#2 Kit
Asi jsem to špatně vysvětlil. Když vytvořím strom přidáváním prvků

Připojen obrázek.

a poté dám vypisStromu() vše se vypíše. To samé ale potřebuji aby fungovalo ne pomocí klikání myší, ale pomocí textového rozhraní.

Připojen obrázek.

public void spustit()
    {
        int volba = -1;
        do{
            menu();
                                
            try{
                volba = scan.nextInt();
            }
            catch(Exception e){
                // vybere z klavesnice znak NL
                scan.nextLine();
            }

            switch (volba) {
                case 1:  prvni(); break;
                case 2:  druhy(); break;
                case 3:  treti(); break;
                case 4:  ctvrty(); break;
                case 5:  paty(); break;
            }
            
        } while(volba != 0);
        konec();
    }

private static void treti()
    {
        Strom str = new Strom();
        System.out.println("Strom obsahuje:");
        str.vypisStromu();
    }

Tento kód je špatně a potřebuji aby se po vytvoření hodnot spustila metoda v jiné třídě a udělala to co má.

public void vypisStromu()
    {
        vypisStromuDva(koren);
    }
    
    private void vypisStromuDva(Prvek oznaceny)
    {
        if(oznaceny == null) // výpis rodič, levý, pravý
        {
            return;
        }
        System.out.println(oznaceny.getHodnota());
        vypisStromuDva(oznaceny.getLevy());
        vypisStromuDva(oznaceny.getPravy());
    }

#3 Marek Musil
Ahoj.

Problém bych viděl v tom, že v metodě treti() vytváříš nový strom (který samozřejmě bude prázdný, protože jsi ho teď vytvořit a vytváříš vždy nový opakovaným voláním této metody).

Má rada zní, vytvoř si jeden strom pro běh celého programu. Tedy vytvoř si jedinou instanci třídy Strom pro třídu Start. Vytvoř si proměnou private Strom strom a v konstruktoru vytvořit instanci, viz následující kus kódu. 

public Start {
	private Strom strom;

	# Konstruktor
	public Start() {
		strom = new Strom();
	}
	.
	.
	.
}

Nyní v metodě jako je treti() už nevytvářej novou instanci třídy Strom, ale pracuj s právě vytvořenou proměnnou. (Proměnná je private, a tudíž se na ní dostaneš právě v rámci celé třídy.

Pokud bys chtěl v běhu svého programu "vyčistit" strom, nebo jinak řečeno vytvořit novou instanci, stačí přidat například šestou volbu s metodou sest() a v ní prostě k aktuální proměnné strom přiřadit novou instanci (nový strom).

public void sest() {
	strom = new Strom();
}

#4 JirkaEr
Děkuji, to je přesně co jsem potřeboval. Včera večer mě to také napadlo a když jsem dnes viděl váš příspěvek a vyzkoušel fungovalo to. Myslel jsem, že je potřeba při každém volání psát new Strom, ale teď už chápu co to vlastně znamená a proč je logické použít to jen jednou.

Děkuji.