правильное программирование окна свойств , отмена изменений без проверки значений

[Cfon]

Здрасте, тут такое дело я запуталси

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

если надо могу прилепить проект, а пока вот часть кода хотя она не очем не скажет

void CPage4::OnEnKillfocusFontsize()
{
    TRACE("CPage4::OnEnKillfocusFontsize\n");
    int minVal = 8;
    int maxVal = 24;
    int value = GetDlgItemInt(IDC_FONTSIZE);    
    TRACE("mFontSize = %d\n", mFontSize);
    TRACE("value = %d\n", value);
 
    //if (mCancel) return;
 
    if (value < minVal || value > maxVal)
    {  
        AfxMessageBox(_T("Enter value between 8 and 24"), MB_ICONEXCLAMATION);              
        mPropSheet->SetActivePage(3);
    }
    else
        SetModified(TRUE);
}

[Олег М]

Цитата Cfon @ 11.03.17, 07:36

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

А зачем ты так делаешь в killfocus. Это сообщение посылается всегда, когда ты выходишь из контрола, независимо куда. Там надо выводить не MesageBox, а какое-нибудь неблокирующее сообщение, например в Static.
А MessageBoxы надо делать при нажатии на кнопку OK.

[Cfon]

Цитата Олег М @ 11.03.17, 09:02

А зачем ты так делаешь в killfocus. Это сообщение посылается всегда, когда ты выходишь из контрола, независимо куда. Там надо выводить не MesageBox, а какое-нибудь неблокирующее сообщение, например в Static.
А MessageBoxы надо делать при нажатии на кнопку OK.

нет не помогло также не закрывает окно при отмене

void CPage4::OnEnKillfocusFontsize()
{
    TRACE("CPage4::OnEnKillfocusFontsize\n");
    const int minVal = 8;
    const int maxVal = 24;
    //int value = GetDlgItemInt(IDC_FONTSIZE);  
    //TRACE("mFontSize = %d\n", mFontSize);
    TRACE("value = %d\n", value);
 
    //if (mCancel) return;
 
    if (value < minVal || value > maxVal)
    {  
        //AfxMessageBox(_T("Enter value between 8 and 24"), MB_ICONEXCLAMATION);
        SetDlgItemText(IDC_ERRORMESSAGE, _T("Enter value between 8 and 24"));
        mPropSheet->SetActivePage(this);
    }
    else
        SetModified(TRUE);
}

Сообщение отредактировано: Cfon - 11.03.17, 10:53

[ЫукпШ]

Цитата Cfon @ 11.03.17, 07:36

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

Так если ты выбрал "отмена", значит должны быть возвращено старое значение, которое было.
Значит, перед показом диалога где-то должна быть спасена старая версия предыдущих
"вполне павильных" значений. Отмена означает их возврат в том числе в окна редактирования.
Такая логика работы позволяет использовать "Отмена" даже после многочисленных "Применить".
По-применял -> не понравилось -> Отменил, вернулся к старым значениям.
---
Или делай в обработчике автомат состояний. Если последней операцией была "отмена" - значит проверка
на диапазон правильных значенй не производится.

Сообщение отредактировано: ЫукпШ - 11.03.17, 11:01

[ter_nk_]

Когда знаешь диапазон верных значение, то можно не давать ввести неверное, т.е. обрабатывать WM_CHAR

[Cfon]

Цитата ЫукпШ @ 11.03.17, 10:56

Так если ты выбрал "отмена", значит должны быть возвращено старое значение, которое было.
Значит, перед показом диалога где-то должна быть спасена старая версия предыдущих
"вполне павильных" значений. Отмена означает их возврат в том числе в окна редактирования.
Такая логика работы позволяет использовать "Отмена" даже после многочисленных "Применить".
По-применял -> не понравилось -> Отменил, вернулся к старым значениям.
---
Или делай в обработчике автомат состояний. Если последней операцией была "отмена" - значит проверка
на диапазон правильных значенй не производится.

с логикой проблем нет, проблема в том что окно свойств не закрыть в моей случае даже если нажать отмену

Цитата ter_nk_ @ 11.03.17, 11:27

Когда знаешь диапазон верных значение, то можно не давать ввести неверное, т.е. обрабатывать WM_CHAR

сам то пробовал обработку WM_CHAR на проверку ввода диапозона значений, например от 8 до 24?

Прикреплённый файлEx12a.zip (83,69 Кбайт, скачиваний: 98)

это не правильной программирование не качайте его, в самом низу есть другой полное демо как надо

Сообщение отредактировано: Cfon - 17.03.17, 06:51

[ЫукпШ]

Цитата Cfon @ 11.03.17, 11:43

с логикой проблем нет, проблема в том что окно свойств не закрыть в моей случае даже если нажать отмену

К закрытию окна указанная логика работы отношения не имеет.. почти.
Если окно не закрывается, то восстановление первоначальных данных
по событию "Отмена" (в том числе и в окошках ввода) как раз весьма актуально !
---
Что касается сообщения - MessageBox это не удобно.
Если обработчиком обнаружена ошибка в числах, можно изменить неверный параметр
на его граничное значение. Подробное описание назначения параметра, граничных значений
и произошедшей ошибки можно представить в тул-типе.

