acdx

帖子: 132
主題: 56
精華: 0
積分: 190
點名: 0
作業系統: Win10
軟體版本: Office 365
閱讀權限: 20
性別: 男
註冊時間: 2012-5-17
最後登錄: 2024-4-29

1^# 跳轉到 » 倒序看帖

打印

字體大小: tT

發表於 2013-8-12 17:33 | 只看該作者

[發問] 簡化程式

我利用Userform做計算時需要選定要計算的欄位
故設定每一個section的起始欄位-"Start Row"及"End Row"
且在Critera存在時才可輸入起始欄位,當它為"N/A"時起使欄位則不可輸入

於是就寫了一長串的code
想問大家同樣的問題會怎麼寫可以比較簡化?
謝謝
'------------------------------------------
If TextBox3.Locked = True Then
CC = 1
Else
If TextBox5.Locked = True Then
      CC = 2
Else
      If TextBox7.Locked = True Then
         CC = 3
      Else
         If TextBox9.Locked = True Then
            CC = 4
         Else
            If TextBox11.Locked = True Then
                  CC = 5
            End If
         End If
      End If
End If
End If

iC = 1
Do Until iC > CC
If iC = 1 Then
      StartRow = Val(TextBox1.Text)
      EndRow = Val(TextBox2.Text)
End If
If iC = 2 Then
      StartRow = Val(TextBox3.Text)
      EndRow = Val(TextBox4.Text)
End If
If iC = 3 Then
      StartRow = Val(TextBox5.Text)
      EndRow = Val(TextBox6.Text)
End If
If iC = 4 Then
      StartRow = Val(TextBox7.Text)
      EndRow = Val(TextBox8.Text)
End If
If iC = 5 Then
      StartRow = Val(TextBox9.Text)
      EndRow = Val(TextBox10.Text)
End If
If iC = 6 Then
      StartRow = Val(TextBox11.Text)
      EndRow = Val(TextBox12.Text)
End If
中間計算程式

iC = iC + 1
Loop
'----------------------------------

樓主熱帖

acdx

中學生 Rank: 2

帖子: 132
主題: 56
精華: 0
積分: 190
點名: 0
作業系統: Win10
軟體版本: Office 365
閱讀權限: 20
性別: 男
註冊時間: 2012-5-17
最後登錄: 2024-4-29

2^#

發表於 2013-8-12 17:37 | 只看該作者

回復 1# acdx
上面沒講清楚
我在Sub ComboBox1_Change裡有設定當ComboBox1.Value 不等於"N/A"(即C1~C5)時 TextBox才可輸入否則會被Locked
Private Sub ComboBox1_Change()
If ComboBox1.Value = "N/A" Then
TextBox1.Locked = True
TextBox2.Locked = True
TextBox1.BackColor = &H8000000A
TextBox2.BackColor = &H8000000A
Else
TextBox1.Locked = False
TextBox2.Locked = False
TextBox1.BackColor = &H8000000B
TextBox2.BackColor = &H8000000B
End If
End Sub

TOP

stillfish00

大學生 Rank: 4

帖子: 1018
主題: 15
精華: 0
積分: 1058
點名: 0
作業系統: win7 32bit
軟體版本: Office 2016 64-bit
閱讀權限: 50
性別: 男
來自: 桃園
註冊時間: 2012-5-9
最後登錄: 2022-9-28

3^#

發表於 2013-8-12 20:18 | 只看該作者

Dim myControl(1 To 6, 1 To 3) As Control
'設定控制項陣列
For i = 1 To UBound(myControl)
Set myControl(i, 1) = Controls("ComboBox" & i)
Set myControl(i, 2) = Controls("TextBox" & 2 * i - 1)
Set myControl(i, 3) = Controls("TextBox" & 2 * i)
Next
For iC = 1 To UBound(myControl)
If myControl(iC, 2).Locked Then Exit For
StartRow = Val(myControl(iC, 2).Value)
EndRow = Val(myControl(iC, 3).Value)
'中間計算程式
Next

複製代碼

回復 1# acdx

TOP

sunnyso

高中生 Rank: 3 Rank: 3

帖子: 605
主題: 92
精華: 0
積分: 648
點名: 0
作業系統: windows
軟體版本: 7
閱讀權限: 50
性別: 男
來自: macau
註冊時間: 2013-4-5
最後登錄: 2019-2-10

熱烈參與勳章好學求知勳章

4^#

發表於 2013-8-13 13:13 | 只看該作者

回復 1# acdx

可以用select case來代替if else if

ss

TOP

acdx

中學生 Rank: 2

帖子: 132
主題: 56
精華: 0
積分: 190
點名: 0
作業系統: Win10
軟體版本: Office 365
閱讀權限: 20
性別: 男
註冊時間: 2012-5-17
最後登錄: 2024-4-29

5^#

發表於 2013-8-14 11:45 | 只看該作者

回復 3# stillfish00
太感謝了這種寫法是我所沒嘗試過的
另外我想設定成第一個Critera有選擇時(即ComboBox1不等於N/A)下一個ComboBox2才可以選擇
該如何寫呢?

TOP

stillfish00

大學生 Rank: 4

帖子: 1018
主題: 15
精華: 0
積分: 1058
點名: 0
作業系統: win7 32bit
軟體版本: Office 2016 64-bit
閱讀權限: 50
性別: 男
來自: 桃園
註冊時間: 2012-5-9
最後登錄: 2022-9-28

6^#

發表於 2013-8-14 17:15 | 只看該作者

本帖最後由 stillfish00 於 2013-8-14 17:25 編輯

回復 5# acdx
簡單的做法就是在你2#的Change事件中判斷就可以了，因為你只有6個ComboBox還算少，只要寫6個ComboBoxN_Change()就好了。

若是數十、數百個控制項，事件執行動作類似，可考慮用物件類別模組(較複雜)去定義共通的事件：
新增一物件類別模組，命名為clsComboBoxMyEvents

Public WithEvents myComboBox As MSForms.ComboBox
Private cbForLock As MSForms.ComboBox
Public Sub setObject(ByRef ctl As MSForms.ComboBox)
Set myComboBox = ctl
End Sub
Public Sub setLinkForLock(ByRef ctl As MSForms.ComboBox)
Set cbForLock = ctl
End Sub
Private Sub myComboBox_Change()
If cbForLock Is Nothing Then Exit Sub
If myComboBox.Value = "N/A" Then
cbForLock.Enabled = False
Else
cbForLock.Enabled = True
End If
End Sub

複製代碼

你的表單:

Private Const COMBOBOX_COUNT = 6
Dim myCombo(1 To COMBOBOX_COUNT) As clsComboBoxMyEvents
Private Sub UserForm_Initialize()
'............................
'加上下面這段
For i = 1 To COMBOBOX_COUNT
Set myCombo(i) = New clsComboBoxMyEvents
myCombo(i).setObject Controls("ComboBox" & i)
If i < COMBOBOX_COUNT Then myCombo(i).setLinkForLock Controls("ComboBox" & i + 1)
Next i
End Sub

複製代碼

大概類似這樣，我也不是很常用。

TOP

acdx

中學生 Rank: 2

帖子: 132
主題: 56
精華: 0
積分: 190
點名: 0
作業系統: Win10
軟體版本: Office 365
閱讀權限: 20
性別: 男
註冊時間: 2012-5-17
最後登錄: 2024-4-29

7^#

發表於 2013-8-14 18:21 | 只看該作者

回復 6# stillfish00
太感謝了
這裡讓我學到太多東西了

TOP