problème de syntaxe avecif else

**salwa17** · 01/04/2020, 16h11

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** · 01/04/2020, 17h12

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** · 01/04/2020, 17h20

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** · 01/04/2020, 18h05

Bonjour

Envoyé par 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

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** · 02/04/2020, 00h06

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** · 02/04/2020, 00h27

Envoyé par salwa17

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 !!!

Envoyé par salwa17

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

Visiblement ton adaptation a foiré !!!

problème de syntaxe avecif else

Python

Vue hybride

Discussions similaires

Partager

Partager