My range creation macro crashes when there's nothing in range and it's a bummer.

[jmpatrick]

Good afternoon.

I have some code sourced from powerspreadsheets.com that creates a named range called FloorPlanRequestsFormulas.

It works as intended when there is data in the target cells, but gives this error when there is none:

Run-time error '91':

Object variable or With block variable not set

More specifically, it doesn't seem to like this line:

myLastRow = .Find(What:="*", LookIn:=xlValues, LookAt:=xlPart, SearchOrder:=xlByRows, SearchDirection:=xlPrevious).Row

Here is the complete code:

Sub FloorPlanRequestsFormulasRange()

    Application.EnableEvents = False
    Application.ScreenUpdating = False
    Application.DisplayAlerts = False

    Dim myWorksheet As Worksheet
 
    Dim myFirstRow As Long
    Dim myLastRow As Long
    Dim myFirstColumn As Long
    Dim myLastColumn As Long
 
    Dim myNamedRangeDynamic As Range
 
    Dim myRangeName As String
 
    Set myWorksheet = ThisWorkbook.Worksheets("FloorPlanRequests")
 
    myFirstRow = 1
    myFirstColumn = 5
    myLastColumn = 8
 
    myRangeName = "FloorPlanRequestsFormulas"
 
    With myWorksheet.Cells
 
        myLastRow = .Find(What:="*", LookIn:=xlValues, LookAt:=xlPart, SearchOrder:=xlByRows, SearchDirection:=xlPrevious).Row
        myLastColumn = .Find(What:="*", LookIn:=xlValues, LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlPrevious).Column
 
        Set myNamedRangeDynamic = .Range(.Cells(myFirstRow, myFirstColumn), .Cells(myLastRow, myLastColumn))
 
End With
 
    ThisWorkbook.Names.Add Name:=myRangeName, RefersTo:=myNamedRangeDynamic
    
    Application.EnableEvents = True
    Application.ScreenUpdating = True
    Application.DisplayAlerts = True
 
End Sub

 

[jmpatrick]

Here's the current code:

Sub FloorPlanRequestsFormulasRange()

    Application.EnableEvents = False
    Application.ScreenUpdating = False
    Application.DisplayAlerts = False

    Dim myWorksheet As Worksheet
 
    Dim myFirstRow As Long
    Dim myLastRow As Long
    Dim myFirstColumn As Long
    Dim myLastColumn As Long
 
    Dim myNamedRangeDynamic As Range
 
    Dim myRangeName As String
 
    Set myWorksheet = ThisWorkbook.Worksheets("FloorPlanRequests")
 
    myFirstRow = 1
    myFirstColumn = 5
    myLastColumn = 8
 
    myRangeName = "FloorPlanRequestsFormulas"
 
    With myWorksheet.Cells
 
        myLastRow = myWorksheet.[match(2,1/(l:l<>""))]
        myLastColumn = .Find(What:="*", LookIn:=xlValues, LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlPrevious).Column
 
        Set myNamedRangeDynamic = .Range(.Cells(myFirstRow, myFirstColumn), .Cells(myLastRow, myLastColumn))
 
End With
 
    ThisWorkbook.Names.Add Name:=myRangeName, RefersTo:=myNamedRangeDynamic
   
    Application.EnableEvents = True
    Application.ScreenUpdating = True
    Application.DisplayAlerts = True
 
End Sub

Now giving me a Run-time '13' error.

 

[Fluff]

How about

Sub FloorPlanRequestsFormulasRange()

    Application.EnableEvents = False
    Application.ScreenUpdating = False
    Application.DisplayAlerts = False

    Dim myWorksheet As Worksheet
 
    Dim myFirstRow As Long
    Dim myLastRow As Long
    Dim myFirstColumn As Long
    Dim myLastColumn As Long
    Dim Fnd As Range
    
    Dim myNamedRangeDynamic As Range
 
    Dim myRangeName As String
 
    Set myWorksheet = ThisWorkbook.Worksheets("FloorPlanRequests")
 
    myFirstRow = 1
    myFirstColumn = 5
    myLastColumn = 8
 
    myRangeName = "FloorPlanRequestsFormulas"
 
    With myWorksheet.Cells
 
        myLastRow = myWorksheet.[match(2,1/(l:l<>""))]
        Set Fnd = .Find(What:="*", LookIn:=xlValues, LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlPrevious)
        If Fnd Is Nothing Then
            MsgBox "no data"
            Exit Sub
         End If
        myLastColumn = Fnd.Column
        Set myNamedRangeDynamic = .Range(.Cells(myFirstRow, myFirstColumn), .Cells(myLastRow, myLastColumn))
 
