Discussion :

[salwa17]

Je suis débutant en python. J'ai développé une fonction qui récupère des décimaux si le code de la fonction est égal à 1 et qui convertit ces décimaux en bits mais le souci c'est que cette fonction ne fonctionne pas. Il me renvoie une liste vide alors qu'elle doit pas. La liste des bits doit contenir bien des bits.

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
 
def coil_val_list(s):
    v = []
    r = []
    i = []
    for p in s:
        if p.haslayer('ModbusADUResponse'):
            try:
                if p['ModbusADUResponse'][1].funcCode == 1:
                    r = v.extend(p['ModbusADUResponse'][1].coilStatus)
                    for i in r:
                        if i == 1:
                            v.extend([1,0,0])
                        else:
                            v += [int(bit)
                    for bit in str( bin(i) )[2:].zfill(8)] [::-1]
            except AttributeError:
                pass
    return v

J'ignore l'original de cet erreur ?

[VinsS]

Salut,

1. La condition if p.haslayer('ModbusADUResponse'): n'est jamais vraie.
2. La condition if p['ModbusADUResponse'][1].funcCode == 1: n'est jamais vraie
3. Tu as un bug ici r = v.extend(p['ModbusA...

Code :

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

1
2
3
4
5
6
7
8
9
10
 
>>> v = [1, 2]
>>> r = v.extend([3, 4])
>>> r
>>> for i in r:
...     pass
... 
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'NoneType' object is not iterable

Étant donné que cette erreur n'est pas levée, je penche plutôt pour une des deux premières causes.

Tu as aussi une bizarerie de syntaxe ici:

Code :

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

1
2
3
 
                            v += [int(bit)
                    for bit in str( bin(i) )[2:].zfill(8)] [::-1]

[marco056]

Quelques "print" bien placés permettraient sans doute de voir quelles lignes ne correspondent pas à ce que tu attends.

[salwa17]

Les deux conditions:
1. La condition if p.haslayer('ModbusADUResponse')
2. La condition if p['ModbusADUResponse'][1].funcCode == 1
Sont vraies (voir l'image), on a bien une réponse modbus et on a bien une fonction dont le code est 1



Pour : v += [int(bit)
for bit in str( bin(i) )[2:].zfill(8)] [::-1]

C'est pour convertir mes décimaux en bits car les fonctions bin () et encode () ne marchaient pas

[marco056]

Enlève le try except pour commencer.