Help with text box

[dg88]

Hello all,

I new to Win32 programming with C++. I am trying to get a text box to accept integer or decimal numbers only. I was hoping there would some method like Tryparse to do this.
I am not using any MFC or windows forms. Pure C++ only.
I have the following code in place:

TR szText1 = new WCHAR[bufSize];
LPWSTR szText2 = new WCHAR[bufSize];
LPWSTR szText3 = new WCHAR[bufSize];
double num1, num2, num3;
GetWindowText(hwndThickness, szText1, bufSize);
GetWindowText(hwndTheta, szText2, bufSize);
GetWindowText(hwndTexheight, szText3, bufSize);
num1=_wtof(szText1);
num2=_wtof(szText2);
num3=_wtof(szText3);
if(_tcslen(szText1)==0 || _tcslen(szText2)==0 || _tcslen(szText3)==0)
{
MessageBox(hWnd, L"One or more input fields left blank", L"!!!!", MB_OK);
}

Thanks

[Elkvis]

if you're using C++, then you can use boost. boost includes a lexical_cast function that allows the conversion of any type to any other type, so long as they both support insertion and extraction from standard C++ streams.

so you could have something like the following:

#include <boost/lexical_cast.hpp>

const int DoubleConversionSuccess = 0;
const int DoubleConversionError = -1;

int GetDoubleFromString(const std::string& str, double& d)
{
  try
  {
    d = boost::lexical_cast<double>(str);
    return DoubleConversionSuccess;
  }
  catch (boost::bad_lexical_cast& blc)
  {
    return DoubleConversionError;
  }
}

[Salem]

Here's an idea - repaste your code without using HTML, so your code is readable.

It should look like this when you're done.

bool CheckEmpty(string file)
{
  if (s.empty()) {
    return true;                //true if stack is empty when it shouldnt be
  } else {
    return false;               //false if stack is not empty when it should be
  }
}

There's a preview button - use it to make sure your post is good.

[Elysia]

if you're using C++, then you can use boost. boost includes a lexical_cast function that allows the conversion of any type to any other type, so long as they both support insertion and extraction from standard C++ streams.

so you could have something like the following:

#include <boost/lexical_cast.hpp>

const int DoubleConversionSuccess = 0;
const int DoubleConversionError = -1;

int GetDoubleFromString(const std::string& str, double& d)
{
  try
  {
    d = boost::lexical_cast<double>(str);
    return DoubleConversionSuccess;
  }
  catch (boost::bad_lexical_cast& blc)
  {
    return DoubleConversionError;
  }
}

Is there a point in circumventing the exception system? Catching an exception just to return some error is usually a sign of bad design.

Btw, dg88, it isn't possible to create GUI applications with pure C++. You are, in fact, using Win32 in addition to C++. I'd recommend you get a GUI library such as Qt. It will be much easier.

[Elkvis]

Is there a point in circumventing the exception system? Catching an exception just to return some error is usually a sign of bad design.

I couldn't agree more, but since he's using pure win32 api, I assumed that he wouldn't want a bunch of stray try/catch blocks scattered throughout the code, and I doubt win32 gui code is exception-safe. if the system calls your WndProc function, and it throws an exception, where does it get caught?

[Elysia]

Then you put try/catch in your WinProc, and nowhere else.
You can also wrap resources in RAII containers to make your Win32 code exception safe.
There is usually no excuse for putting it in a utility function.