麻辣家族討論版版 - Powered by Discuz! Board

標題: [發問] 昨天開始接觸VBA，想簡化程式... [打印本頁]

作者: bsy4life 時間: 2016-6-2 16:26 標題: 昨天開始接觸VBA，想簡化程式...

大大們好！
從這裡找到非常有用的資訊
拼拼貼貼之下完成了一個簡單的程式
現在有一個欄位工號需要做比對，比對正確後轉換成姓名帶入combobox
原本用case select完成，可是50筆資料我要一筆一筆輸入...實在眼花
程式如下
If IsNumeric(TextBox6.Text) Then
   Dim num As Integer
   num = TextBox6.Text
Else
      If Not IsNumeric(TextBox6.Text) Then
      MsgBox "欄位輸入必須為數值"
      End If
End If
Set y = Sheets("CNC名單(All1)")
Select Case num
Case "1908"
ComboBox1.Value = "黃xx"
Case "2029"
ComboBox1.Value = "陳xx"
Case "2066"
ComboBox1.Value = "林xx"
Case "2145"
ComboBox1.Value = "陳yy"
Case "2167"
ComboBox1.Value = "陳aa"
Case "2645"
ComboBox1.Value = "劉bb"
Case "2732"
ComboBox1.Value = "洪cc"

目前有一個sheet專門放這些清單
所以想從A2開始比對工號，比對正確後將此工號的B欄位姓名帶入combobox1的value
請問該如何進行~~~
[attach]24388[/attach]

有試過這樣寫
If IsNumeric(TextBox6.Text) Then
   Dim num As Integer
   num = TextBox6.Text
Else
      If Not IsNumeric(TextBox6.Text) Then
      MsgBox "欄位輸入必須為數值"
      End If
End If
Set y = Sheets("CNC名單(All1)")
Select Case num
Case num
ComboBox1.Value =  y.Range("B2", y.[B2].End(xlDown)).Value
-------------------------------------------------------------------------
腦袋實在有點打結，這樣寫法有問題嗎?
拜託各位大大指點~~~

作者: luhpro 時間: 2016-6-2 22:32

本帖最後由 luhpro 於 2016-6-2 22:34 編輯

回復 1# bsy4life
基本上對於一對一對應且不會重複的情形,
用 Dictionary 是一種滿容易達到需求的方式 :

[attach]24399[/attach]

== Module==

Public dPeo

複製代碼

== ThisWorkbook ==

Private Sub Workbook_Open()
Dim lRow&
Set dPeo = CreateObject("Scripting.Dictionary")
lRow = 2
With Sheets("Sheet1")
While .Cells(lRow, 1) <> ""
dPeo(CStr(.Cells(lRow, 1))) = .Cells(lRow, 2)
lRow = lRow + 1
Wend
End With
ufMain.Show
End Sub

複製代碼

== UserForm ==

Private Sub cbNo_Change()
lbName.Caption = dPeo(cbNo.Text)
End Sub
Private Sub UserForm_Initialize()
Dim vTmp
For Each vTmp In dPeo
cbNo.AddItem vTmp
Next
cbNo.ListIndex = 0
End Sub

複製代碼

時間因素這裡只能先簡易的表現 : 建立資料對應與查詢實現的方式,
其他就請自己酌予修改套用囉...

[attach]24400[/attach]

作者: GBKEE 時間: 2016-6-3 08:19

本帖最後由 GBKEE 於 2016-6-3 08:22 編輯

回復 1# bsy4life
luhpro 提供的Dictionary 最簡便,也可用工作表函數 CountIf,Match 替代

Private Sub TextBox6_Change()
Dim xNo As Variant, y As Range
Set y = Sheets("CNC名單(All1)").Range("a:a")
xNo = Application.CountIf(y, Val(TextBox6)) '傳回TextBox6相同的內容在y 的個數
Select Case xNo
Case 1
'luhpro 上傳的工號索引.xls 名單內的儲存格有些是文字格式
'y.Value = y.Value '轉為數字格式
xNo = Application.Match(Val(TextBox6), y, 0) '傳回TextBox6在y 的列號
ComboBox1 = y.Cells(xNo, "b")
Case 0
ComboBox1 = ""
Case Is > 1
ComboBox1 = "工號重複"
End Select
End Sub

複製代碼

作者: bsy4life 時間: 2016-6-3 09:43

回復 2# luhpro

謝謝 luhpro 大大
我的資質駑鈍，GBKEE大大的方式解決了我的問題
可是還是想瞭解一下，Dictionary的用法
可是我在置換後，執行程式發生
我才輸入一個數值
下面這行發生型態不符合
Label8.Caption = dPeo(TextBox6.Text)