End With
 
    ThisWorkbook.Names.Add Name:=myRangeName, RefersTo:=myNamedRangeDynamic
   
    Application.EnableEvents = True
    Application.ScreenUpdating = True
    Application.DisplayAlerts = True
 
End Sub

 

[jmpatrick]

How about

Sub FloorPlanRequestsFormulasRange()

    Application.EnableEvents = False
    Application.ScreenUpdating = False
    Application.DisplayAlerts = False

    Dim myWorksheet As Worksheet
 
    Dim myFirstRow As Long
    Dim myLastRow As Long
    Dim myFirstColumn As Long
    Dim myLastColumn As Long
    Dim Fnd As Range
   
    Dim myNamedRangeDynamic As Range
 
    Dim myRangeName As String
 
    Set myWorksheet = ThisWorkbook.Worksheets("FloorPlanRequests")
 
    myFirstRow = 1
    myFirstColumn = 5
    myLastColumn = 8
 
    myRangeName = "FloorPlanRequestsFormulas"
 
    With myWorksheet.Cells
 
        myLastRow = myWorksheet.[match(2,1/(l:l<>""))]
        Set Fnd = .Find(What:="*", LookIn:=xlValues, LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlPrevious)
        If Fnd Is Nothing Then
            MsgBox "no data"
            Exit Sub
         End If
        myLastColumn = Fnd.Column
        Set myNamedRangeDynamic = .Range(.Cells(myFirstRow, myFirstColumn), .Cells(myLastRow, myLastColumn))
 
End With
 
    ThisWorkbook.Names.Add Name:=myRangeName, RefersTo:=myNamedRangeDynamic
  
    Application.EnableEvents = True
    Application.ScreenUpdating = True
    Application.DisplayAlerts = True
 
End Sub

That gives me a Run-time error '13' Type mismatch at this line:

myLastRow = myWorksheet.[match(2,1/(l:l<>""))]

 

[Fluff]

I didn't make any changes to that row, so was it giving you an error before?

 

[jmpatrick]

I didn't make any changes to that row, so was it giving you an error before?

It was, yes.

 

[Fluff]

Then just change that line back to what it was & do it in the same way as I showed for the column.

 

[jmpatrick]

Then just change that line back to what it was & do it in the same way as I showed for the column.

OK, did that. Back to this error:

Run-time error '91':

Object variable or With block variable not set

 

[Fluff]

Did you do it in the same way I showed for the column?

 

[jmpatrick]

Did you do it in the same way I showed for the column?

I did (I think). Here's the current code:

Sub FloorPlanRequestsFormulasRange()

    Application.EnableEvents = False
    Application.ScreenUpdating = False
    Application.DisplayAlerts = False

    Dim myWorksheet As Worksheet
 
    Dim myFirstRow As Long
    Dim myLastRow As Long
    Dim myFirstColumn As Long
    Dim myLastColumn As Long
    Dim Fnd As Range
    
    Dim myNamedRangeDynamic As Range
 
    Dim myRangeName As String
 
    Set myWorksheet = ThisWorkbook.Worksheets("FloorPlanRequests")
 
    myFirstRow = 1
    myFirstColumn = 5
    myLastColumn = 8
 
    myRangeName = "FloorPlanRequestsFormulas"
 
    With myWorksheet.Cells
 
        myLastRow = .Find(What:="*", LookIn:=xlValues, LookAt:=xlPart, SearchOrder:=xlByRows, SearchDirection:=xlPrevious).Row
        Set Fnd = .Find(What:="*", LookIn:=xlValues, LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlPrevious)
        If Fnd Is Nothing Then
            MsgBox "no data"
            Exit Sub
         End If
        myLastColumn = Fnd.Column
        Set myNamedRangeDynamic = .Range(.Cells(myFirstRow, myFirstColumn), .Cells(myLastRow, myLastColumn))
 
End With
 
    ThisWorkbook.Names.Add Name:=myRangeName, RefersTo:=myNamedRangeDynamic
   
    Application.EnableEvents = True
    Application.ScreenUpdating = True
    Application.DisplayAlerts = True
 
End Sub