Простой чат в консоли - проверьте корректность реализации

@kadet_of_viti · Регистрация: 19.12.2013

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

Здравствуйте, подумал может кому то будет полезно или интересно.
Я создал простенький чат в консоли.
Код сервера

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
using System.Text;
using System.Net;
using System.IO;
using System.Net.Sockets;
using System.Threading;
 
namespace server
{
    class Program
    {
        
        
        
        static void Main(string[] args)
        {
            SF ClientName = new SF();
            string clientName = ClientName.GatName();
 
            if (clientName != "error")
            {
                Console.WriteLine("Client name - " + clientName);
 
                SF Chating = new SF();
 
                TcpListener listener =  Chating.TcpList();
 
                Thread resive = new Thread(() => Chating.resivMSG(clientName, listener));
                resive.Start();
 
                Chating.writeMSG(listener);
            }
 
            Console.Read();
        }
    }
}

Функции здесь

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Net;
using System.IO;
using System.Net.Sockets;
 
namespace server
{
    class SF
    {
        public string GatName()
        {
            Console.WriteLine("Whait for connect");
 
            TcpListener listener = new TcpListener(new IPEndPoint(IPAddress.Parse("127.0.0.1"), 12000));
            listener.Start();
 
            try
            {
                // це наш клієнт
                TcpClient client = listener.AcceptTcpClient();
 
                // Прийом повідомлення
                StreamReader sr = new StreamReader(client.GetStream());
                string clientMSG = sr.ReadLine();
                Console.WriteLine("Client test msg  : " + clientMSG);
 
                // Передача повідомлення
                StreamWriter sw = new StreamWriter(client.GetStream());
                sw.AutoFlush = true;
 
                Console.WriteLine("Enter you name");
                sw.WriteLine("Enter you name");
 
                // прийом імені
                string clientName = sr.ReadLine();
 
                Destroy(sr, sw, client, listener);
 
                return clientName;
            }
            catch (Exception er)
            {
                Console.WriteLine(er.ToString());
                return "error";
            }
        }
 
        public void Destroy(StreamReader sr, StreamWriter sw, TcpClient client, TcpListener listener)
        {
            sr.Close();
            sw.Close();
            client.Close();
            listener.Stop();
        }
 
        public void Destroy(StreamWriter sw, TcpClient client, TcpListener listener)
        {
            sw.Close();
            client.Close();
            listener.Stop();
        }
 
        public void Destroy(StreamReader sr, TcpClient client, TcpListener listener)
        {
            sr.Close();
            client.Close();
            listener.Stop();
        }
 
 
        public void writeMSG(TcpListener listener)
        {
            try
            {
                // це наш клієнт
                TcpClient client = listener.AcceptTcpClient();
 
                // Передача повідомлення
                StreamWriter sw = new StreamWriter(client.GetStream());
                sw.AutoFlush = true;
 
                Console.WriteLine("spicing");
 
                while (true)
                {
                    Console.WriteLine("You write");
                    string servMsg = Console.ReadLine();
                    Console.WriteLine("--------------------");
                    if (servMsg == "TheEnd") break;
                    sw.WriteLine(servMsg);
                }
 
                Destroy(sw, client, listener);
 
            }
 
            catch (Exception er)
            {
                Console.WriteLine(er.ToString());
            }
        }
 
        public void resivMSG(string clientName, TcpListener listener)
        {
            try
            {
                // це наш клієнт
                TcpClient client = listener.AcceptTcpClient();
 
                // Прийом повідомлення
                StreamReader sr = new StreamReader(client.GetStream());
                // Передача повідомлення
 
                while (true)
                {
                    string clientMSG = sr.ReadLine();
                    if (clientName == "TheEnd") break;
                    Console.WriteLine("====================");
                    Console.WriteLine(clientName + " write : " + clientMSG);
                    Console.WriteLine("====================");
                }
 
                Destroy(sr, client, listener);
            }
 
            catch (Exception er)
            {
                Console.WriteLine(er.ToString());
            }
        }
 
 
        public TcpListener TcpList()
        {
            TcpListener listener = new TcpListener(new IPEndPoint(IPAddress.Parse("127.0.0.1"), 12000));
            listener.Start();
 
            return listener;
        }
}
}

