Реализация алгоритма цикла

@Игрок_со_Смерть · Регистрация: 06.01.2013

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

Приветствую!
В общем сразу к делу!

Метод подключения клиента TCPClient к серверу.

Условие do .... while почему то не правильно работает или я его организовал не правильно, суть такая я вызываю метод, и он ждет когда появится сервер в сети тогда подключается к нему все остальное время выводит сообщение мол сервер не доступен ожидаем, но вместо этого он выдает 2 раза и все, но когда сервер появляется он бесконечное число раз начинает писать дынные не приняты то есть выполняет вот эту процедуру

C#
1
client.ReciveData += new ClienReciveData(resiveData);

Понять не могу как отработать логику что бы такого не было?

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
void Connecting() //Создание соединения
        {
            string nameClient = textNameClient.Text;
            string ipServer = textIP.Text;
            string port = textPort.Text;
            try
            {
                client = new NetClient(nameClient);//Передаем серверу свое имя
                addItem(client.Status);
                do
                {
                    client.Connect(IPAddress.Parse(ipServer), int.Parse(port)); //Устанавливаем соединение с сервером
                    addItem(client.status);
                    Thread.Sleep(25);
                }
                while (client.isConnect); //Если соединение установлено
 
                client.ReciveData += new ClienReciveData(resiveData);
                textNameServer.Text = client.ServerName;
            }
            catch (SocketException errors)
            {
                string item = String.Format("Ошибка {1}| : {2}", errors.ErrorCode, errors.Message);
                addItem(item);
            }
        }

Тут проект полностью + сервер

@Spectral-Owl · 24.01.2013, 15:32

А ты уверен что именно так и надо: пока есть соединение соединятся? по моему 1 раза вполне хватит, хотя... честно скажу: точно не знаю.

второе: я бы посоветовал обработчик на получение данных выставлять ДО этого странного цикла, который черт на сколько времени может затянуться.

третье: проект у меня не открылся, бо твоя студия новее, но екзеха запускается, обмена данными между клиентом и сервером идёт вроде нормельно. даже между 1 сервером и 2 клиентами

@Игрок_со_Смерть · 24.01.2013, 15:47 **[ТС]**

Смотри обмениваться данными мы можем только после соединения с сервером иначе ошибка.

Ты скорее всего запустил NetServer как клиент и как сервер? если да то это не то=) мой клиент в папке Debug.

Суть приложения такая это будущее ПО для удалённого администрирования =), то есть он должен всегда обмениваться данные с сервером, а если связь с нетом или сервером оборвалась делать перекподключение.

Пытался как то перестроить всю логику приложения и создать метод переподключения ну что то не как не получилось отобрать и сформулировать логику.

@Spectral-Owl · 24.01.2013, 17:00

начну с малого, постепенно буду добавлять новые лузлы:

C#
1
2
3
4
5
6
7
8
9
10
11
        public bool Connect(IPAddress ip, int port) /// Соединение с сервером.
        {
            if (isConnect == true) return false;
 
            try
            {
                client.Connect(ip, port);
                status = "Устанавливаем соединение...";
                if (isConnect)
                {
...

Добавлено через 51 секунду
при этом isConnect это не поле, возвращающее isConnected объекта client

Добавлено через 12 минут

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
public NetClient(string clientName)         /// Создать клиент.
        {
            this.name = clientName;
            client = new TcpClient();
            status = "Создаём клиента ...";
            ThreadReciveData = new Thread(new ThreadStart(_ThreadReciveData));
        }
......
void _ThreadReciveData() /// Поток проверки полученых данных.
        {
            while (true)
            {
                if (client.Available == 0)
                {
                    oldReciveDataSize = 0;
                    status = "Данных от клиента нет ...";               
                }
 
                if (client.Available > oldReciveDataSize)
                {
                    oldReciveDataSize = client.Available;
                    if (ReciveData != null) ReciveData.Invoke(this);
                }
 
                Thread.Sleep(TimeOutUpdate);
            }
        }

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

Добавлено через 1 минуту
так что рой метод NetClient.Connect(), всё дело в нём. у тебя, если обратишь внимание, флаг isConnect вообще никогда true не будет.

@Игрок_со_Смерть · 25.01.2013, 03:04 **[ТС]**

Ты имел в виду строку

C#
1
if (isConnect == true) return false;

что никогда не будет true?

Ранее процедура выглядела так

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
public bool Connect(IPAddress ip, int port) /// Соединение с сервером.
        {
            if (isConnect == true) return false;
 
            try
            {
                //client.Connect(ip, port);
                status = "Устанавливаем соединение...";
                writer = new BinaryWriter(client.GetStream());
                reader = new BinaryReader(client.GetStream());
                writer.Write(name);
                Thread.Sleep(50);
                serverName = reader.ReadString();
                ThreadReciveData.Start();
                isConnect = true;
                status = "Соединение установлено ...";
                return true;
            }
            catch 
            {
                isConnect = false;
                return false;                
            }

Сообщение от Spectral-Owl

запуск потока находится в недостижимом коде

объясни почему код недостижим? Ранее этот код работал как часики в архиве приложение NETServer там и клиент и сервер.

@Spectral-Owl · 25.01.2013, 10:04

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
 public bool Connect(IPAddress ip, int port) /// Соединение с сервером.
{
            if (isConnect == true) //если подключено...
                     return false;     //то выход, возвращая лож.
 
//т.е. в эту область мы заходим только если уже не подключено.
            try
            {
                client.Connect(ip, port);//что-то там соединяем...
                status = "Устанавливаем соединение...";//что-то там устанавливаем...
               
//===============бесполезные строки...===========================================//
                
                //вот эта строчка меня немного даже веселит.
                if (isConnect) //если программа сдесь, то isConnect НЕ РАВНО true
                {
                    writer = new BinaryWriter(client.GetStream());
                    reader = new BinaryReader(client.GetStream());
                    writer.Write(name);
                    Thread.Sleep(50);
                    serverName = reader.ReadString();
                    ThreadReciveData.Start();
                    isConnect = true;
                    status = "Соединение установлено ...";
                    return true;
                }
//==========================================================================//
 
 
            //полезные строки
                else
                {
                    //то что мне писало при любой попытки соединения
                    status = "Соединение не установлено, повторная попытка через 2 секунды";
                    return false;
                }
                
            }
            catch //тут я бы посоветовал обработать так, чтобы оповещалось об ошибке.
            {
                isConnect = false;
                return false;                
            }
}

Добавлено через 21 минуту

Сообщение от Spectral-Owl

C#
1
if (isConnect) //если программа сдесь, то isConnect НЕ РАВНО true

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

C#
1
2
3
if  (a) {return false;}
if  (a) {a=true;  return true;}
if (!a) {return false;}

@Игрок_со_Смерть · 25.01.2013, 10:16 **[ТС]**

Ты малость не понял =)

Это метод который возвращает если клиент подключен значение False, мол не нужно подключаться, но если он не подключен то идет выполнение кода

C#
1
2
3
4
5
6
7
8
9
 writer = new BinaryWriter(client.GetStream());
                    reader = new BinaryReader(client.GetStream());
                    writer.Write(name);
                    Thread.Sleep(50);
                    serverName = reader.ReadString();
                    ThreadReciveData.Start();
                    isConnect = true;
                    status = "Соединение установлено ...";
                    return true;

Условие

C#
1
2
3
4
5
6
7
8
9
 
                if (isConnect) 
                {
                   ...............
                else
                {
                    status = "Соединение не установлено, повторная попытка через 2 секунды";
                    return false;
                }

Я просто забыл удалить перед тем как выложил, и вопрос был в правильной реализации алгоритма

вот что у меня получилось
NETClient
Form1

@Spectral-Owl · 25.01.2013, 14:59

очень смущает, теперь, метод:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
public void Connecting() //Создание содинения
        {
            do
            {
                Connected(textNameClient.Text, textIP.Text, textPort.Text);
                Thread.Sleep(int.Parse(textTimeReconnection.Text)*1000);
                if (Connected(textNameClient.Text, textIP.Text, textPort.Text) == true) 
                {
                    break;
                }
            }
            while (Connected(textNameClient.Text, textIP.Text, textPort.Text) == true);
        }

он 3 раза пробует выполнять метод Connect, при этом, на мой взгляд, цикл там совершенно ни к чему.
в условии if он выйден из цикла если метод вернёт 1, в условии while он выйдет из цикла если метод вернёт 0.

вообще грамотнее этот метод, на мой взгляд, было сделать какнибудь так:

C#
1
2
3
4
5
6
bool isCon=false;
while (!isCon)
{
isCon=Connect(...);
Thread.Sleep(50);
}

а так, опыта работа с сетевыми программами у меня мало, но вопросы возникают ещё по поводу этого:

C#
1
2
3
4
5
6
7
...
client.ReciveData += new ClienReciveData(resiveData);
...
void resiveData(NetClient c)
{
     addItem("Получены данные: " + client.Reader.ReadString());
}

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

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

@Игрок_со_Смерть · 25.01.2013, 15:18 **[ТС]**

Спасибо большое за помощь, но это не решает проблемы=)

Тут как бы 2 задачи ожидать появления сервака, и при появлении подключаться, а при разрыве пере подключаться.

Согласно этому

C#
1
2
3
4
5
6
bool isCon=false;
while (!isCon)
{
isCon=Connect(...);
Thread.Sleep(50);
}

Клиент один раз пробует подключится и все, после того как сервак появляется то никаких действий от клиента не производятся =((

метод

C#
1
2
3
4
5
6
client.ReciveData += new ClienReciveData(resiveData);
...
void resiveData(NetClient c)
{
     addItem("Получены данные: " + client.Reader.ReadString());
}

Вызывает чтение всех данных с сокета и выводит их в listBox.

@Spectral-Owl · 25.01.2013, 18:03

Если ты не переписал метод NetClient.ConnectToServer() то проблемма в нём. хоть ты и сказал что всего-лишь забыл удалить раньше и перевыложил, но в новом файле та-же проблема: стоит первое странное условие.

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
public bool ConnectToServer(IPAddress ip, int port) /// Соединение с сервером.
{
            if (isConnect == true) return false; //вот оно.
 
            try
            {
                client.Connect(ip, port);
                status = "Устанавливаем соединение...";
                writer = new BinaryWriter(client.GetStream());
                reader = new BinaryReader(client.GetStream());
                writer.Write(name);
                Thread.Sleep(50);
                serverName = reader.ReadString();
                ThreadReciveData.Start();
                isConnect = true;
                status = "Соединение установлено ...";
                return true;
            }
            catch
            {
                isConnect = false;
                return false;
            }
}

Второе:

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
        void _ThreadReciveData() 
        {
            while (true)
            {
                if (client.Available == 0)
                {
                    oldReciveDataSize = 0;
                    status = "Данных от клиента нет ...";
                }
 
                if (client.Available > oldReciveDataSize)
                {
                    oldReciveDataSize = client.Available;
                    if (ReciveData != null) ReciveData.Invoke(this);
                }
                Thread.Sleep(TimeOutUpdate);
            }
        }
...
void resiveData(NetClient c)
        {
            addItem("Получены данные: " + client.Reader.ReadString());//почему не C.Reader.ReadString()?
//хотя в плане одного объекта должно быть всё равно норм.
        }

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

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
void _ThreadReciveData() 
        {
            while (true)
            {
                if (client.Available == 0)
                {
                    oldReciveDataSize = 0;
                    status = "Данных от клиента нет ...";
                }
 
                if (client.Available > oldReciveDataSize)
                {
                    oldReciveDataSize = client.Available;
                    string recString=this.reader.ReadString();
                    if (ReciveData != null) ReciveData(this, recString);
                }
                Thread.Sleep(TimeOutUpdate);
            }
        }
...
void resiveData(object sender, string recString)
        {
            addItem("Получены данные: "+recString);
        }

ну и следовательно делегат для ReciveData будет иметь вид

C#
1
 public delegate void ClienReciveData(object sender,string recivedString);

третье:

C#
1
2
3
4
5
6
7
8
9
10
11
if (client.Available == 0)
                {
                    oldReciveDataSize = 0;
                    status = "Данных от клиента нет ...";
                }
 
                if (client.Available > oldReciveDataSize)
                {
                    oldReciveDataSize = client.Available;
                    if (ReciveData != null) ReciveData.Invoke(this);
                }

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

А вообще, есть 1 хороший способ организовать поток чтения. некоторые компоненты позволяют выполнять 1 свой метод непозволительно долгое время (банально сколько в тайм-ауте).
к примеру у TcpClient есть свойство Client, а у него есть метод bool Recive(byte[] Buffer), который может выполняться время тайм-аута.

в общем обрати внимание на вот эти строки

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
/// <summary>
        /// Метод на основной поток чтения
        /// </summary>
        private void Read()
        {
            //пока флаг установлен...
            while (WantRead)
            {
                int i = 0;
                //чтение данных с сервера
                try { i = Sock.Receive(bBuffer); }
                //если любая ошибка - остановка
                catch (SocketException e) { Stop(e); return; }
 
                //если принятых байт больше нуля...
                if (i > 0)
                {
                        //получение строки
                        string messaga = Encoding.UTF8.GetString(bBuffer);
 
                        try
                        {
                            //нахождение индекса окончания строки
                            int stopInd = messaga.IndexOf("\0");
                            //выделение сообщения
                            messaga = messaga.Substring(0, stopInd);
                        }
                        catch { }
//вызов соответствющего события
                            if (OnReviceString != null)
                                OnReviceString(this, messaga);
                        }
 
                        //обнуление буффера
                        bBuffer = new byte[1024];
                }
                else
                {
                    //если принятых байт каким-то багом оказалось меньше нуля..
                    //значит сервер обтух и можно отключатся
                    Stop(new Exception("Server shutdown"));
                }
            }
        }

я пару дней не появлюсь

Новые блоги и статьи Все статьи Все блоги /
http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит переходные токи и напряжения на элементах схемы. . . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .

@Spectral-Owl 608 / 583 / 157 Регистрация: 29.06.2010 Сообщений: 1,620
	24.01.2013, 15:32
	А ты уверен что именно так и надо: пока есть соединение соединятся? по моему 1 раза вполне хватит, хотя... честно скажу: точно не знаю. второе: я бы посоветовал обработчик на получение данных выставлять ДО этого странного цикла, который черт на сколько времени может затянуться. третье: проект у меня не открылся, бо твоя студия новее, но екзеха запускается, обмена данными между клиентом и сервером идёт вроде нормельно. даже между 1 сервером и 2 клиентами 0

@Игрок_со_Смерть 37 / 37 / 6 Регистрация: 06.01.2013 Сообщений: 195
	24.01.2013, 15:47 [ТС]
	Смотри обмениваться данными мы можем только после соединения с сервером иначе ошибка. Ты скорее всего запустил NetServer как клиент и как сервер? если да то это не то=) мой клиент в папке Debug. Суть приложения такая это будущее ПО для удалённого администрирования =), то есть он должен всегда обмениваться данные с сервером, а если связь с нетом или сервером оборвалась делать перекподключение. Пытался как то перестроить всю логику приложения и создать метод переподключения ну что то не как не получилось отобрать и сформулировать логику. 0