Discussion :

[Duanrad]

Bonjour, Bonsoir,

Je travaille sur Visual Studio 2022 avec pour modèle : Application Windows Forms (.NET Framework).
Je communique par port série avec une alimentation KORAD KA6002P et le problème est que lorsque je lui envoi la commande pour savoir quelle tension il délivre il me répond bien et la valeur s'affiche dans une TextBox, mais lorsque je relance une seconde fois, tout s'efface de la TextBox et plus rien ne s'affiche après.
---------------------------------------------------------------------------------

---------------------------------------------------------------------------------

---------------------------------------------------------------------------------

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
string DataOUT;
private void si_DataReceived(string data) { tBox_ReceiveData.Text = data.Trim(); }
private delegate void SetTextDeleg(string text);      
 
private void bSendData_Click(object sender, EventArgs e)
{
             if (serialPort1.IsOpen) 
            {
                DataOUT = tBox_SendData.Text;
                serialPort1.Write(DataOUT);
                serialPort1.DataReceived += new SerialDataReceivedEventHandler(DataReceivedHandler);
            }
}
 
private void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e) 
{
            string indata = serialPort1.ReadExisting();
            Console.Write(indata);
            BeginInvoke(new SetTextDeleg(si_DataReceived), new object[] { indata });
}

Le but étant que, lorsque je viens changer la tension qu'il doit délivrer et que je lui renvoi la commande d'information, cela vient modifier le texte dans la TextBox et non l'effacer.
Je ne sais pas si j'ai été assez claire et compréhensible, n'hésitez pas à me demander plus d'informations pour être plus précis.

Je vous remercie d'avance de votre aide !

[popo]

Je vois plusieurs problèmes.

Le premier concerne DataReceivedHandler où tu réutilise serialPort1 au lieu de sender.
Donc dans un premier temps :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
private void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e) 
{
     SerialPort sp = (SerialPort)sender;
     String indata = sp .ReadExisting();
     Console.Write(indata);
     BeginInvoke(new SetTextDeleg(si_DataReceived), new object[] { indata });
}

Le deuxième concerne le branchement de l'écoute.
Il faut le faire lors de la création de l'objet SerialPort et non à chaque fois que tu cliques sur le bouton.
Car += new SerialDataReceivedEventHandler(DataReceivedHandler) ajoute un événement à chaque appel.

Ce qui se passe.
Tu cliques une première fois
=> cela branche un premier évènement d'écoute
=> Puis tu envoies une information
=> L'évènement d'écoute se déclenche
=> ReadExisting() obtient l'info.
=> L'info est placée dans le texte

Tu cliques une seconde fois.
=> cela branche un deuxième évènement d'écoute (en plus du premier)
=> Puis tu envoies une information
=> Le PREMIER évènement d'écoute se déclenche
=> ReadExisting() obtient l'info.
=> L'info est placée dans le texte
=> Le SECOND évènement d'écoute se déclenche
=> ReadExisting() n'obtient pas d'info CAR DEJA LUE.
=> Un texte vide est placé dans le texte

[Duanrad]

Merci pour votre aide Popo !

Maintenant, lors de l'appui du bouton d'envoi, j'ai dans la TextBox "0".
----------------------------------------------------------------------------------

----------------------------------------------------------------------------------

----------------------------------------------------------------------------------
Dois-je rappeler le branchement de l'écoute lors d'un autre événement ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

serialPort1.DataReceived += new SerialDataReceivedEventHandler(serialPort1_DataReceivedHandler);