module

Public dPeo
Sub 新增資料()
fcckeyin.Show
End Sub

複製代碼

ThisWorkBook

Private Sub Workbook_Open()
Dim lRow&
Dim dPeo
Set dPeo = CreateObject("Scripting.Dictionary")
lRow = 2
With Sheets("CNC名單(All)")
While .Cells(lRow, 1) <> ""
dPeo(CStr(.Cells(lRow, 1))) = .Cells(lRow, 2)
lRow = lRow + 1
Wend
End With
fcckeyin.Show
End Sub

複製代碼

userform

Private Sub TextBox6_Change() '輸入工號讀取操作人
TextBox6.IMEMode = fmIMEModeOff
If IsNumeric(TextBox6.Text) Then
Dim num As Integer
num = TextBox6.Text
Else
If Not IsNumeric(TextBox6.Text) Then
MsgBox "欄位輸入必須為數值"
End If
End If
Label8.Caption = dPeo(TextBox6.Text) '本來使用combobox，不知道如何取代所以用label測試
End Sub
Private Sub fcckeyin_Initialize()
Dim vTmp
For Each vTmp In dPeo
TextBox6.Text = vTmp
Next
'本來此行是cbNo.ListIndex = 0 可是我是用textbox，所以此行拿掉
End Sub

複製代碼

請問是不是我的寫法有誤?
謝謝各位大大~~
也謝謝GBKEE幫我解決問題唷~~!!

作者: bsy4life 時間: 2016-6-3 10:27

回復 3# GBKEE

不好意思，雖然問題解決了
但對這段程式碼還是沒有很理解

Dim xNo As Variant, y As Range '這邊為什麼xNo要設定成 Variant類型呢?
Set y = Sheets("CNC名單(All1)").Range("a:a") '這裡的a:a 不需要定義嗎? 那它代表的值為何?所有A行A列的資料嗎~~
xNo = Application.CountIf(y, Val(TextBox6)) '傳回TextBox6相同的內容在y 的個數<<這一段是要判斷我工號名單是否是唯一值嗎?
Select Case xNo
Case 1
'luhpro 上傳的工號索引.xls 名單內的儲存格有些是文字格式 <我還不能載><
'y.Value = y.Value '轉為數字格式 <這一行是什麼意思呢? 這有轉格式嗎?
xNo = Application.Match(Val(TextBox6), y, 0) '傳回TextBox6在y 的列號
ComboBox1 = y.Cells(xNo, "b")
Case 0
ComboBox1 = ""
Case Is > 1
ComboBox1 = "工號重複"

複製代碼

麻煩GBKEE大大為我解說一下><
其實本來問題更多
然後才剛發現可以用excel的函數，實際測試完就比較清楚了
謝謝前輩~~

作者: bsy4life 時間: 2016-6-3 13:15

另外還有一個小問題...
我還做了一個載入清單的部分
因為有可能輸入者不是本人，所以不會知道工號
分開來用都是正常的
但只要先載入清單兩次以上
再去輸入工號查詢
ComboBox1 = y.Cells(xNo, "b")
就會錯誤~~
[attach]24406[/attach]
[attach]24405[/attach]

Private Sub CommandButton4_Click() '讀取操作人員
If OptionButton1.Value = True Then
Set x = Sheets("操作人員(早班)")
End If
If OptionButton2.Value = True Then
Set x = Sheets("操作人員(中班)")
End If
If OptionButton3.Value = True Then
Set x = Sheets("操作人員(晚班)")
End If
ComboBox1.List = x.Range("B2", x.[B2].End(xlDown)).Value
End Sub
'Private Sub CommandButton5_Click() '清除操作人員
'TextBox6.SetFocus
'ComboBox1.Value = ""
'TextBox6.Text = ""
'For j = ComboBox1.ListCount To 1 Step -1
'Me.ComboBox1.RemoveItem j - 1
'Next
'End Sub
Private Sub TextBox6_Change() '輸入工號讀取操作人
TextBox6.IMEMode = fmIMEModeOff
If IsNumeric(TextBox6.Text) Then
Dim num As Integer
num = TextBox6.Text
Else
If Not IsNumeric(TextBox6.Text) Then
MsgBox "欄位輸入必須為數值"
End If
End If
Dim xNo As Variant, y As Range
Set y = Sheets("CNC名單(All)").Range("a:a")
xNo = Application.CountIf(y, Val(TextBox6)) '傳回TextBox6相同的內容在y 的個數
Select Case xNo
Case 1
xNo = Application.Match(Val(TextBox6), y, 0) '傳回TextBox6在y 的列號
ComboBox1 = y.Cells(xNo, "b") <<<偵測到此行
Case 0
ComboBox1 = ""
Case Is > 1
ComboBox1 = "工號重複"
End Select
End Sub

