Модераторы: feodorv, GremlinProg, xvr, Fixin
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Вызов глобальной функции из функции потока 
V
    Опции темы
lv151
Дата 8.11.2011, 16:44 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


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

Репутация: 1
Всего: 2
Не возникнет проблема при вызове BuildString при запуске нескольких потоков ThreadFunc?

Код

void BuildString(TCHAR* message)
{

}

unsigned __stdcall ThreadFunc(void* pvParam)
{            
    TCHAR* message = new TCHAR[charCount];
    BuildString(message);
        
    if (message != NULL)
    {
          delete[] message;
          message = NULL;
    }
    
    _endthreadex(0);
    return 0;
}


Это сообщение отредактировал(а) lv151 - 8.11.2011, 16:44
PM MAIL   Вверх
azesmcar
Дата 8.11.2011, 16:48 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


uploading...
****


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

Репутация: 5
Всего: 211
Нет. Память выделяется для каждого потока и каждый поток работает с отдельными данными.
PM   Вверх
lv151
Дата 8.11.2011, 16:51 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


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

Репутация: 1
Всего: 2
thx
PM MAIL   Вверх
azesmcar
Дата 8.11.2011, 16:51 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


uploading...
****


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

Репутация: 5
Всего: 211
Цитата(lv151 @  8.11.2011,  16:44 Найти цитируемый пост)
 if (message != NULL)
    {
          delete[] message;
          message = NULL;
    }

проверка лишняя, delete и так проверяет на null. Достаточно написать
Код

delete[] message;
message = NULL;


Добавлено через 10 секунд
Цитата(lv151 @  8.11.2011,  16:51 Найти цитируемый пост)
thx 

пожалуйста.
PM   Вверх
Alca
Дата 8.11.2011, 17:01 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


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

Репутация: 5
Всего: 50
Цитата

Код

_endthreadex(0);


тоже лишний, эта функуция вызывается автоматом

Добавлено через 5 минут и 34 секунды
Цитата

Не возникнет проблема при вызове BuildString при запуске нескольких потоков ThreadFunc?

главное, чтоб в этой функции не было глобальных и статических данных.
И функций, которые могут эти данные использовать (strtok, ...), которые не thread-safe.


--------------------
C++ cross-platform library
PM WWW ICQ Skype Jabber   Вверх
lv151
Дата 8.11.2011, 17:10 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


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

Репутация: 1
Всего: 2
Цитата(azesmcar @ 8.11.2011,  16:51)
Цитата(lv151 @  8.11.2011,  16:44 Найти цитируемый пост)
 if (message != NULL)
    {
          delete[] message;
          message = NULL;
    }

проверка лишняя, delete и так проверяет на null. Достаточно написать
Код

delete[] message;
message = NULL;


Добавлено @ 16:52
Цитата(lv151 @  8.11.2011,  16:51 Найти цитируемый пост)
thx 

пожалуйста.

Падает приложение если не проверяю на нулл.  Я смотрю народ в примерах и в мсдн юзает эту проверку.

Это сообщение отредактировал(а) lv151 - 8.11.2011, 17:20
PM MAIL   Вверх
Alca
Дата 8.11.2011, 17:41 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


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

Репутация: 5
Всего: 50
Может так?
Код

void BuildString(TCHAR **message)
{
    *message = ... ;
}

unsigned __stdcall ThreadFunc(void* pvParam)
{            
    TCHAR* message = new TCHAR[charCount];
    BuildString(&message);
        
    delete[] message;
    message = NULL;
    
    return 0;
}


Это сообщение отредактировал(а) Alca - 8.11.2011, 17:46


--------------------
C++ cross-platform library
PM WWW ICQ Skype Jabber   Вверх
azesmcar
Дата 8.11.2011, 17:52 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


uploading...
****


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

Репутация: 5
Всего: 211
Цитата(lv151 @  8.11.2011,  17:10 Найти цитируемый пост)
Падает приложение если не проверяю на нулл.  Я смотрю народ в примерах и в мсдн юзает эту проверку.

не верю ©
пример покажи.

Это сообщение отредактировал(а) azesmcar - 8.11.2011, 17:53
PM   Вверх
lv151
Дата 9.11.2011, 08:24 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


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

Репутация: 1
Всего: 2
Ты прав. Я не обнулял в NULL, поэтому и валилось.
PM MAIL   Вверх
lv151
Дата 9.11.2011, 08:40 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


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

Репутация: 1
Всего: 2
Цитата(Alca @  8.11.2011,  17:01 Найти цитируемый пост)
Цитата

Выделить всёкод C++
1:
    
_endthreadex(0);


тоже лишний, эта функуция вызывается автоматом


Так в примере из мсдн.


Цитата(Alca @  8.11.2011,  17:41 Найти цитируемый пост)
Может так?
Выделить всёкод C++
void BuildString(TCHAR **message)
{
    *message = ... ;
}
unsigned __stdcall ThreadFunc(void* pvParam)
{            
    TCHAR* message = new TCHAR[charCount];
    BuildString(&message);
        
    delete[] message;
    message = NULL;
    
    return 0;
}


Так лучше?
PM MAIL   Вверх
azesmcar
Дата 9.11.2011, 09:28 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


uploading...
****


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

Репутация: 5
Всего: 211
Цитата(lv151 @  9.11.2011,  08:40 Найти цитируемый пост)
Так в примере из мсдн.


Цитата(MSDN)

You can call _endthread or _endthreadex explicitly to terminate a thread; however, _endthread or _endthreadex is called automatically when the thread returns from the routine passed as a parameter to _beginthread or _beginthreadex. 

из MSDN


Цитата(lv151 @  9.11.2011,  08:40 Найти цитируемый пост)
Так лучше?

Да.
PM   Вверх
lv151
Дата 9.11.2011, 11:02 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


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

Репутация: 1
Всего: 2
Насчёт NULL.
В деструкторах есть смысл проверять на NULL или так же удалять без проверки?
PM MAIL   Вверх
Alca
Дата 9.11.2011, 11:17 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


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

Репутация: 5
Всего: 50
внутри delete уже есть проверка на NULL


--------------------
C++ cross-platform library
PM WWW ICQ Skype Jabber   Вверх
  
 Ответ в темуСоздание новой темы Создание опроса
Правила форума "C/C++: Системное программирование и WinAPI"
Fixin
GremlinProg
xvr
feodorv
  • Большое количество информации и примеров с использованием функций WinAPI можно найти в MSDN
  • Описание сообщений, уведомлений и примеров с использованием компонент WinAPI (BUTTON, EDIT, STATIC, и т.п.), можно найти в MSDN Control Library
  • Непосредственно, перед созданием новой темы, проверьте заголовок и удостоверьтесь, что он отражает суть обсуждения.
  • После заполнения поля "Название темы", обратите внимание на наличие и содержание панели "А здесь смотрели?", возможно Ваш вопрос уже был решен.
  • Приводите часть кода, в которой предположительно находится проблема или ошибка.
  • Если указываете код, пользуйтесь тегами [code][/code], или их кнопочными аналогами.
  • Если вопрос решен, воспользуйтесь соответствующей ссылкой, расположенной напротив названия темы.
  • Один топик - один вопрос!
  • Перед тем как создать тему - прочтите это .

На данный раздел распространяются Правила форума и Правила раздела С++:Общие вопросы .

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, Chipset, Step, Fixin, GremlinProg, xvr. feodorv.

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

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0908 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.