ExcelBanter

ExcelBanter (https://www.excelbanter.com/)
-   Excel Programming (https://www.excelbanter.com/excel-programming/)
-   -   Why does the line of code NOT work? (https://www.excelbanter.com/excel-programming/385916-why-does-line-code-not-work.html)

Corey

Why does the line of code NOT work?
 
So far i can get this code to work as required, except for 1 line.
The highlighted line is suppose to eliminate any values that are LESS than the Textbox1.value, but
it DOES'NT, Why ?

Private Sub TextBox1_Change()
Application.ScreenUpdating = False
ListBox3.Clear
If TextBox1.Value < "" Then
Dim lastcell As Long
Dim myrow As Long
Dim i As Long
On Error Resume Next
lastcell = Worksheets("InspectionData").Cells(Rows.Count, "A").End(xlUp).Row
With ActiveWorkbook.Worksheets("InspectionData")
For myrow = 1 To lastcell
If .Cells(myrow, 1) < "" Then
If .Cells(myrow, 1).Offset(-1, 2).Value = ListBox1.Value Then
If .Cells(myrow, 1).Offset(-1, 6).Value = ListBox2.Value Then
For i = 2 To 22
If .Cells(myrow, 3).Offset(i, 0).Font.Strikethrough = False And Cells(myrow, 3).Offset(i,
0).Value < "" _
And IsNumeric(.Cells(myrow, 3).Offset(i, 0)) Then
If .Cells(myrow, 3).Offset(i, 0).Value < "" And .Cells(myrow, 3).Offset(i, 0).Value =
TextBox1.Value Then ' <==== NOT WORKING ??

ListBox3.AddItem Cells(myrow, 3).Offset(i, 0)
ListBox3.List(ListBox3.ListCount - 1, 1) = Cells(myrow, 3).Offset(i, 0).Address
End If
End If
Next i
End If
End If
End If
Next
End With
End If
Label8 = ListBox3.ListCount
Application.ScreenUpdating = True
End Sub



merjet

Why does the line of code NOT work?
 
Comparing a number to text doesn't work. Use Val(TextBox1.Value).

Merjet



Corey

Why does the line of code NOT work?
 
Thank you for the reply.
Perfect.
Would not have solved that in 1Mil yrs.
Thanks

Corey....

"merjet" wrote in message
oups.com...
Comparing a number to text doesn't work. Use Val(TextBox1.Value).

Merjet





All times are GMT +1. The time now is 01:05 AM.

Powered by vBulletin® Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.
ExcelBanter.com