Question basique : Comment répéter un appel à une fonction ?

Invité · 16/01/2023, 19h15

Bonjour,

Je m'amuse avec SpeechRecognition pour en faire un Cortana en moins bien mais ma passion c'est de réinventer la roue

!
Bref, j'aimerais répéter la dernière commande que j'ai transmise vocalement à mon script. J'utilise une liste et lambda mais je me demande si je fais les choses correctement.

Exemple bidon :

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
commands = []
 
def main_thread(audio):
    if audio == 'toto':
        func = lambda : do_something_1(1,2,3)
    elif audio == 'tata':
        func = lambda : do_something_2("a","b")
    elif audio == 'répète':
        commands[-1]()
 
    func()
    command.append(func)
 
def do_something_1(arg1, arg2, arg3):
    '''do something'''
    return True
 
def do_something_2(arg1, arg2):
    '''do something else'''
    return False

Je ne vois pas d'autres manières de faire... Dites-moi comment vous auriez géré la chose si le cœur vous en dit !?

Merci d'avance et bonne soirée/journée !

**Sve@r** · 16/01/2023, 19h34

Bonjour

Envoyé par LeNarvalo

Dites-moi comment vous auriez géré la chose si le cœur vous en dit !?

C'est juste un souci de persistance des valeur entre chaque appel (style le "static" du C). Ca ce résoud assez facilement.

Si tu ne veux pas en passer par l'objet, alors moi je passerais par un attribut de fonction (ce qui "lie" visuellement la variable avec la fonction)

Code python :

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
from functools import partial
 
def main_thread(audio):
	if audio == 'toto':
		func=partial(do_something_1, 1,2,3)
		main_thread.history.append(func)
	elif audio == 'tata':
		func=partial(do_something_2, "a","b")
		main_thread.history.append(func)
	elif audio == 'répète':
		func=main_thread.history.pop()
	func()
# main_thread()
 
def do_something_1(arg1, arg2, arg3):
	'''do something'''
	print(arg1, arg2, arg3)
	return True
 
def do_something_2(arg1, arg2):
	'''do something else'''
	print(arg1, arg2)
	return False
 
main_thread.history=list()

Remarque qq part ça reste de l'objet (une fonction étant un objet)...

Invité · 16/01/2023, 19h52

Merci pour ta réponse, je vais me pencher un peu sur functools que je n'utilise jamais.

Par contre squatter la fonction j'en connais un ou deux qui ne devrait pas être content ! Et puis le pop() ça m'empêcher de "re-répéter"...

Je n'ai pas compris le "Si tu ne veux pas en passer par l'objet", il faudrait que je passe par une class dans l'idéal ?

**Sve@r** · 16/01/2023, 20h38

Envoyé par LeNarvalo

Par contre squatter la fonction j'en connais un ou deux qui ne devrait pas être content !

Sisi, c'est autorisé. La fonction c'est un objet donc tu as le droit de lui créer des attributs.

Envoyé par LeNarvalo

Et puis le pop() ça m'empêcher de "re-répéter"...

Une fois que le principe est montré...

Code python :

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
def main_thread(audio):
	if audio == 'toto':
		func=partial(do_something_1, 1,2,3)
		main_thread.history.append((audio, func))
		func()
	elif audio == 'tata':
		func=partial(do_something_2, "a","b")
		main_thread.history.append((audio, func))
		func()
	elif audio == "history":
		print(tuple("%d: %s" % (i, x[0]) for (i, x) in enumerate(main_thread.history)))
	elif isinstance(audio, int):
		(label, func)=main_thread.history[audio]
		print("repeat %d (%s)" % (audio, label))
		func()
# main_thread()

Envoyé par LeNarvalo

Je n'ai pas compris le "Si tu ne veux pas en passer par l'objet", il faudrait que je passe par une class dans l'idéal ?

A la base il n'y a rien d'obligé. Mais avec une classe tu peux commencer une vraie gestion...

Invité · 16/01/2023, 20h44

Envoyé par Sve@r

Sisi, c'est autorisé. La fonction c'est un objet donc tu as le droit de lui créer des attributs.

Hum, ok ! Je pensais que c'était déconseillé comme pour les instances de classe.

Envoyé par Sve@r

A la base il n'y a rien d'obligé. Mais avec une classe tu peux commencer une vraie gestion...

Okay ! Ca me paraît un peu lourd à mettre en œuvre pour le moment.

Merci encore.

**fred1599** · 16/01/2023, 20h56

Envoyé par LeNarvalo

Hum, ok ! Je pensais que c'était déconseillé comme pour les instances de classe.

Déconseillé pour des fonctions de modules dont tu ne connais pas les détails d'implémentation...

**Sve@r** · 16/01/2023, 21h05

