Diskuze: synchronizace

V předchozím kvízu, Online test znalostí Java, jsme si ověřili nabyté zkušenosti z kurzu.

Aktivity

dinokino

Člen

dinokino:15.12.2015 14:57

Dobrý den, učím se vlákna v javě a u "snychronizovaných metod" jsem narazil na problém. Mám Hlavní třídu:

public class Vlakna {

        public static void main(String[] args) throws InterruptedException {
                TridaVlakna tridaVlakna = new TridaVlakna();
                TridaVlakna tridaVlakna2 = new TridaVlakna();
        }

}

A dále třídu s názvem "TridaVlakna":

public class TridaVlakna extends Thread {
        Thread v;
TridaVlakna(){
        v = new Thread(this);
        v.start();
}
        @Override
        public void run() {
                this.zobraz();
        }
        synchronized void zobraz(){
                System.out.print("(");
                try {
                        Thread.sleep(1000);
                } catch (InterruptedException e) {
                        // TODO Auto-generated catch block
                        e.printStackTrace();
                }
                System.out.print("ab");
                try {
                        Thread.sleep(1000);
                } catch (InterruptedException e) {
                        // TODO Auto-generated catch block
                        e.printStackTrace();
                }
                System.out.println(")");
        }

}

Problém je ten, že mi program do konzole vypíše text:
""((abab)
)" místo očekávaného:
"(ab)
(ab)"
Nevím proč mi "nefunguje" synchronizace metody "zobraz". Neví někdo co s tím??

Odpovědět

15.12.2015 14:57

Neaktivní uživatel

Tvůrce

Neaktivní uživatel:15.12.2015 15:12

jsi si jisty ze se ma vypsat (ab) (ab) ? protoze ja co si dobre pamatuju tak vidim beh programu takhle ... spustis jedno vlakno (ulohu), jelikoz v konstruktoru vlakno startujes, pak uz se to bere samostatne ... vypises v prvnim vlaknu zavorku ...spustis druhy vlakno ...zase konstruktor a jedes ... druha zavorka ... ubehla vterina na prvnim vlaknu..napise se ab ...ubehla vterina na druhem... tedy dalsi ab ... dobehla vterina na prvnim ... zavorka..vterina na druhem dalsi zavorka... a pokud se vubec nepletu.. tak nevidim, ze bys nekde synchronizoval... na to jsou prece v jave veci jako zamky a podobne.. treba pro pristup ke zdrojum a tak.. no hadam ze se k tomu urcite vyjadri i nekdo, kdo ti poradi jak dosahnout toho co chces...ja ti zatim jen reknu, ze tady nevidim problem ...proste se deje to, cos napsal..

Nahoru Odpovědět

15.12.2015 15:12

Neaktivní uživatelský účet

coells

Tvůrce

coells:15.12.2015 15:53

Protože když napíšeš synchronized void zobraz(), provádí se synchronizace nad instancí třídy, ve které je metoda zobraz() definovaná.
Pokud vytvoříš dvě různé instance, máš dva různé objekty, nad kterými zamykáš, a které spolu nesouvisí.

Na chvíli budeme předpokládat, že metoda zobraz() je public a pak stačí přepsat main do tvaru, kdy budeš mít jedinou instanci a už dostaneš požadovaný efekt.

public static void main(String[] args) throws InterruptedException {
        TridaVlakna tridaVlakna = new TridaVlakna();
        tridaVlakna.zobraz();
}

Druhou chybu máš v kódu při vytváření vlákna.
V Javě nikdy nevytvářej vlákna v konstruktoru, je složité vysvětlit, proč se to nesmí dělat, takže to jednoduše nedělej.

Akceptované řešení
+20 Zkušeností
+2,50 Kč

Nahoru Odpovědět

15.12.2015 15:53

Lukáš Hruda

Tvůrce

Lukáš Hruda:15.12.2015 16:20

"V Javě nikdy nevytvářej vlákna v konstruktoru..."
Tím vytvořením myslíš skutečně vytvoření nebo spuštění?

Nahoru Odpovědět

15.12.2015 16:20

coells

Tvůrce

coells:15.12.2015 16:32

V terminologii Javy spuštění, v terminologii vláken vytvoření.

Nahoru Odpovědět

15.12.2015 16:32

Lukáš Hruda

Tvůrce

Lukáš Hruda:15.12.2015 17:18

Jsem si říkal, co by vadilo na vytvoření objektu v konstruktoru

Nahoru Odpovědět

15.12.2015 17:18

Neaktivní uživatel

Tvůrce

Neaktivní uživatel:15.12.2015 18:05

Prehlidl jsem, ze je metoda synchonized ... omlouvam se tim padem za mystifikaci

Nahoru Odpovědět

15.12.2015 18:05

Neaktivní uživatelský účet

B42P6

Člen

B42P6:15.12.2015 18:31

Ako už bolo spomenuté voláš inštančnú metodu, čo znamená že každá inštancia má "vlastnú" metodu. Riešením by mohlo byť deklarovať metodu zobraz() statickú čo znamená, že by patrila triede a tým pádom zdieľajú všetky inštancie tú istú metodu. BTW kód metódy je uložený v pamäti len raz, v tomto prípade ide o to ,na ktorom objekte bola metóda volaná.

Editováno 15.12.2015 18:33

Nahoru Odpovědět

15.12.2015 18:31

'long long long' is too long for GCC

dinokino

Člen

dinokino:15.12.2015 20:24

Děkuji za odpovědi, neuvědomil jsem si, že vytvářím 2 instance, které mají každá svoji metodu zobraz().

Nahoru Odpovědět

15.12.2015 20:24

Naučíme tě pracovat na home-office.

Zjistit více...

Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 9 zpráv z 9.

Nejčastěji vyhledáváné

Diskuze: synchronizace