@Elroy: Think about Form2.Show vbModal line. This reduces Form2 to a dummy View in the Model-View-Presenter design-pattern, a module which has no code at all (besides simple client-side validation probably).

If Form2 was coded as a Presenter then there has to be a method that accepts input data, fills the controls, calls Show vbModal and returns true/false if the form is confirmed along with output data. The modal interaction is encapsulated and the public methods on the form that other components are calling are reduced to a single method. There is no need for anyone to know what TextBoxes are there in the UI and all the gory details of the UI. This way the UI can be refactored separately in insolation without breaking a bunch of other modules/forms that access controls on Form2 for whatever reason.

cheers,
</wqw>