Visual Basic 2005 programming question

[Zap Brannigan]

I'm having trouble converting some currencies in this program I made.

Anyone know what I need ot get this working properly? So far here is what I have,




Option Explicit On
Option Strict Off



Public Class Form1

Private Sub xClear_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles xClear.Click

Me.xDollarsTextBox.Text = String.Empty
Me.xPoundsTextBox.Text = String.Empty
Me.xPesosTextBox.Text = String.Empty

End Sub

Private Sub xExit_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles xExit.Click
Me.Close()

End Sub

Private Sub xCalc_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles xCalc.Click

Dim Dollars As Integer
Dim Pounds As Integer
Dim Pesos As Integer

Pounds = Dollars * 0.571505
Pesos = Dollars * 10.7956




Me.xPoundsTextBox.Text = Pounds.ToString

Me.xPesosTextBox.Text = Pesos.ToString

 

[Crusty]

Can you tell us what errors you are getting and/or what doesn't work? Just posting code and asking people to fix it won't get you anywhere. Sure, you might get working code but you won't learn anything in the process.

From what I can tell your code is incomplete, as in you haven't finished writing everything for the program to even attempt to work.

 

[joutlaw]

Where are your inputs.

You probably need to have

Dim Dollars as Integer = CInt(xDollarsTextBox.Text)
etc...

 

[joutlaw]

Also after I looked at it... I dont think you need to be using integer as the type. I would use Double. Integers are whole numbers I believe.


convert you textbox inputs as so:
CDbl(textbox.text)

 

[joutlaw]

I don't know if this is for a class or a program you would like to use in the future... but I'd put the exchange rate in a app.config file so you won't have re-compile when the exchange rate changes.

 

[KLin]

Originally posted by: joutlaw
I don't know if this is for a class or a program you would like to use in the future... but I'd put the exchange rate in a app.config file so you won't have re-compile when the exchange rate changes.

Definitely homework question.

 

[Zap Brannigan]

When I try to add ,

Const poundsrate As Decimal = 0.571505 it underlines pounds rate in green and says unused local variable

I made 3 text boxes on the form, xDollarsTextBox, etc....

 

[Zap Brannigan]

Originally posted by: KLin

Originally posted by: joutlaw
I don't know if this is for a class or a program you would like to use in the future... but I'd put the exchange rate in a app.config file so you won't have re-compile when the exchange rate changes.

Definitely homework question.

Oh you're a big help. lol

 

[brandonbull]

I also think it's not a good practice to mix integer and decimal data types. Integers like to to get of decimals.

I'd also have some input validation for your text boxes. If you input alphas, makes it hard to multiple.

 

[Zap Brannigan]

Originally posted by: brandonbull
I also think it's not a good practice to mix integer and decimal data types. Integers like to to get of decimals.

I'd also have some input validation for your text boxes. If you input alphas, makes it hard to multiple.

Thanks! Yeah the decimals and integers were clashing. What can I say, I'm learning as I go. lol

 

[Zap Brannigan]

Well the program is running as intended now.