Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cuidar duplicados de nombres de variables #92

Open
asanzo opened this issue Nov 6, 2017 · 2 comments
Open

Cuidar duplicados de nombres de variables #92

asanzo opened this issue Nov 6, 2017 · 2 comments

Comments

@asanzo
Copy link

asanzo commented Nov 6, 2017

Cuando se cambia un nombre, hay que eliminar duplicados

@afska afska changed the title Cuidar duplicados de nombres de procedimiento, parámetros y variables Cuidar duplicados de nombres de parámetros y variables Dec 6, 2017
@afska afska changed the title Cuidar duplicados de nombres de parámetros y variables Cuidar duplicados de nombres de variables Dec 6, 2017
@afska
Copy link
Member

afska commented Dec 6, 2017

Edité el título porque lo que es parámetros y procedimientos lo arreglé.

Con las variables no entiendo muy bien qué habría que hacer...
Si renombro el lado izquierdo de una asignación, no sé si quiero que me renombre la variable o que elimine sus otros usos. Capaz que solo me confundí y en vez de usar A usé B.

Tal vez podríamos pensar esto como "eliminar los usos de variables que no hayan al menos una asignación dentro de un scope"

@asanzo
Copy link
Author

asanzo commented Dec 11, 2017

Blockly ya resuelve casi todo problema de nombres.

Blockly.GobstonesLanguage.variableDB_.getName("nombre de variable", Blockly.VARIABLE_CATEGORY_NAME);

Eso debería hacer magia con cosas duplicadas evitando que sucedan. Con procedimientos y etcéteras es similar, e incluso hay muchos problemas que ya blockly resuelve por defecto y es buena idea chusmearlos y no volver a reinventar la rueda.

Pero igual no termino de entender el problema original del issue, ni el problema actual.

Respecto de lo que mencionás de cambiar el nombre de la variable, podemos charlar mejor qué querríamos hacer, si cambiar todos los variable get, meter un warning, no hacer nada, etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants