added MyThread class and synchronized Methods

[aliviaVi]

No description provided.

[sergeylukichev]

this is wrong: main thread is waiting for the t1 to finish and only then you start t2. We need 2 threads started almost simultaneously. So, you do not have 2 threads competing for the same Counter instance and you will always get correct result (0) because threads run in a sequence, not in parallel.

[sergeylukichev]

This is also wrong: one thread (t1) should increment, second, t2, should decrement.

[aliviaVi]

bd838cf Сергей, добрый день! чт, 5 дек. 2019 г. в 11:53, Sergey Lukichev <notifications@github.com>:

…

***@***.**** commented on this pull request. ------------------------------ In HW_Multithreading/src/MyThread.java <#2 (comment)> : > @@ -0,0 +1,22 @@ +public class MyThread implements Runnable { + private Counter count; + // Thread t; + + public MyThread(Counter count) { + this.count = count; + /* t=new Thread(); + t.start();*/ + } + + + @OverRide + public void run() { + for (int i = 0; i <10 ; i++) { + count.increment(); This is also wrong: one thread (t1) should increment, second, t2, should decrement. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2?email_source=notifications&email_token=AL22PHM2HQS3TDXV45MNWVTQXDMR5A5CNFSM4JVJASZKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCOCKYYY#pullrequestreview-327461987>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AL22PHPJP6ZYHYOYIIRQEYTQXDMR5ANCNFSM4JVJASZA> .