Клиент

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Net;
using System.Net.Sockets;
using System.IO;
using System.Threading;
 
namespace client
{
    class Program
    {
        static void Main(string[] args)
        {
            CF ClientName = new CF();
 
            if (ClientName.SendName() == 1)
            {
                Console.WriteLine("Go spicking!");
 
                CF Chating = new CF();
 
                Thread resive = new Thread(Chating.resivMSG);
                resive.Start();
 
                Chating.writePSG();
 
                Console.Read();
            }
 
        }  
    }
}

Функции

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Net;
using System.Net.Sockets;
using System.IO;
using System.Threading;
 
namespace client
{
    class CF
    {
        public int SendName()
        {
            try
            {
                TcpClient client = new TcpClient();
                client.Connect(new IPEndPoint(IPAddress.Parse("127.0.0.1"), 12000));
 
                // відправка повідомлення
                StreamWriter sw = new StreamWriter(client.GetStream());
                sw.AutoFlush = true;
                Console.WriteLine("write test mesage");
                string clientMsg = Console.ReadLine();
                sw.WriteLine(clientMsg);
 
                // прийом повідомлення
                StreamReader sr = new StreamReader(client.GetStream());
                string serverMSG = sr.ReadLine();
                Console.WriteLine(" From Server : " + serverMSG);
 
                // відправка імені
                string myName = Console.ReadLine();
                sw.WriteLine(myName);
 
                Destroy(sr, sw, client);
 
                return 1;
            }
 
            catch (Exception er)
            {
                Console.WriteLine(er.ToString());
                return 0;
            }
        }
 
       
 
        public void Destroy(StreamReader sr, StreamWriter sw, TcpClient client)
        {
            sr.Close();
            sw.Close();
            client.Close();
        }
 
 
        public void Destroy(StreamWriter sw, TcpClient client/*, TcpListener listener*/)
        {
            sw.Close();
            client.Close();
        }
 
        public void Destroy(StreamReader sr, TcpClient client/*, TcpListener listener*/)
        {
            sr.Close();
            client.Close();
        }
 
        public void writePSG()
        {
            try
            {
                TcpClient client = new TcpClient();
                client.Connect(new IPEndPoint(IPAddress.Parse("127.0.0.1"), 12000));
 
                // відправка повідомлення
                StreamWriter sw = new StreamWriter(client.GetStream());
                sw.AutoFlush = true;
 
                while (true)
                {
                    Console.WriteLine("You mesage");
                    string clientMSG = Console.ReadLine();
                    Console.WriteLine("--------------------");
                    if (clientMSG == "TheEnd") break;
                    sw.WriteLine(clientMSG);
                }
 
                Destroy(sw, client);
            }
 
            catch (Exception er)
            {
                Console.WriteLine(er.ToString());
            }
        }
 
 
        public void resivMSG()
        {
            try
            {
                TcpClient client = new TcpClient();
                client.Connect(new IPEndPoint(IPAddress.Parse("127.0.0.1"), 12000));
 
                // прийом повідомлення
                StreamReader sr = new StreamReader(client.GetStream());
 
                while (true)
                {
                    string serverMSG = sr.ReadLine();
                    if (serverMSG == "TheEnd") break;
                    Console.WriteLine("====================");
                    Console.WriteLine("Server mesage: " + serverMSG);
                    Console.WriteLine("====================");
                }
 
                Destroy(sr, client);
            }
 
            catch (Exception er)
            {
                Console.WriteLine(er.ToString());
            }
        }
}
}

Как видете клиент и серевер почти идентичны.

Обращиюсь к "знатокам", мне интересны ващи замечания и предложения покоду.
Может кто то имеет предложения по усоверщенствоватию програмы. Просто я студент и мне хочется реализовать интересные задачи.
Кстати этот вариан програмы работаэт локально. Если нужно в сети то на серверной часть там где "127.0.0.1" прописать адрес клиента, а на клиентской - наоборот.
Спасибо за внимание.

