закрытие формы - в чем ошибка?

[Bape}l{ka]

привет всем))
подскажите пожалуйста, есть код, но почему-то на закрытие формы выдает ошибку "Run-time error 1004: Введено недопустимое имя" и отсылка идет на саб, который должен выполняться, если форма все еще открыта. в чем может быть дело??

Sub FinishVvod()
            k = 0
            Do
                If MsgBox("Новый респондент?", vbYesNo) = vbNo Then
                    If MsgBox("Завершить ввод данных?", vbYesNo) = vbYes Then
                        Unload Me
                        ActiveWorkbook.Save 'сохранение книги
                        Exit Sub
                    Else: k = 1
                    End If
                Else
                    k = 0
                    Colm_ = 1
                    Roww_ = Roww_ + 1
                    id_ = id_ + 1
                    Cells(Roww_, 1).Value = id_
                    Exit Do
                End If
            Loop While k = 1
End Sub

[EducatedFool]

Unload Me
ActiveWorkbook.Save 'сохранение книги

Переставьте эти строки местами

Или вместо Unload Me
напишите Me.Hide

[Bape}l{ka]

ты же катавасия..
почему-то продолжают выполняться действия для этой формы (другой саб этой формы), как-будто она не закрылась О.о ничего не понимаю

[Bape}l{ka]

в итоге просто в другом сабе поставила проверку, но все-же не понятно, с какой стати он перескакивает отсюда туда при том, что тут есть команда "Exit Sub"

[nerv]

Bape}l{ka,
1. Откуда вызывается данная процедура?
2. Почему не объявлены переменные? Неужели они все приватные/публичные?

[SAS888]

Если выполняемая процедура находится в программном модуле, то ключевое слово "Me" относится не к открытой форме. Поэтому, к форме, которую требуется закрыть, нужно обращаться по имени. Например:

Unload UserForm1

[Bape}l{ka]

nerv
спасибо, подсказали))
я ж этот саб вызываю из другой процедуры, соответственно он выходит из этого и возвращается обратно.
а подскажите, есть ли такая команда, которая прекращала бы выполнение макроса вообще?? без возвращений к предыдущим процедурам =ь

п.с.: наверно проще объединить сабы
п.п.с.: да, тут все переменные публичные - это плохо?

[nerv]

Сообщение от Bape}l{ka

п.п.с.: да, тут все переменные публичные - это плохо?

Мое мнение - чем меньше публичных переменных, тем лучше.

[SAS888]

есть ли такая команда, которая прекращала бы выполнение макроса вообще?? без возвращений к предыдущим процедурам

Если из одной процедуры вызывать другую, то после выполнения команды "Exit Sub" будет осуществлен переход к следующей команде того макроса (процедуры), из которого мы вызывали данный. Поэтому, для того, чтобы завершить выполнение каких-либо дальнейших действий, Вы должны остановить их в вызывающей процедуре. Либо безусловно, либо по условию. Это зависит от того, что конкретно требуется.

[Bape}l{ka]

SAS888, спасибо, я поняла, в итоге сделала FinishVvod не сабом а функцией Boolean, и уже в главной процедуре осуществляю выход)))

nerv, я думала, что может существуют какие-то объективные причины... а так, мне гораздо удобнее с публичными переменными) конечно, специфические я объявляю. а постоянно используемые так и висят))