648

I've heard much about the understandable abhorrence of using .Select in Excel VBA, but I am unsure of how to avoid using it.

I am finding that my code would be more reusable if I were able to use variables instead of Select functions. However, I am not sure how to refer to things (like the ActiveCell, etc.) if not using Select.

I have found this article on ranges and this example on the benefits of not using select, but I can't find anything on how.

4
  • 23
    It's important to note that there are instances when using Select and/or ActiveSheet etc etc is completely unavoidable. Here's an example that I found: stackoverflow.com/questions/22796286/…
    – Rick
    CommentedMay 28, 2014 at 14:00
  • 18
    And there are occasions - editing chart data in ppt with an underlying excel file being one - where activate or select are required.
    – brettdj
    CommentedDec 21, 2014 at 5:37
  • @brettdj - here's a recent example. To set all the sheets in a workbook to the same value, it seems .Select / .Selection is required.CommentedNov 15, 2019 at 15:49
  • 4
    @bruce from the same QA seems it's notCommentedDec 13, 2019 at 4:48

15 Answers 15

677

Some examples of how to avoid select

Use Dim'd variables

Dim rng as Range 

Set the variable to the required range. There are many ways to refer to a single-cell range:

Set rng = Range("A1") Set rng = Cells(1, 1) Set rng = Range("NamedRange") 

Or a multi-cell range:

Set rng = Range("A1:B10") Set rng = Range("A1", "B10") Set rng = Range(Cells(1, 1), Cells(10, 2)) Set rng = Range("AnotherNamedRange") Set rng = Range("A1").Resize(10, 2) 

You can use the shortcut to the Evaluate method, but this is less efficient and should generally be avoided in production code.

Set rng = [A1] Set rng = [A1:B10] 

All the above examples refer to cells on the active sheet. Unless you specifically want to work only with the active sheet, it is better to Dim a Worksheet variable too:

Dim ws As Worksheet Set ws = Worksheets("Sheet1") Set rng = ws.Cells(1, 1) With ws Set rng = .Range(.Cells(1, 1), .Cells(2, 10)) End With 

If you do want to work with the ActiveSheet, for clarity it's best to be explicit. But take care, as some Worksheet methods change the active sheet.

Set rng = ActiveSheet.Range("A1") 

Or better

Dim ws As Worksheet Set ws = ActiveSheet Set rng = ws.Range("A1") 

Again, this refers to the active workbook. Unless you specifically want to work only with the ActiveWorkbook or ThisWorkbook, it is better to Dim a Workbook variable too.

Dim wb As Workbook Set wb = Application.Workbooks("Book1") Set rng = wb.Worksheets("Sheet1").Range("A1") 

If you do want to work with the ActiveWorkbook, for clarity it's best to be explicit. But take care, as many WorkBook methods change the active book.

Set rng = ActiveWorkbook.Worksheets("Sheet1").Range("A1") 

You can also use the ThisWorkbook object to refer to the book containing the running code.

Set rng = ThisWorkbook.Worksheets("Sheet1").Range("A1") 

A common (bad) piece of code is to open a book, get some data then close again

This is bad:

Sub foo() Dim v as Variant Workbooks("Book1.xlsx").Sheets(1).Range("A1").Clear Workbooks.Open("C:\Path\To\SomeClosedBook.xlsx") v = ActiveWorkbook.Sheets(1).Range("A1").Value Workbooks("SomeAlreadyOpenBook.xlsx").Activate ActiveWorkbook.Sheets("SomeSheet").Range("A1").Value = v Workbooks(2).Activate ActiveWorkbook.Close() End Sub 

And it would be better like:

Sub foo() Dim v as Variant Dim wb1 as Workbook Dim wb2 as Workbook Set wb1 = Workbooks("SomeAlreadyOpenBook.xlsx") Set wb2 = Workbooks.Open("C:\Path\To\SomeClosedBook.xlsx") v = wb2.Sheets("SomeSheet").Range("A1").Value wb1.Sheets("SomeOtherSheet").Range("A1").Value = v wb2.Close() End Sub 

