application.match errors

wwbwb

Well-known Member
Joined
Oct 20, 2003
Messages
513
Greetings all,

I have a question about the following code erroring out.

Code:
dim res9 as variant
dim res4 as variant

res9 = application.match(combobox1.value, Range("B:B"), 0)
res4 = application.match(combobox2.value, Range("B:B"), 0)

Range("e" & res9).value = userform4.textbox1.value
Range("e" & res4).value = userform4.textbox1.value

Col B is a list of serial numbers. Either 7 digits long (1234567) or 8 alphanumeric (W1234567). The combo boxes are loaded with col B when the userform is initialized. When I run the code, any serial number that begins with a letter, works. Any serial number I select that is strictly a number, errors out (Error 2042). The serial number is indeed there and if I put a letter in front of a 'number only' serial, it works perfectly. Any reason why it can't find the 'number only' serials?
 
Last edited:

Excel Facts

Create a chart in one keystroke
Select the data and press Alt+F1 to insert a default chart. You can change the default chart to any chart type
I believe you're likely trying to compare an integer value and a string value, which are completely different. To convert every value to a string you can simply use something similar to below:

Code:
Dim x as Int
Dim xStr as String

x = 1234
xStr = CStr(x)

On Error Resume Next
If x = xStr Then
    'Do Stuff
End If
If Err.Number <> 0
    MsgBox("Error, you tried to compare a string and an integer!")
End If
 
Upvote 0
You should make some kind of check first for the ComboBox1.value, like below:
Code:
 If IsNumeric(ComboBox1.Value) Then
  res9 = Application.Match([COLOR=#ff0000][B]Val[/B][/COLOR](ComboBox1.Value), Range("B:B"), 0)
 Else: res9 = Application.Match(ComboBox1.Value, Range("B:B"), 0)
 End If
 
Upvote 0
You should make some kind of check first for the ComboBox1.value, like below:
Code:
 If IsNumeric(ComboBox1.Value) Then
  res9 = Application.Match([COLOR=#ff0000][B]Val[/B][/COLOR](ComboBox1.Value), Range("B:B"), 0)
 Else: res9 = Application.Match(ComboBox1.Value, Range("B:B"), 0)
 End If

Thank you. This seems to work on a test sample I have. I'll try it more once I get back to work. I'll check to see if the numbers are actually setup as text.
 
Last edited:
Upvote 0

Forum statistics

Threads
1,215,307
Messages
6,124,168
Members
449,146
Latest member
el_gazar

We've detected that you are using an adblocker.

We have a great community of people providing Excel help here, but the hosting costs are enormous. You can help keep this site running by allowing ads on MrExcel.com.
Allow Ads at MrExcel

Which adblocker are you using?

Disable AdBlock

Follow these easy steps to disable AdBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the icon in the browser’s toolbar.
2)Click on the "Pause on this site" option.
Go back

Disable AdBlock Plus

Follow these easy steps to disable AdBlock Plus

1)Click on the icon in the browser’s toolbar.
2)Click on the toggle to disable it for "mrexcel.com".
Go back

Disable uBlock Origin

Follow these easy steps to disable uBlock Origin

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back

Disable uBlock

Follow these easy steps to disable uBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back
Back
Top