Клиент - сервер. Где использовать Using для TcpClient?

@Defences · Регистрация: 28.12.2016

Author24 — интернет-сервис помощи студентам

Вот мы запускаем Server

C#

1	new Server(80);

C#

internal class Server
    {
        private readonly TcpListener Listener;
 
        public Server(int Port)
        {
            Listener = new TcpListener(IPAddress.Any, Port);
            Listener.Start();
            while (true)
            {
                ThreadPool.QueueUserWorkItem(new WaitCallback(ClientThread), Listener.AcceptTcpClient());
                int workerThreads = Environment.ProcessorCount * 8;
                ThreadPool.SetMaxThreads(workerThreads, workerThreads);
                ThreadPool.SetMinThreads(2, 2);
            }
        }
 
        public void ClientThread(object StateInfo)
        {
            try
            {
                new Client((TcpClient)StateInfo);
            }
            catch(Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
        }
 
        ~Server()
        {
            Listener?.Stop();
        }
    }

// Тут использовать using?

C#

  class Client
        {
            public Client(TcpClient client)
            {
                using (client)
                {
                    // код
                }
            }
        }

@Usaga · 30.05.2017, 06:01

Defences, класс Client не имеет права закрывать\диспозить объект TcpClient так как он получает его извне, а значит кто-то другой ответственен за управление жизнью данного объекта:

C#

        public void ClientThread(object StateInfo)
        {
            try
            {
                using(var tcpClient = StateInfo as TcpClient) {
                   new Client(tcpClient);
                }
            }
            catch(Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
        }

Замечу, что конструктор класса должен выполнять только инициализацию объекта, но уж ни как не являться полноценным методом с какой-то логикой. Вся логика в классе должна вызываться явно:

C#

        public void ClientThread(object StateInfo)
        {
            try
            {
                using(var tcpClient = StateInfo as TcpClient) {
                   var client = new Client(tcpClient); // только инициализируется, но ничего не делает
                   client.ProcessRequest();
                }
            }
            catch(Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
        }

Применять финализатор (~Server()) нужно только в купе с реализацией интерфейса IDisposable. Это ни разу не тот же деструктор, что и в С++!

	30.05.2017, 06:01

Опции темы