Обращение к открытой книге

@Nsekins · Регистрация: 05.10.2020

Author24 — интернет-сервис помощи студентам

Всем доброго времени суток!
С помощью интернета я смог написать небольшой макрос, задача которого состоит в том, чтобы открыть определенную книгу и с ней сравнить активную.
Макрос работает, но никак не могу исправить одну строчку.

Visual Basic

Sub Sravni()
 
Dim i As Long
Set w2 = Workbooks.Open("Y:\Мин макс детали\Min_max.xlsx")
 
 
    Application.ScreenUpdating = False
    Workbooks("Spec.xlsx").Sheets(1).Activate
    With Workbooks("Min_max.xlsx").Sheets(1)
        Columns("B").Interior.ColorIndex = xlNone:
        .Columns("A").Interior.ColorIndex = xlNone:
        For i = 1 To Cells(Rows.Count, "A").End(xlUp).Row
            If Cells(i, "B") <> "" Then
                If Cells(i, "B") = .Cells(i, "A") Then
                    Cells(i, "B").Interior.ColorIndex = 6: Cells(i, "C") = 0
                    .Cells(i, "A").Interior.ColorIndex = 6: .Cells(i, "C") = 0
                End If
            End If
        Next
    End With
 
Workbooks("Min_max.xlsx").Close False
 
End Sub

Visual Basic

1	Workbooks("Spec.xlsx").Sheets(1).Activate

в этой строчке макрос берет определенную книгу, а мне нужно чтобы он работал с активной книгой и не привязывался к названию

@Punkt5 · 17.03.2021, 17:34

Сообщение от Nsekins

Workbooks("Spec.xlsx").Sheets(1).Activate

Ну, замените на

Код

ActiveWorkbook.Sheets(1).Activate

Но я бы советовал взять 2 переменные, в одну (наприvер назову её aWbSh1) записать проверяемый лист текущей активной книги: aWbSh1=ActiveWorkBook.Sheets(НомерНужногоЛиста)
а в другую (назову например MinMaxWbSh1) MinMaxWbSh1 = Workbooks("Min_max.xlsx").Sheets(1)
И дальше обращаясь к Cells указывать конкретно переменную нужного листа, на котором эта ячейка. Поверьте - гораздо удобнее будет и меньше придётся искать почему вдруг перестало работать, хоть до этого работало как часы на другой машине у другого пользователя.

@Alex77755 · 17.03.2021, 17:39

на коленке писано: (не проверено)

Visual Basic

Dim w1 As Workbook
Set w1 = ActiveWorkbook
 
    Set w2 = Workbooks.Open("Y:\Мин макс детали\Min_max.xlsx")
    Application.ScreenUpdating = False
    Workbooks(w1.Name).Sheets(1).Activate

Добавлено через 1 минуту
Punkt5,
после этого активной будет вновь открытая книга. ИМХО

Сообщение от Nsekins

Set w2 = Workbooks.Open

Добавлено через 1 минуту

Сообщение от Alex77755

Visual Basic

1	Workbooks(w1.Name).Sheets(1).Activate

и, скорей всего надо будет сначала активировать книгу, а потом лист

@Nsekins · 17.03.2021, 18:07 **[ТС]**

Сообщение от Punkt5

ActiveWorkbook.Sheets(1).Activate

Так я пробовал, у меня была ошибка

Сообщение от Alex77755

на коленке писано: (не проверено)

и все же оно работает. Спасибо большое, очень выручили!

@Zeag · 17.03.2021, 18:10

Зачем вообще активировать? Если открываем по указателям, активная вроде ж не изменится и останется на экране после обработки и закрытия того файла?
Select и Activate при обработке макросом - зло. Исключение - некоторые оконные операции, типа ActiveWindow.Zoom = 85

@Alex77755 · 17.03.2021, 18:36

Сообщение от Zeag

Зачем вообще активировать?

Zeag,
Справедливо! Но не на 100%
Некоторые методы не работают

@Zeag · 17.03.2021, 18:50

Alex77755, например? Я себе сохраню, чтобы в другой раз не рыпаться. Помню, что на ActiveWindow.Zoom = 85 нарвался и пока не прочел, не работало. А что еще?

@NewMaNew · 22.03.2021, 16:21

Еще ActiveWorkbook приходится применять после Workbooks(*).Sheets(*).Copy, когда создаётся некий Book1.

@Punkt5 малоболт 1314 / 498 / 211 Регистрация: 30.01.2020 Сообщений: 1,218
	17.03.2021, 17:34	2
	Сообщение от Nsekins Workbooks("Spec.xlsx").Sheets(1).Activate Ну, замените на Код ActiveWorkbook.Sheets(1).Activate Но я бы советовал взять 2 переменные, в одну (наприvер назову её aWbSh1) записать проверяемый лист текущей активной книги: aWbSh1=ActiveWorkBook.Sheets(НомерНужногоЛиста) а в другую (назову например MinMaxWbSh1) MinMaxWbSh1 = Workbooks("Min_max.xlsx").Sheets(1) И дальше обращаясь к Cells указывать конкретно переменную нужного листа, на котором эта ячейка. Поверьте - гораздо удобнее будет и меньше придётся искать почему вдруг перестало работать, хоть до этого работало как часы на другой машине у другого пользователя. 0

@Nsekins 0 / 0 / 0 Регистрация: 05.10.2020 Сообщений: 12
	17.03.2021, 18:07 [ТС]	4
	Сообщение от Punkt5 ActiveWorkbook.Sheets(1).Activate Так я пробовал, у меня была ошибка Сообщение от Alex77755 на коленке писано: (не проверено) и все же оно работает. Спасибо большое, очень выручили! 0

@Zeag 834 / 484 / 186 Регистрация: 09.03.2009 Сообщений: 1,664
	17.03.2021, 18:10	5
	Зачем вообще активировать? Если открываем по указателям, активная вроде ж не изменится и останется на экране после обработки и закрытия того файла? Select и Activate при обработке макросом - зло. Исключение - некоторые оконные операции, типа ActiveWindow.Zoom = 85 0

@Alex77755 11513 / 3799 / 681 Регистрация: 13.02.2009 Сообщений: 11,217
	17.03.2021, 18:36	6
	Сообщение от Zeag Зачем вообще активировать? Zeag, Справедливо! Но не на 100% Некоторые методы не работают 0

@Zeag 834 / 484 / 186 Регистрация: 09.03.2009 Сообщений: 1,664
	17.03.2021, 18:50	7
	Alex77755, например? Я себе сохраню, чтобы в другой раз не рыпаться. Помню, что на ActiveWindow.Zoom = 85 нарвался и пока не прочел, не работало. А что еще? 0

@NewMaNew 5 / 5 / 0 Регистрация: 02.09.2020 Сообщений: 32
	22.03.2021, 16:21	8
	Еще ActiveWorkbook приходится применять после Workbooks().Sheets().Copy, когда создаётся некий Book1. 0