Tool Strip Menu And Check Box Sync'd Together?

Ok, I'm a very inexperienced programmer writing my first "non-example" program without step by step instructions provided by the learning material. That said, I am still trying to grasp all the different concepts, syntaxes, tools available, etc. So I wanted someone to take a look at my code and give me some input as to how I can make the code more efficient because I can already envision some bugs that could come up from the code I have written right now.

So I'm writing a program that will have two controls (as of now, which I might think of other controls that might become useful) that will be in a boolean state. I have a checked tool strip menu item and a check box item. I need to have the clicked event run some statements that will either set a textbox's visible property. But my question is whether I should keep the code as it is, if I should create a new module, use a sub procedure, or what is the most efficient way of dong this? Here's my code but I can already see some functionality problems if I decide more controls that accomplish the same task as having to go back and add code to the existing controls. Should I not try to keep the code in a seperate module so all I have to do is change the code getting called in that seperate module instead of actually going through the event code and changing/adding statements repetitively? Anyway:

[Code]...