How to remove rows that contain current date (mm/dd/yyyy)

[joshuarobbins]

Hello everyone,

Below is a Macro that I am trying to create that will automatically take and format data on two newly created Workbooks. The problem I am running into is that I need to remove all the rows that contain the current date. I have bolded the portion of the code that is giving me trouble. I am new to VBA within excel, and any help would be greatly appreciated.

Please Login or Register  to view this content.

[daffodil11]

Looks pretty good so far. I usually use range.entirerow myself.

Please Login or Register  to view this content.

[joshuarobbins]

Hey daffodil11,

Thank you for the very quick reply. I added your portion to my code, but for some reason it still will not remove my rows containing today's date. My dates are located in Column E on my first workbook. Is my range incorrect? Below is my updated portion. Do I take the "Dim lRow as Long" and "Dim iCntr As Long" out?

Please Login or Register  to view this content.

[daffodil11]

I see the issue now. The argument of Cells is (row #, column #).

Your dates are in Column E, which is Column 5. Therefore

Please Login or Register  to view this content.

should be

Please Login or Register  to view this content.

The proceeding line should specify only a single cell.

Range("E" & iCntr).EntireRow.Delete

E:E would be all of the cells. That would be bad. It would delete everything.

Please Login or Register  to view this content.

This should do it just fine.

And technically, the Column value of the EntireRow.Delete line is irrelevant because all Columns of a given cell are on the same row of that cell.

This snippet would function no different

Please Login or Register  to view this content.

[joshuarobbins]

Thanks again for the prompt response, and especially spelling it out. That makes a lot more sense. For some reason though, it is still not removing the rows with today's date. Below is my updated portion. My other rows contain things such as, names and email addresses. This shouldn't conflict, correct?

Please Login or Register  to view this content.

[daffodil11]

lRow needs to be defined

Please Login or Register  to view this content.

This identifies the last row so it knows where to start counting.

Without the variable, you could always use something static such as

Please Login or Register  to view this content.

[joshuarobbins]

That seems to have done the trick! I really appreciate the help. At first it was not working, because I had the range as "Z", when my values were actually in "E". So I just switched them around and it worked. This is what I ended up with:

Please Login or Register  to view this content.

[daffodil11]

No problem. Glad I was able to help you out.

I only started teaching myself code last year.