2

Как передать выбранные параметры текста из второй формы в первую, по нажатию кнопки "применить" во второй форме, не нарушая принципов ООП?

Form1:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.IO;

namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void font_button_Click(object sender, EventArgs e)
        {
            Form2 font_dialog = new Form2();
            font_dialog.ShowDialog();
        }

        private void textBox1_TextChanged(object sender, EventArgs e)
        {

        }

        // Кнопка сохранения файла в заранее указанную папку
        private void save_button_Click(object sender, EventArgs e)        
        {
            Directory.CreateDirectory("C:\\my_dir"); // Создание директории

            // Запись
            string lines = "" + textBox1.Text;
            File.WriteAllText(@"C:\\my_dir\my_txt.txt", lines);

            MessageBox.Show("Файл сохранен!");            
        }

        private void exit_button_Click(object sender, EventArgs e)
        {
            this.Close();
        }
    }
}

Form2:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace WindowsFormsApplication1
{
    public partial class Form2 : Form
    {
        public Form2()
        {
            InitializeComponent();
        }

        // Список шрифтов
        private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
        {
            switch (comboBox1.Text)
            {
                case "Arial":
                    Font font_1 = new Font("Arial", this.Font.Size);
                    comboBox1.Font = font_1;
                    break;

                case "Courier New":
                    Font font_2 = new Font("Courier New", this.Font.Size);
                    comboBox1.Font = font_2;
                    break;

                case "Times New Roman":
                    Font font_3 = new Font("Times New Roman", this.Font.Size);
                    comboBox1.Font = font_3;
                    break;

                default:
                    Font def_font = new Font("Arial", this.Font.Size);
                    comboBox1.Font = def_font;
                    break;
            }

        }

        // Кнопка применения параметров шрифта
        private void OK_Click(object sender, EventArgs e)
        {

        }

        private void numericUpDown1_ValueChanged(object sender, EventArgs e)
        {

        }       
    }
}
1
  • Да, идея с тремя захардкоженными шрифтами и текстбоксом по меньшей мере странная.
    – Qwertiy
    13 апр 2015 в 7:27

3 ответа 3

8

ИМХО, предложенный вариант с глобальным FormValuesHolder противоречит всем принципам ООП: он добавляет временную связанность между объектами и может привести к проблемам сопровождения.

  • Решение нарушает принцип инкапсуляции. Где здесь сокрытие информации? Открытое поле без поведения не способствует сокрытию информации.

  • Отсутствие абстракции. Статическое поле не повышает уровень абстракции.

  • Отсутствие поведения. В ООП данные сами по себе не хранятся. С ними должно быть связано некоторое поведение. Поведение у свойства типа string с точки зрения моделирования решения отсутствует.

Теперь, альтернативные решение.

  1. Если нажатие кнопки "применить" второй формы закрывает ее.

В этом случае, в классе Form2 достаточно объвить открытое свойтсов нужного типа и устанавливать его в событии OK_Click:

public partial class Form1 : Form
{
        private void font_button_Click(object sender, EventArgs e)
        {
            Form2 font_dialog = new Form2();
            if (font_dialog.ShowDialog() == DialogResult.OK)
            {
               // Мы знаем, что кнопка OK была нажата, пользователь 
               // не отменил ввод.
               // Используем новые свойства шрифта
               Console.WriteLine(font_dialog.NewFont);
            }
        }
}

Обычно первый вариант подходит, поскольку кнопка OK обычно закрывает форму.

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

// Да, имена форм тоже разумно сразу же давать осмысленные
public partial class MainForm : Form
{
  public void FontButtonClick(object sender, EventArgs e)
  {
    // Да, подчеркивания не приветствуются в именах полей/переменных
    var fontDialog = new FontDialog();
    fontDialog.NewFontSelected += HandleNewFont;
    fontDialog.ShowDialog();
  }

  private void HandleNewFont(object sender, NewFontSelectedEventArgs ea)
  {
    // Обратаываем выбранный новый шрифт
  }
}

public class NewFontSelectedEventArgs : EventArgs
{
  public string NewFont {get; set;}
}

public partial class FontDialogForm : Form
{
  public event EventHandler<NewFontSelectedEventArgs> NewFontSelected;

