Форум программистов, компьютерный форум CyberForum.ru

Mutex блокировка приватного поля класса - C++

Восстановить пароль Регистрация
 
Shaman163
4 / 4 / 0
Регистрация: 22.12.2011
Сообщений: 134
27.09.2013, 21:28     Mutex блокировка приватного поля класса #1
Доброго времени суток.
Использую Qt Creator 5.1.1 MinGW 32
Вопрос в следующем..
Есть класс "A" в котором есть поля private и public, информация которую нужно блокировать находится в поле private.
Для наглядности код:
C++
1
2
3
4
5
6
7
8
class A
{
private:
   Pixel** pixels;
   std::mutex door;
public:
   Pixel** GetPixels();
}
Как я понял Mutex должен находится в поле которое должно быть захвачено но что делать с private?
Дописывание публичных методов Lock() и Unlock() которые будут 'общаться' с приватным Мьютексом в моей задаче не выход, так как существует вероятность рассинхрона потоков.
То есть мне нужно обращаться напрямую к Мьютексу door. Вопрос в том как это сделать?
Ну или аналогичное решение проблемы..
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Tulosba
:)
Эксперт C++
4378 / 3221 / 297
Регистрация: 19.02.2013
Сообщений: 9,044
27.09.2013, 21:44     Mutex блокировка приватного поля класса #2
Shaman163, опишите лучше задачу, что Вы хотите сделать.
Shaman163
4 / 4 / 0
Регистрация: 22.12.2011
Сообщений: 134
27.09.2013, 22:45  [ТС]     Mutex блокировка приватного поля класса #3
Реализовать фильтрацию изображения. Класс "A" это так называемый Bitmap содержащий массив пикселей, мне нужно заблокировать этот массив, затем вытащить массив указателей на пиксели методом GetPixels() ну а после разблокировать массив для дальнейших операций. Всё это должно осуществляться в отдельном потоке, из за чего собственно весь геморой с мьютексами.
Вот класс который работает криво. При объявлениии нового потока Мьютекс должен захватываться сразу же, а здесь он захватывается только после вызова Lock() в этом месте и происходит рассинхрон.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
    class Bitmap
    {
        private:
            Pixel** pixels;
            Vector2 Size;
            mutex door;
 
        public:
            //= Конструкторы / Деструкторы ============================
            Bitmap();
            ~Bitmap();
            Bitmap(Vector2 size);
            Bitmap(Pixel** pixels);
            Bitmap(Byte* pixels, int bpp);
 
            //= Методы замены =========================================
            void SetPixel (const Vector2 position, const Pixel color);
            void SetPixels(Pixel** pixels);
            void SetBytes(Byte* Pixels, int bpp);
 
            //= Методы получения =======================================
            Pixel  GetPixel(Vector2 position);
            Pixel** GetPixels();
            Byte*   GetBytes(int bpp);
            Vector2 GetSize();
 
            //= Обработка Mutex ========================================
            bool closed = false;// флаг показывающий заблокированы ли ресурсы изображения
 
            void Lock()
            {
                closed = true;
                door.lock();
            }
            bool try_Lock()
            {
                return closed = door.try_lock();
            }
            void Unlock()
            {
                closed = false;
                door.unlock();
            }
 
    };
Ну и вот ещё объект нового потока:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
        void Binary::Build(Bitmap* face, float treshold)
        {
            // дожидаемся освобождения ресурсов.
            while(!face->try_Lock());
            // как только они освобождаются блокируем и забираем их для просчёта
            Pixel** pixels = face->GetPixels();
            Vector2 Size = face->GetSize();
 
            for(int x = 0; x < Size.x; x++)
            {
                for(int y = 0; y < Size.y; y++)
                {
                    if(pixels[x][y].GetBrightness() <  treshold)
                    {
                        pixels[x][y] = Pixel(0,0,0);
                    }
                    else
                    {
                        pixels[x][y] = Pixel(255,255,255);
                    }
                }
            }
            face->Unlock();// разблокирование после просчёта.
        }
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
27.09.2013, 23:07     Mutex блокировка приватного поля класса #4
Почему бы захват мьютекса не проводить внутри GetPixels() ? публичный метод lock/try_lock убрать, оставить только unlock

Добавлено через 4 минуты
Можно еще сделать метод
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
void Bitmap::process_pixels(std::function<void(Pixel&)> i_process)
{
  std::lock_guard<std::mutex> lock(door);
 
  // дальше прежний цикл
  for(int x = 0; x < Size.x; x++)
  {
     for(int y = 0; y < Size.y; y++)
     {
        i_process(pixels[x][y]);
     }
  }
}
OhMyGodSoLong
~ Эврика! ~
 Аватар для OhMyGodSoLong
