Macro has a compile error and cant work out why

ghrek

Active Member
Joined
Jul 29, 2005
Messages
317
Hi

I have the following macro and keep getting a compile error.

Any Ideas?


Sub test()

Dim sName As Variant, bFolder As String, dest As String

sName = InputBox("Subfolder name")
If sName = "" Then Exit Sub

bFolder = "X:\backup"


If Dir(bFolder & "" & sName, vbDirectory) = "" Then
MkDir bFolder & "" & sName
End If

dest = bFolder & "" & sName

ActiveWorkbook.Sheets(Array("Week1", "Week 2", "Week 3", "Week 4” "Period Summary", "Sheet1")).Copy

ActiveWorkbook.SaveAs dest & "\NewWorkbook", ActiveWorkbook.FileFormat

End Sub
 

Excel Facts

How to change case of text in Excel?
Use =UPPER() for upper case, =LOWER() for lower case, and =PROPER() for proper case. PROPER won't capitalize second c in Mccartney

shg

MrExcel MVP
Joined
May 7, 2008
Messages
21,824
Office Version
  1. 2010
Platform
  1. Windows
VBA uses straight quotes, and you're missing a comma after "Week 4"

Code:
ActiveWorkbook.Sheets(Array("Week1", "Week 2", "Week 3", "Week 4", "Period Summary", "Sheet1")).Copy
 

Marcelo Branco

MrExcel MVP
Joined
Aug 23, 2010
Messages
16,907
Office Version
  1. 2019
  2. 2010
Platform
  1. Windows
This is wrong
ActiveWorkbook.Sheets(Array("Week1", "Week 2", "Week 3", "Week 4 "Period Summary", "Sheet1")).Copy

and missing a comma (,)

Maybe...

Code:
ActiveWorkbook.Sheets(Array("Week1", "Week 2", "Week 3", "Week 4[B][COLOR=#ff0000]",[/COLOR][/B] "Period Summary", "Sheet1")).Copy

M.
 

Forum statistics

Threads
1,148,108
Messages
5,744,874
Members
423,907
Latest member
zerocool88

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
Top