Блог

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

У меня есть приложение Telnet. Form1 подключается и предварительно формирует содержимое сокета через класс. Класс ссылается на форму один следующим образом

 //form1
telnet tc;

private void connect_Click(object sender, EventArgs e)
{
  tc = new telnet (IP, Port);
}
  

То, что я пытаюсь сделать, это использовать этот же метод в form2. form2 — это окно конфигурации, требующее получения данных с сервера telnet для сохранения в XML-файл. через

  //form 1
 tc.read();
  

Я мог бы вызвать новый экземпляр класса telnet, но это было бы пустой тратой ресурсов, и я думаю, что есть лучший способ его закодировать.

Итак, что я хотел бы знать, находясь в form2. Как мне сообщить событию нажатия кнопки для доступа к tc из form1?

До сих пор я не смог понять, как получить ссылку на form1 tc или кнопку connect_click для формы 2.

  //form2
 private void read_click(object sender, EventArgs e)
 {
   tc.read();  // how do i call it? or even call a method of form 1 to button click?
   ... write data from tc.read()
  }
  

Я знаю, что это шаг за пределы вашей текущей проблемы, но это будет важно в будущем и решит эту проблему, связанную с тем, где создавать экземпляр класса и как совместно использовать этот экземпляр.

Не создавайте экземпляр telnet класса ни в одной из форм. Используйте Dependency Injection вместо этого. Создайте экземпляр Telnet на верхнем уровне вашего приложения и передайте его обеим формам.

Это поможет вам разделить проблемы в вашем приложении и упростит модульное тестирование вашего приложения в будущем (потому что вы можете создать подделку Telnet для их использования).

Что-то вроде:

 public static void Main(string[] args)
{
    // maybe get these from the command line, or from app.config
    const string ipAddress = "10.0.0.1";
    const string port = "80";

    var telnet = new Telnet(ipAddress, port);

    var mainForm = new MainForm(telnet);
    mainForm.ShowDialog();
}

// ...

public class MainForm : Form
{
    private Telnet telnet;

    public MainForm(Telnet telnet)
    {
        this.telnet = telnet;
    }

    // Todo: Use telnet in other methods

    private void ShowSubForm()
    {
        var subForm = new SubForm(telnet);
        subForm.Show();
    }
}
  

Это лучше, чем одноэлементный или статический экземпляр, потому что это позволит вам легче изменять ваше приложение в будущем, позволит вам повторно использовать ваш Telnet класс в будущем, не ограничит вас наличием только одного Telnet экземпляра для каждой программы и значительно упростит написание модульных тестов для вашего приложения.код, когда вы дойдете до этой точки.

Если вам нужно получить информацию о том, какой IP и порт использовать из основной формы, используйте шаблон Factory, создайте factory Main и передайте его в форму. Затем вызовите factory в конструкторе MainForm :

 public class TelnetFactory
{
    public Telnet Create(string ipAddress, string portNumber)
    {
        return new Telnet(ipAddress, portNumber);
    }
}

// ...

public class MainForm : Form
{
    private TelnetFactory telnetFactory;
    private Telnet telnet;

    public MainForm(TelnetFactory telnetFactory)
    {
        this.telnetFactory = telnetFactory;
    }

    // Called by a UI action of some sort...
    private void Connect(string ipAddress, string portNumber)
    {
        if(this.telnet != null)
        {
            this.telnet = telnetFactory.Create(ipAddress, portNumber);
        }
    }

    // Todo: Use telnet in other methods

    // Todo: Just pass the existing telnet instance to SubForm
}
  

Если вы создаете обе формы в третьем промежуточном классе, вместо создания одной формы из другой формы, вы все равно можете использовать эти шаблоны. Просто создайте Telnet экземпляр в этом промежуточном классе или передайте фабрику этому промежуточному классу.

Одним из вариантов является создание одноэлементного объекта, которому принадлежат коммуникации.

http://msdn.microsoft.com/en-us/library/ff650316.aspx

Итак, если у вас было что-то вроде:

 public class Communicator
{
    Telnet m_telnet;

    private static Communicator instance;

    private Communicator() {m_telnet = new Telnet()}

    public static Communicator Instance
    {
        get {
            if (instance == null) {
                instance = new Communicator();
         }
         return instance;
     }

     public string DoSomething()
     {
         return m_telnet.DoSomething();
     }
   }
}
  

Затем в ваших функциях формы вы могли бы просто:

 Communicator.Instance.DoSomething();
  

Поскольку к этому объекту обращаются разные формы, он не должен быть потокобезопасным. Если это так, вы можете использовать шаблон двойной блокировки.

Почему бы не поместить код из первой формы в свой собственный метод и объявить этот метод как общедоступный статический, например.

 public static void connect()
{
  tc = new telnet (IP, Port);
}
  

обновление на основе комментария брайанмака. я предлагал статический метод в интересах быстрого выполнения чего-либо. вот более подходящая, не одноэлементная реализация (на основе кода Брайана)

 public static class Communicator
{
    private static Communicator _instance;
    private Communicator() {}

    public static Telnet Telnet 
    { 
      get
      {
         if (_instance == null) 
         { 
            throw new InvalidOperationException("No telnet session established"); 
         }

         return _instance;
      }
    }

     public static void OpenSession(string ip, int port)
     {
         _instance = new telnet(ip, port);
     }
}

// form1
private void connect_Click(object sender, EventArgs e)
{
  Communicator.OpenSession(IP, Port);
}

//form2
 private void read_click(object sender, EventArgs e)
 {
   Communicator.Telnet.Read();
  }