Неправильно работает strcat

@razer89 · Регистрация: 20.06.2010

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

Всем привет! Прошу помощи у знающих, сам никак не могу справиться... Имеется код:

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
char* editor (nodeStruct nodei)
{
    char* tempChar;
    tempChar = new char;
    char tempBirthday2[3];
    char tempBirthday4[4];
 
    //Формирование строки
    strcpy (tempChar, nodei.firstName);
    strcat (tempChar, " ");
    strcat (tempChar, nodei.secondName);
    strcat (tempChar, " ");
    strcat (tempChar, nodei.lastName);
    strcat (tempChar, ", ");
 
    tempBirthday2[0] = AnsiString(nodei.birthday).c_str()[6];
    tempBirthday2[1] = AnsiString(nodei.birthday).c_str()[7];
    strcat (tempChar, tempBirthday2);
    strcat (tempChar, ".");
    tempBirthday2[0] = AnsiString(nodei.birthday).c_str()[4];
    tempBirthday2[1] = AnsiString(nodei.birthday).c_str()[5];
    strcat (tempChar, tempBirthday2);
    strcat (tempChar, ".");
    tempBirthday4[0] = AnsiString(nodei.birthday).c_str()[0];
    tempBirthday4[1] = AnsiString(nodei.birthday).c_str()[1];
    tempBirthday4[2] = AnsiString(nodei.birthday).c_str()[2];
    tempBirthday4[3] = AnsiString(nodei.birthday).c_str()[3];
    tempBirthday4[4] = '\0';
    strcat (tempChar, tempBirthday4);
    return tempChar;
    delete tempChar;
}
void __fastcall TForm2::Button3Click(TObject *Sender)
{
    char charDate[8];
    int date;
    char* tempChar3;
    tempChar3 = new char;
    bool isFinded = false;
 
    //Преобразование даты в строку
    charDate[0] = AnsiString(DateTimePicker1->Date.DateString()).c_str()[6];
    charDate[1] = AnsiString(DateTimePicker1->Date.DateString()).c_str()[7];
    charDate[2] = AnsiString(DateTimePicker1->Date.DateString()).c_str()[8];
    charDate[3] = AnsiString(DateTimePicker1->Date.DateString()).c_str()[9];
    charDate[4] = AnsiString(DateTimePicker1->Date.DateString()).c_str()[3];
    charDate[5] = AnsiString(DateTimePicker1->Date.DateString()).c_str()[4];
    charDate[6] = AnsiString(DateTimePicker1->Date.DateString()).c_str()[0];
    charDate[7] = AnsiString(DateTimePicker1->Date.DateString()).c_str()[1];
 
    //Преобразование даты в long
    date = atol (charDate);
 
    for (int i = 0; i < treeVector.size(); i++)
    {
        if (treeVector[i].birthday < date)
        {
            if (!isFinded)
            {
                strcpy (tempChar3, "Finded:\n");
                strcat (tempChar3, editor(treeVector[i]));
            }
            else
            {
                strcat (tempChar3, "\n");
                strcat (tempChar3, editor(treeVector[i])); //Вот здесь возникает проблема
            }
            isFinded = true;
        }
    }
    if (isFinded)
    {
        StaticText1->Caption = tempChar3;
        isFinded = false;
    }
 
    delete tempChar3;
}

Коротко поясню: функция editor преобразует данные из структуры nodeStruct в строку (ФИО и дата рождения), и возвращает эту строку типа char *. Далее, в цикле, эта строка несколько раз добавляется (если нужно вывести несколько строк), и вот тут возникает проблема - при обычном копировании значения, возвращаемого editor, все нормально, но стоит использовать strcat, как в поле StaticText1 начинает выводиться помимо нужной информации еще и всякие кракозябры. Никак не могу понять, в чем проблема? Догадываюсь, что дело в каком-то указателе... В общем, буду рад если ткнете носом в ошибку. =)
Заранее благодарен!

@cmath · 20.01.2013, 17:29

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

еще и всякие кракозябры.

Ставьте \0 в том месте, где заканчивается нужная информация в строке. Ошибка - вы не учитываете длину строки. Если строка получается короче, то вылезают всякие "крякозябы". По умолчанию строка после инициализации забивается черт знаем чем. Когда вы записываете что-то в строку (если не использовали её целиком), то ведь символы в "хвосте" не затираются автоматически.

@razer89 · 20.01.2013, 17:36 **[ТС]**

Уважаемый cmath, прошу Вас обратить на строку 28, там вставляется символ окончания строки. Или так поступать не верно?

@cmath · 20.01.2013, 17:50

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

Или так поступать не верно?

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

tempBirthday4[4] = '\0';

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

strcat (tempChar, tempBirthday4);

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

return tempChar;

А в строке

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

char* tempChar;

?
Кст, в Builder есть средство, позволяющее просмотреть выполнение программы пошагово. Не пробовали использовать?

@razer89 · 20.01.2013, 19:27 **[ТС]**

cmath, пробовал, теперь вообще все изменилось. При первом вызове editor возвращается нормальная строка, но при 2-ом вообще вылетает access violation. Ничего не понимаю! Ошибка вылетает при повторном определении char переменных.

Добавлено через 55 минут
Нагуглил, что удалять char нужно не просто delete, а delete[], однако и это не помогает...

Croessmah · 20.01.2013, 20:45

Не по теме:

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

Нагуглил, что удалять char нужно не просто delete, а delete[], однако и это не помогает...

Если выделяете один элемент, то delete, если массив, то delete []

@razer89 · 20.01.2013, 21:11 **[ТС]**

Croessmah, прошу прощения, но я не совсем понимаю как писать в данном случае? Ведь по сути char - это массив символов, тогда получается что delete []?

@Том Ардер · 20.01.2013, 21:11

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

char* tempChar; tempChar = new char;

Здесь описан только указатель на строку, но память для её размещения не выделена. Нужно так:

C++
1
2
3
4
5
tempChar = new char[256];  // нужный размер
 
{всякие действия}
 
delete [] tempChar;

@razer89 · 20.01.2013, 21:52 **[ТС]**

Том Ардер, спасибо Вам огромное! Теперь все работает как надо!

Новые блоги и статьи Все статьи Все блоги /
[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .
Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .

@razer89 15 / 15 / 0 Регистрация: 20.06.2010 Сообщений: 142
	20.01.2013, 17:36 [ТС]
	Уважаемый cmath, прошу Вас обратить на строку 28, там вставляется символ окончания строки. Или так поступать не верно? 0

@razer89 15 / 15 / 0 Регистрация: 20.06.2010 Сообщений: 142
	20.01.2013, 19:27 [ТС]
	cmath, пробовал, теперь вообще все изменилось. При первом вызове editor возвращается нормальная строка, но при 2-ом вообще вылетает access violation. Ничего не понимаю! Ошибка вылетает при повторном определении char переменных. Добавлено через 55 минут Нагуглил, что удалять char нужно не просто delete, а delete[], однако и это не помогает... 0

@razer89 15 / 15 / 0 Регистрация: 20.06.2010 Сообщений: 142
	20.01.2013, 21:11 [ТС]
	Croessmah, прошу прощения, но я не совсем понимаю как писать в данном случае? Ведь по сути char - это массив символов, тогда получается что delete []? 0

@razer89 15 / 15 / 0 Регистрация: 20.06.2010 Сообщений: 142
	20.01.2013, 21:52 [ТС]
	Том Ардер, спасибо Вам огромное! Теперь все работает как надо! 0