We value your feedback.
Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!
COURSE of the MONTH
11 days, 1 hour left to enrollBecome a Premium Member and unlock a new, free course in leading technologies each month.
Solved
Select Case Statement vs If ElseIf Statement
Posted on 2006-10-30
Simple form to send chosen report to printer with CRViewer91. Form has a Frame control with 3 Option Buttons, 2 Command Buttons. Following is code for the cmdSelect Report button.
Private Sub cmdSelectReport_Click()
'View Report and Print user choice
Dim Value As Boolean
Select Case Value
Case optAssist(0).Value = True
frmCRViewerAssist.Show
Case optBoard(1).Value = True
frmCRViewerMonth.Show
Case optAnnual(2).Value = True
frmCRViewerYear.Show
Case Else
'If optAssist(0).Value = True Then
' frmCRViewerAssist.Show
'ElseIf optBoard(1).Value = True Then
' frmCRViewerMonth.Show
'ElseIf optAnnual(2).Value = True Then
' frmCRViewerYear.Show
' ElseIf optGraph(3).Value = True Then
' frmCRViewerGraph.Show
' Else
MsgBox "You have not made a selection" & vbCrLf & _
"Please select one option or press Cancel", 32, "Make Print Selection"
'End If
End Select
End Sub
The Select Case code drops to he MsgBox when the click event is fired regardless is a button is checked or not checked.
The If ElseIf code that is commented out works like a charm with no problems but I know that the Case Statement will work. I'm missisng something simple in the Select Case statement I think. Any help appreciated
Private Sub cmdSelectReport_Click()
'View Report and Print user choice
Dim Value As Boolean
Select Case Value
Case optAssist(0).Value = True
frmCRViewerAssist.Show
Case optBoard(1).Value = True
frmCRViewerMonth.Show
Case optAnnual(2).Value = True
frmCRViewerYear.Show
Case Else
'If optAssist(0).Value = True Then
' frmCRViewerAssist.Show
'ElseIf optBoard(1).Value = True Then
' frmCRViewerMonth.Show
'ElseIf optAnnual(2).Value = True Then
' frmCRViewerYear.Show
' ElseIf optGraph(3).Value = True Then
' frmCRViewerGraph.Show
' Else
MsgBox "You have not made a selection" & vbCrLf & _
"Please select one option or press Cancel", 32, "Make Print Selection"
'End If
End Select
End Sub
The Select Case code drops to he MsgBox when the click event is fired regardless is a button is checked or not checked.
The If ElseIf code that is commented out works like a charm with no problems but I know that the Case Statement will work. I'm missisng something simple in the Select Case statement I think. Any help appreciated
[X]
Welcome to Experts Exchange
Add your voice to the tech community where 5M+ people just like you are talking about what matters.
- Help others & share knowledge
- Earn cash & points
- Learn & ask questions
-
2
-
2
- +2
7 Comments
Active 2 days ago
this is because the default value of the variable "Value" is false.
Private Sub cmdSelectReport_Click()
'View Report and Print user choice
Dim Value As Boolean
Select Case true
Case optAssist(0).Value = True
frmCRViewerAssist.Show
Case optBoard(1).Value = True
frmCRViewerMonth.Show
Case optAnnual(2).Value = True
frmCRViewerYear.Show
Case Else
'If optAssist(0).Value = True Then
' frmCRViewerAssist.Show
'ElseIf optBoard(1).Value = True Then
' frmCRViewerMonth.Show
'ElseIf optAnnual(2).Value = True Then
' frmCRViewerYear.Show
' ElseIf optGraph(3).Value = True Then
' frmCRViewerGraph.Show
' Else
MsgBox "You have not made a selection" & vbCrLf & _
"Please select one option or press Cancel", 32, "Make Print Selection"
'End If
End Select
End Sub
Private Sub cmdSelectReport_Click()
'View Report and Print user choice
Dim Value As Boolean
Select Case true
Case optAssist(0).Value = True
frmCRViewerAssist.Show
Case optBoard(1).Value = True
frmCRViewerMonth.Show
Case optAnnual(2).Value = True
frmCRViewerYear.Show
Case Else
'If optAssist(0).Value = True Then
' frmCRViewerAssist.Show
'ElseIf optBoard(1).Value = True Then
' frmCRViewerMonth.Show
'ElseIf optAnnual(2).Value = True Then
' frmCRViewerYear.Show
' ElseIf optGraph(3).Value = True Then
' frmCRViewerGraph.Show
' Else
MsgBox "You have not made a selection" & vbCrLf & _
"Please select one option or press Cancel", 32, "Make Print Selection"
'End If
End Select
End Sub
Why are you doing this Select Case Value?
value boolean only has two value and that is true and false therefore you do not use case statement for this.
This is what I would have done.
if Case optAssist(0).Value = True then
frmCRViewerAssist.Show
elseif Case optBoard(1).Value = True then
frmCRViewerMonth.Show
elseif Case optAnnual(2).Value = True then
frmCRViewerYear.Show
else
'If optAssist(0).Value = True Then
' frmCRViewerAssist.Show
'ElseIf optBoard(1).Value = True Then
' frmCRViewerMonth.Show
'ElseIf optAnnual(2).Value = True Then
' frmCRViewerYear.Show
' ElseIf optGraph(3).Value = True Then
' frmCRViewerGraph.Show
' Else
MsgBox "You have not made a selection" & vbCrLf & _
"Please select one option or press Cancel", 32, "Make Print Selection"
'End If
end if
value boolean only has two value and that is true and false therefore you do not use case statement for this.
This is what I would have done.
if Case optAssist(0).Value = True then
frmCRViewerAssist.Show
elseif Case optBoard(1).Value = True then
frmCRViewerMonth.Show
elseif Case optAnnual(2).Value = True then
frmCRViewerYear.Show
else
'If optAssist(0).Value = True Then
' frmCRViewerAssist.Show
'ElseIf optBoard(1).Value = True Then
' frmCRViewerMonth.Show
'ElseIf optAnnual(2).Value = True Then
' frmCRViewerYear.Show
' ElseIf optGraph(3).Value = True Then
' frmCRViewerGraph.Show
' Else
MsgBox "You have not made a selection" & vbCrLf & _
"Please select one option or press Cancel", 32, "Make Print Selection"
'End If
end if
correction
if optAssist(0).Value = True then
frmCRViewerAssist.Show
elseif optBoard(1).Value = True then
frmCRViewerMonth.Show
elseif optAnnual(2).Value = True then
frmCRViewerYear.Show
else
'If optAssist(0).Value = True Then
' frmCRViewerAssist.Show
'ElseIf optBoard(1).Value = True Then
' frmCRViewerMonth.Show
'ElseIf optAnnual(2).Value = True Then
' frmCRViewerYear.Show
' ElseIf optGraph(3).Value = True Then
' frmCRViewerGraph.Show
' Else
MsgBox "You have not made a selection" & vbCrLf & _
"Please select one option or press Cancel", 32, "Make Print Selection"
'End If
end if
if optAssist(0).Value = True then
frmCRViewerAssist.Show
elseif optBoard(1).Value = True then
frmCRViewerMonth.Show
elseif optAnnual(2).Value = True then
frmCRViewerYear.Show
else
'If optAssist(0).Value = True Then
' frmCRViewerAssist.Show
'ElseIf optBoard(1).Value = True Then
' frmCRViewerMonth.Show
'ElseIf optAnnual(2).Value = True Then
' frmCRViewerYear.Show
' ElseIf optGraph(3).Value = True Then
' frmCRViewerGraph.Show
' Else
MsgBox "You have not made a selection" & vbCrLf & _
"Please select one option or press Cancel", 32, "Make Print Selection"
'End If
end if
A Select Case would be useful if you were looking for several different VALUES of the same VARIABLE:
Select Case color
Case "blue"
...
Case "green"
...
End Select
You're looking for the same value in each of several different variables. A Select Case can't handle that; your original If statement is the "correct" (and best) syntax for it.
Select Case color
Case "blue"
...
Case "green"
...
End Select
You're looking for the same value in each of several different variables. A Select Case can't handle that; your original If statement is the "correct" (and best) syntax for it.
Active today
You can still use the neater Select Case in your case (No puns intended)
Select Case True
Case optAssist(0).Value
frmCRViewerAssist.Show
Case optBoard(1).Value
frmCRViewerMonth.Show
Case optAnnual(2).Value
frmCRViewerYear.Show
Case Else
'...
Select Case True
Case optAssist(0).Value
frmCRViewerAssist.Show
Case optBoard(1).Value
frmCRViewerMonth.Show
Case optAnnual(2).Value
frmCRViewerYear.Show
Case Else
'...
lunchbyte, yes that works. I think that is what I have commented out that I said worked, but thank anyway.
GrahamSkan, "You Are the Man", your code is neater and straight forward. I should have realized what was wrong, there is only one varaible involved.
Thanks, Guys for all your replys. CopperNose
GrahamSkan, "You Are the Man", your code is neater and straight forward. I should have realized what was wrong, there is only one varaible involved.
Thanks, Guys for all your replys. CopperNose
I'd strongly encourage the simpler (and more expected) format of
If optAssist(0).Value Then
frmCRViewerAssist.Show
ElseIf optBoard(1).Value Then
frmCRViewerMonth.Show
ElseIf optAnnual(2).Value
frmCRViewerYear.Show
End If
It doesn't take up any more space than a Select Case (less if you count the "Select Case" statement itself), and it's the more appropriate construct for this type of situation. Clearly a Select Case can be made to work, but it takes longer to adjust to an unusual use of a construct, and thus longer to understand the code.
If optAssist(0).Value Then
frmCRViewerAssist.Show
ElseIf optBoard(1).Value Then
frmCRViewerMonth.Show
ElseIf optAnnual(2).Value
frmCRViewerYear.Show
End If
It doesn't take up any more space than a Select Case (less if you count the "Select Case" statement itself), and it's the more appropriate construct for this type of situation. Clearly a Select Case can be made to work, but it takes longer to adjust to an unusual use of a construct, and thus longer to understand the code.
Featured Post
An intuitive utility to help find the CSS path to UI elements on a webpage. These paths are used frequently in a variety of front-end development and QA automation tasks.
One of a set of tools we're offering as a way of saying thank you for being a part of the community.
Question has a verified solution.
If you are experiencing a similar issue, please ask a related question
This article describes some techniques which will make your VBA or Visual Basic Classic code easier to understand and maintain, whether by you, your replacement, or another Experts-Exchange expert.
If you need to start windows update installation remotely or as a scheduled task you will find this very helpful.
Get people started with the utilization of class modules. Class modules can be a powerful tool in Microsoft Access. They allow you to create self-contained objects that encapsulate functionality. They can easily hide the complexity of a process from…
Show developers how to use a criteria form to limit the data that appears on an Access report. It is a common requirement that users can specify the criteria for a report at runtime. The easiest way to accomplish this is using a criteria form that a…
Suggested Courses
Course of the Month11 days, 1 hour left to enroll
770 members asked questions and received personalized solutions in the past 7 days.
Join the community of 500,000 technology professionals and ask your questions.