[RESOLVED] Menustrip Function

[VBAppleCoder]

Hello everyone,

In my project, which is for a final in my class, I am needing some help with a function. This is what my instructor is asking of us.

" Deposit (menu item) - after the user inputs value in the textbox for the "Enter Amount" and selects this menu item, it will read and convert the text into a
value and send it to a function, which will calculate the deposit. In addition, this function will keep track of the number of deposits made (counter) and
running total of deposits (accumulator). The menu item will also display the calculated value in the account balance label."

I would greatly appreciate any help! I have finished programming a lot of other coding for this program yet this one item is holding me back.

I do have code that displays the entered amount but when you enter a different amount it does not add it to the previous amount entered.

Code for that is:

' Add the entered amount as deposit
intAmount = txtAmount.Text
If txtAmount.Text = True Then
decAccountBalance += intAmount + decDeposit
End If

' Display amount in the label
lblBalance.Text = decAccountBalance.ToString("c")

[OptionBase1]

Based solely on the snippet of code you included, my best guess would be that you are running in to an issue with variable scope. Where/how is the decAccountBalance variable declared? If it is declared with "Dim" inside the function that is processing the deposit, change "Dim" to "Static". That will initialize the variable only the first time the function runs. With Dim, it will reset the value to 0 each time the function runs. Your textbook should have some explanation of Dim vs. Static.

Also the "If txtAmount.Text = True" line is wonky in my opinion. If the purpose of that line is to ensure that the user entered a valid numeric value, there are more proper ways of doing that.

Edit: Looking at your variables:

- intAmount is assigned the value entered by the user in the textbox
- decAccountBalance is your running total balance
- decDeposit it isn't clear what purpose it has or where it is getting a value assigned.

[jmcilhinney]

In addition, this function will keep track of the number of deposits made (counter) and running total of deposits (accumulator).

If you need to store values that persist between method calls then you need variables declared outside of any method.

it will read and convert the text into a value and send it to a function

If you intend to call a method and pass it a number then you need to have a method with a numeric parameter. If you're dealing with monetary values then you would generally use Decimal. If there will only be whole values then you can use Integer.

So, declare appropriate variables and declare an appropriate method. Next, write code to convert the String from the TextBox to an appropriate numerical type and call the method. If you are going to provide proper validation then you should use something like Decimal.TryParse. If you're allowed to assume that input will be valid then you can use Convert.ToDecimal, Decimal.Parse or CDec. Etc.

[VBAppleCoder]

Ok so how would I fix this issue? I have taken out decDeposit because it does not have any function.

[OptionBase1]

Where/how is the decAccountBalance variable declared? If it is declared with "Dim" inside the function that is processing the deposit, change "Dim" to "Static". That will initialize the variable only the first time the function runs. With Dim, it will reset the value to 0 each time the function runs. Your textbook should have some explanation of Dim vs. Static.

See bolded.

[VBAppleCoder]

I changed it to static like you said and it worked! Thanks!

[alexE]

Any chance you can spare the code you used?

[VBAppleCoder]

Any chance you can spare the code you used?

Yes but it will have to be tomorrow!