Envoyé par LeNarvalo

Hum, ok ! Je pensais que c'était déconseillé comme pour les instances de classe.

C'est vrai que depuis P3, on peut faire autrement. Sauf que j'ai encore du mal à voir comment appliquer l'exemple de ce tuto à ton cas...

[edit]Ok, j'y suis arrivé, sans squatter la fonction

Code python :

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
from functools import partial
 
def thread():
	def action(m):
		nonlocal h
		if m == "toto":
			func=partial(do_something_1, 1, 2, 3)
			h.append((m, func))
			func()
		elif m == "titi":
			func=partial(do_something_2, "x", "y")
			h.append((m, func))
			func()
		elif m == "history":
			print(tuple("%d: %s" % (i, x[0]) for (i, x) in enumerate(h)))
		elif isinstance(m, int):
			(label, func)=h[m]
			print("repeat %d (%s)" % (m, label))
			func()
		# if
	# action()
	h=list()
	return action
# thread()
 
def do_something_1(arg1, arg2, arg3):
	'''do something'''
	print(arg1, arg2, arg3)
	return True
 
def do_something_2(arg1, arg2):
	'''do something else'''
	print(arg1, arg2)
	return False
 
audio=thread()
audio("toto")
audio("titi")
audio("titi")
audio("toto")
audio("history")
audio(-1)
audio(2)

L'astuce c'est que la fonction qu'on appelle (thread) se contente d'initialiser l'historique et de renvoyer une autre fonction qui, elle, fera appel à cet historique lorsqu'elle sera appelée.
Ensuite, quand on appelle l'item créé (audio) on appelle en réalité la fonction "action".

**papajoker** · 17/01/2023, 12h22

bonjour

Envoyé par LeNarvalo

Ca me paraît un peu lourd à mettre en œuvre pour le moment.

Oui, le bon mot est bien "le moment" : Puisque tu ne sais pas où tu vas, il n'est pas encore possible de définir des classes. Autant rester avec un code facilement remplaçable (et pas forcément propre "pour le moment"). Donc, pour moi, ta question me semble beaucoup trop anticipée.

Si tu as besoin d'un historique, tu peux aussi regarder du côté des décorateurs ?

Puisque tu désires ratisser "large", il serait bien d'envisager comme commande 2 mots (lancer/fermer navigateur...). Si tu as le dernier python, la nouvelle forme switch case peut être intéressante pour ton cas.

**fred1599** · 17/01/2023, 09h50

Il y a aussi possibilité avec les dictionnaires comme dis par @josmiley,

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 func_1(): return 1
 
def func_2(): return 42
 
 
def execute_command(audio):
    if audio == "repeat" and commands["repeat"] is not None:
        func = commands["repeat"]
        commands["repeat"] = None
        return func()
 
     if audio == "repeat" and commands["repeat"] is None:
        return None
 
     if audio in commands:
        commands["repeat"] = commands[audio]
        return commands[audio]()
 
    return None
 
 
commands = {
    "toto": func_1,
    "tata": func_2,
    "repeat": None
}
 
assert execute_command("repeat") is None
assert execute_command("tata") == 42
assert commands["repeat"].__name__ == "func_2"
assert execute_command("repeat") == 42
assert commands["repeat"] is None
assert execute_command("tata") == 42
assert execute_command("toto") == 1
assert commands["repeat"].__name__ == "func_1"
assert execute_command("repeat") == 1

**wiztricks** · 22/01/2023, 11h43

Envoyé par LeNarvalo

Je ne vois pas d'autres manières de faire... Dites-moi comment vous auriez géré la chose si le cœur vous en dit !?

On étudie le command pattern pour en comprendre la structure, comment l'adapter à son code et les simplifications dans sa réalisation avec Python.

- W

Invité · 16/01/2023, 22h19

Envoyé par Beginner.

Ben ta méthode est bien je trouve mais pourquoi utiliser une liste, une simple variable (genre last_commande) suffirait, non ?

Oui mais je ne sais pas du tout où je vais donc je prévois large ! ^^

Oki doki Sve@r !

**josmiley** · 17/01/2023, 07h52

Pourquoi pas un dictionnaire ?

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
actions = {"toto":lambda:do_something1(1,2,3),
           "tata":lambda:do_something2("a","b")}
 
def main_thread(audio):
    action = actions.get(audio)
    if action:
        actions["répète"] = action
        action()
    else:
        do_default()
 
 
def do_default():
    print("pas compris ou rien à répéter ")
 
def do_something1(*args):
    print(*args)
 
main_thread("toto")
main_thread("répète")

Peut être même qu'un générateur aurait fait l'affaire.

Question basique : Comment répéter un appel à une fonction ?

Python

Vue hybride

Discussions similaires

Partager

Partager