Discussion :

[salwa17]

Je suis novice en python. Je fais ce bout de code pour calculer le nombre de registre des fonction Modbus Pour cela j'ai besoin de faire une vérification qui est la suivant:
Si funcCode est différent de 5 et 6 faire ceci: r.append (p ['ModbusADURequest'] [1] .startAddr) sinon faire r.append (p ['ModbusADURequest'] [1]. RegisterAddr ) mais j'ai une erreur:

File" parserz.py ", ligne 868 else: ^ SyntaxError: syntaxe invalide

ça doit être une erreur d'indentation ou autre mais je vois pas où.

Une idée s'il vous plait ?

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
def reg_no_list(s):
	r = []
	for p in s:
		if (p['ModbusADUResponse'][1].funcCode != int("0x06", 16)) and (p['ModbusADUResponse'][1].funcCode != int("0x05", 16)):
 
			if p.haslayer('ModbusADURequest'):
				try:
					r.append(p['ModbusADURequest'][1].startAddr)
				except AttributeError:
					pass
			return r
		else:
			r.append(p['ModbusADURequest'][1]. registerAddr)
		except AttributeError:
				pass
	return r
 
def reg_val_list(s):
	v = []
	for p in s:
		if (p['ModbusADUResponse'][1].funcCode != int("0x06", 16)) and (p['ModbusADUResponse'][1].funcCode != int("0x05", 16)):
 
 
		if p.haslayer('ModbusADUResponse'):
			try:
				v += p['ModbusADUResponse'][1].registerVal
			except AttributeError:
				pass
	return v
 
	else:
 
                v += p['ModbusADUResponse'][1]. registerValue
            except AttributeError:
                pass
    return v

[marco056]

le else à la ligne 31 est un peu étrange, non ?
et le except ligne 34 ?
Sinon, pas de message d'erreur ?
Code incompréhensible pour moi sinon.

[salwa17]

En fait, Le code initial était celui là (code 1):

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 reg_no_list(s):
	r = []
	for p in s:
		if p.haslayer('ModbusADURequest'):
			try:
				r.append(p['ModbusADURequest'][1].startAddr)
			except AttributeError:
				pass
	return r
 
def reg_val_list(s):
	v = []
	for p in s:
		if p.haslayer('ModbusADUResponse'):
			try:
				v += p['ModbusADUResponse'][1].registerVal
			except AttributeError:
				pass
	return v

Puis maintenant je chercher à ajouter la condition suivante :

Code :

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

if (p['ModbusADUResponse'][1].funcCode != int("0x06", 16)) and (p['ModbusADUResponse'][1].funcCode != int("0x05", 16)):

Si la condition est satisfaite, on fait le code 1 sinon on fait le code 2 :

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 reg_no_list(s):
    r = []
    for p in s:
        if p.haslayer('ModbusADURequest'):
            try:
                r.append(p['ModbusADURequest'][1]. registerAddr)
            except AttributeError:
                pass
    return r
 
def reg_val_list(s):
    v = []
    for p in s:
        if p.haslayer('ModbusADUResponse'):
            try:
                v += p['ModbusADUResponse'][1]. registerValue
            except AttributeError:
                pass
    return v

[Sve@r]

Bonjour

En fait, Le code initial était celui là (code 1):

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 reg_no_list(s):
	r = []
	for p in s:
		if p.haslayer('ModbusADURequest'):
			try:
				r.append(p['ModbusADURequest'][1].startAddr)
			except AttributeError:
				pass
	return r
 
def reg_val_list(s):
	v = []
	for p in s:
		if p.haslayer('ModbusADUResponse'):
			try:
				v += p['ModbusADUResponse'][1].registerVal
			except AttributeError:
				pass
	return v

Puis maintenant je chercher à ajouter la condition suivante :

Code :

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

if (p['ModbusADUResponse'][1].funcCode != int("0x06", 16)) and (p['ModbusADUResponse'][1].funcCode != int("0x05", 16)):

Si la condition est satisfaite, on fait le code 1 sinon on fait le code 2

Ce que tu appelles "code" sont en réalité des fonctions. Il faut déjà nommer les choses correctement.
Et le souci, c'est que les fonctions sont appelées sans savoir si la condition sera/ne sera pas validée.
Donc soit tu répètes la condition dans chacune des fonctions en l'inversant dans la seconde (et en te souvenant que quand on inverse une condition on inverse aussi ses connecteurs "and"/"or")

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 reg_no_list(s):
	r = []
	for p in s:
		if p.haslayer('ModbusADURequest') and p['ModbusADUResponse'][1].funcCode != int("0x06", 16) and p['ModbusADUResponse'][1].funcCode != int("0x05", 16):
			try:
				r.append(p['ModbusADURequest'][1].startAddr)
			except AttributeError:
				pass
	return r
 