Сообщение отредактировано: ЫукпШ - 11.03.17, 16:53

[Cfon]

Цитата ЫукпШ @ 11.03.17, 16:48

К закрытию окна указанная логика работы отношения не имеет.. почти.
Если окно не закрывается, то восстановление первоначальных данных
по событию "Отмена" (в том числе и в окошках ввода) как раз весьма актуально !
---
Что касается сообщения - MessageBox это не удобно.
Если обработчиком обнаружена ошибка в числах, можно изменить неверный параметр
на его граничное значение. Подробное описание назначения параметра, граничных значений
и произошедшей ошибки можно представить в тул-типе.

ясно спс, будем думать
кстати наш диалог будет более продуктивнее если ты запустишь мой примерчик и поймёшь о чем я толкую ж

[ter_nk_]

Цитата Cfon @ 11.03.17, 11:43

сам то пробовал обработку WM_CHAR на проверку ввода диапозона значений, например от 8 до 24?

А в чем проблема? Когда символ уже есть то второй либо игнорируется, если число уже не будет соответствовать или ставится.

[ЫукпШ]

Цитата Cfon @ 11.03.17, 17:32

кстати наш диалог будет более продуктивнее если ты запустишь мой примерчик и поймёшь о чем я толкую ж

я и не против - но исполнимого файла в архиве нет,
а собирать из исходников не буду.

[Cfon]

Цитата ter_nk_ @ 11.03.17, 17:38

А в чем проблема? Когда символ уже есть то второй либо игнорируется, если число уже не будет соответствовать или ставится.

нет проблем нет, только этот подход не правилен, т.к. каждый символ будет проверяться на соответствие диапазону ввода, а не все введённое число целиком, не?

Цитата ЫукпШ @ 11.03.17, 18:12

я и не против - но исполнимого файла в архиве нет,
а собирать из исходников не буду.

ок позже запостю ехе файл

Сообщение отредактировано: Cfon - 12.03.17, 02:37

[Cfon]

много гуглил ничего толком не нашел, в основном лажу

решил по другому точнее по старому через DDX/DDV( валидация DDV мне подходит).
нужно было решить как быть чтобы изменения, которые сохранялись при переходе с вкладки на вкладку(речь идет о диалоге в несколькими вкладками), при отмене не сохранялись, решение простое хранить старое значение, а потом восстанавливать, если оно было отменено

поскольку в CPropertySheet нет обработчиков на OK, Cancel, Apply, то обработал OnCommand:

BOOL CPropSheet::OnCommand(WPARAM wParam, LPARAM lParam)
{
    TRACE(_T("CPropSheet::OnCommand\n"));
    UINT nID = LOWORD(wParam);
        
    if (nID == IDCANCEL)
    {
        TRACE(_T("IDCANCEL\n"));
        TRACE("mEditValue = %d\n", mPage2.EditValue());
        TRACE("mEditOldValue = %d\n", mPage2.EditOldValue());
        mPage2.EditValue(mPage2.EditOldValue());
    }
    
    return CPropertySheet::OnCommand(wParam, lParam);
}

Вопрос решен, но если у вас есть что сказать вельком!

Сообщение отредактировано: Cfon - 12.03.17, 07:13

[Cfon]

возникла одна трабла мой вариант обрабатывал нажатие кнопки Отмена, но не нажатие кнопки Закрыть, та что в правом верхнем углу
короче запуталси я уж слишком много чего завязано тут надо снава разбирать
Вопрос не решен однако

[ЫукпШ]

Цитата Cfon @ 12.03.17, 11:14

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

Всё же от тебя зависит.
Какую именно логику работы ты желаешь - такую и реализуй.
Что именно необходимо сделать по событию "закрыть окно" ?
В простейшем случае это эквивалент кнопки "Отмена".
Значит, напижем процедуру "OnCancel" и вызовем её из обоих
обработчиков.
При этом очевидно, что OnCancel это OnApply, но с изначальными
(спасёнными перед показом диалога) параметрами...
(Но не всегда, а только тогда, когда новые параметры не равны спасённым или
использовалась кнопка "Применить" с параметрами, не равными спасённым)
Если кнопка "Закрыть" вызывает у тебя сложности понимания, вообще
создай диалог без неё.

Сообщение отредактировано: ЫукпШ - 12.03.17, 13:20

[Cfon]

Цитата ЫукпШ @ 12.03.17, 13:17

Всё же от тебя зависит.
Какую именно логику работы ты желаешь - такую и реализуй.
Что именно необходимо сделать по событию "закрыть окно" ?
В простейшем случае это эквивалент кнопки "Отмена".
Значит, напижем процедуру "OnCancel" и вызовем её из обоих
обработчиков.
При этом очевидно, что OnCancel это OnApply, но с изначальными
(спасёнными перед показом диалога) параметрами...
(Но не всегда, а только тогда, когда новые параметры не равны спасённым или
использовалась кнопка "Применить" с параметрами, не равными спасённым)
Если кнопка "Закрыть" вызывает у тебя сложности понимания, вообще
создай диалог без неё.

все что ты говоришь правильно, но это всего лишь на бумаге а на практике мы имеем дело с MFC.
по тому что ты пишешь я сделал вывод, что ты никогда не писал на MFC верно?