Не проходит

Вопрос по задаче

Java Multithreading, 7 уровень, 6 лекция

Решен

Правда когда меняю местами объекты o1, o2, всегда возращает тру...

Реализуй логику метода isLockOrderNormal, который должен определять:
соответствует ли порядок synchronized блоков в методе someMethodWithSynchronizedBlocks - порядку передаваемых в него аргументов.
В случае, если сначала происходит синхронизация по o1, а потом по o2, метод должен вернуть true.
Если наоборот - false.

Требования:

Метод isLockOrderNormal должен возвращать true в случае, если синхронизация в методе someMethodWithSynchronizedBlocks происходит сначала по объекту o1, а потом по o2.
Метод isLockOrderNormal должен возвращать false в случае, если синхронизация в методе someMethodWithSynchronizedBlocks происходит сначала по объекту o2, а потом по o1.
Метод isLockOrderNormal НЕ должен быть приватным.
Класс Solution НЕ должен быть объявлен с модификатором final.

package com.javarush.task.task27.task2707; /* Определяем порядок захвата монитора */ public class Solution { public void someMethodWithSynchronizedBlocks(Object obj1, Object obj2) { synchronized (obj1) { synchronized (obj2) { System.out.println(obj1 + " " + obj2); } } } public static boolean isNormalLockOrder(final Solution solution, final Object o1, final Object o2) throws Exception { Thread thread1 = new Thread(); Thread thread2 = new Thread(); synchronized (o1){ new Thread().sleep(500); thread1.sleep(100); synchronized (o2){ solution.someMethodWithSynchronizedBlocks(o2, o1); thread1.start(); thread2.start(); new Thread().sleep(2000); if(thread2.getState() != Thread.State.BLOCKED){ return true; } else return false; } } } public static void main(String[] args) throws Exception { final Solution solution = new Solution(); final Object o1 = new Object(); final Object o2 = new Object(); while (true){ System.out.println(isNormalLockOrder(solution, o1, o2)); }} }

Комментарии (8)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

hidden #1278682

Уровень 32

5 мая 2018, 07:31

полезный

Ваши объекты, по которым происходит синхронизация, используются только главным потоком. Другие потоки их не используют

Юрий Кузнецов

Уровень 35

5 мая 2018, 10:35

Сделал так, но после первого прохода у меня дедлок

public static boolean isNormalLockOrder(final Solution solution, final Object o1, final Object o2) throws Exception {
    Thread thread1 = new Thread();
        Thread thread2 = new Thread();
        synchronized (o1){
            new Thread().sleep(500);

            synchronized (o2){
                solution.someMethodWithSynchronizedBlocks(o1, o2);
                new Thread() {
                    @Override
                    public void run() {
                        try {
                            isNormalLockOrder(solution, o1, o2); //expected boolean b
                        } catch (Exception ignored) {
                        }
                    }
                }.start();

                new Thread() {
                    @Override
                    public void run() {
                        try {
                            isNormalLockOrder(solution, o2, o1); //expected boolean !b
                        } catch (Exception ignored) {
                        }
                    }
                }.start();
                new Thread().sleep(1000);
                if(thread2.getState() != Thread.State.BLOCKED){
                    return true;
                }
                else return false;
            }
        }
    }

Юрий Кузнецов

Уровень 35

5 мая 2018, 11:08

все, решил) все равно спасибо)

hidden #1278682

Уровень 32

5 мая 2018, 12:12

я верил в вас!

Юрий Кузнецов

Уровень 35

5 мая 2018, 13:05

ахах спасиб =))

Юрий Кузнецов

Уровень 35

4 мая 2018, 19:22

Не понимаю, что не так делаю...(

hidden #1278682

Уровень 32

5 мая 2018, 07:09

какой исходный код задачи?

Юрий Кузнецов

Уровень 35

5 мая 2018, 10:32

package com.javarush.task.task27.task2707;

/*
Определяем порядок захвата монитора
*/
public class Solution {
    public void someMethodWithSynchronizedBlocks(Object obj1, Object obj2) {
        synchronized (obj1) {
            synchronized (obj2) {
                System.out.println(obj1 + " " + obj2);
            }
        }
    }

    public static boolean isNormalLockOrder(final Solution solution, final Object o1, final Object o2) throws Exception {
        //do something here
        return false;
    }

    public static void main(String[] args) throws Exception {
        final Solution solution = new Solution();
        final Object o1 = new Object();
        final Object o2 = new Object();

        System.out.println(isNormalLockOrder(solution, o1, o2));
    }
}

исходник