handling blank input box

[daviddoria]

If I do this:

salesgoal = Application.InputBox("What is the sales goal for the month?", Title:="Holidays", Type:=1)

If the user clicks "OK" without entering anything, it produces an error. I tried handling it with

If salesgoal = vbNullString Then
    Exit Sub
End If

but it doesn't get there, excel throws an error before it runs that "if null" line.

Is there a way to fix this?

Thanks,
Dave

[VBA Noob]

Your post does not comply with Rule 3 of our Forum RULES. Use code tags around code. Posting code without them makes your code hard to read and difficult to be copied for testing. Highlight your code and click the # at the top of your post window. For more information about these and other tags, found here

VBA Noob

[daviddoria]

I fixed it, but generally 1-3 line things aren't any easier to read with code tags, are they?

Also, the

type:=1

was forcing it to check if the value was numeric - is the idea just not to use the built in error checking, simply read the value as a string, and convert it to a numeric value when needed?

Dave

[shg]

You can use either, just be consistent.

If you use the InputBox method (i.e., Application.InputBox) with type = 1, Excel will force you to input a number or numeric formula, the method will always return a number, and you can receive the result in a numeric variable.

If you just use InputBox, you should return the result to string variable and do whatever error checking is appropriate to your application. If the user enters a formula, you need to evaluate it yourself.

A numeric variable can never be vbNullString. And if you want to test if a string variable is a null string, you should use the Len function -- it's faster.

I fixed it, but generally 1-3 line things aren't any easier to read with code tags, are they?

It is for us.

[Leith Ross]

Hello Dave,

Just for future reference, vbNullString and an empty string "" are not the same thing. vbNullString is a special string that contains Chr$(0). This is useful when writing code with API functions, but should not be used to test if a string is empty (no characters in the string).

Sincerely,
Leith Ross

[daviddoria]

ah ok, i'll keep that in mind, thanks