Страницы: 1
RSS
Разбиение таблицы с отфильтровкой нужных полей и удалением ненужных
 
Всем привет!!!!!! Столкнулся с проблемой отработки макроса, который нашел в сети. Аналогов не нашел, поэтому прошу помочь его реанимировать...

Возвращает ошибку:
Method Range of object _Worksheet failed

Макрос:
Код
Public Sub SplitToFiles()
' MACRO SplitToFiles
' Last update: 2012-03-04
' Author: mtone
' Version 1.1
' Description:
' Loops through a specified column, and split each distinct values into a separate file by making a copy and deleting rows below and above
'
' Note: Values in the column should be unique or sorted.
'
' The following cells are ignored when delimiting sections:
' - blank cells, or containing spaces only
' - same value repeated
' - cells containing "total"
'
' Files are saved in a "Split" subfolder from the location of the source workbook, and named after the section name.
Dim osh As Worksheet ' Original sheet
Dim iRow As Long ' Cursors
Dim iCol As Long
Dim iFirstRow As Long ' Constant
Dim iTotalRows As Long ' Constant
Dim iStartRow As Long ' Section delimiters
Dim iStopRow As Long
Dim sSectionName As String ' Section name (and filename)
Dim rCell As Range ' current cell
Dim owb As Workbook ' Original workbook
Dim sFilePath As String ' Constant
Dim iCount As Integer ' # of documents created
iCol = Application.InputBox("Enter the column number used for splitting", "Select column", 2, , , , , 1)
iRow = Application.InputBox("Enter the starting row number (to skip header)", "Select row", 5, , , , , 1)
iFirstRow = iRow
Set osh = Application.ActiveSheet
Set owb = Application.ActiveWorkbook
iTotalRows = osh.UsedRange.Rows.Count
sFilePath = Application.ActiveWorkbook.Path
If Dir(sFilePath + "\Split", vbDirectory) = "" Then
    MkDir sFilePath + "\Split"
End If
'Turn Off Screen Updating  Events
Application.EnableEvents = False
Application.ScreenUpdating = False
Do
    ' Get cell at cursor
    Set rCell = osh.Cells(iRow, iCol)
    sCell = Replace(rCell.Text, " ", "")
    If sCell = "" Or (rCell.Text = sSectionName And iStartRow <> 0) Or InStr(1, rCell.Text, "total", vbTextCompare) <> 0 Then
        ' Skip condition met
    Else
        ' Found new section
        If iStartRow = 0 Then
            ' StartRow delimiter not set, meaning beginning a new section
            sSectionName = rCell.Text
            iStartRow = iRow
        Else
            ' StartRow delimiter set, meaning we reached the end of a section
            iStopRow = iRow - 1
            ' Pass variables to a separate sub to create and save the new worksheet
            CopySheet osh, iFirstRow, iStartRow, iStopRow, iTotalRows, sFilePath, sSectionName, owb.fileFormat
            iCount = iCount + 1
            ' Reset section delimiters
            iStartRow = 0
            iStopRow = 0
            ' Ready to continue loop
            iRow = iRow - 1
        End If
    End If
    ' Continue until last row is reached
    If iRow < iTotalRows Then
            iRow = iRow + 1
    Else
        ' Finished. Save the last section
        iStopRow = iRow
        CopySheet osh, iFirstRow, iStartRow, iStopRow, iTotalRows, sFilePath, sSectionName, owb.fileFormat
        iCount = iCount + 1
        ' Exit
        Exit Do
    End If
Loop
'Turn On Screen Updating  Events
Application.ScreenUpdating = True
Application.EnableEvents = True
MsgBox Str(iCount) + " documents saved in " + sFilePath

End Sub
Public Sub DeleteRows(targetSheet As Worksheet, RowFrom As Long, RowTo As Long)
Dim rngRange As Range
Set rngRange = Range(targetSheet.Cells(RowFrom, 1), targetSheet.Cells(RowTo, 1)).EntireRow
rngRange.Select
rngRange.Delete
End Sub

