hey im new to VB & mostly understand pseudo code, i have a program which has a sub (method? - function?) what it does is calculate a number when you hit a "calculate" button. i created a menu item called calculate and i am trying to call the function that calculates without duplicating the code...heres what it looks like, should i create a totally seperate function that calcualtes and call this function from both the botton click and the menu select items?


    Private Sub Button1_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles CalculateButton.Click
        Dim BCuser As Long
        Dim IMuser As Long
        Dim BLuser As Long
        Dim AVuser As Long
        Dim ENuser As Long
        Dim MAuser As Long
        Dim PMuser As Long
        Dim EMuser As Long
        Dim BC2user As Long
        Dim AMVCost As Long
        Dim MRR As Long


        EMuser = txtAMuser.Text
        BCuser = txtBCuser.Text
        IMuser = txtIMuser.Text
        BLuser = txtBLuser.Text
        AVuser = txtAVuser.Text
        ENuser = txtENuser.Text
        MAuser = txtMAuser.Text
        PMuser = txtPMuser.Text
        BC2user = txtBC2user.Text


        MyCost= ((7 * EMuser) + (2.1 * (BCuser + BC2user + PMuser + AVuser)) + (3.5 * (IMuser + BLuser)) + (2.5 * ENuser) + (4.2 * MAuser))
        MRR = ((10 * EMuser) + (3 * (AVuser + BCuser + BC2user)) + (5 * (IMuser + BLuser)) + (3.5 * ENuser) + (6 * MAuser) + (4 * PMuser))

        MsgBox("This deal will cost Me $" & MyCost & " per month. Your Client will be charged $" & MRR & " per month.")

End Sub


    Private Sub CalculateToolStripMenuItem_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles CalculateToolStripMenuItem.Click

//Heres where i want to call the previous function//

    End Sub

First of all: Both functions and subs are methods and can also jointly be called "subroutines". In C#/Java/C++, there are only functions, and the difference is only in whether they return void (no value) or not. VB has a long legacy of the two different names, but truly the only difference is whether the method returns a value (function) or not (sub).

Now...

Do you want both methods to do exactly the same thing, *INCLUDING* displaying that MsgBox?

If so, you could just code
[code]
Private Sub CalculateToolStripMenuItem_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) _
        Handles CalculateToolStripMenuItem.Click
    Button1_Click( sender, e )
End Sub
[code]

But another trick is to just have one method handle multiple events:
Nobody said that the name of the sub has to match the name of the event it is handling, at all. So why not use a name that is meaningful to you?