VPF::Потоки в с++ - Форум программистов

XaverOz

Дата 9.3.2011, 21:46 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 35
Регистрация: 9.3.2011

Репутация: нет
Всего: нет

Учусь работать с потоками на линуксовом с++. Решил написать тестовую программу устроена следующим образом: существует класс при создании объекта класса происходит запуск потока который все время в бесконченом цикле увеличивает значение переменной если значение больше 532, переменная становится равна 0, так же в классе есть метод который позволяет получить значение этой переменной. Для организации взаимного исключения использовались мьютексы.

Код


class Devemu {
    int VarInc;
    pthread_t thread_id;
    pthread_mutex_t inc_mutex;
    
    void Increm() {
        for(;;) {
            pthread_mutex_lock(&inc_mutex);
            if (VarInc > 532) VarInc = 0;
            VarInc++;
            pthread_mutex_unlock(&inc_mutex);
        }
    }

public:
    static void* IncWrapper(void* thisPtr) {
        ((Devemu*) thisPtr)->Increm();
        return NULL;
    }
    Devemu() {
        VarInc = 0;
        pthread_mutex_init(&inc_mutex, NULL);
        pthread_create(&thread_id, NULL, &Devemu::IncWrapper, NULL);
    }
    int Var() {
        int i;
        pthread_mutex_lock(&inc_mutex);
        i =  VarInc;
        pthread_mutex_unlock(&inc_mutex);
        return i;
    }
}; 

int main(int argc, char** argv) {

    Devemu* em = new Devemu();
    int i = 0;
    for(; i < 35; i++) {
        printf("%d\n", em->Var());
    }
    return (EXIT_SUCCESS);
}

Данный код возвращает ошибку сегментации не могу понять почему

azesmcar

Дата 9.3.2011, 21:58 (ссылка)

(нет голосов)

Загрузка ...

uploading...

Профиль
Группа: Участник Клуба
Сообщений: 6291
Регистрация: 12.11.2004
Где: Армения

Репутация: 1
Всего: 211

Цитата(XaverOz @ 9.3.2011, 21:46

)

Данный код возвращает ошибку сегментации не могу понять почему

потому-что аргументом в поток передается NULL вместо this.

XaverOz

Дата 9.3.2011, 22:10 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 35
Регистрация: 9.3.2011

Репутация: нет
Всего: нет

Предлагаете переделать вот так?

Код


Devemu() {
        VarInc = 0;
        pthread_mutex_init(&inc_mutex, NULL);
        pthread_create(&thread_id, NULL, &Devemu::IncWrapper, this);
    }

Это сообщение отредактировал(а) XaverOz - 9.3.2011, 22:10

azesmcar

Дата 9.3.2011, 22:34 (ссылка)

(нет голосов)

Загрузка ...

uploading...

Профиль
Группа: Участник Клуба
Сообщений: 6291
Регистрация: 12.11.2004
Где: Армения

Репутация: 1
Всего: 211

Цитата(XaverOz @ 9.3.2011, 22:10

)

Предлагаете переделать вот так?

Да.
Не уверен, что это единственная ошибка, но во всяком случае пока что это попалось на глаза.

Это сообщение отредактировал(а) azesmcar - 9.3.2011, 22:35

fish9370

Дата 10.3.2011, 10:36 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 663
Регистрация: 15.4.2007
Где: Москва

Репутация: 2
Всего: 1

ну вообще поток перед выходом нужно прибить.. используя pthread_cancel() или вставить в цикл проверку на завершение и дождаться пока он сам умрет с помощью pthread_join()

а потом выходить..

Это сообщение отредактировал(а) fish9370 - 10.3.2011, 10:46

--------------------

undefined

fish9370

Дата 10.3.2011, 10:52 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 663
Регистрация: 15.4.2007
Где: Москва

Репутация: 2
Всего: 1

да и мютекс выделеный динамически нужно освобождать функцией pthread_mutex_destroy()..

