Simple line of code generating run time error

[Scott JS]

The following code, which works in other macros, is causing a “Run-time error ‘1004’: Application-defined or object-defined error"

1. Cells(20000,1).Select
2. Selection.End(x1Up).Select
3. Let lastrow = Activecell.Row

The debugger is highlighting row 2 -- the equivalent of a manual “end then up arrow” as the problem code. ?????????

 

[JEC]

Not x1 up, but xlup.

Furthermore, you don’t need selects in VBA and “let” is actually always omitted.

 

[Scott JS]

Not x1 up, but xlup.

Furthermore, you don’t need selects in VBA and “let” is actually always omitted.

Re the let -- thanks, will amend, but I'm not even getting that far
re the 1 vs. the l, that fixed it. Thank you!

 

[Joe4]

The following code, which works in other macros, is causing a “Run-time error ‘1004’: Application-defined or object-defined error"

1. Cells(20000,1).Select
2. Selection.End(x1Up).Select
3. Let lastrow = Activecell.Row

The debugger is highlighting row 2 -- the equivalent of a manual “end then up arrow” as the problem code. ?????????

That can all be simplified to a single line:

lastrow = Cells(20000,1).End(xlUp).Row