Public Sub CopySheet(osh As Worksheet, iFirstRow As Long, iStartRow As Long, iStopRow As Long, iTotalRows As Long, sFilePath As String, sSectionName As String, fileFormat As XlFileFormat)
     Dim ash As Worksheet ' Copied sheet
     Dim awb As Workbook ' New workbook
     ' Copy book
     osh.Copy
     Set ash = Application.ActiveSheet
     ' Delete Rows after section
     If iTotalRows > iStopRow Then
         DeleteRows ash, iStopRow + 1, iTotalRows
     End If
     ' Delete Rows before section
     If iStartRow > iFirstRow Then
         DeleteRows ash, iFirstRow, iStartRow - 1
     End If
     ' Select left-topmost cell
     ash.Cells(1, 1).Select
     ' Clean up a few characters to prevent invalid filename
     sSectionName = Replace(sSectionName, "/", " ")
     sSectionName = Replace(sSectionName, "\", " ")
     sSectionName = Replace(sSectionName, ":", " ")
     sSectionName = Replace(sSectionName, "=", " ")
     sSectionName = Replace(sSectionName, "*", " ")
     sSectionName = Replace(sSectionName, ".", " ")
     sSectionName = Replace(sSectionName, "?", " ")
     ' Save in same format as original workbook
     ash.SaveAs sFilePath + "\Split\" + sSectionName, fileFormat
     ' Close
     Set awb = ash.Parent
     awb.Close SaveChanges:=False
End Sub
Изменено: direget - 27.09.2018 20:23:28
 
Т.е. Вы всерьез уверены, что проблема в самом коде, а не в файле? И при этом строку, на которой появляется ошибка тоже нам знать не обязательно? Я думаю, что без Вашего файла, на котором возникает ошибка и без указания строки, на которую указывает VBA - решить Вашу проблему сложно и вряд ли кто захочет сидеть и гадать.
Даже самый простой вопрос можно превратить в огромную проблему. Достаточно не уметь формулировать вопросы...
 
Файл ниже.. Но строку не подсвечивает...Поэтому не могу определить, где фол(
 
Вероятно ошибка возникает из-за неверного синтаксиса в строках с 85 по 89. В VBA я не силен, поэтому не могу определить точно.
 
Ну да, не подсвечивает, потому что в этом файле ваще ни одного макроса нет  :)  
 
Цитата
_Igor_61 написал:
Ну да, не подсвечивает, потому что в этом файле ваще ни одного макроса нет    
Я же говорю, что не силен)! Но можно же отработать его макрос выше)))!
 
:) Там три макроса, который из них Вы запускаете и как?
 
_Igor_61, ALT+F8, запускаю все вместе, так как они должны отработать в цикле.
 
Цитата
direget написал:
запускаю все вместе
Так не бывает. Напишите, в какой очерёдности запускаете.
 
Юрий М, 1->2->3 на 1 ошибка 1004, на 2 и 3 - Method Range of object _Worksheet failed
 
Добавил в начало макроса
Код
Application.DisplayAlerts = False

а в конец
Код
Application.DisplayAlerts = True

И ни каких ошибок нет. У Вас макрос пытается перезаписать уже имеющиеся файлы а вы ему в этом отказываете, поэтому на экран и выводится ошибка, а вот ошибка 1004 вообще не проявила себя никоим образом.
Изменено: Nordheim - 27.09.2018 08:31:54
"Все гениальное просто, а все простое гениально!!!"
 
Цитата
_Igor_61 написал:
Там три макроса, который из них Вы запускаете и как?
Там один макрос (отображается при запуске), остальные не запустишь через
Код
Alt+F8
, потому что для них нужно значения аргументам передать.
Код
Public Sub DeleteRows(targetSheet As Worksheet, RowFrom As Long, RowTo As Long)

Код
Public Sub CopySheet(osh As Worksheet, iFirstRow As Long, iStartRow As Long, iStopRow As Long, iTotalRows As Long, sFilePath As String, sSectionName As String, fileFormat As XlFileFormat)

Что и делает основной макрос вот тут для третьего:
Код
CopySheet osh, iFirstRow, iStartRow, iStopRow, iTotalRows, sFilePath, sSectionName, owb.fileFormat

а третий вот тут для второго:
Код
DeleteRows ash, iStopRow + 1, iTotalRows
Изменено: Nordheim - 27.09.2018 08:40:18
"Все гениальное просто, а все простое гениально!!!"
 
