Мусор и оператор new

@SEGNET · Регистрация: 28.12.2012

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

Создаю глобальный указатель на структуру, инициализирую некоторые поля после выделения оператором new.
Вызываю функцию (callback) создающую диалоговое окно, пытаюсь заполнить оставшиеся поля. При выходе из нее значения действительные, но при повторном входе в эту функцию там мусор. Что это может быть?

@castaway · 18.09.2014, 17:24

Может быть что угодно. Нужно код смотреть.

@worldown · 18.09.2014, 17:24

прогугли "static" может поможет

@aLarman · 18.09.2014, 17:33

Сообщение от SEGNET

Что это может быть?

копирование...например

@SEGNET · 18.09.2014, 17:42 **[ТС]**

Кода много. Те поля которые были иниц. значениями - не изменились, а оставшиеся мусор. Бред какой-то.

@aLarman · 18.09.2014, 17:43

Сообщение от SEGNET

Вызываю функцию (callback) создающую диалоговое окно, пытаюсь заполнить оставшиеся поля. При выходе из нее значения действительные, но при повторном входе в эту функцию там мусор

Вы хотя бы ее покажите

@castaway · 18.09.2014, 17:54

Покажи структуру, как ты выделяешь под неё память и как изменяешь её поля.

@SEGNET · 18.09.2014, 18:01 **[ТС]**

C++

    mystruct *pFSP;
 
BOOL CALLBACK DlgSetParam(HWND hDlg, UINT iMsg, WPARAM wParam, LPARAM lParam)
{
    char tempbuf[50] = { 0 };
    switch (iMsg)
    {
    case WM_INITDIALOG:
        // Выводим данные структуры 
        //pFSP = new mystruct[xxx]
 
        _itoa_s(pFSP[iIdItem].iSecTake, tempbuf, 10);
        SetWindowText(GetDlgItem(hDlg, IDC_EDIT1), tempbuf);
        ZeroMemory(tempbuf, sizeof(tempbuf));
        _itoa_s(pFSP[iIdItem].iSecBusy, tempbuf, 10);
        SetWindowText(GetDlgItem(hDlg, IDC_EDIT2), tempbuf);
        ZeroMemory(tempbuf, sizeof(tempbuf));
        _itoa_s(pFSP[iIdItem].iSecNotAvail, tempbuf, 10);
        SetWindowText(GetDlgItem(hDlg, IDC_EDIT3), tempbuf);
        ZeroMemory(tempbuf, sizeof(tempbuf));
        return TRUE;
 
    case WM_COMMAND:
        switch (wParam)
        {
        case IDOK:
            /* Получаем данные и заносим в структуру */
            GetWindowText(GetDlgItem(hDlg, IDC_EDIT1), tempbuf, sizeof(tempbuf));
            ZeroMemory(tempbuf, sizeof(tempbuf));
            GetWindowText(GetDlgItem(hDlg, IDC_EDIT2), tempbuf, sizeof(tempbuf));
            ZeroMemory(tempbuf, sizeof(tempbuf));
            GetWindowText(GetDlgItem(hDlg, IDC_EDIT3), tempbuf, sizeof(tempbuf));
 
        case IDCANCEL:
            EndDialog(hDlg, 0);
            return TRUE;
        }
        break;
    }
    return FALSE;
}

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

@castaway · 18.09.2014, 18:05

В каком месте выделяется память под массив структур?

@SEGNET · 18.09.2014, 20:39 **[ТС]**

Сообщение от castaway

В каком месте выделяется память под массив структур?

В функций обработки сообщений (WndProc). В определенном мною сообщений. Вход производится один раз после создания окон. Примерно так.

C++

    if (fopen_s(&hfile, "listphone.txt", "r")) {
            MessageBox(0, "Ошибка файла listphone.txt", 0, 0);
            return 0;
        }
            for (int i = 0; fgets(tempbuf, sizeof(tempbuf), hfile);CountPhone++) {  }
            fseek(hfile,NULL,SEEK_SET);
            pFSP = new mystruct[CountPhone];
            if (pFSP == 0) MessageBoxA(0,"Ошибка выделения памяти",0,0);

Добавлено через 43 минуты
Мое сообщение конфликтовало с другими и поэтому память выделялась несколько раз. Спасибо всем отписавшимся.

@aLarman · 19.09.2014, 09:35

Сообщение от SEGNET

C++

1
2
3

for (int i = 0; fgets(tempbuf, sizeof(tempbuf), hfile);CountPhone++) 
{
 }

пустой for О_О

@DrOffset · 19.09.2014, 09:51

Сообщение от aLarman

пустой for О_О

Ну фактически он не пустой, там каждую итерацию выполняется fgets(tempbuf, sizeof(tempbuf), hfile).

Tulosba · 19.09.2014, 10:27

Сообщение от DrOffset

там каждую итерацию выполняется fgets(tempbuf, sizeof(tempbuf), hfile).

и инкремент CountPhone

Вообще, конечно любопытный цикл, что в каждой его секции несвязанные переменные.

@aLarman · 19.09.2014, 10:40

DrOffset, точно...

Сообщение от SEGNET

C++

1 2	pFSP = new mystruct[CountPhone]; if (pFSP == 0) MessageBoxA(0,"Ошибка выделения памяти",0,0);

