2016-11-08 196 views
0

ご協力いただければ幸いです。私はエラーが発生しているランタイムエラー1004 Rangeクラスのオートフィルタメソッドが失敗しましたVBA実行時エラー1004 Rangeクラスのオートフィルタメソッドが失敗しました

私はPublic Sub TestThis()コードを実行すると面白いことは、それは完璧に動作しますが、私はそれを他のコードに入れて、エラー私はバグそれを呼び出したときに呼び出されると、それ自体では問題を実行しないとき言ったように、エラーがこの行

.Range("A:K").AutoFilter Field:=11, Criteria1:="<>", Operator:=xlFilterValues 

に起こっている

失敗したRangeクラスのファイル名を指定して実行時エラー1004オートフィルタのメソッドを提供します。どんな助けでも大歓迎です。

私のコードは以下の通りです。

Sub Open_Workbook_Dialog() 

Dim my_FileName As Variant 
Dim my_Workbook As Workbook 

    MsgBox "Pick your CRO file" '<--| txt box for prompt to pick a file 

    my_FileName = Application.GetOpenFilename(FileFilter:="Excel Files,*.xl*;*.xm*") '<--| Opens the file window to allow selection 

    If my_FileName <> False Then 
    Set my_Workbook = Workbooks.Open(Filename:=my_FileName) 

    Call TestThis 

    Call Filter(my_Workbook) '<--|Calls the Filter Code and executes 

    End If 
End Sub 

Public Sub Filter(my_Workbook As Workbook) 
    Dim rCountry As Range, helpCol As Range 
    Dim wb As Workbook 
    With my_Workbook.Sheets(1) '<--| refer to data worksheet 
    With .UsedRange 
     Set helpCol = .Resize(1, 1).Offset(, .Columns.Count) '<--| get a "helper" column just at the right of used range, it'll be used to store unique country names in 
    End With 

    With .Range("A1:Y" & .Cells(.Rows.Count, 1).End(xlUp).Row) '<--| refer to its columns "A:Y" from row 1 to last non empty row of column "A" 
      .Columns(11).AdvancedFilter Action:=xlFilterCopy, CopyToRange:=helpCol, Unique:=True '<-- call AdvancedFilter on 11th column of the referenced range and store its unique values in "helper" column 
      Set helpCol = Range(helpCol.Offset(1), helpCol.End(xlDown)) '<--| set range with unique names in (skip header row) 
      For Each rCountry In helpCol '<--| iterate over unique country names range (skip header row) 
       .AutoFilter 11, rCountry.Value2 '<--| filter data on country field (11th column) with current unique country name 
       If Application.WorksheetFunction.Subtotal(103, .Cells.Resize(, 1)) > 1 Then '<--| if any cell other than header ones has been filtered... 
        Set wb = Application.Workbooks.Add '<--... add new Workbook 
         wb.SaveAs Filename:=rCountry.Value2 '<--... saves the workbook after the country 
          .SpecialCells(xlCellTypeVisible).Copy wb.Sheets(1).Range("A1") 
           ActiveSheet.Name = rCountry.Value2 '<--... rename it 
          .SpecialCells(xlCellTypeVisible).Copy ActiveSheet.Range("A1") 'copy data for country under header 
          Sheets(1).Range("A1:Y1").WrapText = False 'Takes the wrap text off 
          ActiveWindow.Zoom = 55 
         Sheets(1).UsedRange.Columns.AutoFit 'Autofits the column 
        wb.Close SaveChanges:=True '<--... saves and closes workbook 
       End If 
      Next 
     End With 
     .AutoFilterMode = False '<--| remove autofilter and show all rows back 
    End With 
    helpCol.Offset(-1).End(xlDown).Clear '<--| clear helper column (header included) 
End Sub 

Public Sub TestThis() 
Dim wks As Worksheet 

Set wks = ThisWorkbook.Sheets(1) 

With wks 
.AutoFilterMode = False 
.Range("A:K").AutoFilter Field:=11, Criteria1:="<>", Operator:=xlFilterValues 
.Range("A:C").SpecialCells(xlCellTypeBlanks).Interior.Color = 65535 
.AutoFilterMode = False 
End With 
End Sub 

答えて

1

Set wks = ThisWorkbook.Sheets(1)は、あなたがしたいシートを与えた場合、あなたはをワークブックからを確認してください。 ThisWorkbook.ステートメントはマクロ(VBAプロジェクト)が配置されているブックを参照します。たぶん、あなたは

Set wks = ActiveWorkbook.Sheets(1) 

に変更したりmy_WorkbookTestThis()にマクロを渡す必要があります。

+0

リマック:素晴らしいもの私の友人私はそれを逃した方法はありません。新鮮な目のセットは不思議をすることができます。ダブリンからこのような敬意を込めてありがとうございました:-) –

関連する問題