Странное поведение строковых переменных (меняются с того ни с сего)

@4lice · Регистрация: 19.07.2014

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

Код такой:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
UnicodeString gorod; // глобальная
void __fastcall TForm1::DBLookupComboBox2Click(TObject *Sender)
{   wchar_t *other =L"Другой";
    UnicodeString pr=ADOQuery2->FieldByName("Название")->AsString;
    if(wcscmp(other,pr.w_str())!=0)
    {
        gorod=ADOQuery2->FieldByName("Идентификатор")->AsString;
    }
    else
    {
        Form3->Visible = true;
    }
    wchar_t *s1 = L"http://xoap.weather.com/weather/local/";
    wchar_t *s2 = L"?cc=*&dayf=5&link=xoap&prod=xoap&unit=m&par=1171574135&key=a3740f391c3a72ee";
    wchar_t *s3; s3 = gorod.w_str();
    wcscat(s1,s3);
    wcscat(s1,s2);
    wchar_t *s4 = GetCurrentDir().w_str();
    wchar_t *s5 = L"\\pogoda.xml";
    wcscat(s4,s5);
    URLDownloadToFile(0, s1, s4, 0, 0);
}

Первый проход:
other="Другой";
pr="Мурманск";
gorod="RSXX0452";
s3="RSXX0452";
s2 = L"?cc=*&dayf=5&link=xoap&prod=xoap&unit= m&par=1171574135&key=a3740f391c3a72ee" ; // после выполнения 16-ой строчки значение s2="SXX0452", после 17-ой s2="SXX0452SXX0452". С какого перепуга??
s1=L"http://xoap.weather.com/weather/local/"; // из-за глюка в s2 прибавилось "SXX0452SXX0452" на выполнении 17-ой строки.
s4=L"C:\\Users\\Алиса\\Documents\\RAD Studio\\Projects\\Pro2\\Win32\\Debug"; // после 20-ой строчки L"C:\\Users\\Алиса\\Documents\\RAD Studio\\Projects\\Pro2\\Win32\\Debug\\po goda.xml" - как и должно быть.
s5=L"\\pogoda.xml";
Второй проход: выбираем любой другой город, в начале функции s1,s2,s3 имеют просто кракозябры, потом присваиваются значения, которые заданы в функции, но почему-то присваиваются предыдущие значения (и обнулять пыталась, и команду delete применять - все бесполезно). С s4 и s5, вроде, все нормально. Когда я пыталась менять строчки местами, тоже такие симптомы имелись, от безысходности делала переменные глобальными, меняла эти кодировки, тогда возникали ошибки хлеще. Естественно, на втором прогоне программа вылетает, так как возникает перегрузка памяти.
Из-за чего такая аномалия? Два дня уже этим страдаю.

Добавлено через 2 минуты
Для справки: Builder XE5.

@D1973 · 24.07.2014, 14:30

Сообщение от 4lice

gorod=ADOQuery2->FieldByName("Идентификатор")->AsString;

Сообщение от 4lice

UnicodeString pr=ADOQuery2->FieldByName("Название")->AsString;

Что-то я нигде не увидел цикла по записям ADOQuery2 типа такого:

C++
1
2
3
4
5
6
ADOQuery2->First();
for(int i = 0; i < ADOQuery2->RecordCount; i++)
{
  //Потом твой код
  ADOQuery2->Next()
}

А иначе твой код и топчется на первой строке запроса.

@4lice · 24.07.2014, 14:43 **[ТС]**

Цикл здесь не нужен. С этим мне помогли в этой теме Как получить значения из таблицы БД. Что со строками?

@D1973 · 24.07.2014, 14:54

Упс... Не увидел, что у Вас DBLookupComboBox2Click. Sorry.

@c0rvax · 24.07.2014, 14:54

4lice, ты же не выделил память под буфер, в который добавляешь строки. s1 у тебя изначально эквивалентен wchar_t s1[39], т.е., длина "http://xoap.weather.com/weather/local/" + завершающий "\0". После того, как выполняется wcscat, новая строка записывается за пределами границ этого массива, перетирая содержимое других локальных переменных.

@Avazart · 24.07.2014, 15:05

Почему не писать используя String:

C++
1
2
3
4
5
6
    String other = "Другой";
    UnicodeString pr=ADOQuery2->FieldByName("Название")->AsString;
    if(other==pr)
    {
        gorod=ADOQuery2->FieldByName("Идентификатор")->AsString;
    }

C++
1
2
3
4
5
6
7
    
    String url = "http://xoap.weather.com/weather/local/";
    String queryString = "?cc=*&dayf=5&link=xoap&prod=xoap&unit=m&par=1171574135&key=a3740f391c3a72ee";
    String request= url+gorod+queryString;
 
    String pogodaFile = GetCurrentDir()+"\\pogoda.xml";
    URLDownloadToFile(0, request.c_str(), pogodaFile.c_str(), 0, 0);

@4lice · 24.07.2014, 15:36 **[ТС]**

Завтра попробую. Насчет выделения памяти под буфер...В универе в консольных прогах так писала, вроде, нормально работало.

Добавлено через 7 минут
Хотя было бы полезным почитать статьи про выделение памяти, чтобы в будущем таких проблем не возникало.

@4lice · 26.07.2014, 04:52 **[ТС]**

Всем спасибо! Использовала String - работает, как по маслу.

Новые блоги и статьи Все статьи Все блоги /
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 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@4lice 0 / 0 / 0 Регистрация: 19.07.2014 Сообщений: 10
	24.07.2014, 14:43 [ТС]
	Цикл здесь не нужен. С этим мне помогли в этой теме Как получить значения из таблицы БД. Что со строками? 0

@D1973 Модератор 9905 / 6442 / 2455 Регистрация: 21.01.2014 Сообщений: 27,347 Записей в блоге: 3
	24.07.2014, 14:54
	Упс... Не увидел, что у Вас DBLookupComboBox2Click. Sorry. 0

@c0rvax 91 / 91 / 22 Регистрация: 02.07.2013 Сообщений: 332
	24.07.2014, 14:54
	4lice, ты же не выделил память под буфер, в который добавляешь строки. s1 у тебя изначально эквивалентен wchar_t s1[39], т.е., длина "http://xoap.weather.com/weather/local/" + завершающий "\0". После того, как выполняется wcscat, новая строка записывается за пределами границ этого массива, перетирая содержимое других локальных переменных. 1

@4lice 0 / 0 / 0 Регистрация: 19.07.2014 Сообщений: 10
	24.07.2014, 15:36 [ТС]
	Завтра попробую. Насчет выделения памяти под буфер...В универе в консольных прогах так писала, вроде, нормально работало. Добавлено через 7 минут Хотя было бы полезным почитать статьи про выделение памяти, чтобы в будущем таких проблем не возникало. 0

@4lice 0 / 0 / 0 Регистрация: 19.07.2014 Сообщений: 10
	26.07.2014, 04:52 [ТС]
	Всем спасибо! Использовала String - работает, как по маслу. 0

Странное поведение строковых переменных (меняются с того ни с сего)

Решение