def reg_val_list(s):
	v = []
	for p in s:
		if p.haslayer('ModbusADUResponse') and (p['ModbusADUResponse'][1].funcCode == int("0x06", 16) or p['ModbusADUResponse'][1].funcCode == int("0x05", 16):
			try:
				v += p['ModbusADUResponse'][1].registerVal
			except AttributeError:
				pass
	return v

soit tu réorganise tes deux fonctions (qui en plus sont quand-même très ressemblantes) en une seule qui pourra alors faire toutes les actions selon les conditions

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
def reg_list(s):
	r = []
	for p in s:
		if p['ModbusADUResponse'][1].funcCode != int("0x06", 16) and p['ModbusADUResponse'][1].funcCode != int("0x05", 16):
			if p.haslayer('ModbusADURequest'):
				try:
					r.append(p['ModbusADURequest'][1].startAddr)
				except AttributeError:
					pass
				# try
			# if
		else:
			if p.haslayer('ModbusADUResponse'):
				try:
					r += p['ModbusADUResponse'][1].registerVal
				except AttributeError:
					pass
				# try
			# if
		# if
	# for
	return r
# reg_list()

[salwa17]

Merci de votre aide précieuse.

J'ai adapté les solutions que vous avez postées en fonction de mon besoin et voici le 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
def reg_no_list(s):
	r = []
	for p in s:
		if p['ModbusADURequest'][1].funcCode != int("0x06", 16) and p['ModbusADURequest'][1].funcCode != int("0x05", 16):
			if p.haslayer('ModbusADURequest'):
				try:
					r.append(p['ModbusADURequest'][1].startAddr)
				except AttributeError:
					pass
				# try
			# if
		else:
			if p.haslayer('ModbusADURequest'):
				try:
					r.append(p['ModbusADURequest'][1].registerAddr)
				except AttributeError:
					pass
				# try
			# if
		# if
	# for
	return r
def reg_val_list(s):
	v = []
	for p in s:
		if p['ModbusADURequest'][1].funcCode != int("0x06", 16) and p['ModbusADURequest'][1].funcCode != int("0x05", 16):
			if p.haslayer('ModbusADUResponse'):
				try:
					r.append(p['ModbusADUResponse'][1].registerVal)
				except AttributeError:
					pass
				# try
			# if
		else:
			if p.haslayer('ModbusADUResponse'):
				try:
					r.append(p['ModbusADUResponse'][1].registerValue)
				except AttributeError:
					pass

ça compile bien mais quand je compile le main, j'ai ce message:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Traceback (most recent call last):
  File "main.py", line 334, in <module>
    main()
  File "main.py", line 323, in main
    data = retrieve(parameters, sbs)
  File "main.py", line 78, in retrieve
    p["Register_number"] = parser.reg_no_list(s)
  File "C:\Users\ensibs\Documents\V14-copie\Final\parserz.py", line 860, in reg_no_list
    if p['ModbusADURequest'][1].funcCode != int("0x06", 16) and p['ModbusADURequest'][1].funcCode != int("0x05", 16):
  File "C:\Users\ensibs\AppData\Local\Programs\Python\Python37\lib\site-packages\scapy\packet.py", line 1171, in __getitem__
    raise IndexError("Layer [%s] not found" % lname)
IndexError: Layer ['ModbusADURequest'] not found

Avez-vous une idée ?

[Sve@r]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Traceback (most recent call last):
  File "main.py", line 334, in <module>
    main()
  File "main.py", line 323, in main
    data = retrieve(parameters, sbs)
  File "main.py", line 78, in retrieve
    p["Register_number"] = parser.reg_no_list(s)
  File "C:\Users\ensibs\Documents\V14-copie\Final\parserz.py", line 860, in reg_no_list
    if p['ModbusADURequest'][1].funcCode != int("0x06", 16) and p['ModbusADURequest'][1].funcCode != int("0x05", 16):
  File "C:\Users\ensibs\AppData\Local\Programs\Python\Python37\lib\site-packages\scapy\packet.py", line 1171, in __getitem__
    raise IndexError("Layer [%s] not found" % lname)
IndexError: Layer ['ModbusADURequest'] not found

Avez-vous une idée ?

Oui, on a tous une idée et tous probablement la même: la clef "ModbusADURequest" n'existe pas dans ton dico. Proablement parce que dans la fonction "reg_val_list(s)" tu lui passes pas le même "s" que dans la fonction "reg_no_list(s)"
Et faudrait un peu te sortir les doigts quoi. Parce que si tu dois venir sur le forum demander de t'expliquer les messages Python élémentaires que tu auras durant ta phase de dev tu n'iras pas bien loin !!!

J'ai adapté les solutions que vous avez postées en fonction de mon besoin

Visiblement ton adaptation a foiré !!!

[marco056]

Oui, on a tous une idée et tous probablement la même: la clef "ModbusADURequest" n'existe pas dans ton dico. Proablement parce que dans la fonction "reg_val_list(s)" tu lui passes pas le même "s" que dans la fonction "reg_no_list(s)"
Et faudrait un peu te sortir les doigts quoi. Parce que si tu dois venir sur le forum demander de t'expliquer les messages Python élémentaires que tu auras durant ta phase de dev tu n'iras pas bien loin !!!


Visiblement ton adaptation a foiré !!!

Moi, non, je ne comprends même pas p['string']

[flapili]

bonjour,

à défaut d'avoir le temps de comprendre le problème j'ai vu int("0x06", 16) dans le code, ce n'es pas quelque chose de mal en soit mais il y à mieux: 0x06, l'utilisation de int(x, 16) est bien pour convertir une chaîne de caractères qu'on récupère mais dans le cas de constante autant utiliser la notation qui va 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
>>> int("0x06", 16)
6
>>> int("0xA6", 16)
166
>>> 0xA6
166
>>> 0xa6 # n'est pas sensible à la case
166
>>> 1e5 # une notation scientifique, par contre ça retourne un float
100000.0
>>> 1E5 # idem, pas sensible à la case
100000.0
>>> hex(int(1E3))
'0x3e8'
>>> int(1E3) == int("0X3E8", 16) == 0x3E8 == 1000
True

comme vous le voyez c'est 4 façons différentes d'écrire la même chose

[salwa17]

Vous proposez alors quelque chose de ce genre ?

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
def reg_no_list(s):
	r = []
	for p in s:
		if p['ModbusADURequest'][1].funcCode != 0x06 and p['ModbusADURequest'][1].funcCode != 0x05
			if p.haslayer('ModbusADURequest'):
				try:
					r.append(p['ModbusADURequest'][1].startAddr)
				except AttributeError:
					pass
				# try
			# if
		else:
			if p.haslayer('ModbusADURequest'):
				try:
					r.append(p['ModbusADURequest'][1].registerAddr)
				except AttributeError:
					pass
				# try
			# if
		# if
	# for
	return r
def reg_val_list(s):
	v = []
	for p in s:
		if p['ModbusADUResponse'][1].funcCode != 0x06 and p['ModbusADUResponse'][1].funcCode != 0x05:
			if p.haslayer('ModbusADUResponse'):
				try:
					r.append(p['ModbusADUResponse'][1].registerVal)
				except AttributeError:
					pass
				# try
			# if
		else:
			if p.haslayer('ModbusADUResponse'):
				try:
					r.append(p['ModbusADUResponse'][1].registerValue)
				except AttributeError:
					pass

[flapili]

Déjà il y a un oublier de : pour le if ... je ne suis pas votre interpréteur ... si vous aviez lancé le code vous auriez vu (ou alors c'est que vous avez vous une erreur en rouge sans même la lire)


pour le reste je n'ai aucune idée du contexte, mais vous avez aussi le droit de faire des variables intermédiaires, en plus comme ça vous pourrez utiliser des noms plus explicites pour p et s puisque vous n'allez pas les répéter 30 fois:

Code :

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

1
2
3
4
5
6
func_code = p['ModbusADURequest'][1].funcCode # faut il encore savoir ce qu'est funCode ? function_code ? je met func_code comme je sais pas
 
if func_code != 0x05 and func_code !=0x06:
    foo()
else:
    bar()

oh tiens, je vois que je peux améliorer ma condition, même mieux prendre la logique dans l'autre sens (bar et foo échangent de place)

dire si x n'est pas égal à 1 ni égal à 2 je fait foo sinon bar reviens au même que si x égal 1 ou si x = 2 je fais bar, sinon foo

Code :

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

1
2
3
4
5
6
 
if func_code == 0x05 or func_code == 0x06:
    bar()
 
else:
    foo()

après on peux condenser, x égal a ou x égal b -> x dans [a, b] if func_code == 0x05 or func_code == 0x06: -> if func_code in (0x05, 0x06):donc

Code :

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

1
2
3
4
5
6
 
if if func_code in (0x05, 0x06):
    bar()
 
else:
    foo()

là c'est déjà beaucoup plus clair !
à voir si un travail similaire peut être appliqué ou non

ps: tous les # if / # try n'ont rien à faire dans le code, ça devrait sauter au yeux avec l'indentation et les saut de ligne, par contre commenter les cas avec les if pourquoi pas

Code :

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

1
2
3
4
5
6
 
 
if retour == 1: # donc mon capteur me retourne 1, ça veux dire qu'il a capté une barbe à papa
    foo()
 
elif retour ==2: # ce code de retour là signifie qu'il détecte de la neige au mois de juin

bon c'est la fin de semaine j'ai été folklorique dans mon dernier bout

ps2: après pour ce qui est de la notation hexadécimale il faut l'utiliser s'il y en a besoin, et là ça dépend de ce qu'est funcCode, de comment il est documenté, d'où il provient, ect
si funcCode ne peux prendre que 8 valeurs et qu'elles sont documentées dans un tableau avec pour chaque ligne 0x00 -> code d'erreur; 0x01 -> inactif .... 0x07 -> est en train de faire quelque chose alors autant utilisé 0, 1 ..., 7 si par contre il y a 230 codes documentés en hexadécimal dans un tableau alors là autant utiliser les notations décimales.

[salwa17]

@flapili: woow ça se voit que tu es passionné par Python. Tu as une méthode facile pour expliquer les choses.
Merci pour ton aide. ça marche.

PS: j'ai bien aimé ton coup folklorique vers la fin