Pass ranges to your Subs and Functions as Range variables:

Sub ClearRange(r as Range) r.ClearContents '.... End Sub Sub MyMacro() Dim rng as Range Set rng = ThisWorkbook.Worksheets("SomeSheet").Range("A1:B10") ClearRange rng End Sub 

You should also apply Methods (such as Find and Copy) to variables:

Dim rng1 As Range Dim rng2 As Range Set rng1 = ThisWorkbook.Worksheets("SomeSheet").Range("A1:A10") Set rng2 = ThisWorkbook.Worksheets("SomeSheet").Range("B1:B10") rng1.Copy rng2 

If you are looping over a range of cells it is often better (faster) to copy the range values to a variant array first and loop over that:

Dim dat As Variant Dim rng As Range Dim i As Long Set rng = ThisWorkbook.Worksheets("SomeSheet").Range("A1:A10000") dat = rng.Value ' dat is now array (1 to 10000, 1 to 1) for i = LBound(dat, 1) to UBound(dat, 1) dat(i,1) = dat(i, 1) * 10 ' Or whatever operation you need to perform next rng.Value = dat ' put new values back on sheet 

This is a small taster for what's possible.

12
  • 12
    adding to this brilliant answer that in order to work wit a range you don't need to know its actual size as long as you know the top left ... e.g. rng1(12, 12) will work even though rng1 was set to [A1:A10] only.
    – MikeD
    CommentedJan 12, 2015 at 18:07
  • 4
    @chrisneilsen Chris, I believe you can also use worksheet prefix before shorthand cell reference notation to save you from typing Range like this: ActiveSheet.[a1:a4] or ws.[b6].CommentedJul 15, 2016 at 18:20
  • 7
    @AndrewWillems ...or 48 times in this post, but who's counting. ☺ ...but seriously, it's an easy one to forget when working with variables holding objects. A variant variable doesn't require Setuntil you assign an object to it. For example, Dim x: x = 1 is okay, but Dim x: x = Sheets("Sheet1") will generate Error 438. However just to confuse/clarify Dim x: x = Range("A1") will not create an error. Why? ...because it's assigned the value of the object to the variable, not a reference to the object itself (since it's the equivalent of Dim x: x = Range("A1").Value)CommentedApr 7, 2018 at 11:38
  • 3
    @user3932000 that would make an interesting Q. I'm sure there are ways to handle it.. you've been on SO long enough to know hijacking a comment thread on an old Q isn't the way to goCommentedDec 18, 2018 at 3:42
  • 4
    One small addition: I found it to be very useful to not name the variables "wb1" and "wb2" for example, but "wbTarget" and "wbSource" if I'm copying over. Helps to keep track of which argument goes where.
    – LWChris
    CommentedApr 26, 2021 at 13:56
257

Two main reasons why .Select, .Activate, Selection, Activecell, Activesheet, Activeworkbook, etc. should be avoided

  1. It slows down your code.
  2. It is usually the main cause of runtime errors.

How do we avoid it?

1) Directly work with the relevant objects

Consider this code

Sheets("Sheet1").Activate Range("A1").Select Selection.Value = "Blah" Selection.NumberFormat = "@" 

This code can also be written as

With Sheets("Sheet1").Range("A1") .Value = "Blah" .NumberFormat = "@" End With 

2) If required declare your variables. The same code above can be written as

Dim ws as worksheet Set ws = Sheets("Sheet1") With ws.Range("A1") .Value = "Blah" .NumberFormat = "@" End With 

To explain the above in detail, I have created a video How to avoid .Select, .Activate, Selection. Feel free to have a look.

That's a good answer, but what I am missing on this topic is when we actually need Activate. Everyone says it is bad, but no one explains any cases where it makes sense to use it.

Situation when you can't avoid using .Activate/.Select. (Will add more links as and when I come across them)

  1. When you want to present a worksheet to a user so that the user can see it.
  2. Scenarios like Working macro returns error when run from form control where you are forced to use .Activate
  3. When usual methods of Text To Columns / .Formula = .Formula doesn't work then you may have to resort to .Select
