Question

I have a Form class

    partial class ProgressMainForm : Form
    {
      public ProgressMainForm()
      {
        InitializeComponent();
      }
    }

And then a class that uses that class and contains all functionality for the user

    public class ProgressForm
    {
        public ProgressMainForm myProgressForm;

        public ProgressForm(string title)
        {
            myProgressForm = new ProgressMainForm();
            myProgressForm.Text = title;
        }

        public void SetProgressBar(int min, int max)
        {
        ....
    }

I then use this ProgressForm class in my project like this

            progresswindow = new ProgressForm("Replacing All Strings");

This way progresswindow only contains members that are related to the functionality of the ProgressForm and all those Form members are hidden from the user.

But sometimes I need to access those Form members, for example when I need Invoke method.

Is there a way to make myProgressForm in ProgressForm accessible to users without making ProgressMainForm public?

Or is this approach wrong?

Answer 1

In my opinion you should not work with the form directly. If I read your setup correctly, you want to show progress indicator while some job is being done. ProgressForm should expose methods to set the counters and increment them; as you run it on another thread, form manipulation should be done from inside the methods of ProgressForm. Your Invokes belong there, wrapped in suitable methods. If you want to change some visual properties of ProgressMainForm relay those properties to ProgressForm.

To resume, calling code should have no clue what ProgressForm does other than setting progress boundaries, starting, setting current percentage and stopping. This way, if you are asked to port the application to another UI system the amount of code you will need to change will be drastically reduced.