I started to just add CODE tags, White space and general 'paragraph' formatting to your code, but I got carried away and removed those "Ands" between value assignments. My bad

There are still some logic errors that need correcting.

Have you ever heard of a positive withdrawal? Or a negative deposit? So we don't have to check for Negative AND Withdrawal, just one or the other.


Start with this and see what you can do
Const v1 = "Vanguard"
Const V2 = "RobingTheHood"
Const v3 = ?
Const V4 = ?

If Cells(i,1) < 0 And(Cells(i, 3) = V1 Or Cells(i, 3) = V2 Or Cells(i, 3) = V3 Or Cells(i, 3) =V4) Then
Cells(i, 1).Value = Abs(Cells(i, 1)
Cells(i, 2) = ?
End If 
Next
This is a tiny bit faster
Const Tester As String = "Vanguard, Robinhood, ******776, ******732"
'
'
... And InStr(Tester, Cells(i, 3)) > 0 Then