7
  • 19
    That's a good answer, but what I am missing on this topic is when we actually need Activate. Everyone says it is bad, but no one explains any cases where it makes sense to use it. For example I was working with 2 workbooks and could not start a macro on one of the workbooks without activating it first. Could you elaborate a bit maybe? Also if for example I do not activate sheets when copying a range from one sheet to another, when I execute the program, it seems to activate the respective sheets anyways, implicitly.CommentedFeb 2, 2016 at 10:04
  • 1
    I find that you may sometimes need to activate a sheet first if you need to paste or filter data on it. I would say its best to avoid activating as much as possible but there are instances where you need to do it. So keep activating and selecting to an absolute minimum as per the answer above.
    – Nick
    CommentedNov 25, 2016 at 14:34
  • 8
    i think the point is not to completely avoid using them, but just as much as possible. if you want to save a workbook, so that when someone opens it a certain cell in a certain sheet is selected, then you have to select that sheet and cell. copy/paste is a bad example, at least in the case of values, it can be done faster by a code such as Sheets(2).[C10:D12].Value = Sheets(1).[A1:B3].Value
    – robotik
    CommentedJun 20, 2017 at 12:52
  • 3
    @Nick You don't need to Activate sheets to paste to them or filter them. Use the sheet object in your paste or filter commands. It becomes easier as you learn the Excel object model through practice. I believe the only time I use .Activate is when I create a new sheet, but I want the original sheet to appear when the code is done.
    – phrebh
    CommentedJan 9, 2018 at 13:30
  • 3
    @phrebh You don't need to use .Activate to move to the original sheet, just use Application.Goto
    – GMalc
    CommentedFeb 28, 2019 at 18:44
112

One small point of emphasis I'll add to all the excellent answers given previously:

Probably the biggest thing you can do to avoid using Select is to as much as possible, use named ranges (combined with meaningful variable names) in your VBA code. This point was mentioned above, but it was glossed over a bit; however, it deserves special attention.

Here are a couple of additional reasons to make liberal use of named ranges, though I am sure I could think of more.

Named ranges make your code easier to read and understand.

Example:

Dim Months As Range Dim MonthlySales As Range Set Months = Range("Months") ' E.g, "Months" might be a named range referring to A1:A12 Set MonthlySales = Range("MonthlySales") ' E.g, "Monthly Sales" might be a named range referring to B1:B12 Dim Month As Range For Each Month in Months Debug.Print MonthlySales(Month.Row) Next Month 

It is pretty obvious what the named ranges Months and MonthlySales contain, and what the procedure is doing.

Why is this important? Partially because it is easier for other people to understand it, but even if you are the only person who will ever see or use your code, you should still use named ranges and good variable names because you will forget what you meant to do with it a year later, and you will waste 30 minutes just figuring out what your code is doing.

Named ranges ensure that your macros are far less likely to break when (not if!) the configuration of the spreadsheet changes.

Consider, if the above example had been written like this:

Dim rng1 As Range Dim rng2 As Range Set rng1 = Range("A1:A12") Set rng2 = Range("B1:B12") Dim rng3 As Range For Each rng3 in rng1 Debug.Print rng2(rng3.Row) Next rng3 

This code will work just fine at first - that is until you or a future user decides "gee wiz, I think I'm going to add a new column with the year in Column A!", or put an expenses column between the months and sales columns, or add a header to each column. Now, your code is broken. And because you used terrible variable names, it will take you a lot more time to figure out how to fix it than it should take.

If you had used named ranges to begin with, the Months and Sales columns could be moved around all you like, and your code would continue working just fine.

