Учимся вводить только цифры в wxTextCtrl

September 8th, 2008 Begemot Posted in Программирование

Поразительная фигня, казалось бы простая задача  и должно быть готовое решение. И правда есть, только поразительно кривое:(

Итак, задача – необходимо дать возможность пользователю указать число, например в настройках, количество записей  для отображения где-нибудь… Первое что приходит в голову – wxSlider, ну знаете такой контрол со стрелочками. Так как у меня диапазон от нуля до 10 000, а возможность задать шаг инкрементации для стрелочек я не нашел, и равен он 1 – то этот вариант, оказывается издевательством над юзером.

Думаем дальше и вспоминаем, про wxTextCtrl и валидаторы. Радуемся и пишем код, что-то типа

m_Count = new wxTextCtrl( this, ID_MAXCLIPTEXT, _T(""), wxDefaultPosition, wxSize(50, -1), 0, wxTextValidator(wxFILTER_NUMERIC, &count));

Я ожидал что контрол будет давать вводить в себя только цифры. Я ошибся, он также пропускал точку и запятую, что еще хоть как-то можно понять, хотя его и не просили. Но еще оказались разрешенными: ‘+’ и ‘–’. Также он спокойно пропускал все буквы кирилицы. Похоже что контрол отфильтровывал только пробел, спецсимволы и латиницу. Хотя даже это можно ввести туда пользуясь драг анд дропом или простой вставкой. Вообщем я разочаровался.

Попробывал задать более конкретно wxFILTER_INCLUDE_CHAR_LIST и разрешил только цифры. “голосуй, не голосуй, все равно … “ (с). Теже траблы со вставкой текста из буфера и …. с кирилицей. Прямо наваждение какое-то.

Решил писать сам, вот что получилось

 
h:
	void OnMaxClipTextUpdated( wxCommandEvent& event );

	wxTextCtrl* m_Count;
	
	unsigned long maxCount;
	long maxInsertionPoint;
	wxString maxCountStr;
	bool maxSetManually;

cpp:
	EVT_TEXT( ID_MAXCLIPTEXT, COPDatabase::OnMaxClipTextUpdated )

	m_Count = new wxTextCtrl( this, ID_MAXCLIPTEXT, _T(""), wxDefaultPosition, wxSize(50, -1), 0);
	m_Count->SetMaxLength(4);


void COPDatabase::OnMaxClipTextUpdated( wxCommandEvent& event )
{
	if (maxSetManually) return;
	wxString str=m_Count->GetValue();
	
	if (str.empty() || (str==wxT(" "))) // вторая проверка нужна для случая когда юзер выделяет все в контроле и жмет пробел
	{
		maxCount=0;
		str=wxT("0");
		maxInsertionPoint=1;
		// or if you want to let empty string - replace last 2 string to these - maxCountStr=wxEmptyString; maxInsertionPoint=0;
	}
		
	if (str.ToULong(&maxCount))
	{ // successfully converted to ulong, check range now
		if (maxCount>10000) maxCount=10000; 
		else if (maxCount<0) maxCount=0;

		maxInsertionPoint=m_Count->GetInsertionPoint();
		maxCountStr=wxString::Format(wxT("%i"), maxCount); // we should do like this! иначе траблы с пробелами, нулями..
	}

	maxSetManually=true;
	m_Count->SetValue(maxCountStr);
	maxSetManually=false;
	m_Count->SetInsertionPoint(maxInsertionPoint);
}

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

Поведение (юзабилити) не идеально, но твердую 4-ку я бы поставил. Хотя в случае если диапазон снизу ограничен не нулем, тогда поведение хреново будет, но у меня такого случая пока не было. Как выход можно исключить динамическую проверку диапазона, и перенести ее в код сохраняющий значение.

Еще совет, лучше поставить ограничение на длину вводимых символов, типа m_Count->SetMaxLength(4); для верхней границы 10000 и т.д.

Пользуйтесь. Высказывайтесь.


Если пост полезен для вас вы можете подписаться на RSS или мы можем доставлять вам новые посты прямо в ваш почтовый ящик.

Related:

14 Responses to “Учимся вводить только цифры в wxTextCtrl”

  1. wxSpinCtrl чем не подошел, по моему он для того и предназначен.

  2. wxSpinCtrl не разрешает ввести число с клавиатуры, только стрелочками. У стрелочек шаг изменения = 1 и его нельзя изменить, у меня диапазон от 0 до 10 000, и я не хочу над юзерами издеватся:)

  3. Alexey Says:

    wxSpinCtrl позволяет вводить значение с клавиатуры. Что-то ты не так делаешь.

  4. ну попробуй скомпили стандартный пример с wxSpinCtrl.
    Хотя может я и путаю что-то, я сам был в шоке когда обнаружил что с клавы нельзя цифры ввести.

  5. wxSpinCtrl – вроде замечен баг в последней ветке (2.8.8), легко исправляется. Посмотрите в баг репортах.

  6. спасибо, нашел.
    Пропатчил, скомпили. Действительно позволяет вводить цифры теперь. А еще можно писать всякие нехорошие слова – буквы не фильтрует:( решение еще хуже чем wxTextCtrl с валидатором.

  7. можно еще посмотреть в сторону wxFormatValidator…

  8. Попробывал, тоже не устраивает поведенье. Но тут хоть понятно, он все-таки заточен немного для другой задачи.
    В принципе меня мой изврат пока устраивает:)

  9. # if (maxSetManually) return;
    # …
    # # maxSetManually=true;
    # m_Count->SetValue(maxCountStr);
    # maxSetManually=false;
    Эту конструкцию можно заменить на
    # m_Count->ChangeValue(maxCountString)
    не вызывающую эвент EVT_TEXT

  10. Да, действительно, здорово:)

    Заменяем SetValue() на ChangeValue(), и выкидываем все что связанно с maxSetManually.

    Я себе уже исправил, спасибо.

  11. Если уж так хочется именно wxTextCtrl для ввода только цифр:

    class wxNumberValidator : public wxTextValidator {
    public:
    wxNumberValidator:wxTextValidator(wxFILTER_INCLUDE_CHAR_LIST)
    {
    wxArrayString a;
    for (int i=0;i<10;i++) a.Add(wxString::Format(“%d”,i));
    SetIncludes(a);
    }
    };

    /* … */
    wxTextCtrl* m_pNumberCtrl = new wxTextCtrl(this, wxID_ANY, wxT(“0”), wxDefaultPosition, wxDefaultSize, 0, wxNumberValidator());

  12. Вариант с wxFILTER_INCLUDE_CHAR_LIST я пробывал, и о проблемах с ним писал выше

  13. На самом деле если верить утверждениям некоторых написавших специальный класс валидатора для этого, делов там на 30 минут, и класс работает лучше чем у меня:(

  14. Сергей Says:

    Не знаю, в каком году появилась вот эта штука, но эту страничку гугл находит и поэтому кому-нибудь явно может быть полезно:
    http://docs.wxwidgets.org/trunk/classwx_num_validator.html

Leave a Reply