Оцените плиз мой первый макрос. Что можно подкрутить - VB
Формулировка задачи:
Добрый день,
написал свой первый макрос, хотелось бы узнать мнение профи, чтобы сразу избежать ошибок и на будущее запомнить это.
Задача:
есть заполненный столбец в листе, необходимо для каждой записи из этого столбца подтянуть некоторые другие данные из БД.
Делаю:
коннект к БД и в цикле работаю с этим листом, вызывая хранимую процедуру и каждый раз передавая ей в качестве параметра значение из этого столбца. Все,что возвращает БД, помещаю в recordset и оттуда вставляю в лист.
Во время выборки максимальной даты придумал временное решение: засунуть в ячейку, потом в переменную и очистить ячейку.
Вот листинг макроса:
спасибо всем большое.
написал свой первый макрос, хотелось бы узнать мнение профи, чтобы сразу избежать ошибок и на будущее запомнить это.
Задача:
есть заполненный столбец в листе, необходимо для каждой записи из этого столбца подтянуть некоторые другие данные из БД.
Делаю:
коннект к БД и в цикле работаю с этим листом, вызывая хранимую процедуру и каждый раз передавая ей в качестве параметра значение из этого столбца. Все,что возвращает БД, помещаю в recordset и оттуда вставляю в лист.
Во время выборки максимальной даты придумал временное решение: засунуть в ячейку, потом в переменную и очистить ячейку.
Вот листинг макроса:
спасибо всем большое.
Решение задачи: «Оцените плиз мой первый макрос. Что можно подкрутить»
textual
Листинг программы
' промежуточное сохранение на лист - лишнее: ' Call ActiveSheet.Cells(100, 100).CopyFromRecordset(rst2) ' rep_date = ActiveSheet.Cells(100, 100).Value ' ActiveSheet.Cells(100, 100).Clear rep_date = rst2(0).Value
ИИ поможет Вам:
- решить любую задачу по программированию
- объяснить код
- расставить комментарии в коде
- и т.д