JavaRush/Помощь по Java задачам/Почему method4 синхронизированный?

40 уровень

18.09.2021
707просмотров
16комментариев

Почему method4 синхронизированный?

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

Решен

Почему method4 синхронизированный? Ведь поле StringBuilder приватное, и сам метод method4 тоже приватный, как к ним смогут получить доступ другие нити?

Установить модификатор synchronized только тем методам, которым необходимо.
Объект класса Solution будет использоваться нитями.

Требования:

В method0, если необходимо, используй synchronized.
В method1, если необходимо, используй synchronized.
В method2, если необходимо, используй synchronized.
В method3, если необходимо, используй synchronized.
В method4, если необходимо, используй synchronized.
В method5, если необходимо, используй synchronized.
В method6, если необходимо, используй synchronized.
В method7, если необходимо, используй synchronized.

package com.javarush.task.task17.task1716; /* Синхронизированные методы */ public class Solution { private double param = Math.random(); private StringBuilder sb = new StringBuilder(); private void method0() { Double d = method3(); } protected synchronized void method1(String param1) { Solution solution = new Solution(); solution.method0(); } public synchronized void method2(int param1) { param1++; } synchronized double method3() { double random = Math.random(); param += 40.7; return random + param; } private void method4() { sb.append(1).append(1).append(1).append(1); } protected synchronized void method5(String param2) { new StringBuffer().append(param2).append(param2).append(param2); } public synchronized String method6(int param2) { System.out.println("Thinking...."); method7(5e-2); sb = new StringBuilder("Got it!."); return sb.toString(); } synchronized String method7(double param2) { return "" + param2; } public static void main(String[] args) { } }

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

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

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

Ksenia Volkova Java Developer в DXC

Master

18 сентября 2021, 07:40

Не очень понятный вопрос... А как связаны между собой модификаторы доступа и синхронизированность?

Ksenia Volkova Java Developer в DXC

Master

18 сентября 2021, 07:52

полезный

public class Test {
    static class TestThread extends Thread {
        private Solution solution;
        public TestThread(Solution solution) {
            super();
            this.solution = solution;
        }

        @Override
        public void run() {
            solution.methodXXX(); // какой-то метод, вызывающий method4()
        }
    }

    public static void main(String[]args){
        Solution solution = new Solution();
        new TestThread(solution).start();
        new TestThread(solution).start();
    }
}

Две нити работают с один объектом Solution.

Vadimayer

Уровень 40

18 сентября 2021, 11:29

то есть я правильно понял, модификатор privat на методе никак не мешает нескольким нитям с ним работать?

Ksenia Volkova Java Developer в DXC

Master

18 сентября 2021, 12:08

Конечно, нет.

Алексей

Уровень 32

15 октября 2021, 12:53

solution.methodXXX(); // какой-то метод, вызывающий method4() Нет методов в Solution, вызывающих method4() - т .е., из вашего кода его не достать, т. е. synchronized не нужен.

Ksenia Volkova Java Developer в DXC

Master

15 октября 2021, 13:06

полезный

А если будет? А если я вызову его через рефлексию?

@Override
public void run() {
    try {
        Method method4 = solution.getClass().getDeclaredMethod("method4");
        method4.setAccessible(true);
        method4.invoke(solution);
    } catch (NoSuchMethodException | InvocationTargetException | IllegalAccessException e) {
        e.printStackTrace();
    }
}

Вопрос же не в этом, а в том, что private не влияет на работу потоков.

Алексей

Уровень 32

18 октября 2021, 12:32

За рефлексию спасибо, изучил. Не в качестве спора: на мой взгляд, если уж ты целенаправленно ломаешь ООП таким образом, убедись, что у тебя метод synchronized, считав его модификаторы доступа. Ведь если я ставлю private - я фактически запрещаю к нему извне обращаться.

Ksenia Volkova Java Developer в DXC

Master

18 октября 2021, 12:39

На мой взгляд, если уж ты хочешь, чтобы твой метод не мог использоваться несколькими нитями - пользуйся ключевым словом synchronized, которое специально для этого предназначено. А не надейся на то, что модификатор private при определенном наборе обстоятельств (которые не всегда могут зависеть от тебя) не даст разным нитям дергать метод.

Алексей

Уровень 32

18 октября 2021, 12:57

Понял, спасибо!

Vadimayer

Уровень 40

18 октября 2021, 19:14

а почему тогда этот метод нельзя сделать synchronized , если предположить, что он будет использоваться другими нитями?

protected synchronized void method1(String param1) {
        Solution solution = new Solution();
        solution.method0();
    }

Ksenia Volkova Java Developer в DXC

Master

18 октября 2021, 19:17

полезный

Можно. Но смысла в этом нет - потому что даже если две нити зайдут в метод одновременно, все равно каждая будет работать со своим объектом Solution, который в этом методе создается.

Serg

Уровень 37

18 сентября 2021, 07:08

private StringBuilder sb это же не static. Private дает защиту от доступа извне. А если у нас два объекта класса Solution? У каждого объекта класса Solution будет доступ и вполне может быть ситуация когда две параллельные нити создают Solution solution = new Solution() и пытаются одновременно изменить StringBuilder sb. Посмотри НЕленивую реализацию паттерна Singlrton (Одиночка) там как раз защита от доступа при работе с нитями. И там специально synhronized в getInstance();

Vadimayer

Уровень 40

18 сентября 2021, 07:31

Тут поле Singleton INSTANCE статическое, а метод getInstance() публичный.

public class Singleton {
    private static Singleton INSTANCE;

  private Singleton() {
  }

    public static Singleton getInstance() {
        if (INSTANCE == null) {
            synchronized (Singleton.class) {
                if (INSTANCE == null) {
                    INSTANCE = new Singleton();
                }
            }
        }
        return INSTANCE;
    }
}

Vadimayer

Уровень 40

18 сентября 2021, 07:34

не совсем представляю эту ситуацию: У каждого объекта класса Solution будет доступ и вполне может быть ситуация когда две параллельные нити создают Solution solution = new Solution() и пытаются одновременно изменить StringBuilder sb.

Serg

Уровень 37

18 сентября 2021, 11:05

synchronized (Singleton.class) { if (INSTANCE == null) { INSTANCE = new Singleton(); } Видите, что у вас идет синхронизация по классу? А в задаче идет синхронизация самого метода. Вы путаете понятия модификаторы доступа и собственно что такое синхронизация. Они никаким боком к друг другу не относятся. Выше Ксения указала ситуацию когда две нити работают с одним обьектом Solution. Почитайте внимательно что такое многопоточность в джаве и как нити могут одновременно работать с одним и тем же обьектом или переменной. Для чего придумали volatile, synhronized и тд.

Vadimayer

Уровень 40

18 сентября 2021, 12:01

да читал я и не раз, на практике возникают моменты, о которых чаще всего не пишут