Многопоточность, чтение и запись переменной типа стринг

@алексашк · Регистрация: 21.01.2014

Author24 — интернет-сервис помощи студентам

всем привет!
реализовал небольшой алгоритм, суть задания была в следующем:
Реализовать «синхронные» методы чтения/записи для переменной типа String. Правила обращения следующие:
a. Записывать в одно время может только один поток, любой следующий «писатель» должен ждать (блокироваться).
b. Если один поток выполняет запись, то любой читатель должен ждать завершения этой записи.
c. Пока происходит чтение, запись невозможна.
d. Однако требуется избежать такой ситуации, когда запись невозможна из-за сменяющих друг друга операций чтения.
e. Любое количество читающих потоков могут читать одновременно

вот что получилось у меня:

Java

package Potoki;
 
public class MainClass {
 
    public static volatile String s = "Start: ";
 
    public static void main(String args[]) {
        ThreadWrite thrWr1 = new ThreadWrite('x'), thrWr2 = new ThreadWrite('y');
        ThreadRead thrRd1 = new ThreadRead(1), thrRd2 = new ThreadRead(2);
        thrWr1.start();
        thrWr2.start();
        thrRd1.start();
        thrRd2.start();
        try {
            thrWr1.join();
            thrWr2.join();
            thrRd1.join();
            thrRd2.join();
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }
}

Java

package Potoki;
 
public class ThreadWrite extends Thread {
    private char simbol;
 
    public ThreadWrite(char c) {
        simbol = c;
    }
 
    @Override
    public void run() {
        while (true) {
            synchronized (MainClass.s) {
                MainClass.s += simbol;
                System.out.println("запись потоком " + simbol + " строка: " + MainClass.s);
            }
            try {
                Thread.sleep(500);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
    }
 
}

Java

package Potoki;
 
public class ThreadRead extends Thread {
    private int name;
 
    ThreadRead(int i) {
        name = i;
    }
 
    public void run() {
 
        try {
            while (true) {
                System.out.println("чтение потоком " + name + " строка: " + MainClass.s);
                Thread.sleep(500);
            }
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }
}

у меня вопрос в следующем, когда просмотрел консольку, то увидел следующее:
...
чтение потоком 2 строка: Start: yxyxyxyxy
запись потоком y строка: Start: yxyxyxyxy
запись потоком x строка: Start: yxyxyxyxyx
запись потоком x строка: Start: yxyxyxyxyxx
запись потоком y строка: Start: yxyxyxyxyxxy
чтение потоком 2 строка: Start: yxyxyxyxyxx
чтение потоком 1 строка: Start: yxyxyxyxyxx
чтение потоком 1 строка: Start: yxyxyxyxyxxyx
чтение потоком 2 строка: Start: yxyxyxyxyxxy
запись потоком x строка: Start: yxyxyxyxyxxyx
запись потоком y строка: Start: yxyxyxyxyxxyxy
...
почему у меня в пятой строчке записывается Start: yxyxyxyxyxxy, а в шестой строчке выводится чтение потоком 2 строка: Start: yxyxyxyxyxx, то-есть на один символ меньше, по идее он же уже его записал, и переменная типа volatile, и синхронизацию сделал, что ему ещё не хватает? или так должно быть? если да, то почему?
возможно я неправильно вообще код реализовал, в общем помогите исправить либо код, либо подскажите, почему у меня так программка работает, всем спасибо

@turbanoff · 10.10.2015, 03:38

Вообще судя по описанию задания вам отлично подойдёт ReentrantReadWriteLock. Нет ограничений на те классы которые можно использовать?

Пункт b у вас не совсем честно выполнен.

Сообщение от алексашк

и так должно быть? если да, то почему?

Да, так и должно быть. Чтение переменной и вывод на экран - две разные операции, за счёт этого получается такой результат.

@алексашк · 10.10.2015, 03:43 **[ТС]**

про ReentrantReadWriteLock почитаю, ограничений по классам нет, спасибо за ответ

@turbanoff 4092 / 3826 / 745 Регистрация: 18.05.2010 Сообщений: 9,331 Записей в блоге: 11
	10.10.2015, 03:38	2
	Вообще судя по описанию задания вам отлично подойдёт ReentrantReadWriteLock. Нет ограничений на те классы которые можно использовать? Пункт b у вас не совсем честно выполнен. Сообщение от алексашк и так должно быть? если да, то почему? Да, так и должно быть. Чтение переменной и вывод на экран - две разные операции, за счёт этого получается такой результат. 0

@алексашк 0 / 0 / 0 Регистрация: 21.01.2014 Сообщений: 73
	10.10.2015, 03:43 [ТС]	3
	про ReentrantReadWriteLock почитаю, ограничений по классам нет, спасибо за ответ 0