Multiplication not working out

Question

1.00/5 (2 votes)

See more:

VB

Dim PlayaSeptemberWeekday As Integer = "100"

    Dim PlayaSeptemberWeekend As Integer = "150"

    Private Sub cmbHotel_SelectedIndexChanged(sender As Object, e As EventArgs) Handles cmbHotel.SelectedIndexChanged

        If (cmbHotel.Text = "Playa del Mar") And (cmbMonth.Text = "September") Then
            txtWeekday.Text = PlayaSeptemberWeekend
        
        End If


    End Sub


    Private Sub cmbWeekdays_SelectedIndexChanged(sender As Object, e As EventArgs) Handles cmbWeekdays.SelectedIndexChanged

        Dim cmbWeekdays As Integer = "2" Or "3" Or "4" Or "5" Or "1"

        If cmbWeekdays = "1" Then
            txtWeekday.Text = cmbWeekdays * PlayaSeptemberWeekday

        End If

    End Sub

    
End Class

Basically I have a combo box, and I need to make it when a 1 or 2 or 3 or 4 or 5 is selected in the combobox I need it to multiply by 100. Anyone know where I'm going wrong here??

Posted 19-Nov-15 6:18am

Member 12150503

Updated 19-Nov-15 6:48am

Patrice T

v2

Add a Solution

Comments

Richard MacCutchan 19-Nov-15 12:40pm

You have not explained what the problem is.

Member 12150503 19-Nov-15 12:53pm

The problem is that they don't multiply, when I have a eg. 2 selected in combo box and 100 is displayed in the text box, the dont multiply.

Richard MacCutchan 19-Nov-15 12:58pm

Yes, but you are only doing the multiplication if cmbWeekdays equals 1, so the answer will only ever be 100.

Member 12150503 19-Nov-15 13:00pm

I just put in cmbWeekdays = "1" or "2" Or "3" Or "4" Or "5"
and now im getting an answer of 700, no matter what number I press in the combo box :(

Matt T Heffron 19-Nov-15 13:03pm

What does cmbWeekdays = "1" or "2" Or "3" Or "4" Or "5" actually do?
It sets cmbWeekdays to the integer value of the bitwise OR of those values. That number is 7

Richard MacCutchan 19-Nov-15 13:08pm

lol. I thought it was some special VB thing that worked like a case statement.

Member 12150503 19-Nov-15 13:08pm

Ah, right..
So is there a way of multiplying those values by whatever comes up in the textbox or do I need to take a different approach?

ZurdoDev 19-Nov-15 13:19pm

Yes, but which value. Just explain in short clear English what you want and then translate that into VB.

Member 12150503 19-Nov-15 13:41pm

never mind, somehow managed to get it working haha. Thanks for your help though, appreciate it sir! Have a great day.

ZurdoDev 19-Nov-15 13:42pm

Please post something as a solution so that this no longer shows unanswered.

Philippe Mori 19-Nov-15 12:42pm

Learn language and use a debugger.

Member 12150503 19-Nov-15 12:54pm

there aren't any bugs, its just that when I try and multiply them they don't multiply.

Matt T Heffron 19-Nov-15 13:01pm

If it doesn't work, by definition that means there are bugs!

Member 12150503 19-Nov-15 13:08pm

haha, thats true unfortunately! :(

Philippe Mori 19-Nov-15 13:09pm

You probably don't know what does the following line:
Dim cmbWeekdays As Integer = "2" Or "3" Or "4" Or "5" Or "1"

In binary 2 is 010b, 3 is 011b, 4 is 100b and 5 is 101b. Or will returns a value that contains bits from both operands. Thus you get :
2 : 010b
2 or 3 : 010b or 011b = 011b = 3
... or 4 : 011b or 100b = 111b = 7
... or 5 : 111b or 101b = 111b = 7

cmbWeekday will never be 1 thus the multiplication won't be executed.

If you would have used a debugger, then in less than 30 seconds, you would know that the program does not do what you think because you don't understand logical operators. Thus as I already said, you should learn the langage first. Take a beginner book and read it.

Member 12150503 19-Nov-15 13:14pm

but is there a way of preforming a task like this?

Matt T Heffron 19-Nov-15 13:24pm

Yes, of course there is.
It just isn't as easy as you want it to be.

Philippe Mori 19-Nov-15 13:33pm

Uses cmbWeekdays.SelectedItem to get selected item and get selected value from there and then do the multiplication...

You might use a sorted set to remember week and weekend days and simply do 2 comparisons.

Patrice T 19-Nov-15 12:51pm

Need to learn the language and programming.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Patrice T · Answer 1 · 2015-11-19T09:29:00

Solution 1

You need some serious reading of the language documentation/tutorials.

Your code look like if you throw almost random pieces of code and get surprised it don't work. I see at least 6 errors in your code.

I guess you need to improve your programming skills too.

Posted 19-Nov-15 9:29am

Patrice T