Je met le code en entier pour voir plus clair.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
 
    public partial class Form1 : Form 
    {
        string DataOUT;
        private void si_DataReceived(string data) { tBox_ReceiveData.Text = data.Trim(); } // Permet d'ecrire dans la tBox_ReceiveData - Ports COM
        private delegate void SetTextDeleg(string text);
 
        public Form1() 
        {
            InitializeComponent();
        }
 
        private void Form1_Load(object sender, EventArgs e) 
        {
            string[] ports = SerialPort.GetPortNames();
            cBox_ComPort.Items.AddRange(ports);
        }
 
        private void bRefresh_Click(object sender, EventArgs e) 
        {
            cBox_ComPort.Items.Clear();
            string[] ports = SerialPort.GetPortNames();
            cBox_ComPort.Items.AddRange(ports);
        }
 
        private void bConnect_Click(object sender, EventArgs e) 
        {            
            try 
            { 
                serialPort1.PortName = cBox_ComPort.Text;
                serialPort1.BaudRate = Convert.ToInt32(cBox_Baudrate.Text);
                serialPort1.DataBits = Convert.ToInt32(cBox_DataBits.Text);
                serialPort1.StopBits = (StopBits)Enum.Parse(typeof(StopBits), cBox_StopBits.Text);
                serialPort1.Parity = (Parity)Enum.Parse(typeof(Parity), cBox_Parity.Text);
 
                serialPort1.DataReceived += new SerialDataReceivedEventHandler(serialPort1_DataReceivedHandler);
 
                serialPort1.Open();
 
                progressBar_Connect.Value = 100;
            } 
            catch (Exception err)  
            {
                MessageBox.Show(err.Message, "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
        }
 
        private void bDisconnect_Click(object sender, EventArgs e) 
        {
            if (serialPort1.IsOpen) 
            {
                serialPort1.Close();
                progressBar_Connect.Value = 0;
            }
        }
 
        private void bSendData_Click(object sender, EventArgs e)
        {
            if (serialPort1.IsOpen) 
            {
                DataOUT = tBox_SendData.Text; // recupere le texte ecrit dans la textbox d'envoi
                serialPort1.Write(DataOUT);  
            }
        }
        private void serialPort1_DataReceivedHandler(object sender, SerialDataReceivedEventArgs e) 
        {
            SerialPort sp = (SerialPort)sender;
            string indata = sp.ReadExisting();
            Console.Write(indata);
            BeginInvoke(new SetTextDeleg(si_DataReceived), new object[] { indata }); // ecrit dans tBox_ReceiveData - Ports COM
        }
    }
}

[Duanrad]

J'ai trouvé quelque chose, ajouter un délais dans DataReceivedHandler... Je sais pas si c'est la meilleur solution mais ça fonctionne !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
Thread.Sleep(1000);

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
private void serialPort1_DataReceivedHandler(object sender, SerialDataReceivedEventArgs e) 
{
     SerialPort sp = (SerialPort)sender;
     Thread.Sleep(1000);
     string indata = sp.ReadExisting();
     Console.Write(indata);
     BeginInvoke(new SetTextDeleg(si_DataReceived), new object[] { indata }); // ecrit dans tBox_ReceiveData - Ports COM
}

Merci popo de m'avoir aidé et de m'avoir éclairé !!

[robun]

Bonsoir à tous
J'ai beaucoup appris de ce post mais je n'arrive pas à résoudre ce problème
Dans mon cas BeginInvoke est souligné en rouge
avec comme msg Le BeginInvoke n'existe pas dans ce contexte
Par ailleurs émission réception fonctione bien

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
 
namespace Test_liaisonPc_Arduino {
    /// <summary>
    /// Logique d'interaction pour MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
 
    {
        SerialPort _serialPort = new SerialPort("COM10", 115200, Parity.None, 8, StopBits.One);
        StringComparer stringComparer = StringComparer.OrdinalIgnoreCase;
 
        private delegate void SetTextDeleg(string text);
 
        public MainWindow()
        {
            _serialPort.Handshake = Handshake.None;
            // "sp_DataReceived" is a custom method that I have created
            _serialPort.DataReceived += new SerialDataReceivedEventHandler(sp_DataReceived);
            InitializeComponent();
            // Set the read/write timeouts
            //_serialPort.ReadTimeout  = 500;
            _serialPort.WriteTimeout = 500;
            _serialPort.Open();
        }
 
        private void sp_DataReceived(object sender, SerialDataReceivedEventArgs e)
        {
            SerialPort sp = (SerialPort)sender ;
            Thread.Sleep(1000);
            String indata = sp.ReadExisting();
            BeginInvoke(new SetTextDeleg(si_DataReceived), new object[] { indata });
        }
        private void si_DataReceived(string data) {
            textBox1.Text = data.Trim();
        }
        private void Button_Click(object sender, RoutedEventArgs e)
        {
            _serialPort.Write("0123456789");
        }
    }
}

Merci de votre aide

[Pol63]

tu es en wpf alors que le gars du début était en winforms (différents moteurs d'UI en .net, wpf étant plus récent)

en wpf c'est this.Dispatcher.BeginInvoke()

[robun]

Merci beaucoup Bonne journée