Nordheim, спасибо за помощь!
Книгу использую с поддержкой макросов...
Выдает теперь 400 ошибку...( Вы макросы запускаете отдельно, или через ALT+F8?
Вставлены строки, о которых вы говорили, но результата нет...
Вы используете на той таблице, которую я вам прислал или на другой?!
Код
Public Sub SplitToFiles()
' MACRO SplitToFiles
' Last update: 2012-03-04
' Author: mtone
' Version 1.1
' Description:
' Loops through a specified column, and split each distinct values into a separate file by making a copy and deleting rows below and above
'
' Note: Values in the column should be unique or sorted.
'
' The following cells are ignored when delimiting sections:
' - blank cells, or containing spaces only
' - same value repeated
' - cells containing "total"
'
' Files are saved in a "Split" subfolder from the location of the source workbook, and named after the section name.
Dim osh As Worksheet ' Original sheet
Dim iRow As Long ' Cursors
Dim iCol As Long
Dim iFirstRow As Long ' Constant
Dim iTotalRows As Long ' Constant
Dim iStartRow As Long ' Section delimiters
Dim iStopRow As Long
Dim sSectionName As String ' Section name (and filename)
Dim rCell As Range ' current cell
Dim owb As Workbook ' Original workbook
Dim sFilePath As String ' Constant
Dim iCount As Integer ' # of documents created
iCol = Application.InputBox("Enter the column number used for splitting", "Select column", 2, , , , , 1)
iRow = Application.InputBox("Enter the starting row number (to skip header)", "Select row", 5, , , , , 1)
iFirstRow = iRow
Set osh = Application.ActiveSheet
Set owb = Application.ActiveWorkbook
iTotalRows = osh.UsedRange.Rows.Count
sFilePath = Application.ActiveWorkbook.Path
If Dir(sFilePath + "\Split", vbDirectory) = "" Then
    MkDir sFilePath + "\Split"
End If
'Turn Off Screen Updating  Events
Application.EnableEvents = False
Application.ScreenUpdating = False
Application.DisplayAlerts = False
Do
    ' Get cell at cursor
    Set rCell = osh.Cells(iRow, iCol)
    sCell = Replace(rCell.Text, " ", "")
    If sCell = "" Or (rCell.Text = sSectionName And iStartRow <> 0) Or InStr(1, rCell.Text, "total", vbTextCompare) <> 0 Then
        ' Skip condition met
    Else
        ' Found new section
        If iStartRow = 0 Then
            ' StartRow delimiter not set, meaning beginning a new section
            sSectionName = rCell.Text
            iStartRow = iRow
        Else
            ' StartRow delimiter set, meaning we reached the end of a section
            iStopRow = iRow - 1
            ' Pass variables to a separate sub to create and save the new worksheet
            CopySheet osh, iFirstRow, iStartRow, iStopRow, iTotalRows, sFilePath, sSectionName, owb.fileFormat
            iCount = iCount + 1
            ' Reset section delimiters
            iStartRow = 0
            iStopRow = 0
            ' Ready to continue loop
            iRow = iRow - 1
        End If
    End If
    ' Continue until last row is reached
    If iRow < iTotalRows Then
            iRow = iRow + 1
    Else
        ' Finished. Save the last section
        iStopRow = iRow
        CopySheet osh, iFirstRow, iStartRow, iStopRow, iTotalRows, sFilePath, sSectionName, owb.fileFormat
        iCount = iCount + 1
        ' Exit
        Exit Do
    End If
Loop
'Turn On Screen Updating  Events
Application.ScreenUpdating = True
Application.EnableEvents = True
Application.DisplayAlerts = True
MsgBox Str(iCount) + " documents saved in " + sFilePath

End Sub
Public Sub DeleteRows(targetSheet As Worksheet, RowFrom As Long, RowTo As Long)
Dim rngRange As Range
Set rngRange = Range(targetSheet.Cells(RowFrom, 1), targetSheet.Cells(RowTo, 1)).EntireRow
rngRange.Select
rngRange.Delete
End Sub

Public Sub CopySheet(osh As Worksheet, iFirstRow As Long, iStartRow As Long, iStopRow As Long, iTotalRows As Long, sFilePath As String, sSectionName As String, fileFormat As XlFileFormat)
     Dim ash As Worksheet ' Copied sheet
     Dim awb As Workbook ' New workbook
     ' Copy book
     osh.Copy
     Set ash = Application.ActiveSheet
     ' Delete Rows after section
     If iTotalRows > iStopRow Then
         DeleteRows ash, iStopRow + 1, iTotalRows
     End If
     ' Delete Rows before section
     If iStartRow > iFirstRow Then
         DeleteRows ash, iFirstRow, iStartRow - 1
     End If
     ' Select left-topmost cell
     ash.Cells(1, 1).Select
     ' Clean up a few characters to prevent invalid filename
     sSectionName = Replace(sSectionName, "/", " ")
     sSectionName = Replace(sSectionName, "\", " ")
     sSectionName = Replace(sSectionName, ":", " ")
     sSectionName = Replace(sSectionName, "=", " ")
     sSectionName = Replace(sSectionName, "*", " ")
     sSectionName = Replace(sSectionName, ".", " ")
     sSectionName = Replace(sSectionName, "?", " ")
     ' Save in same format as original workbook
     ash.SaveAs sFilePath + "\Split\" + sSectionName, fileFormat
     ' Close
     Set awb = ash.Parent
     awb.Close SaveChanges:=False
End Sub

Изменено: direget - 27.09.2018 10:13:04
 
Цитата
direget написал:
Выдает теперь 400 ошибку...( Вы макросы запускаете отдельно, или через ALT+F8?
Запускал по разному.
Цитата
direget написал:
Книгу использую с поддержкой макросов...
макрос находится в стандартном модуле обрабатываемой книги?
Цитата
direget написал:
Вы используете на той таблице, которую я вам прислал или на другой?!
Файл использовал из сообщения №3, после запуска макроса, в InputBox оставлял значения уже указанные т.е. 2 и 5
макрос отработал корректно.
"Все гениальное просто, а все простое гениально!!!"
 
Nordheim, я правильно вставил строки, о которых вы говорили? или быть может что-то дописать нужно к ним?!
 
Вот этот файл у меня работает корректно.
"Все гениальное просто, а все простое гениально!!!"
 
Nordheim, может я коряво дописал, но уже не знаю в чем проблема?
 
А что с файлом из сообщения #16?
"Все гениальное просто, а все простое гениально!!!"
 
Nordheim, к сожалению, аналогично(!
Падает на
Строка 89 Set rngRange = Range(targetSheet.Cells(RowFrom, 1), targetSheet.Cells(RowTo, 1)).EntireRow
 
Аналог макроса №2
Код
Public Sub DeleteRows(targetSheet As Worksheet, RowFrom As Long, RowTo As Long)
       Range(targetSheet.Cells(RowFrom, 1), targetSheet.Cells(RowTo, 1)).EntireRow.Delete
End Sub
Изменено: Nordheim - 27.09.2018 11:30:06
"Все гениальное просто, а все простое гениально!!!"
 
Выдает ошибку на одном и том же месте
Range(targetSheet.Cells(RowFrom, 1), targetSheet.Cells(RowTo, 1)).EntireRow.Delete

Ошибка: Method 'Range' of object '_Worksheet' failed
 
Посмотрите в окне Locals, чему равны переменные.
"Все гениальное просто, а все простое гениально!!!"
 
Всем спасибо за помощь! Коллега помог разобраться...не хватало всего лишь Activesheet. перед Range!

Незаменимая вещь!!!!!!!!!!!!!!

А кто знает, как дописывать "...нарезка" к каждому файлу?!
 
И чтобы сохранял файлы при нарезке в формате .xls
 
Отдельно огромное спасибо за поддержку, Nordheim!
 
Цитата
direget написал:
не хватало всего лишь Activesheet.
Вот это странно, код без использования  Activesheet по умолчанию работает с активным листом, только если ваш макрос не помещен  в модуль листа.
"Все гениальное просто, а все простое гениально!!!"
 
direget, не нужно "разговаривать с собой", плодя сообщения. Можно ведь отредактировть предыдущее.
Страницы: 1
Наверх