да че уж там проверять то?) исключение будет если не выделится память). Если Вам хочется так то

C++

1 2	pFSP = new(std::nothrow) mystruct[CountPhone]; if (pFSP == 0) MessageBoxA(0,"Ошибка выделения памяти",0,0);

Tulosba · 19.09.2014, 10:45

Сообщение от aLarman

исключение будет если не выделится память)

если используется стандартная либа, собранная без поддержки исключений (а такие бывают), то можно и без placement версии new (если я не ошибаюсь).

@aLarman · 19.09.2014, 10:49

Сообщение от Tulosba

без placement версии new

а это разве placement?

Добавлено через 1 минуту

Не по теме:

Сообщение от Tulosba

собранная без поддержки исключений

судя по тому, что приложение c WinApi я думаю TC вряд ли, пытался пересобирать стандартные либы :)

Tulosba · 19.09.2014, 11:01

Сообщение от aLarman

TC вряд ли, пытался пересобирать стандартные либы

а это и не требуется, просто она уже может идти в комплекте.

Сообщение от aLarman

а это разве placement?

18.6.1.1/5

C++

1	void* operator new(std::size_t size, const std::nothrow_t&) noexcept;

Effects: Same as above, except that it is called by a placement version of a new-expression when a C++
program prefers a null pointer result as an error indication, instead of a bad_alloc exception.

5.3.4/11

The new-placement syntax is used to supply additional arguments to an allocation function. If used, overload resolution is performed on a function call created by assembling an argument list consisting of the amount of space requested (the first argument) and the expressions in the new-placement part of the new-expression (the second and succeeding arguments). The first of these arguments has type std::size_t and the remaining arguments have the corresponding types of the expressions in the new-placement.

@aLarman · 19.09.2014, 11:15

Не по теме:

Tulosba, схоронил :)

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	19.09.2014, 10:27	13
	Сообщение от DrOffset там каждую итерацию выполняется fgets(tempbuf, sizeof(tempbuf), hfile). и инкремент CountPhone Вообще, конечно любопытный цикл, что в каждой его секции несвязанные переменные. 0

@aLarman 654 / 575 / 164 Регистрация: 13.12.2012 Сообщений: 2,124
	19.09.2014, 10:49	16
	Сообщение от Tulosba без placement версии new а это разве placement? Добавлено через 1 минуту Не по теме: Сообщение от Tulosba собранная без поддержки исключений судя по тому, что приложение c WinApi я думаю TC вряд ли, пытался пересобирать стандартные либы :) 0

@SEGNET 68 / 68 / 11 Регистрация: 28.12.2012 Сообщений: 471
		1
	Мусор и оператор new 18.09.2014, 17:19. Показов 1247. Ответов 17 Метки нет (Все метки) Создаю глобальный указатель на структуру, инициализирую некоторые поля после выделения оператором new. Вызываю функцию (callback) создающую диалоговое окно, пытаюсь заполнить оставшиеся поля. При выходе из нее значения действительные, но при повторном входе в эту функцию там мусор. Что это может быть? 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	18.09.2014, 17:24	2
	Может быть что угодно. Нужно код смотреть. 0

@worldown 189 / 177 / 111 Регистрация: 22.06.2009 Сообщений: 533
	18.09.2014, 17:24	3
	прогугли "static" может поможет 0

@aLarman 654 / 575 / 164 Регистрация: 13.12.2012 Сообщений: 2,124
	18.09.2014, 17:33	4
	Сообщение от SEGNET Что это может быть? копирование...например 1

@SEGNET 68 / 68 / 11 Регистрация: 28.12.2012 Сообщений: 471
	18.09.2014, 17:42 [ТС]	5
	Кода много. Те поля которые были иниц. значениями - не изменились, а оставшиеся мусор. Бред какой-то. 0

@aLarman 654 / 575 / 164 Регистрация: 13.12.2012 Сообщений: 2,124
	18.09.2014, 17:43	6
	Сообщение от SEGNET Вызываю функцию (callback) создающую диалоговое окно, пытаюсь заполнить оставшиеся поля. При выходе из нее значения действительные, но при повторном входе в эту функцию там мусор Вы хотя бы ее покажите 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	18.09.2014, 17:54	7
	Покажи структуру, как ты выделяешь под неё память и как изменяешь её поля. 0

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	18.09.2014, 18:05	9
	Сообщение было отмечено SEGNET как решение Решение В каком месте выделяется память под массив структур? 0

@DrOffset 19409 / 10028 / 2443 Регистрация: 30.01.2014 Сообщений: 17,678
	19.09.2014, 09:51	12
	Сообщение от aLarman пустой for О_О Ну фактически он не пустой, там каждую итерацию выполняется fgets(tempbuf, sizeof(tempbuf), hfile). 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	19.09.2014, 10:45	15
	Сообщение от aLarman исключение будет если не выделится память) если используется стандартная либа, собранная без поддержки исключений (а такие бывают), то можно и без placement версии new (если я не ошибаюсь). 0

@aLarman
	19.09.2014, 11:15 Мусор и оператор new #18
	Не по теме: Tulosba, схоронил :) 0

Мусор и оператор new

Решение