  private void OnNewFontSelected(string newFont)
  {
    var handler = NewFontSelected;
    if (handler != null)
      handler(this, new NewFontSelectedEventArgs{NewFont = newFont};
  }
  // все тоже самое

  // Кнопка применения параметров шрифта
  private void OK_Click(object sender, EventArgs e)
  {
    // уведомляем подписчиков о выбранном новом шрифте
    OnNeFontSelected(newFont);
  }
}

Да, если посмотреть на стандартный FontDialog, то легко можно будет заметить в нем сходство с предложенным решением. Вот пример использования стандартного диалога для изменения шрифта:

private void button1_Click(object sender, System.EventArgs e)
 {
    fontDialog1.ShowColor = true;

    fontDialog1.Font = textBox1.Font;
    fontDialog1.Color = textBox1.ForeColor;

    if(fontDialog1.ShowDialog() != DialogResult.Cancel )
    {
       textBox1.Font = fontDialog1.Font ;
       textBox1.ForeColor = fontDialog1.Color;
    }
 }

[Cap mode on]Никаких глобальных статических классов, которые содержат промежуточный новый фонт![Cap mode off].

16
  • Эм, я как-то не вижу здесь нарушения инкапсуляции, полиморфизма, наследования и абстракции. Это нехорошее решение, не спорю, но разве тут есть нарушение принципов ООП? Там есть нарушение принципов хорошей архитектуры, которые к ООП никакого отношения не имеют.
    – etki
    11 апр 2015 в 1:42
  • Я так понимаю, речь идет о моей фразе в начале ответа... Глобальный ответ - это не-ОО-решение. Совсем. Это из мира структурного программирования. И да, глобальные объект с открытым свойством нарушает ООП: это нарушение инкапсуляции (где здесь сокрытие информации?), отсутствие абстракции (где она в этом решении?), отсутствие поведения (есть только данные). Так что оно очень много чего нарушает;) 11 апр 2015 в 2:11
  • Существование принципа абстракции не подразумевает его обязательную реализацию, равно как и с инкапсуляцией. И вообще там формально сеттер и геттер.
    – etki
    11 апр 2015 в 2:30
  • Так из всех классических принципов ООП здесь не нарушается только наследование, которое здесь не применимо. Другими словами, здесь нарушаются все применимые в данном случае каноны ООП, и вам этого мало? 11 апр 2015 в 5:12
  • Они не нарушаются, их здесь нет, потому что они не нужны, как и наследование. Здесь не нужна абстракция, потому что нет многоуровневости. Здесь нет наследования, потому что пример банальный и наследовать нечего. Единственное поле скрыто за аксессорами.
    – etki
    11 апр 2015 в 5:47
3

Я бы больше говорил не о нарушении ООП, а о нарушении общих принципов дизайна программ.

Проблема вашего кода заключается в том, что бизнес-логика и представление объединены и перемешаны. В результате окно почему-то знает о том, где и как хранятся данные, хотя это на самом деле вопрос для уровня модели.

Таким образом, правильный подход мне кажется следующим:

  1. Первая форма запрашивает параметры у своей модели/VM (в зависимости от выбранного паттерна организации программы).
  2. Модель/VM обнаруживает, что эти данные нужно запросить в UI, и инициирует показ второй формы.
  3. После закрытия второй формы информация есть у модели/VM, и она доставляется первой форме.
9
  • 1
    Это верно, но для очень простых случаев (наподобие выбора шрифта) таким разделением, как мне кажется, можно пренебречь. Особенно в WinForms, где с MVVM не так чтобы очень.
    – Memoizer
    11 апр 2015 в 9:23
  • @Memoizer: Для простой программы не обязательно и следование принципам ООП. Я расценивал код из вопроса как упрощённый пример.
    – VladD
    11 апр 2015 в 9:38
  • @Memoizer для WinForms очень хорошо работает MVP.
    – andreycha
    11 апр 2015 в 10:13
  • 1
    @Memorize, наоборот код упростится, если ввести отдельный объект бизнес-логики, который будет управлять вызовом форм и собирать данные.
    – hardsky
    11 апр 2015 в 11:04
  • 1
    @SergeyTeplyakov: Тогда ответ будет ему «на вырост» :-)
    – VladD
    11 апр 2015 в 18:23
1

Cделайте отдельный static-класс - хранилище.

public static class FormValuesHolder
{
   static string _str1 = string.Empty;

   public static string Str1
   {
       get { return _str1; }
       set { _str1 = value; }
   }
}

И обращайтесь из разных форм к этому полю: FormValuesHolder.Str1

1
  • 6
    А разве это не глобальный объект, который нарушает ключевые принципы ООП? 10 апр 2015 в 18:57

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.