返回列表 上一主題 發帖

[發問] 簡化程式

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

帖子
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

帖子
1018 
主題
15 
精華
0 
積分
1058 
點名
0  
作業系統
win7 32bit 
軟體版本
Office 2016 64-bit 
閱讀權限
50 
性別
男 
來自
桃園 
註冊時間
2012-5-9 
最後登錄
2022-9-28 
3#
發表於 2013-8-12 20:18 | 只看該作者
  1.   Dim myControl(1 To 6, 1 To 3) As Control
  2.   
  3.   '設定控制項陣列
  4.   For i = 1 To UBound(myControl)
  5.     Set myControl(i, 1) = Controls("ComboBox" & i)
  6.     Set myControl(i, 2) = Controls("TextBox" & 2 * i - 1)
  7.     Set myControl(i, 3) = Controls("TextBox" & 2 * i)
  8.   Next
  9.   
  10.   For iC = 1 To UBound(myControl)
  11.     If myControl(iC, 2).Locked Then Exit For
  12.    
  13.     StartRow = Val(myControl(iC, 2).Value)
  14.     EndRow = Val(myControl(iC, 3).Value)
  15.    
  16.     '中間計算程式
  17.   Next
複製代碼
回復 1# acdx

TOP

sunnyso

帖子
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

帖子
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

帖子
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
  1. Public WithEvents myComboBox As MSForms.ComboBox
  2. Private cbForLock As MSForms.ComboBox

  4. Public Sub setObject(ByRef ctl As MSForms.ComboBox)
  5.   Set myComboBox = ctl
  6. End Sub

  8. Public Sub setLinkForLock(ByRef ctl As MSForms.ComboBox)
  9.   Set cbForLock = ctl
  10. End Sub

  12. Private Sub myComboBox_Change()
  13.   If cbForLock Is Nothing Then Exit Sub
  14.   
  15.   If myComboBox.Value = "N/A" Then
  16.     cbForLock.Enabled = False
  17.   Else
  18.     cbForLock.Enabled = True
  19.   End If
  20. End Sub
複製代碼
你的表單:
  1. Private Const COMBOBOX_COUNT = 6
  2. Dim myCombo(1 To COMBOBOX_COUNT) As clsComboBoxMyEvents

  4. Private Sub UserForm_Initialize()

  6.   '............................
  7.   
  8.   '加上下面這段
  9.   For i = 1 To COMBOBOX_COUNT
  10.     Set myCombo(i) = New clsComboBoxMyEvents
  11.     myCombo(i).setObject Controls("ComboBox" & i)
  12.     If i < COMBOBOX_COUNT Then myCombo(i).setLinkForLock Controls("ComboBox" & i + 1)
  13.   Next i
  14. End Sub
複製代碼
大概類似這樣，我也不是很常用。

TOP

acdx

帖子
132 
主題
56 
精華
0 
積分
190 
點名
0  
作業系統
Win10 
軟體版本
Office 365 
閱讀權限
20 
性別
男 
註冊時間
2012-5-17 
最後登錄
2024-4-29 
7#
發表於 2013-8-14 18:21 | 只看該作者
回復 6# stillfish00
太感謝了
這裡讓我學到太多東西了

TOP

        靜思自在 : 生氣，就是拿別人的過錯來懲罰自己。
返回列表 上一主題