[發問] 昨天開始接觸VBA，想簡化程式...

GBKEE

超級版主 Rank: 8 Rank: 8

帖子: 5923
主題: 13
精華: 1
積分: 5986
點名: 0
作業系統: win10
軟體版本: Office 2010
閱讀權限: 150
性別: 男
來自: 台灣基隆
註冊時間: 2010-5-1
最後登錄: 2022-1-23

2017 雞年全勤獎章 2016猴年全勤獎章 2015羊年全勤獎章 2014馬年全勤獎章 2013蛇年全勤獎章 2012龍年全勤獎章論壇貢獻勳章熱烈參與勳章 2011兔年全勤獎章

3^#

發表於 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

複製代碼

感恩的心......(在麻辣家族討論區.用心學習會有進步的)
但資源無限,後援有限, 一天1元的贊助,人人有能力.

帖子: 22
主題: 2
精華: 0
積分: 19
點名: 0
作業系統: Win7
軟體版本: 2007
閱讀權限: 10
註冊時間: 2016-6-2
最後登錄: 2016-6-22

4^#

發表於 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幫我解決問題唷~~!!

帖子: 22
主題: 2
精華: 0
積分: 19
點名: 0
作業系統: Win7
軟體版本: 2007
閱讀權限: 10
註冊時間: 2016-6-2
最後登錄: 2016-6-22

5^#

發表於 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的函數，實際測試完就比較清楚了
謝謝前輩~~

帖子: 22
主題: 2
精華: 0
積分: 19
點名: 0
作業系統: Win7
軟體版本: 2007
閱讀權限: 10
註冊時間: 2016-6-2
最後登錄: 2016-6-22

6^#

發表於 2016-6-3 13:15 | 只看該作者

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

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

超級版主 Rank: 8 Rank: 8

帖子: 5923
主題: 13
精華: 1
積分: 5986
點名: 0
作業系統: win10
軟體版本: Office 2010
閱讀權限: 150
性別: 男
來自: 台灣基隆
註冊時間: 2010-5-1
最後登錄: 2022-1-23

2017 雞年全勤獎章 2016猴年全勤獎章 2015羊年全勤獎章 2014馬年全勤獎章 2013蛇年全勤獎章 2012龍年全勤獎章論壇貢獻勳章熱烈參與勳章 2011兔年全勤獎章

7^#

發表於 2016-6-4 05:22 | 只看該作者

回復 6# bsy4life

有錯誤,上傳檔案看看

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

感恩的心......(在麻辣家族討論區.用心學習會有進步的)
但資源無限,後援有限, 一天1元的贊助,人人有能力.

帖子: 22
主題: 2
精華: 0
積分: 19
點名: 0
作業系統: Win7
軟體版本: 2007
閱讀權限: 10
註冊時間: 2016-6-2
最後登錄: 2016-6-22

8^#

發表於 2016-6-4 08:00 | 只看該作者

回復 7# GBKEE

已附上檔案~~

生產實績效率運算0604.rar (116.23 KB)
謝謝前輩!!

luhpro

高中生 Rank: 3 Rank: 3

帖子: 835
主題: 6
精華: 0
積分: 915
點名: 16
作業系統: Win 10,7
軟體版本: 2019,2013,2003
閱讀權限: 50
性別: 男
註冊時間: 2010-5-3
最後登錄: 2024-11-14

9^#

發表於 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) 上動態建立清單.

帖子: 22
主題: 2
精華: 0
積分: 19
點名: 0
作業系統: Win7
軟體版本: 2007
閱讀權限: 10
註冊時間: 2016-6-2
最後登錄: 2016-6-22

10^#

發表於 2016-6-6 10:40 | 只看該作者

回復 9# luhpro

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

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

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