19
  • 7
    The debate about whether named ranges are good or bad spreadsheet design continues - I'm firmly in the no camp. In my experience they increase errors (for standard users who have no need of code).
    – brettdj
    CommentedFeb 27, 2015 at 8:15
  • 6
    one reference do Range Names Hinder Novice Debugging Performance?
    – brettdj
    CommentedFeb 28, 2015 at 7:23
  • 13
    I agree with your development philosophy; however I think the paper is nonsense. It talks about how range names can confuse novices who are debugging spreadsheets, but anyone who uses novices to look at complex spreadsheets gets what they deserve! I used to work for a firm who reviewed financial spreadsheets, and I can tell you that it is not the sort of job you give to a novice.
    – DeanOC
    CommentedMar 25, 2015 at 1:18
  • 11
    There is no meaningful debate. Anyone who argues against defined names has not taken the time to fully understand their ramifications. Named formulas may be the single most profound and useful construct in all of Excel.CommentedAug 21, 2015 at 0:28
  • 13
    @brettdj: Your citation is correct, but you forgot to mention that it is followed by six "Except..." phrases. One of them being: "Except as a substitute for cell references in macro coding Always use Excel Names as a substitute for cell references when constructing macros. This is to avoid errors arising from the insertion of additional rows or columns whereby the macro coding no longer points to the intended source data."CommentedNov 3, 2017 at 10:35
56

I'm going to give the short answer since everyone else gave the long one.

You'll get .select and .activate whenever you record macros and reuse them. When you .select a cell or sheet it just makes it active. From that point on whenever you use unqualified references like Range.Value they just use the active cell and sheet. This can also be problematic if you don't watch where your code is placed or a user clicks on the workbook.

So, you can eliminate these issues by directly referencing your cells. Which goes:

'create and set a range Dim Rng As Excel.Range Set Rng = Workbooks("Book1").Worksheets("Sheet1").Range("A1") 'OR Set Rng = Workbooks(1).Worksheets(1).Cells(1, 1) 

Or you could

'Just deal with the cell directly rather than creating a range 'I want to put the string "Hello" in Range A1 of sheet 1 Workbooks("Book1").Worksheets("Sheet1").Range("A1").value = "Hello" 'OR Workbooks(1).Worksheets(1).Cells(1, 1).value = "Hello" 

There are various combinations of these methods, but that would be the general idea expressed as shortly as possible for impatient people like me.

