BUG: incorrect lambda closure

Abierto
#667 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
javascript, python
Área
compilers

Línea de trabajo

Start with the Python reproducer and compare its generated JavaScript, especially the lambda's intent and method bindings inside the loop. Trace the lambda translation and scope handling; done means the generated code preserves each iteration's captured values and the reproducer calls method_one successfully.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

STATE: under consideration

see javascript-closure-inside-loops

  1. the following code
def method_one(intent, *args):
    print(f'method_one {intent}: {args}')


def method_two(intent, *args):
    print(f'method_two {intent}: {args}')


config_list = [
    ('one', method_one),
    ('two', method_two),
]

method_dict = dict()

for intent, method in config_list:
    print(f"intent={intent} method={method}")
    method_dict[intent] = lambda *args, intent = intent, method = method : method(intent, args)

method_dict['one']()

  1. produces output in python:
intent=one method=<function method_one at 0x7f7cb7111830>
intent=two method=<function method_two at 0x7f7cb7139950>
method_one one: ((),)
  1. and produces error in javascript:
tester.py:28 Uncaught TypeError: method is not a function
  1. due to translation:
export var method_one = function (intent) {
	var args = tuple ([].slice.apply (arguments).slice (1));
	print ('method_one {}: {}'.format (intent, args));
};
export var method_two = function (intent) {
	var args = tuple ([].slice.apply (arguments).slice (1));
	print ('method_two {}: {}'.format (intent, args));
};
export var config_list = [tuple (['one', method_one]), tuple (['two', method_two])];
export var method_dict = dict ();
for (var [intent, method] of config_list) {
	print ('intent={} method={}'.format (intent, method));
	method_dict [intent] = (function __lambda__ () {
		var intent = intent;
		var method = method;
		var args = tuple ([].slice.apply (arguments).slice (0));
		return method (intent, args);
	});
}
method_dict ['one'] ();
  1. where both intent and method are undefined via incorrect lambda closure:
		var intent = intent;
		var method = method;

6 . and one way to transform is:

for (var [intent, method] of config_list) {
	const intent_ = intent // const
	const method_ = method // const
	print ('intent={} method={}'.format (intent, method));
	method_dict [intent] = (function __lambda__ () {
		var intent = intent_;
		var method = method_;
		var args = tuple ([].slice.apply (arguments).slice (0));
		return method (intent, args);
	});
}

Lenguaje dominante
Python
Estrellas
2.9k
Forks
218
Métricas de merge de PR
Sin PR fusionados en 30 d

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de TranscryptOrg/Transcrypt

Todos los issues de TranscryptOrg/Transcrypt

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.