Discussion :

[Invité]

Bonjour , svp j'ai mon code qui doit m'afficher des registres modbus pour 6 ip

Voila mon code :

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
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using EasyModbus;
 
namespace ConsoleApp7
{
    class Program
    {
        static void Main(string[] args)
        {
            var ipadresses = new List<string>();
            int i = 1;
            ipadresses.Add("194.176.3.14");
            ipadresses.Add("194.176.3.10");
            ipadresses.Add("194.176.3.15");
            ipadresses.Add("194.176.3.19");
            ipadresses.Add("194.176.3.20");
            ipadresses.Add("194.176.3.24");
 
            List<List<int>> valeurs = new List<List<int>>();
            List<int> registrevaleurs = new List<int>();
            List<int> registrevaleurs1 = new List<int>();
            List<int> registrevaleurs2 = new List<int>();
 
            foreach (string ip in ipadresses)
            {
                {
 
                    ModbusClient modbusClient = new ModbusClient(ip, 502);
                    modbusClient.Connect();
                    int[] readimputregistre = modbusClient.ReadInputRegisters(1, 10);
                    bool[] readCoils = modbusClient.ReadCoils(1, 5);
                    bool[] readDiscreteInputs = modbusClient.ReadDiscreteInputs(1, 9);
 
                    foreach (int y in readimputregistre)
                    {
                        registrevaleurs.Add(y);
                    }
 
                    valeurs.Add(registrevaleurs);
 
                    foreach (bool x in readCoils)
                    {
                        registrevaleurs1.Add(Convert.ToInt32(x));
                    }
                    valeurs.Add(registrevaleurs1);
 
                    foreach (bool z in readDiscreteInputs)
                    {
                        registrevaleurs2.Add(Convert.ToInt32(z));
                    }
                    valeurs.Add(registrevaleurs2);
                    modbusClient.Disconnect();
                }
                foreach (int x in registrevaleurs)
                {
                    Console.WriteLine("ReadInputRegisters = "+ x);
                }
                i++;
 
                foreach (int y in registrevaleurs1)
                {
                    Console.WriteLine("ReadCoils = " + y);
                }
                foreach (int z in registrevaleurs2)
                {
                    Console.WriteLine("ReadDiscreteInputs = " + z);
                }
            }
 
 
            /* for (i = 1; i < registrevaleurs2.Count; i++)
             {
                 Console.WriteLine("Value of imputregistre " + i + " est " + registrevaleurs2[i].ToString());
                 // Console.WriteLine("Value of HoldingRegisters" + j + " est " + readHoldingRegisters[j].ToString());
             }*/
            /*  for (i = 1; i < registrevaleurs1.Count; i++)
              {
                 // Console.WriteLine("ReadDiscreteInputs " + i + " est " + registrevaleurs[i].ToString());
              }
 
              for (i = 0; i < registrevaleurs1.Count; i++)
              {
                  Console.WriteLine("imputregistre_wago " + i + " est " + registrevaleurs1[i].ToString());
              }*/
            //int valeur = valeurs[5][1];
            //   Console.WriteLine("test "+ valeur);
 
            Console.Write("Press any key to continue . . . ");
            Console.ReadKey(true);
 
        }
    }
}

et ce qui m'affiche en haut de consule qui est bon et qui correspondant à ce que je veux



mais juste en dessus il m'affiches d'autres registres que je sais pas d'ou il les a pris




Je sais que c'est un probléme d’accolades quelque part , y a quelqu'un qui peut m'aider à bien afficher mes registres pour chaque ip de la liste des ip

[KiLVaiDeN]

Bonjour,

En regardant ton code, j'ai remarqué plusieurs choses, mais tout d'abord, je crois qu'il faudrait que tu renommes certaines variables pour y voir plus clair :

Si tu renommes :

• registrevaleurs en registreInputValeurs
• registrevaleurs1 en registreCoilValeurs
• registrevaleurs2 en registreDiscreteValeurs

Et :

• x en myInputValue
• y en myCoilValue
• z en myDiscriteValue

Tu devrais y voir beaucoup plus clair et repérer les endroits où potentiellement, selon ce que j'ai observé, tu as mélangés les variables ;-)

A+ :-)

[Invité]

Merci pour votre réponse , le voila mon code aprés modification , j'ai pu avoir les registres que je veux pour chaque ip

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using EasyModbus;
 
