Sorry, didn't know you wanted to move to the other sheets. I wrote it to NOT
move on purpose.
One of the slowest actions Excel can do is to constantly update/refresh the
video display, so unless there's a need to go another sheet, I generally
don't do it. Plus some users find it annoying to see a bunch of sheets
flashing on the screen in front of them.

"kevlarmcc" wrote:

With both solutions I still have the same result. The macro never moves on to
the next sheet.

"JLatham" wrote:

That will work, but (in my opinion) bad form to try to use reserved words as
variables or constants. But I think I know why you did it in this case.

Probably better for OP to do it this way:

Sub CalcDates()
Dim anySheet as Worksheet
For Each anySheet in ThisWorkbook.Worksheets
'.... the other code here
Next ' end of anySheet loop
End Sub

"Gary Brown" wrote:

I'd say you probably need to declare the worksheet....

Sub CalcDates()
Dim worksheet as worksheet
'stuff
End Sub

--
Hope this helps.
If it does, please click the Yes button.
Thanks in advance for your feedback.
Gary Brown



"kevlarmcc" wrote:

I have some simple code that selects a cell, enters a value, selects some
more cells and enters a formula. When i follow suggestions to configure the
code to run on all worksheets I cannot get it right. What it does is run the
code on the active sheet as many times as there are sheets. So it's looping
but not going on to the next sheet. Can someone tell me what I have wrong? I
have included the code that works (for the active sheet only) and the code i
think would work but won't.

Works:

Sub CalcDates()
ActiveSheet.Cells(2, 7).Select
Selection.Value = "Days"
ActiveSheet.Cells(3, 7).Select
Do While Not IsEmpty(ActiveCell.Offset(0, -1))
ActiveCell.FormulaR1C1 = "=DAYS360(RC[-4],RC[-6])"
ActiveCell.Offset(1, 0).Select
Loop
End Sub

Doesn't work:

Sub CalcDates()
For Each Worksheet In Worksheets
Cells(2, 7).Select
Selection.Value = "Days"
Cells(3, 7).Select
Do While Not IsEmpty(ActiveCell.Offset(0, -1))
ActiveCell.FormulaR1C1 = "=DAYS360(RC[-4],RC[-6])"
ActiveCell.Offset(1, 0).Select
Loop
Next Worksheet
End Sub