1234 / 983 / 42
Регистрация: 24.07.2012
Сообщений: 2,002
27.09.2013, 23:17     Mutex блокировка приватного поля класса #5
Цитата Сообщение от Shaman163 Посмотреть сообщение
Вопрос в том как это сделать?
В лоб:
C++
1
2
3
4
5
Pixel A::GetPixel(int x, int y)
{
    std::lock_guard<std::mutex> lock(door);
    return pixels[x][y];
}
std::library<std::way>:
C++
1
2
3
4
5
std::pair<Pixel**, std::unique_lock<std::mutex>>&& A::GetPixels()
{
    typedef std::unique_lock<std::mutex> Lock;
    return std::move(std::pair<Pixel**, Lock>(pixels, Lock(door)));
}
Функциональнее:
C++
1
2
3
4
5
template <class Func>
auto A::WithPixels(Func&& thunk) -> decltype(thunk(pixels)) {
    std::lock_guard<std::mutex> lock(door);
    return thunk(pixels);
}
Shaman163
4 / 4 / 0
Регистрация: 22.12.2011
Сообщений: 134
28.09.2013, 00:00  [ТС]     Mutex блокировка приватного поля класса #6
Какую роль играет lock_guard???
Цитата Сообщение от I.M. Посмотреть сообщение
публичный метод lock/try_lock убрать, оставить только unlock
Если try_lock убрать, как тогда выловить момент освобождения ресурсов?
Цитата Сообщение от I.M. Посмотреть сообщение
Почему бы захват мьютекса не проводить внутри GetPixels() ?
Но это по сути тоже самое что делал я..
Мне нужно при создании потока сразу же заблокировать ресурсы..
OhMyGodSoLong
~ Эврика! ~
 Аватар для OhMyGodSoLong
1234 / 983 / 42
Регистрация: 24.07.2012
Сообщений: 2,002
28.09.2013, 00:09     Mutex блокировка приватного поля класса #7
Цитата Сообщение от Shaman163 Посмотреть сообщение
Какую роль играет lock_guard???
Роль механизма, который никогда не забывает сделать unlock() после завершения работы с разделяемыми данными.
Shaman163
4 / 4 / 0
Регистрация: 22.12.2011
Сообщений: 134
28.09.2013, 00:10  [ТС]     Mutex блокировка приватного поля класса #8
Вот код непосредственно фильтрации:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
        void Binary::Build(Bitmap* face, float treshold)
        {
            while(!face->try_lock());
            Pixel** pixels = face->GetPixels();
            Vector2 Size = face->GetSize();
 
            for(int x = 0; x < Size.x; x++)
            {
                for(int y = 0; y < Size.y; y++)
                {
                    if(pixels[x][y].GetBrightness() <  treshold)
                    {
                        pixels[x][y] = Pixel(0,0,0);
                    }
                    else
                    {
                        pixels[x][y] = Pixel(255,255,255);
                    }
                }
            }
            face->Unlock();
        }
 
 
        Binary::Binary(Bitmap* bitmap, float treshold)
        {
            thread th0(Build, bitmap, treshold);
            th0.detach();
        }
Вся проблема в том, что если сразу же после вызова Binary(Bitmap,float) вызвать GetPixel() поток не успеет заблокировать пиксели до вызова метода GetPixel().
OhMyGodSoLong
~ Эврика! ~
 Аватар для OhMyGodSoLong
1234 / 983 / 42
Регистрация: 24.07.2012
Сообщений: 2,002
28.09.2013, 00:24     Mutex блокировка приватного поля класса #9
Цитата Сообщение от Shaman163 Посмотреть сообщение
Вся проблема в том, что если сразу же после вызова Binary(Bitmap,float) вызвать GetPixel() поток не успеет заблокировать пиксели до вызова метода GetPixel().
Вам надо передать в поток Bitmap с уже захваченным мьютексом и там передать владение. Захватите unique_lock для мьютекса передаваемого Bitmap в конструкторе Binary, передайте их обоих в новый поток, где Build создаёт у себя локальный unique_lock и забирает себе захваченный мьютекс из параметра.

(И почему while(mutex.trylock()) вместо mutex.lock()?..)
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
28.09.2013, 01:18     Mutex блокировка приватного поля класса
Еще ссылки по теме:

C++ Друзья, подскажите, говорят, что можно поля класса инициализировать в момент объявления класса. Как это сделать?
C++ Mutex ограничение доступа к переменным класса
Передача поля класса C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Shaman163
4 / 4 / 0
Регистрация: 22.12.2011
Сообщений: 134
28.09.2013, 01:18  [ТС]     Mutex блокировка приватного поля класса #10
Цитата Сообщение от OhMyGodSoLong Посмотреть сообщение
(И почему while(mutex.trylock()) вместо mutex.lock()?..)
мы ожидаем освобождения, если try_lock возвращает false цикл продолжается, если true mutex наш, и мы можем обращаться к ресурсам Bitmap
Цитата Сообщение от OhMyGodSoLong Посмотреть сообщение
Захватите unique_lock для мьютекса передаваемого Bitmap в конструкторе Binary, передайте их обоих в новый поток, где Build создаёт у себя локальный unique_lock и забирает себе захваченный мьютекс из параметра.
Понимаю что это немного нагло, но всё же, не могли бы вы дать пример как подобное сотворить?
Yandex
Объявления
28.09.2013, 01:18     Mutex блокировка приватного поля класса
Ответ Создать тему
Опции темы

Текущее время: 01:49. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru