Неправильно работает 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 **[ТС]**

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

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Ниже машинный перевод статьи The Thinkpad X220 Tablet is the best budget school laptop period . Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы,. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .

@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