Search a worksheet, find matches, copy row data to another worksheet

sarasotavince

New Member
Joined
May 14, 2011
Messages
24
Hello All. Trying to get some help with this problem. I have two worksheets (LABOR) and (SEARCH). From a userform I call an input box and ask for the search criteria. I then search column A on the LABOR worksheet for all matches --and want to copy each row's data of the LABOR worksheet to the SEARCH worksheet. I am using the following code, which does not produce an error but it does not produce any results either. After the search criteria is entered I see the message box telling me the operation is complete. I have tinkered with this code but obviously I am missing something. I may also have "more" code than I need so if something can be deleted please REM it out....thanks in advance. Here is the non-working code:

Code:
Private Sub CommandButton4_Click()
Dim strLastRow As String
Dim rngC As Range
Dim strToFind As String, FirstAddress As String
Dim wSht As Worksheet
Dim rngtest As String
Application.ScreenUpdating = False
Set wSht = Worksheets("Labor")
strToFind = InputBox("Enter Search Criteria")
With wSht.Range("A:A")
Set rngC = .Find(what:=strToFind, LookAt:=xlPart)
If Not rngC Is Nothing Then
FirstAddress = rngC.Address
Do
strLastRow = Sheets("Search").Range("A" & Rows.Count).End(xlUp).Row + 1
rngC.EntireRow.Copy .Cells(strLastRow, 1)
Set rngC = .FindNext(rngC)
Loop While Not rngC Is Nothing And rngC.Address <> FirstAddress
End If
End With
MsgBox ("Finished")
End Sub
 

Excel Facts

Bring active cell back into view
Start at A1 and select to A9999 while writing a formula, you can't see A1 anymore. Press Ctrl+Backspace to bring active cell into view.
The .Cells in this line:

rngC.EntireRow.Copy .Cells(strLastRow, 1)

refers to the wrong sheet (With wSht.Range("A:A")).

Try:

rngC.EntireRow.Copy Sheets("Search").Cells(strLastRow, 1)
 
Upvote 0
Try I did and it works perfectly! That one simple change made all the difference. So, here is the working code.

Thanks to John_w.

Code:
Private Sub CommandButton1_Click()
Dim strLastRow As String
Dim rngC As Range
Dim strToFind As String, FirstAddress As String
Dim wSht As Worksheet
Dim rngtest As String
Application.ScreenUpdating = False
Set wSht = Worksheets("Labor")
strToFind = InputBox("Enter Search Criteria")
With wSht.Range("A:A")
Set rngC = .Find(what:=strToFind, LookAt:=xlPart)
If Not rngC Is Nothing Then
FirstAddress = rngC.Address
Do
strLastRow = Sheets("Search").Range("A" & Rows.Count).End(xlUp).Row + 1
rngC.EntireRow.Copy Sheets("Search").Cells(strLastRow, 1)
Set rngC = .FindNext(rngC)
Loop While Not rngC Is Nothing And rngC.Address <> FirstAddress
End If
End With
MsgBox ("Finished")
 
End Sub

Hope this can save someone a few hours sleep. Thanks again.
 
Upvote 0

Forum statistics

Threads
1,224,518
Messages
6,179,248
Members
452,900
Latest member
LisaGo

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