Как упростить код с помощью LOOPS в VBA?

Question

Во-первых, добро пожаловать.
Во-вторых, первое, чего вы могли бы легко избежать, это повторение одного и того же ifснова и снова.

Таким образом, вместо того, чтобы проверять If Application.WorksheetFunction.IsText(Worksheets("Investor Data").range("E2"))13 раз и каждый раз выполнять одно действие, вы можете просто проверить один раз и выполнить все 13 действий.

Затем, конечно, наступает та часть, о которой вы спрашиваете, циклирование.
Есть много мест, где можно почитать о том, как циклировать вещи разными способами, но пока у нас есть шаблон, мы можем сделать цикл.

Итак, для первой части, E2у вас может быть такой цикл;

If Application.WorksheetFunction.IsText(Worksheets("Investor Data").Range("E2")) = True Then
j = 114
For i = 3 To 15
    Worksheets("Investor Data").Range("E" & i).copy Worksheets("Data Base").Range("C" & j & ":C" & j + 6)
    j = j + 7
Next i
End If

Это будет выполнено в соответствии с E3-E15 for loop, путем замены номера строки на используемую переменную.(Заметка: разве вам не следует скопировать и E2?) Вероятно, есть еще более эффективные способы сделать это, но это пример простого цикла.

Поскольку количество строк в каждом столбце разное, мы не можем просто включить это в больший цикл и на этом закончить.

Вы можете сделать несколько меньших петель, но я оставляю это на ваше усмотрение. Удачи.

редактировать

В одном из комментариев говорится о функциях, которые являются одним из способов делать одно и то же много раз, но с гораздо меньшим количеством кода.

Sub callCopy()

Dim startRow As Long
startRow = 114
Call copySub(startRow, "E", 15)
Call copySub(startRow, "F", 5)
Call copySub(startRow, "G", 6)
Call copySub(startRow, "H", 13)
Call copySub(startRow, "I", 5)
Call copySub(startRow, "J", 8)
Call copySub(startRow, "K", 11)
Call copySub(startRow, "L", 7)

End Sub

Эта первая подпрограмма вызывает вторую приватную подпрограмму. Это могла бы быть функция, но они используются, когда вы хотите что-то вернуть. Вместо этого мы используем приватную подпрограмму:

Private Sub copySub(startRow As Long, iCol As String, iRows As Long)
Dim i As Long
If Application.WorksheetFunction.IsText(Worksheets("Investor Data").Range(iCol & "2")) = True Then
For i = 2 To iRows
    Worksheets("Investor Data").Range(iCol & i).copy Worksheets("Data Base").Range("C" & startRow & ":C" & startRow + 6)
    startRow = startRow + 7
Next i
End If

End Sub

Вторая подпрограмма — это всего лишь мой первый пример, но с большим количеством переменных. Затем они устанавливаются в скобках при вызове подпрограммы.
Я заменил фиксированные числа для столбца C в базе данных на переменную, поэтому вычисления будут проще. Но вы можете просто изменить их на фиксированное число при вызове, если это больше подходит для ваших нужд.

Answer 1