Ненужная синхронизация edit box' ов. MFC C++

@011 · Регистрация: 28.11.2013

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

Занимаюсь реализацией шифра Цезаря.

Проблема: незапланированно обновляется один из edit' ов.
Ввожу ключ (целое число, например, 3). Ввожу зашифрованный текст в левое поле. Нажимаю на кнопку Decrypt и расшифрованный текст записывается в оба edit' а.
Подскажите, пожалуйста, в чем дело.

P.S.
cstringTextOUT -- переменная CString, отвечающая за правый edit. cstringTextIN -- аналогично.

pastebin.com/YkfRDsst

Кликните здесь для просмотра всего текста

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
void CaesarDlg::OnDecrypt()
{
 
    UpdateData(TRUE);   
    int size = cstringTextIN.GetLength()+1;
    char * text = new char [size];
    // исходный текст в чар-массиве
    text = const_cast<TCHAR*>((LPCTSTR)cstringTextIN); 
    // дешифровка по ключу cipher_key
    for (int i = 0; i<size+1; i++)
    {
        if  (65<=text[i] && text[i]<=90)
            text[i] = (char)(((int)(text[i])-65-cipher_key+26)%26+65); 
        if  (97<=text[i] && text[i]<=122)
            text[i] = (char)(((int)(text[i])-97-cipher_key+26)%26+97);
 
        // обработка конца файла
        if ((int)(text[i])==-1)
            text[i] = (char)(0);
        else
            text[i] = text[i];
    }   
 
    CString tmp(text);
    cstringTextOUT = tmp;
    UpdateData(FALSE);  
}

@Людвиг Бодмер · 27.02.2015, 12:51

011, ну исходя из логики работы функции так и должно быть (на мой взгляд дилетанта), оба Edit'а должны поменяться. Я так понимаю Edit'ы через визард привязаны к значениям переменных cstringTextIN и cstringTextOUT. Значение cstringTextIN меняется через указатели. Можно пофиксить таким образом: переменные не привязывать к Edit'ам, просто сделать по такой схеме:

C++
1
2
3
4
5
6
CString cstringTextIN, cstringTextOUT;
GetDlgItemText(IDC_EDIT1,cstringTextIN);
   .
   .
   .
SetDlgItemText(IDC_EDIT2,cstringTextOUT);

P.S. У вас в коде утечка памяти.

Добавлено через 1 час 20 минут
Пардон, не увидел, что в соседней ветке уже нашлось решение

@011 · 28.02.2015, 00:43 **[ТС]**

Людвиг Бодмер, да, решение этой проблемы предложили.
Спасибо за Ваш вариант!
Вы упомянули утечку памяти. Можно подробнее? Речь о том, что массив text не удаляю?
Ну и вообще, буду рад, если еще дадите советов

Добавлено через 2 часа 40 минут

Не по теме:

Людвиг Бодмер, может, подскажете как реализовать побуквенный вывод во второй edit? Т.е. хочу добиться эффекта постепенного формирования сообщения, а не сразу результат. Использую Sleep(5) и прогресс бар. Но не получается обновлять второй edit в цикле кодирования (там, где я с чарами работаю). Каким методом можно обновлять содержание второго поля? Пробовал так:

Кликните здесь для просмотра всего текста

IDC_EDIT3 -- поле вывода. Работает так: прогресс бар заполняется постепенно, за ощутимое время, т.е. как надо. После его заполнения сразу появляется зашифрованное сообщение. Целиком.O_o

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
for (int i = 0; i<size+1; i++)
    {       
        if (progress_bar.GetPos()!=size)
        {
            progress_bar.StepIt();
            if (size<100)
                Sleep(50);
            Sleep(1);
        }
        if  (65<=text[i] && text[i]<=90)
            text[i] = (char)(abs(((int)(text[i])-65+cipher_key))%26+65); 
        if  (97<=text[i] && text[i]<=122)
            text[i] = (char)(abs(((int)(text[i])-97+cipher_key))%26+97);
 
        cstringTextOUT += text[i];      
        SetDlgItemText(IDC_EDIT3,cstringTextOUT);           
    }

@Людвиг Бодмер · 28.02.2015, 18:23

011, по поводу утечки памяти, да, я имел ввиду, что для каждого вызова new должен быть соответствующий вызов delete, для освобождения выделенной памяти. В том коде

C++
1
delete [] text;

По поводу побуквенного вывода набросал такой код (у меня работает):

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
    progress_bar.SetRange (0,size);
    progress_bar.SetPos(0);
    progress_bar.SetStep(1);
    for (int i = 0; i<size+1; i++)
    {       
        if (progress_bar.GetPos()!=size)
        {
            progress_bar.StepIt();
            Sleep(500);
            if  (65<=text[i] && text[i]<=90)
            text[i] = (char)(abs(((int)(text[i])-65+cipher_key))%26+65); 
            if  (97<=text[i] && text[i]<=122)
            text[i] = (char)(abs(((int)(text[i])-97+cipher_key))%26+97);
 
            cstringTextOUT += text[i];      
            SetDlgItemText(IDC_EDIT3,cstringTextOUT); 
            GetDlgItem(IDC_EDIT3)->UpdateWindow();
        }
 
    }

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .