Discussion :

[Invité]

Bonjour, svp voici mon code pour récupere deux tableau de registres modbus ReadHoldingRegisters & ReadInputRegisters , sachant que j'ai 6 ip ou je dois se connecter pour avoir les 2 tableaux pour chaques ip

mon probléme c'est au niveau affichage il me donne
une seul valeur répéter deux fois pour chaque tableau de l'ip 1
2 valeurs répter 2 fois pour chaqe tableau pour ip 2
3 valeurs répter 3 fois pour chaqe tableau pour ip 3

et ainsi de suite , alors que je veux afficher toutes les lignes des mes 2 tableaux un en dessus de un

Que dois-je changer svp

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using EasyModbus;
 
namespace ConsoleApp6
{
    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>> valeurs = new List<List<int>>();
            foreach (string ip in ipadresses)
            {
                ModbusClient modbusClient = new ModbusClient(ip, 502);
                modbusClient.Connect();
                int[] readHoldingRegisters = modbusClient.ReadHoldingRegisters(0, 8);
                int[] readimputregistre = modbusClient.ReadInputRegisters(0, 4);
 
                List<int> i = new List<int>();
 
                foreach(int x in readHoldingRegisters)
                {
                    i.Add(x);
                }
                valeurs.Add(i);
 
                foreach (int y in readimputregistre)
                {
                    i.Add(y);
                }
                valeurs.Add(i);
 
 
                int j =  0;
                foreach (List<int> registrevaleurs in valeurs)
                {
                    Console.WriteLine("Value of imputregistre " + j + " est " + readimputregistre[j].ToString());
                    Console.WriteLine("Value of HoldingRegisters" + j + " est " + readHoldingRegisters[j].ToString());
                }
                j++;
 
                modbusClient.Disconnect();
            }
 
            int valeur = valeurs[0][0];
            Console.WriteLine("La valeurs selectionnée est " + valeur);
 
 
            Console.Write("Press any key to continue . . . ");
            Console.ReadKey(true);
        }
    }
}

[Momoth]

Salut,

Tu pourrai te créer une classe pour avoir tes tableaux au même niveau que ton adresse, voir même un dictionnaire avec tes adresses en clé, vu qu'à priori, elles sont uniques.

[Invité]

Salut,

Tu pourrai te créer une classe pour avoir tes tableaux au même niveau que ton adresse, voir même un dictionnaire avec tes adresses en clé, vu qu'à priori, elles sont uniques.

y a moyen de me détailler un peu votre réponse puisque je suis loin du domaine infos et j'ai besoin de cette partie juste pour récupérer des registres modbus
merci

[Invité]

j'ai mis à jour mon programme

[Momoth]

Je suis assez réticent à donner des réponses toutes faites à des étudiants. Le but du forum est d'aider à progresser et du coup, on est d'accord pour te donner toutes les pistes pour que tu puisses t'en sortir. Après, si tu me dis que tu n'es pas dans le domaine du développement et que tu ne souhaites pas spécialement progresser dans le domaine, on pourrai t'aider oui. C'est pas la première fois que des gens se retrouve à faire du dev sans le vouloir.

Pour la classe, ça pourrai donner quelque chose comme ça :

Code :

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

1
2
3
4
5
6
public class ModbusRegister
    {
        public string Adresse { get; set; }
        public List<int> HoldingRegister { get; set; }
        public List<int> InputRegister { get; set; }
    }

Et a l'utilisation :

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
var valeurs = new List<ModbusRegister>();
 
            foreach (string ip in ipadresses)
            {
                ModbusClient modbusClient = new ModbusClient(ip, 502);
                modbusClient.Connect();
 
                var modbusRegister = new ModbusRegister();
 
                modbusRegister.Adresse = ip;
                modbusRegister.HoldingRegister = modbusClient.ReadHoldingRegisters(0, 8);
                modbusRegister.InputRegister = modbusClient.ReadInputRegisters(0, 8);
 
                valeurs.Add(modbusRegister);
 
                modbusClient.Disconnect();
            }

[Invité]

je vous remercie , c'est gentil