複製代碼

我是不是把使用者想得太白目XD

作者: GBKEE 時間: 2016-6-4 05:22

回復 6# bsy4life

有錯誤,上傳檔案看看

Application.Match ,查無時傳回錯誤值,所以變數習慣 Type As Variant
這程式Application.Match 有搭配Application.CountIf 傳回的值,可避開錯誤值.
所以xNo可以不設定成 Variant類型.

作者: bsy4life 時間: 2016-6-4 08:00

回復 7# GBKEE

已附上檔案~~
[attach]24415[/attach]
謝謝前輩!!

作者: luhpro 時間: 2016-6-5 00:39

本帖最後由 luhpro 於 2016-6-5 00:54 編輯

回復 luhpro
謝謝 luhpro 大大
我的資質駑鈍，GBKEE大大的方式解決了我的問題
可是還是想瞭解一下， ...
bsy4life 發表於 2016-6-3 09:43

個人看到的情形提出淺見供你參考:
1.
Module 上設定 Public dPeo 為 "全域變數", (整個檔案任何地方都是使用同一個物件或變數)
但你在 ThisWorkBook 又設了一個 Dim dPeo,
Dim 的 dPeo 則為區域變數,
只有這個 Sub 看的到,
執行到 End Sub 後 -> 出去 Sub 就自動被消滅了,

而因與 Module 的 dPeo 變數名稱相同,
這個 Sub 中使用到的 dPeo 一律預設使用 Dim 出來的這個 "區域變數",
也就是這個 Sub 中的賦值作業結果在其他任一 Sub 完全無法使用)

你的 Label8.Caption = dPeo(TextBox6.Text) 會發生錯誤是因為 UserForm 這個 Sub 看到的 dPeo 是空的.
(可以在執行到此行發錯誤時點 "偵錯" 按鈕, 再於 "即時運算視窗" 下此指令: debug.print dpeo.count    你會看到結果是 0 )
所以 ThisWorkBook 的 Dim dPeo 必須拿掉, 其他 Sub 才能正常使用到 ThisWorkBook 賦予 dPeo 的資料.

2. userform 中:
If IsNumeric(TextBox6.Text) Then
....
Else
      If Not IsNumeric(TextBox6.Text) Then  <- 這一行是多餘的 If  X  Then ... Else 指令的 Else 就是當 Not X 的條件為 "真" 時執行.
...
      End If    <-  搭配 IF 的指令, 當 IF 拿掉時也要跟著拿掉

End If
Label8.Caption = dPeo(TextBox6.Text)  '本來使用combobox，不知道如何取代所以用label測試
'不知為何要用 combobox ?
當使用者在 TextBox 輸入內容後,
這裡應該只有唯一的值(用 Label 物件就可以了),
而不是有好幾個值的清單物件.
除非你想要讓使用者接著可以再在 combobox 點選操作人以再次變更 TextBox 的內容.
註 : 若有此需求, 可以用 combobox.AddItem ... 以增加清單項目
End Sub

最後... 8# 的檔案有設密碼, 程式看不到...

額外建議 :
CNC名單(All) 如果應該是包含全部  操作人員(早, 中, 晚班) 的話,
可以考慮用 Dictionary 在開檔時在三個 Sheet 分別抓取,
然後在 CNC名單(All) 上動態建立清單.

作者: bsy4life 時間: 2016-6-6 10:40

回復 9# luhpro

不知為何要用 combobox ?
當使用者在 TextBox 輸入內容後,
這裡應該只有唯一的值(用 Label 物件就可以了),
而不是有好幾個值的清單物件.
除非你想要讓使用者接著可以再在 combobox 點選操作人以再次變更 TextBox 的內容.
註 : 若有此需求, 可以用 combobox.AddItem ... 以增加清單項目

因為現在還不確定程式會到現場供操作人使用
還是給辦公室的內勤人員使用
如果是給現場的使用，那輸入工號會是最快的方式
如果是內勤人員，也不太可能把每一個人的工號熟記
所以為了方便他們靈活運用，才設計成工號查詢，跟讀取清單兩種模式\
應該還會再把它變成掃條碼可以直接讀取的方式
還有還有
密碼是1215，抱歉忘了解開@@

謝謝luhpro 大大耐心的解答~~
你的建議我會試試看!!

歡迎光臨麻辣家族討論版版 (http://forum.twbts.com/)