Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor template function to validate numeric wxPG properties

Browse Source 
Duplicating the code of the entire NumericValidation() template function just for one specialization (for double value type) seems to be an overkill. Only small fraction of the code (to round the validated value) is really dependent on the value type and this part can be moved to the template function GetRoundedValue() which in turn can be specialized as required.
This commit is contained in:
Artur Wieczorek
2019-06-25 21:37:19 +02:00
parent dbb727a716
commit e96bc37066
1 changed files with 36 additions and 114 deletions
Download Patch File Download Diff File Expand all files Collapse all files

150
src/propgrid/props.cpp
View File

@@ -336,15 +336,33 @@ bool wxIntProperty::IntToValue( wxVariant& variant, int value, int WXUNUSED(argF
return false;
}
//
namespace {
// This function by default doesn't modify the value.
// For argument 'value' of type 'double' there is a specialized function (below).
template<typename T>
T GetRoundedValue(const wxPGProperty* prop, T value)
{
wxUnusedVar(prop);
return value;
}
// Specialized function for floating-point values
// takes into account required precision of the numbers
// to avoid rounding and conversion errors.
template<>
double GetRoundedValue(const wxPGProperty* prop, double value)
{
// Round value to the required precision.
wxVariant variant = value;
wxString strVal = prop->ValueToString(variant, wxPG_FULL_VALUE);
strVal.ToDouble(&value);
return value;
}
// Common validation code to be called in ValidateValue()
// implementations.
//
// Note that 'value' is reference on purpose, so we can write
// back to it when mode is wxPG_PROPERTY_VALIDATION_SATURATE or wxPG_PROPERTY_VALIDATION_WRAP.
// For argument 'value' of type 'double' there is a specialized function (below).
//
namespace {
template<typename T>
bool NumericValidation( const wxPGProperty* property,
T& value,
@@ -357,17 +375,30 @@ bool NumericValidation( const wxPGProperty* property,
bool minOk = false;
bool maxOk = false;
// Round current value to the required precision, if applicable
value = GetRoundedValue<T>(property, value);
variant = property->GetAttribute(wxPG_ATTR_MIN);
if ( !variant.IsNull() )
{
minOk = variant.Convert(&min);
}
if ( minOk )
{
// Round minimal value to the required precision, if applicable
min = GetRoundedValue<T>(property, min);
}
variant = property->GetAttribute(wxPG_ATTR_MAX);
if ( !variant.IsNull() )
{
maxOk = variant.Convert(&max);
}
if ( maxOk )
{
// Round maximal value to the required precision, if applicable
max = GetRoundedValue<T>(property, max);
}
if ( minOk )
{
@@ -432,115 +463,6 @@ bool NumericValidation( const wxPGProperty* property,
}
return true;
}
// Template specialization for argument 'value' of type 'double'.
// It takes into account required precision of the numbers
// to avoid rounding and conversion errors.
template<>
bool NumericValidation( const wxPGProperty* property,
double& value,
wxPGValidationInfo* pValidationInfo,
int mode, double defMin, double defMax)
{
double min = defMin;
double max = defMax;
wxVariant variant;
bool minOk = false;
bool maxOk = false;
variant = property->GetAttribute(wxPG_ATTR_MIN);
if ( !variant.IsNull() )
{
minOk = variant.Convert(&min);
}
variant = property->GetAttribute(wxPG_ATTR_MAX);
if ( !variant.IsNull() )
{
maxOk = variant.Convert(&max);
}
if ( minOk || maxOk )
{
// Round current value to the required precision.
variant = value;
wxString strVal = property->ValueToString(variant, wxPG_FULL_VALUE);
strVal.ToDouble(&value);
// Round minimal value to the required precision.
variant = min;
strVal = property->ValueToString(variant, wxPG_FULL_VALUE);
strVal.ToDouble(&min);
// Round maximal value to the required precision.
variant = max;
strVal = property->ValueToString(variant, wxPG_FULL_VALUE);
strVal.ToDouble(&max);
}
if ( minOk )
{
if ( value < min )
{
if ( mode == wxPG_PROPERTY_VALIDATION_ERROR_MESSAGE )
{
wxString msg;
wxVariant vmin = min;
wxString smin = property->ValueToString(vmin);
if ( !maxOk )
msg = wxString::Format(
_("Value must be %s or higher."),
smin);
else
{
wxVariant vmax = max;
wxString smax = property->ValueToString(vmax);
msg = wxString::Format(
_("Value must be between %s and %s."),
smin, smax);
}
pValidationInfo->SetFailureMessage(msg);
}
else if ( mode == wxPG_PROPERTY_VALIDATION_SATURATE )
value = min;
else
value = max - (min - value);
return false;
}
}
if ( maxOk )
{
if ( value > max )
{
if ( mode == wxPG_PROPERTY_VALIDATION_ERROR_MESSAGE )
{
wxString msg;
wxVariant vmax = max;
wxString smax = property->ValueToString(vmax);
if ( !minOk )
msg = wxString::Format(
_("Value must be %s or less."),
smax);
else
{
wxVariant vmin = min;
wxString smin = property->ValueToString(vmin);
msg = wxString::Format(
_("Value must be between %s and %s."),
smin, smax);
}
pValidationInfo->SetFailureMessage(msg);
}
else if ( mode == wxPG_PROPERTY_VALIDATION_SATURATE )
value = max;
else
value = min + (value - max);
return false;
}
}
return true;
}
} // namespace
#if wxUSE_LONGLONG