#excel #vba #userform
#excel #vba #пользовательская форма
Вопрос:
Я пытаюсь заполнить два разных поля списка одним и тем же словарем сценариев, но это не работает.
У меня есть два вспомогательных модуля, которые я запускаю, один вызывает другой.
Вот исходный подраздел (рабочий код):
Option Explicit
Option Base 1
Sub NewWire()
Call PopulateClientName
NewWireForm.Show
End Sub
Sub PopulateClientName()
Dim FileName As String
Dim twb As Workbook, awb As Workbook
Dim aws As Worksheet, tws As Worksheet, ws As Worksheet
Dim rg1 As Range, rg2 As Variant
Dim rCell As Range
'//Clear combobox
DirectCalls.ClientName.Clear
ThirdParty.ClientName.Clear
Transfers.ClientName.Clear
'Open Master Client List
Set twb = ThisWorkbook
Set tws = ThisWorkbook.Worksheets("New")
tws.Visible = True
FileName = "C:UsersnleonDocumentsClient Master List.xlsx"
Application.ScreenUpdating = False
Workbooks.Open FileName
Set awb = ActiveWorkbook
Set aws = awb.Worksheets("Client Info")
Set ws = awb.Worksheets("Fund Info")
Set rg1 = aws.Range("A1").CurrentRegion
Set rg2 = ws.Range("A2").CurrentRegion
With CreateObject("Scripting.Dictionary")
For Each rCell In aws.Range("B2", aws.Cells(Rows.Count, "B").End(xlUp))
If Not .exists(rCell.Value) Then
.Add rCell.Value, Nothing
End If
Next rCell
DirectCalls.ClientName.List = .Keys
ThirdParty.ClientName.List = .Keys
Transfers.ClientName.List = .Keys
End With
Call PopulateClientNo(rg1, aws)
Call PopulateFund(rg2, ws)
awb.Close SaveChanges:=False
Application.ScreenUpdating = True
End Sub
Sub PopulateClientNo(rg1 As Range, aws As Worksheet)
Dim ws As Worksheet, wb As Workbook
Dim rCell As Range
'//Clear combobox
DirectCalls.ClientNo.Clear
ThirdParty.ClientNo.Clear
Transfers.ClientNo.Clear
With CreateObject("Scripting.Dictionary")
For Each rCell In aws.Range("C2", aws.Cells(Rows.Count, "C").End(xlUp))
If Not .exists(rCell.Value) Then
.Add rCell.Value, Nothing
End If
Next rCell
DirectCalls.ClientNo.List = .Keys
ThirdParty.ClientNo.List = .Keys
Transfers.ClientNo.List = .Keys
End With
End Sub
вот подзаголовок, который не работает:
Sub PopulateFund(rg2 As Variant, ws As Worksheet)
Dim dict As Object, d As Variant
Dim result As Long
Dim i As Integer, j As Integer, v As Variant
CapitalCalls.DCMFund.Clear
CapitalCalls.FundDesc.Clear
d = rg2.Value2
If Not IsArray(d) Then d = Array(d)
Debug.Print IsArray(d)
Set dict = CreateObject("scripting.dictionary")
On Error Resume Next
For i = 2 To UBound(d, 1)
For j = 2 To 2
If dict.exists(d(i, j)) = False Then
dict.Add Key:=d(i, j), Item:=d(i, j 1)
Else
result = result d(i, j)
End If
Next j
Next i
CapitalCalls.DCMFund.List = dict.Keys -----> this isn't actually working
CapitalCalls.FundDesc.List = dict.Item -----> this isn't actually working
End Sub
Внутри пользовательской формы, называемой «NewWireForm», существует следующий код:
Option Base 1
Private Sub CancelButton_Click()
Unload Me
End
End Sub
Private Sub UserForm_Initialize()
With Frame1
With .Font
.Bold = False
.Name = "Ariel"
.Size = 14
End With
End With
End Sub
Private Sub NextButton_Click()
If TPExpense = True Then
ThirdParty.Show
ElseIf DCMCC = True Then
CapitalCalls.Show
ElseIf DirectCC = True Then
DirectCalls.Show
ElseIf Transfer = True Then
Transfers.Show
Else: MsgBox ("You have not selected a wire type. Please try again!.")
End If
End Sub
Я ожидаю, что списки в пользовательской форме «capital call» будут заполнены ключами и элементами из словаря, но оно отображается пустым.
Есть идеи, почему?
Комментарии:
1. Для чего это
On Error Resume Next
?2. … если вы удалите его, что произойдет при запуске вашего кода?
3. Вы должны добавить элементы в том же цикле; в поле списка есть . Метод AddItem , который должен быть вызван перед добавлением другого элемента; например (.AddItem: .List(.listCount-1) = «texthere»).
4. @TimWilliams ошибки нет, я вижу через окно locals, что элементы добавляются в словарь. Это заполнение списка, которое вызывает проблему.
5. Взгляните на это rubberduckvba.wordpress.com/2017/10/25/userform1-show и посмотрите, поможет ли изменение вашего подхода с использованием этих рекомендаций
Ответ №1:
Следуя ссылке, которую я разместил выше:
Sub UserFormStuff()
'setup: "UserForm1" is a userform with 2 textboxes
'this code relies on the "default instance"
UserForm1.TextBox1 = "Test1"
Test2
UserForm1.Show '<< only textbox1 has content
'this code uses an explicit instance of the userform
Dim uf As UserForm1
Set uf = New UserForm1
uf.TextBox1 = "Test1 new"
Test2_new uf '<< pass our explicit instance to Test2_new
uf.Show '<< now both textboxes are populated
End Sub
Sub Test2()
UserForm1.TextBox2 = "Test2" 'this just creates a *new* UserForm1
' which is not the same as the copy used
' in the sub which calls this
End Sub
Sub Test2_new(uf As UserForm1)
uf.TextBox2 = "Test2" '<< use the provided instance
End Sub
Комментарии:
1. но пользовательская форма («CapitalCalls») содержит 2 списка со списком (один называется «DCMFund», а другой — «FundDesc»), и он (что означает пользовательскую форму) уже существует. Я не следую вашему примеру и не понимаю сообщение, которое вы пытаетесь передать с помощью примера.
2.При
Test2
вызове оно не использует то жеUserForm1
, что используется вUserFormStuff
— оно «автоматически создает» новый экземпляр формы пользователя и изменяет этот экземпляр: когда Test2 завершается, этот новый экземпляр просто уничтожается, а экземпляр, используемый вUserFormStuff
, остается незатронутым.3. думаю, теперь я понял. вы предполагаете, что экземпляр по умолчанию — это то, что облажалось, правильно?
4. Да, именно это пытается охватить ссылка, которую я опубликовал — никогда не используйте экземпляр по умолчанию: объявите переменную для формы пользователя и явно создайте экземпляр формы для этой переменной.
5. итак, я добавил [Dim UF1 как пользовательскую форму] и установил UF1 = CapitalCalls, заменил все «CapitalCalls» в коде на UF1 и по-прежнему ничего -_-