--------------------

undefined

XaverOz

Дата 10.3.2011, 14:52 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 35
Регистрация: 9.3.2011

Репутация: нет
Всего: нет

спасибо заработало

boostcoder

Дата 10.3.2011, 16:16 (ссылка)

(нет голосов)

Загрузка ...

pattern`щик

Профиль
Группа: Завсегдатай
Сообщений: 5458
Регистрация: 1.4.2010

Репутация: 16
Всего: 110

Цитата(fish9370 @ 10.3.2011, 10:36

)

поток перед выходом нужно прибить.. используя pthread_cancel()

не нужно.

--------------------

компилятор онлайн с поддержкой C++11
сборки компиляторов MinGW

fish9370

Дата 10.3.2011, 17:38 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 663
Регистрация: 15.4.2007
Где: Москва

Репутация: 2
Всего: 1

Цитата(boostcoder @ 10.3.2011, 16:16

)

не нужно

не уверен, что это безопасно.. прошу аргументированный ответ..

--------------------

undefined

boostcoder

Дата 10.3.2011, 17:44 (ссылка)

(нет голосов)

Загрузка ...

pattern`щик

Профиль
Группа: Завсегдатай
Сообщений: 5458
Регистрация: 1.4.2010

Репутация: 16
Всего: 110

Цитата(fish9370 @ 10.3.2011, 17:38

)

прошу аргументированный

RTFM

--------------------

компилятор онлайн с поддержкой C++11
сборки компиляторов MinGW

fish9370

Дата 10.3.2011, 21:20 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 663
Регистрация: 15.4.2007
Где: Москва

Репутация: 2
Всего: 1

boostcoder, ты в вопросе не разбираешься.. ты бы хоть сказал на какой ман ты ссылаешься..

Это сообщение отредактировал(а) fish9370 - 10.3.2011, 21:22

--------------------

undefined

boostcoder

Дата 10.3.2011, 21:49 (ссылка)

(нет голосов)

Загрузка ...

pattern`щик

Профиль
Группа: Завсегдатай
Сообщений: 5458
Регистрация: 1.4.2010

Репутация: 16
Всего: 110

Цитата(fish9370 @ 10.3.2011, 21:20

)

ты в вопросе не разбираешься.

раз уж ты так сказал, значит так и есть smile

--------------------

компилятор онлайн с поддержкой C++11
сборки компиляторов MinGW

fish9370

Дата 10.3.2011, 22:21 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 663
Регистрация: 15.4.2007
Где: Москва

Репутация: 2
Всего: 1

Цитата(boostcoder @ 10.3.2011, 21:49

)

раз уж ты так сказал, значит так и есть

я мог бы и промолчать, но это и без того видно.. ты воздух сотрясаешь, а по делу тебе сказать нечего.. мое мнение..

--------------------

undefined

boostcoder

Дата 10.3.2011, 22:23 (ссылка)

(нет голосов)

Загрузка ...

pattern`щик

Профиль
Группа: Завсегдатай
Сообщений: 5458
Регистрация: 1.4.2010

Репутация: 16
Всего: 110

Цитата(fish9370 @ 10.3.2011, 22:21

)

ты воздух сотрясаешь, а по делу тебе сказать нечего

ну... это самый весомый аргумент. даже не поспоришь smile

--------------------

компилятор онлайн с поддержкой C++11
сборки компиляторов MinGW

fish9370

Дата 10.3.2011, 22:29 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 663
Регистрация: 15.4.2007
Где: Москва

Репутация: 2
Всего: 1

Цитата(boostcoder @ 10.3.2011, 22:23

)

ну... это самый весомый аргумент. даже не поспоришь

может, ты все-таки перестанешь умничать, и что-то ответишь?

напаминаю вопрос:

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

Это сообщение отредактировал(а) fish9370 - 10.3.2011, 22:32

--------------------

undefined

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Программирование под Unix/Linux \| Следующая тема »