Неверная работа On Error, он ругается а не переходит.

@P_u_n_i_s_h_e_r · Регистрация: 24.04.2010

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

В столбце A находятся даты, подряд, 01.01.05, 02.01.05 и т.д.
Ниже приведенный фрагмент кода, перебирает некий период, заданный пользователем и вставляет по указанной дате курс баксов в столбец "D"
Что хотелось сделать, чтобы когда в столбце A отсуствует какая-то дата, то прога ее туда добавляет, если же отсустсвует целый диапазон, то добавляет этот диапазон.
Т.е. при ошибке поиска прыгает на NewDay, а если и в NewDay тоже ничего не найдено, то прыгает на NewPeriod.
По работе, после отработки первого NewDay, второй раз по On Error уже не прыгает, просто ругается (как обычно при неудачном поиске). Никогда не переходит на On Error Goto NewPeriod. Как это можно исправить?

Visual Basic
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
Do While myDate <= dtEnd
    
    On Error GoTo NewDay
    Columns("A:A").Select
    Selection.Find(What:=myDate, After:=ActiveCell, LookIn:=xlFormulas, _
    LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlNext, _
    MatchCase:=False, SearchFormat:=False).Activate
    
    Cells(ActiveCell.Row, 5) = SetRate(CStr(myDate))
    GoTo Continue
 
''Если дата не была найдена, необходимо ее добавить
NewDay:
    tmpDate = myDate - 1
    On Error GoTo NewPeriod ''Не пашет, нужно разбираться с On Error
    Selection.Find(What:=tmpDate, After:=ActiveCell, LookIn:=xlFormulas, _
    LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlNext, _
    MatchCase:=False, SearchFormat:=False).Activate
    
    MyRange = CStr(ActiveCell.Row + 1) + ":" + CStr(ActiveCell.Row + 1)
    Rows(MyRange).Select
    Selection.Insert Shift:=xlDown
    
    ''Получение нового курса из процедуры SetRate
    ActiveCell = myDate
    Cells(ActiveCell.Row, 5) = SetRate(CStr(myDate))
    GoTo Continue
    
''Если это новый период, а не просто пропущенная одна дата)
NewPeriod:
    Selection.Find(What:=Null, After:=ActiveCell, LookIn:=xlFormulas, _
    LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlNext, _
    MatchCase:=False, SearchFormat:=False).Activate
 
    ActiveCell = Date
    Cells(ActiveCell.Row, 5) = SetRate(CStr(myDate))
 
 
Continue:
myDate = myDate + 1
Loop

Johny Walker · 01.01.2012, 16:36

Если я не ошибаюсь, после возникновения ошибки и ее обработки перед переходом к обработке новой ошибки предыдущую нужно сбросить: Err.Clear
Не вдаваясь в логику, все же кажется, что алгоритм с использованием обработчика ошибок не совсем верный. IF THEN ELSE и им подобные никто, к счастью, не отменял :-)

@Alexxander · 02.01.2012, 13:07

Вообще-то Find возвращает значение.

Finds specific information in a range, and returns a Range object that represents the first cell where that information is found. Returns Nothing if no match is found.

И не нужно никакого On Error

@P_u_n_i_s_h_e_r · 02.01.2012, 15:12 **[ТС]**

