Как минимизировать затраты рабочего времени при работе со старым проектом WinForms?

Question

Моя главная цель - минимизировать затраты рабочего времени при работе со старым проектом WinForms.Я сделал пример с реализацией MCV + WinForms.Это сработало.У меня есть четкая модель, но у меня есть два метода в Controller.cs (OnSuper () и WorkMethod ()) - и мне они не нравятся.Они не несут функциональности, но они смотрят, когда кто-то намазывает масло на масло без хлеба.Как я могу обновить эту идею без интерфейса, как в MPV и без WPF?Или это не очень хороший подход, и я должен создать новый проект WPF?

Класс Model.cs

class Model
{    
    public int X { get; set; }
    public int Y { get; set; }
    public int R { get; set; }
    public Model() { }
    public Model(int x, int y)
    {
        X = x;
        Y = y;
    }
    public int GetRes()
    {
        return X + Y;            
    }
}

Класс Form1.cs

public partial class Form1 : Form
{        
    public Form1()
    {
        InitializeComponent();            
        button1.Click += delegate
        {
            Controller controller = new Controller
            (
                this, 
                Int32.Parse(this.textBox1.Text),
                Int32.Parse(this.textBox2.Text)
            );
            controller.SuperEvent += Controller_SuperEvent;
            controller.OnSuper();
        };
    }
    private void Controller_SuperEvent(object sender, EventArgs e)
    {
        Controller controller = sender as Controller;
        textBox3.Text = controller.WorkMethod();
        controller.WorkMethod();
    }
}

и класс Controller.cs

class Controller
{        
    public event EventHandler SuperEvent;
    Model model;
    Form1 form1;
    public int Result { get; set; }
    public int CX { get; set; }
    public int CY { get; set; }
    public Controller(Form1 form1, int cX, int cY)
    {
        this.form1 = form1;
        this.model = new Model();            
        CX = cX;
        CY = cY;
    }
    public void OnSuper()
    {
        SuperEvent?.Invoke(this, EventArgs.Empty);
    }
    public string WorkMethod()
    {            
        model.X = CX;
        model.Y = CY;
        return model.GetRes().ToString();
    }
}

Answer 1

Для упрощения сопровождения проведите четкую грань между уровнями ответственности.

Модель, отвечающая за бизнес-логику

class Model
{    
    public int X { get; set; }
    public int Y { get; set; }

    public Model() {}

    public int GetRes() => X + Y
}

ViewModel, отвечающая за подготовку данных для отображения в представлении (если требуется) и длявызов операции бизнес-логики модели, запрошенной из представления.
INotifyPropertyChanged уведомит представление о возможных изменениях без зависимости модели представления от конкретного типа представления.

class ViewModel : INotifyPropertyChanged
{    
    private readonly Model _model;

    public int X { get; set; }

    public int Y { get; set; }

    private string _result = "";
    public string Result 
    { 
        get
        {
            return _result;
        } 

        private set
        {
            if (_result.Equals(value) == false)
            {
                _result = value;
                RaisePropertyChanged();
            }

        }
    }

    public ViewModel(Model model) => _model = model;

    public void CalculateResult()
    {
         Result = _model.GetRes().ToString();
    }

    // Implementation of INotifyPropertyChanged
    public event PropertyChangedEventHandler PropertyChanged;

    private void RaisePropertyChanged([CallerMemberName] string propertyName = "")
    {
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }
}

Представление, ответственность за которое заключается в отображении значений из представления моделиили вызовите операцию viewmodel, когда пользователь запрашивает.

public partial class View : Form
{    
    private ViewModel _viewmodel;

    public View()
    {
        InitializeComponent();   
        var model = new Model { X = 10, Y = 20 }  
        _viewmodel = new ViewModel(model);

        // Bind controls to the viewmodel
        textboxForX.DataBindings.Add("Text", _viewmodel, "X", true);
        textboxForY.DataBindings.Add("Text", _viewmodel, "Y", true);
        labelForResult.DataBindings.Add("Text", _viewmodel, "Result", true);            
    } 

    private void CalculateButton_Click(object sender, EvetnArgs e)
    {
        _viewmodel.Calculate();
    }
}    

Обратите внимание, что в простых случаях (как указано выше) вы можете пропустить «слой» viewmodel и связать вид непосредственно с моделью.Но наличие «буфера» между представлением и бизнес-логикой облегчит вашу жизнь, когда проект начнет расти.