@kirdenoff · 13.08.2014, 23:54

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

C#
1
2
3
4
5
interface IChatService
    {
        void SendMessage(string message);
        void ReceiveMessage();
    }

Клиент так же не слишком силён в возможностях, он может говорить и читать сообщения:

C#
1
2
3
4
5
    interface IChatable
    {
        public string Say();
        public void Read(string message);
    }

Потом бы реализовал интерфейсы:

C#
1
2
3
4
5
6
7
8
9
10
11
12
public class ChatService:IChatService 
    {
        public void SendMessage(string message)
        { 
            //some logic here
        }
 
        public void ReceiveMessage()
        { 
            //return some message
        }
    }

Заметим, что мы можем потом расширить класс нашего сервиса. Приведённый пример только пример.

Создадим класс клиента, реализующий интерфейс IChatable:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
public class ChatUser : IChatable
    {
        public string UserName { get; set; }
        public int UserAge { get; set; }
 
        public string Say()
        {  
        }
 
        public void Read(string message)
        { 
        }
    }

Теперь, собственно, упрощённая модель нашего чата могла бы функционировать следующим образом:

C#
1
2
3
4
5
6
7
ChatUser chatUser = new ChatUser();
            ChatService chatService = new ChatService();
 
            chatService.SendMessage(chatUser.Say());
            chatUser.Read(chatService.ReceiveMessage());
 
            chatService.SendMessage(chatUser.UserName);

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

@XRoy · 14.08.2014, 00:04

Более того, и свойства можно вынести в интерфейс, ведь имена есть и у модераторов, администраторов и тд.

@DataPlanner · 14.08.2014, 03:24

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

@tranquil · 14.08.2014, 04:15

Не по теме:

Сообщение от DataPlanner

как козе баян

:D .

@XRoy · 14.08.2014, 16:46

DataPlanner,
Потом с такой архитектурой без абстракций и получается черт-те что.

@Wolfdp · 14.08.2014, 18:51

Не по теме:

Сообщение от kirdenoff

Абстрагировавшись от Вашего кода, я бы начал с написания интерфейсов для сервиса чата и клиентов. Допустим, у нас простейший чат...

И тут я проржал. =) Нет, серьезно -- интерфейсы это конечно круто и читабельно, но если мы пишем один(!) класс на интерфейс, без дальнейших планов модернизации, это бесполезно. Так что соглашусь, что для начала нужно по нормлаьному разбить классы, а уже потом можно и интерфейсами занятся. Просто, когда ты знаешь возможности, писать от интерфейса к реализации легко, а вот без должных знаний и понимания архитектуры, лучше уже просто реализовывать.

Сообщение от kadet_of_viti

while (true)

попробуйте задействовать асинхронные методы, это облегчит чтение.

Сообщение от kadet_of_viti

string clientMSG = sr.ReadLine();
if (clientName == "TheEnd") break;

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

@DataPlanner · 14.08.2014, 20:22

Сообщение от XRoy

Потом с такой архитектурой без абстракций и получается черт-те что.

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

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

@XRoy 871 / 721 / 304 Регистрация: 15.04.2013 Сообщений: 2,047 Записей в блоге: 5
	14.08.2014, 00:04
	Более того, и свойства можно вынести в интерфейс, ведь имена есть и у модераторов, администраторов и тд. 0

@DataPlanner 158 / 187 / 48 Регистрация: 25.11.2013 Сообщений: 978
	14.08.2014, 03:24
	Не пудрите человеку мозги, ваш интерфейс тут нужен как козе баян. 0

@XRoy 871 / 721 / 304 Регистрация: 15.04.2013 Сообщений: 2,047 Записей в блоге: 5
	14.08.2014, 16:46
	DataPlanner, Потом с такой архитектурой без абстракций и получается черт-те что. 0