Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Системное программирование и WinAPI > Вызов глобальной функции из функции потока

Автор: lv151 8.11.2011, 16:44

Не возникнет проблема при вызове 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;
}

Автор: azesmcar 8.11.2011, 16:48

Нет. Память выделяется для каждого потока и каждый поток работает с отдельными данными.

Автор: lv151 8.11.2011, 16:51

thx

Автор: 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;

Добавлено через 10 секунд

Цитата(lv151 @ 8.11.2011, 16:51

)

thx

пожалуйста.

Автор: Alca 8.11.2011, 17:01

Цитата

Код


_endthreadex(0);

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

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

Цитата

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

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

Автор: lv151 8.11.2011, 17:10

Цитата(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

пожалуйста.

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

Автор: Alca 8.11.2011, 17:41

Может так?

Код


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

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

Автор: azesmcar 8.11.2011, 17:52

Цитата(lv151 @ 8.11.2011, 17:10

)

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

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

Автор: lv151 9.11.2011, 08:24

Ты прав. Я не обнулял в NULL, поэтому и валилось.

Автор: lv151 9.11.2011, 08:40

Цитата(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;
}

Так лучше?

Автор: azesmcar 9.11.2011, 09:28

Цитата(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.

из http://msdn.microsoft.com/en-us/library/hw264s73.aspx

Цитата(lv151 @ 9.11.2011, 08:40

)

Так лучше?

Да.

Автор: lv151 9.11.2011, 11:02

Насчёт NULL.
В деструкторах есть смысл проверять на NULL или так же удалять без проверки?

Автор: Alca 9.11.2011, 11:17

внутри delete уже есть проверка на NULL