looking for VBA solution that changes data

Dark0Prince

Active Member
Joined
Feb 17, 2016
Messages
433
I have old dates in the E column on my sheet called ClientListings. Then on the next sheet Called GJ_Import I have new dates in the C column.
How could I use VBA to update the old dates on SOME of the clients when there client number matches from the A column on both sheets. I also want it to effect a range of cells in between the words startGJ and endGJ

Example data is linked here: https://1drv.ms/x/s!Ap0t6Fmj7XZehBIhhLAgL58G_cFG
 

Excel Facts

Select a hidden cell
Somehide hide payroll data in column G? Press F5. Type G1. Enter. Look in formula bar while you arrow down through G.
Code:
=INDEX(Table_Query_from_GJ[Last_Stat_Date],MATCH(ClientListings!A664,Table_Query_from_GJ[CLIENT_NUM],0))

So far this is as far as I can get without VBA and it gives me the dates but replaces all the data that isn't there with N/A instead of leaving it alone with it's current date.
 
Upvote 0
Hi Dark0Prince,

Try the below code ...

Code:
Sub UpdateDates()

Dim Arr As Variant, Dic As Object
Set Dic = CreateObject("Scripting.Dictionary")

Arr = Sheets("GJ_IMPORT").Range("A2", Sheets("GJ_IMPORT").Range("C" & Rows.Count).End(xlUp)).Value

For x = LBound(Arr) To UBound(Arr)
    If Not Dic.exists(Arr(x, 1)) Then Dic.Add Arr(x, 1), Arr(x, 3)
Next

Arr = Sheets("ClientListings").Range("A5", Sheets("ClientListings").Range("E" & Rows.Count).End(xlUp)).Value

For x = LBound(Arr) To UBound(Arr)
    If Dic.exists(Arr(x, 1)) And Dic(Arr(x, 1)) > Arr(x, 5) Then
        Arr(x, 5) = Dic(Arr(x, 1))
    End If
Next
    
Sheets("ClientListings").Range("A5").Resize(UBound(Arr), UBound(Arr, 2)).Value = Arr

End Sub
 
Upvote 0
It does what it is supposed to but unfortunately it effects the whole sheet instead of the parts between "startGJ" and "endGJ" text and there are duplicate numbers in other sections of my sheet that should not be effected.
 
Upvote 0
I also want it to effect a range of cells in between the words startGJ and endGJ

My bad I didn't notice this part of your original post

Also, what do you mean by "there are duplicate numbers in other sections of my sheet that should not be effected" ? Are these duplicates out of the first range you specify (startGJ and endGJ) ?
 
Last edited:
Upvote 0
There are other sections of my sheet where I have the text startABQ, endABQ and startDEN, and endDEN so that I will only effect the proper sections with the code. some of the client numbers are the same in other sections of the clientlisting which is why I say that there are duplicate numbers.
 
Upvote 0
Does this macro do what you want...
Code:
[table="width: 500"]
[tr]
	[td]Sub ChangeDates()
  Dim R As Long, StartGJ As Range, EndGJ As Range, ChangeMe As Variant
  Set StartGJ = Sheets("ClientListings").Columns("A").Find("startGJ", , , xlWhole, , , False, , False)
  Set EndGJ = Sheets("ClientListings").Columns("A").Find("endGJ", , , xlWhole, , , False, , False)
  ChangeMe = Sheets("GJ_IMPORT").Range("A2", Sheets("GJ_IMPORT").Cells(Rows.Count, "E").End(xlUp))
  For R = 1 To UBound(ChangeMe)
    Range(StartGJ, EndGJ).Find(ChangeMe(R, 1), , , xlWhole, , , , , False).Offset(, 4) = ChangeMe(R, 3)
  Next
End Sub[/td]
[/tr]
[/table]
 
Upvote 0
Try this updated code …

Code:
Sub UpdateDates()
Dim Arr As Variant, Dic As Object, fRow As Long, lRow As Long
Set Dic = CreateObject("Scripting.Dictionary")
Arr = Sheets("GJ_IMPORT").Range("A2", Sheets("GJ_IMPORT").Range("C" & Rows.Count).End(xlUp)).Value
For x = LBound(Arr) To UBound(Arr)
    If Not Dic.exists(Arr(x, 1)) Then Dic.Add Arr(x, 1), Arr(x, 3)
Next
fRow = Sheets("ClientListings").Range("A:B").Find("startGJ").Row
lRow = Sheets("ClientListings").Range("A:A").Find("endGJ").Row
Arr = Sheets("ClientListings").Range(Cells(fRow, 1), Cells(lRow, 5)).Value
For x = LBound(Arr) To UBound(Arr)
    If Dic.exists(Arr(x, 1)) And Dic(Arr(x, 1)) > Arr(x, 5) Then
        Arr(x, 5) = Dic(Arr(x, 1))
    End If
Next
    
Sheets("ClientListings").Cells(fRow, 1).Resize(UBound(Arr), UBound(Arr, 2)).Value = Arr
End Sub
 
Upvote 0

Forum statistics

Threads
1,216,179
Messages
6,129,338
Members
449,504
Latest member
Alan the procrastinator

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