Err.Clear не помогает получается все также.
Во втором случае можно использовать IF, вполне, я использовал On Error специально чтобы убедиться что он не работает, т.е. не передает новое значение On Error.
Более того, получается, что он обрабатывает первое вхождение в ошибку (не находит например 03 число, вставляет его по On Error, а когда в следующий раз не находит например 10 (в том же цикле), то уже рубится на нем.

Вопрос к второму ответившему гражданину. Окей, хелп не смотрел, но пусть так, тогда раскажите плиз как найти каую-то величину, пример:
В столбце А содержатся несколько значений A1=100, A2=200, A3="abc", A4="asd".

Если я напишу вот так:

Visual Basic
1
2
3
4
5
6
7
8
9
10
Dim MyVar As String
 
MyVar = "abc"
 
Columns("A:A").Select
Selection.Find(What:=MyVar, After:=ActiveCell, LookIn:=xlFormulas, _
LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlNext, _
MatchCase:=False, SearchFormat:=False).Activate
 
MsgBox ActiveCell.Row

То все будет окей, пока MyVar будет равно какому-то из перечисленных выше значений, а если я напишу, например: MyVar="bla-bla-bla", то код выдаст ошибку, т.к. значение "bla-bla-bla" в столбце A отсутствует. Ну и как с этим быть? Где ловить Nothing ?

@Alexxander · 02.01.2012, 15:52

Пример из того же хелпа:

This example finds all cells in the range A1:A500 on worksheet one that contain the value 2 and changes it to 5.

Visual Basic
1
2
3
4
5
6
7
8
9
10
With Worksheets(1).Range("a1:a500")
    Set c = .Find(2, lookin:=xlValues)
    If Not c Is Nothing Then
        firstAddress = c.Address
        Do
            c.Value = 5
            Set c = .FindNext(c)
        Loop While Not c Is Nothing And c.Address <> firstAddress
    End If
End With

Т.е. для данного случая:

Visual Basic
1
2
3
4
5
6
...
Set c = Selection.Find(What:=MyVar, After:=ActiveCell, LookIn:=xlFormulas, _
LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlNext, _
MatchCase:=False, SearchFormat:=False)
If Not c Is Nothing Then
...

@P_u_n_i_s_h_e_r · 05.01.2012, 14:09 **[ТС]**

По вышеописаному примеру, при установке значения, которое присутствует в столбце, строка физически находится, но выдается ошибка Type Missmatch.
А если поставить значение, отсутствующее в столбце, то выдается ошибка 91, Object variable or With block variable not set.

Собственно у меня так и было раньше, поэтому по On Error я и работал

@Alexxander · 05.01.2012, 15:06

Visual Basic
1
2
3
4
5
6
7
8
9
Sub exp()
Dim MyVar As String
MyVar = "abc"
Columns("A:A").Select
Set c = Selection.Find(What:=MyVar, After:=ActiveCell, LookIn:=xlFormulas, _
LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlNext, _
MatchCase:=False, SearchFormat:=False)
If Not c Is Nothing Then c.Select
End Sub

у меня работает без всяких ошибок.
Прикрепи свой файл, посмотрю.

@Dimakart · 07.01.2012, 19:54

Не вдаваясь в логику, я всегда использую - On error goto 0.
Т. е. прежде чем цикл ВНОВЬ подойдет к строке On Error GoTo NewDay
Должен быть disable предыдущий error handler, навскидку перед Loop.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

Johny Walker 2 / 2 / 1 Регистрация: 10.04.2011 Сообщений: 415
	01.01.2012, 16:36
	Если я не ошибаюсь, после возникновения ошибки и ее обработки перед переходом к обработке новой ошибки предыдущую нужно сбросить: Err.Clear Не вдаваясь в логику, все же кажется, что алгоритм с использованием обработчика ошибок не совсем верный. IF THEN ELSE и им подобные никто, к счастью, не отменял :-) 0

@P_u_n_i_s_h_e_r 0 / 0 / 0 Регистрация: 24.04.2010 Сообщений: 29
	05.01.2012, 14:09 [ТС]
	По вышеописаному примеру, при установке значения, которое присутствует в столбце, строка физически находится, но выдается ошибка Type Missmatch. А если поставить значение, отсутствующее в столбце, то выдается ошибка 91, Object variable or With block variable not set. Собственно у меня так и было раньше, поэтому по On Error я и работал 0

@Dimakart 0 / 0 / 1 Регистрация: 11.10.2010 Сообщений: 48
	07.01.2012, 19:54
	Не вдаваясь в логику, я всегда использую - On error goto 0. Т. е. прежде чем цикл ВНОВЬ подойдет к строке On Error GoTo NewDay Должен быть disable предыдущий error handler, навскидку перед Loop. 0