namespace ConsoleApp7
{
    class Program
    {
        static void Main(string[] args)
        {
            var ipadresses = new List<string>();
 
            ipadresses.Add("194.176.3.14");
            ipadresses.Add("194.176.3.10");
            ipadresses.Add("194.176.3.15");
            ipadresses.Add("194.176.3.19");
            ipadresses.Add("194.176.3.20");
            ipadresses.Add("194.176.3.24");
 
            List<List<int>> valeursregistres = new List<List<int>>();
            List<int> read_imputregistre = new List<int>();
            List<int> read_Coils = new List<int>();
 
 
            foreach (string ip in ipadresses)
            {
                    ModbusClient modbusClient = new ModbusClient(ip, 502);
                    modbusClient.Connect();
                    int[] readimputregistre = modbusClient.ReadInputRegisters(1, 10);
                    bool[] readCoils = modbusClient.ReadCoils(1, 5);
 
                    foreach (int y in readimputregistre)
                    {
                    read_imputregistre.Add(y);
                    Console.WriteLine("readimputregistre = " + y);
                    }
 
                valeursregistres.Add(read_imputregistre);
 
                    foreach (bool x in readCoils)
                    {
                    read_Coils.Add(Convert.ToInt32(x));
                    Console.WriteLine("readCoils = " + x);
                    }
                valeursregistres.Add(read_Coils);
                 modbusClient.Disconnect();
            }
 
            int valeur = valeursregistres[5][13];
          Console.WriteLine("test "+ valeur);
 
            Console.Write("Press any key to continue . . . ");
            Console.ReadKey(true);
 
        }
    }
}

une capture de ce que j'ai pu avoir pour l'affichage de l'ip 5 et 6 par exemple ,


mon probléme c'est pour parcourir mes listes , quand je mets par exemple la 3 valeur de l'ip 6 je peux avoir ce que je veux





mais que j'essaie la meme chose avec l'ip 5 par exemple il me donne n'importe quoi




vous avez des explications svp

[KiLVaiDeN]

Peut-être que l'erreur vient de l'index des tableaux : il commence à 0. Accéder à la valeur par [4][3] devrait permettre d'obtenir la bonne valeur, non ?

[Invité]

j'ai essayé mais me donne toujours la mème chose :/

Peut-être que l'erreur vient de l'index des tableaux : il commence à 0. Accéder à la valeur par [4][3] devrait permettre d'obtenir la bonne valeur, non ?

le probléme se pose uniquement pour la liste des readimputregistre pour la liste des readcoils il marche parfaitement et me donne exactement la valeur demandé

[KiLVaiDeN]

Ok il semble que le problème vienne de la logique, potentiellement aussi de la récupération des données ne utilisant la syntaxe des tableaux.

Essayez d'utiliser la fonction "debug" de votre IDE, afin de voir ce qui est crée, et de comprendre pourquoi vous obtenez cette valeur. En tout cas, je peux être quasiment catégorique en affirmant qu'il s'agit d'une de ces deux choses : mauvaise méthode de récupération des données, peut-être s'intéresser aux méthodes spécifiques pour récupérer les éléments d'une List, ou problème de logique, les données ne sont pas insérées comme il faut et comme vous vous y attendez : quoiqu'il en soit, le mode debug devrait apporter des réponses :-)

[Invité]

aprés débogage je vois que dans ma liste valeursregistres il 12 tableaux



chaque tableau contient les registres des 6 ip


donc pour les 6 ip il créer 12 tableau chaque tableau contient tout les registres que j'ai (80 registres)
et du coup pour acceder au 3 éme registre de l'ip 4 je dois mettre



alors que mon idée était d'avoir accés au registre en mettant le numéro d'ip et le numéro de registre
vous avez une idée sur ce que je dois changer ?
NB: je suis automatisien et j'ai besoin de cette partie juste pour récuperer des registre en modbus

[ericlm128]

Pourquoi est tu si méchant

Je te propose d'utiliser un objet.
Voici sa définition :

Code c# :

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

1
2
3
4
5
6
public class CDatas
{
    public string Ip;
    public int[] InputRegisters;
    public bool[] Coils;
}

Et le code dans le Main :

Code c# :

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
List<CDatas> Odatas = new List<CDatas>();
 
List<string> ipadresses = new List<string>();
ipadresses.Add("194.176.3.14");
ipadresses.Add("194.176.3.10");
ipadresses.Add("194.176.3.15");
ipadresses.Add("194.176.3.19");
ipadresses.Add("194.176.3.20");
ipadresses.Add("194.176.3.24");
 
foreach (string ip in ipadresses)
{
    CDatas Odata = new CDatas();
    Odata.Ip = ip;
 
    ModbusClient modbusClient = new ModbusClient(Odata.Ip, 502);
    modbusClient.Connect();
    Odata.InputRegisters = modbusClient.ReadInputRegisters(1, 10);
    Odata.Coils = modbusClient.ReadCoils(1, 5);
    modbusClient.Disconnect();
    Odatas.Add(Odata);
}
 
Console.WriteLine(Odatas[0].Ip);
Console.WriteLine(Odatas[0].InputRegisters[2]);
Console.WriteLine(Odatas[0].Coils[1]);
 
Console.Write("Press any key to continue . . . ");
Console.ReadKey(true);

Ça me parait plus simple et plus lisible en utilisant la POO.

[Invité]

Merci bcp , t'es un ange