0
    42

    Avoiding Select and Activate is the move that makes you a bit better VBA developer. In general, Select and Activate are used when a macro is recorded, thus the Parent worksheet or range is always considered the active one.

    This is how you may avoid Select and Activate in the following cases:


    Adding a new Worksheet and copying a cell on it:

    From (code generated with macro recorder):

    Sub Makro2() Range("B2").Select Sheets.Add After:=ActiveSheet Sheets("Tabelle1").Select Sheets("Tabelle1").Name = "NewName" ActiveCell.FormulaR1C1 = "12" Range("B2").Select Selection.Copy Range("B3").Select ActiveSheet.Paste Application.CutCopyMode = False End Sub 

    To:

    Sub TestMe() Dim ws As Worksheet Set ws = Worksheets.Add With ws .Name = "NewName" .Range("B2") = 12 .Range("B2").Copy Destination:=.Range("B3") End With End Sub 

    When you want to copy range between worksheets:

    From:

    Sheets("Source").Select Columns("A:D").Select Selection.Copy Sheets("Target").Select Columns("A:D").Select ActiveSheet.Paste 

    To:

    Worksheets("Source").Columns("A:D").Copy Destination:=Worksheets("Target").Range("a1") 

    Using fancy named ranges

    You may access them with [], which is really beautiful, compared to the other way. Check yourself:

    Dim Months As Range Dim MonthlySales As Range Set Months = Range("Months") Set MonthlySales = Range("MonthlySales") Set Months =[Months] Set MonthlySales = [MonthlySales] 

    The example from above would look like this:

    Worksheets("Source").Columns("A:D").Copy Destination:=Worksheets("Target").[A1] 

    Not copying values, but taking them

    Usually, if you are willing to select, most probably you are copying something. If you are only interested in the values, this is a good option to avoid select:

    Range("B1:B6").Value = Range("A1:A6").Value


    Try always to reference the Worksheet as well

    This is probably the most common mistake in . Whenever you copy ranges, sometimes the worksheet is not referenced and thus VBA considers the wrong sheet the ActiveWorksheet.

    'This will work only if the 2. Worksheet is selected! Public Sub TestMe() Dim rng As Range Set rng = Worksheets(2).Range(Cells(1, 1), Cells(2, 2)).Copy End Sub 'This works always! Public Sub TestMe2() Dim rng As Range With Worksheets(2) .Range(.Cells(1, 1), .Cells(2, 2)).Copy End With End Sub 

    Can I really never use .Select or .Activate for anything?

    • A good example of when you could be justified in using .Activate and .Select is when you want make sure that a specific Worksheet is selected for visual reasons. E.g., that your Excel would always open with the cover worksheet selected first, disregarding which which was the ActiveSheet when the file was closed.

    Thus, something like the code below is absolutely OK:

    Private Sub Workbook_Open() Worksheets("Cover").Activate End Sub 
    3
    • 1
      You can use Application.Goto instead of Worksheets.Activate. Somewhat less risky.CommentedFeb 20, 2020 at 17:53
    • 1
      Late reply FYI - a nice and somewhat unexpected example for a needed .Select - as well as my work around - can be found at How to write identical information to all sheets - @Vityata :)
      – T.M.
      CommentedMar 11, 2020 at 14:17
    • 1
      @T.M. - indeed it is an interesting example, and probably saves some milliseconds for 100+ worksheets, but I would probably discourage it, if I see it somewhere. Anyway, the Selection there is not explicitly written but a result of .FillAcrossSheets, so this is somewhere in between (at least in my idea about VBA taxonomy)
      – Vityata
      CommentedMar 11, 2020 at 14:48
    41

    "... and am finding that my code would be more re-usable if I were able to use variables instead of Select functions."

    While I cannot think of any more than an isolated handful of situations where .Select would be a better choice than direct cell referencing, I would rise to the defense of Selection and point out that it should not be thrown out for the same reasons that .Select should be avoided.

    There are times when having short, time-saving macro sub routines assigned to hot-key combinations available with the tap of a couple of keys saves a lot of time. Being able to select a group of cells to enact the operational code on works wonders when dealing with pocketed data that does not conform to a worksheet-wide data format. Much in the same way that you might select a group of cells and apply a format change, selecting a group of cells to run special macro code against can be a major time saver.

    Examples of Selection-based sub framework:

    Public Sub Run_on_Selected() Dim rng As Range, rSEL As Range Set rSEL = Selection 'store the current selection in case it changes For Each rng In rSEL Debug.Print rng.Address(0, 0) 'cell-by-cell operational code here Next rng Set rSEL = Nothing End Sub Public Sub Run_on_Selected_Visible() 'this is better for selected ranges on filtered data or containing hidden rows/columns Dim rng As Range, rSEL As Range Set rSEL = Selection 'store the current selection in case it changes For Each rng In rSEL.SpecialCells(xlCellTypeVisible) Debug.Print rng.Address(0, 0) 'cell-by-cell operational code here Next rng Set rSEL = Nothing End Sub Public Sub Run_on_Discontiguous_Area() 'this is better for selected ranges of discontiguous areas Dim ara As Range, rng As Range, rSEL As Range Set rSEL = Selection 'store the current selection in case it changes For Each ara In rSEL.Areas Debug.Print ara.Address(0, 0) 'cell group operational code here For Each rng In ara.Areas Debug.Print rng.Address(0, 0) 'cell-by-cell operational code here Next rng Next ara Set rSEL = Nothing End Sub 

    The actual code to process could be anything from a single line to multiple modules. I have used this method to initiate long running routines on a ragged selection of cells containing the filenames of external workbooks.

    In short, don't discard Selection due to its close association with .Select and ActiveCell. As a worksheet property it has many other purposes.

    (Yes, I know this question was about .Select, not Selection but I wanted to remove any misconceptions that novice VBA coders might infer.)

    2
    • 14
      Selection can be anything in the worksheet so might as well test first the type of the object before assigning it to a variable since you explicitly declared it as Range.
      – L42
      CommentedMay 19, 2015 at 22:19
    • 1
      I took this a step further and created an excel Add-In that works on what cells the user has selected, and added a button to the right-click context menu that appears when cells are selected to trigger it. My users absolutely love it. To them, it's as if my macro is a built in feature of Excel, they don't know any different.CommentedJan 22, 2021 at 16:30
    37

    Please note that in the following I'm comparing the Select approach (the one that the OP wants to avoid), with the Range approach (and this is the answer to the question). So don't stop reading when you see the first Select.

    It really depends on what you are trying to do. Anyway, a simple example could be useful. Let's suppose that you want to set the value of the active cell to "foo". Using ActiveCell you would write something like this:

    Sub Macro1() ActiveCell.Value = "foo" End Sub 

    If you want to use it for a cell that is not the active one, for instance for "B2", you should select it first, like this:

    Sub Macro2() Range("B2").Select Macro1 End Sub 

    Using Ranges you can write a more generic macro that can be used to set the value of any cell you want to whatever you want:

    Sub SetValue(cellAddress As String, aVal As Variant) Range(cellAddress).Value = aVal End Sub 

    Then you can rewrite Macro2 as:

    Sub Macro2() SetCellValue "B2", "foo" End Sub 

    And Macro1 as:

    Sub Macro1() SetValue ActiveCell.Address, "foo" End Sub 
    2
    • 1
      Thanks for the excellent response so quickly. So does that mean that if i would normally add cells to range, name the range, and iterate through it, i should jump straight to creating an array?
      – BiGXERO
      CommentedMay 23, 2012 at 6:33
    • I'm not sure I understand what you mean, but you can create a Range with a single instruction (e.g. Range("B5:C14")) and you can even set its value at once (if it has to be the same for every cell in the range), e.g. Range("B5:C14").Value = "abc"CommentedMay 23, 2012 at 6:50
    21

    Always state the workbook, worksheet and the cell/range.

    For example:

    Thisworkbook.Worksheets("fred").cells(1,1) Workbooks("bob").Worksheets("fred").cells(1,1) 

    Because end users will always just click buttons and as soon as the focus moves off of the workbook the code wants to work with then things go completely wrong.

    And never use the index of a workbook.

    Workbooks(1).Worksheets("fred").cells(1,1) 

    You don't know what other workbooks will be open when the user runs your code.

    2
    • 7
      The names of worksheets can change, too, you know. Use codenames instead.
      – Rick
      CommentedNov 23, 2014 at 14:33
    • Worksheet names can change, sure. But I disagree that you should over-complicate your code to try and mitigate that. If a user changes the name of a sheet and their macro stops working, that's on them. I generally just assume worksheet names are going to be the same. For particularly critical macros, I run a little pre-flight check before launching into the macro proper, which just checks to make sure all the sheets it expects to find are indeed there, and if any are missing it notifies the user which one.CommentedMar 2, 2020 at 13:21
    19

    These methods are rather stigmatized, so taking the lead of Vityata and Jeeped for the sake of drawing a line in the sand:

    Call .Activate, .Select, Selection, ActiveSomething methods/properties

    Basically because they're called primarily to handle user input through the application UI. Since they're the methods called when the user handles objects through the UI, they're the ones recorded by the macro-recorder, and that's why calling them is either brittle or redundant for most situations: you don't have to select an object so as to perform an action with Selection right afterwards.

    However, this definition settles situations on which they are called for:

    When to call .Activate, .Select, .Selection, .ActiveSomething methods/properties

    Basically when you expect the final user to play a role in the execution.

    If you are developing and expect the user to choose the object instances for your code to handle, then .Selection or .ActiveObject are apropriate.

    On the other hand, .Select and .Activate are of use when you can infer the user's next action and you want your code to guide the user, possibly saving him/her some time and mouse clicks. For example, if your code just created a brand new instance of a chart or updated one, the user might want to check it out, and you could call .Activate on it or its sheet to save the user the time searching for it; or if you know the user will need to update some range values, you can programmatically select that range.

    1
    • What answer does "Jeeped" refer to? There isn't currently an answer or comment by that user name (user names may change at any time) - including deleted answers. A candidate is user4039065's answer (now deleted user).CommentedSep 29, 2020 at 15:59
    12

    IMHO use of .select comes from people, who like me started learning VBA by necessity through recording macros and then modifying the code without realizing that .select and subsequent selection is just an unnecessary middle-men.

    .select can be avoided, as many posted already, by directly working with the already existing objects, which allows various indirect referencing like calculating i and j in a complex way and then editing cell(i,j), etc.

    Otherwise, there is nothing implicitly wrong with .select itself and you can find uses for this easily, e.g. I have a spreadsheet that I populate with date, activate macro that does some magic with it and exports it in an acceptable format on a separate sheet, which, however, requires some final manual (unpredictable) inputs into an adjacent cell. So here comes the moment for .select that saves me that additional mouse movement and click.

    1
    • 2
      While you are right, there is at least one thing implicitly wrong with select: it is slow. Very slow indeed compared to everything else happening in a macro.
      – vacip
      CommentedNov 22, 2016 at 13:25
    12

    To avoid using the .Select method, you can set a variable equal to the property that you want.

    For instance, if you want the value in Cell A1 you could set a variable equal to the value property of that cell.

    • Example valOne = Range("A1").Value

    For instance, if you want the codename of 'Sheet3' you could set a variable equal to the Codename property of that worksheet.

    • Example valTwo = Sheets("Sheet3").Codename
      11

      How can we avoid copy-paste?

      Let's face it: this one appears a lot when recording macros:

      Range("X1").Select Selection.Copy Range("Y9").Select Selection.Paste 

      While the only thing the person wants is:

      Range("Y9").Value = Range("X1").Value 

      Therefore, instead of using copy-paste in VBA macros, I'd advise using the following simple approach:

      Destination_Range.Value = Source_Range.Value 
      2
      • 2
        Think this corresponds to @Vityata 's fine answer, section "Not copying values, but taking them": Range("B1:B6").Value = Range("A1:A6").Value. Of course some of the answers above don't face OP's core issue :-)
        – T.M.
        CommentedOct 30, 2020 at 18:27
      • This is also addressed in the accepted answer in the section heade If you are looping over a range of cells it is often better (faster) to copy the range values to a variant array first and loop over that:CommentedJan 21, 2023 at 6:15
      6

      The main reason never to use Select or Activesheet is because most people will have at least another couple of workbooks open (sometimes dozens) when they run your macro, and if they click away from your sheet while your macro is running and click on some other book they have open, then the "Activesheet" changes, and the target workbook for an unqualified "Select" command changes as well.

      At best, your macro will crash, at worst you might end up writing values or changing cells in the wrong workbook with no way to "Undo" them.

      I have a simple golden rule that I follow: Add variables named "wb" and "ws" for a Workbook object and a Worksheet object and always use those to refer to my macro book. If I need to refer to more than one book, or more than one sheet, I add more variables.

      For example,

      Dim wb as Workbook Dim ws as Worksheet Set wb = ThisWorkBook Set ws = wb.sheets("Output") 

      The "Set wb = ThisWorkbook" command is absolutely key. "ThisWorkbook" is a special value in Excel, and it means the workbook that your VBA code is currently running from. A very helpful shortcut to set your Workbook variable with.

      After you've done that at the top of your Sub, using them could not be simpler, just use them wherever you would use "Selection":

      So to change the value of cell "A1" in "Output" to "Hello", instead of:

      Sheets("Output").Activate ActiveSheet.Range("A1").Select Selection.Value = "Hello" 

      We can now do this:

      ws.Range("A1").Value = "Hello" 

      Which is not only much more reliable and less likely to crash if the user is working with multiple spreadsheets; it's also much shorter, quicker and easier to write.

      As an added bonus, if you always name your variables "wb" and "ws", you can copy and paste code from one book to another and it will usually work with minimal changes needed, if any.

      4
      • 1
        Not my downvote, but I'm not sure this adds anything new to what has already been proposed in existing answers.
        – BigBen
        CommentedMar 2, 2020 at 15:13
      • 1
        Yeah, my answer is slightly redundant, but the other answers are too long, contain too much superfluous stuff and nobody has mentioned using ThisWorkbook to set your worksheet variable upfront. That is something which, if someone had shown me the first time I dipped a toe into VBA, I would have found incredibly helpful. Others have mentioned using a Worksheet variable but don't really explain why very well, and don't offer an example of code with and without using worksheet and workbook variables.CommentedMar 4, 2020 at 10:13
      • But the accepted answer definitely discusses ThisWorkbook... I'm not sure your comment is accurate.
        – BigBen
        CommentedMar 4, 2020 at 12:11
      • It does, you're not wrong. But not in the context of using it to set a workbook variable and using that workbook variable going forward, or using that workbook variable to set a worksheet variable, as I'm suggesting. My answer is shorter, simpler and more accessible for beginners than the accepted answer.CommentedMar 4, 2020 at 16:23
      6

      I noticed that none of these answers mention the .Offset Property. This also can be used to avoid using the Select action when manipulating certain cells, particularly in reference to a selected cell (as the OP mentions with ActiveCell).

      Here are a couple of examples:

      I will also assume the ActiveCell is J4.

      ActiveCell.Offset(2, 0).Value = 12

      • This will change the value of the cell two rows down from activecell (which is J6) to be a value of 12
      • A minus -2 would have put the value 12 two rows above in J2

      ActiveCell.Offset(0,1).Copy ActiveCell.Offset(,2)

      • This will copy the cell one column to the right (k4) to the cell two columns from the activecell (L4).
      • Note that 0 may be omitted in the offset parameter
        • So: ActiveCell.Offset(,2) is the same as ActiveCell.Offset(0,2)
      • Similar to the previous example a -1 would be one column to the left (i4)

      This isn't to imply these are better than the above options, but it's definitely better than using select. Note that using the EXCEL FUNCTION Offset should be avoided in a worksheet as it is a volatile function.

        5

        Working with the .Parent feature, this example shows how setting only one myRng reference enables dynamic access to the entire environment without any .Select, .Activate, .Activecell, .ActiveWorkbook, .ActiveSheet and so on. (There isn't any generic .Child feature.)

        Sub ShowParents() Dim myRng As Range Set myRng = ActiveCell Debug.Print myRng.Address ' An address of the selected cell Debug.Print myRng.Parent.name ' The name of sheet, where MyRng is in Debug.Print myRng.Parent.Parent.name ' The name of workbook, where MyRng is in Debug.Print myRng.Parent.Parent.Parent.name ' The name of application, where MyRng is in ' You may use this feature to set reference to these objects Dim mySh As Worksheet Dim myWbk As Workbook Dim myApp As Application Set mySh = myRng.Parent Set myWbk = myRng.Parent.Parent Set myApp = myRng.Parent.Parent.Parent Debug.Print mySh.name, mySh.Cells(10, 1).Value Debug.Print myWbk.name, myWbk.Sheets.Count Debug.Print myApp.name, myApp.Workbooks.Count ' You may use dynamically addressing With myRng .Copy ' Pastes in D1 on sheet 2 in the same workbook, where the copied cell is .Parent.Parent.Sheets(2).Range("D1").PasteSpecial xlValues ' Or myWbk.Sheets(2).Range("D1").PasteSpecial xlValues ' We may dynamically call active application too .Parent.Parent.Parent.CutCopyMode = False ' Or myApp.CutCopyMode = False End With End Sub 
        1
        • 3
          Very nice, but not sure what this has to do with OPs question. You don't need "Parent" at all to work in VBA without using Select or ActiveSheetCommentedMar 2, 2020 at 13:18

        Start asking to get answers

        Find the answer to your question by asking.

        Ask question

        Explore related questions

        See similar questions with these tags.