Skip to content

Object methods, "this" - #343

Merged
maksumi merged 16 commits into
javascript-tutorial:masterfrom
joaquinelio:obmeto
Aug 16, 2020
Merged

maksumi merged 16 commits into
javascript-tutorial:masterfrom
joaquinelio:obmeto

Conversation

@joaquinelio

Copy link
Copy Markdown
Member

No description provided.

@javascript-translate-bot

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review πŸ‘»

joaquinelio and others added 9 commits August 14, 2020 07:13
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
…/task.md

Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
@joaquinelio

Copy link
Copy Markdown
Member Author

/done

@maksumi maksumi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hay algo que me causa confusiΓ³n y creo que es bueno que nos lo verifiquemos. LeΓ­ un diccionario y creo que tienes razΓ³n en el "before" pero no estoy seguro ya que hay cΓ³digo de ejemplo que no puedo observar en modo review, asΓ­ que revisarΓ© el artΓ­culo con formato. Por otro lado a veces es difΓ­cil concentrarse en todo el contexto del artΓ­culo mientras haces review, entonces creo que eso me estΓ‘ pasando. Lo leerΓ© y comprenderΓ© antes de regresar a aprobar en caso de que tengas razΓ³n

AquΓ­ el valor de `this` dentro de `makeUser()` es `undefined`, porque es llamado como una funciΓ³n, no como un mΓ©todo con sintaxis de punto.

The value of `this` is one for the whole function, code blocks and object literals do not affect it.
El valor de `this` es uno para la funciΓ³n entera, bloques de cΓ³digo y objetos literales no lo afectan.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
El valor de `this` es uno para la funciΓ³n entera, bloques de cΓ³digo y objetos literales no lo afectan.
El valor de `this` es uno para la funciΓ³n entera. Bloques de cΓ³digo y objetos literales no lo afectan.

Funciona bien asΓ­ como dos oraciones independientes en el mismo pΓ‘rrafo y pienso que se entiende mejor. Si lo leo con la coma me voy de corrido y pienso que es "funciΓ³n entera, bloques de cΓ³digo y objetos literales" hasta que mi cerebro hace corto circuito al encontrarse con "no lo afectan".

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@maksumi la coma es justamente de pausa en este caso. No se recomiendo usa punto para dar pausa o separar oraciones pequeΓ±as.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

creo qu amerita un " ; " !!!
si estΓ‘ mal no me importa.

Creo que se lee bien en una oracion si lee con el ritmo adecuado,
tambien reo que se puede poner en dos oraciones aunque se corte un poquito

pero asi no hay duda:
El valor de this es uno para la funciΓ³n entera; bloques de cΓ³digo y objetos literales no lo afectan.

podria haber puesto
fun entera, ni bloquess ni lieterales...
fun entera (biques y leterales....)
muy feo.

joaquinelio and others added 4 commits August 15, 2020 19:53
…/solution.md

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio

joaquinelio commented Aug 15, 2020 •

Copy link
Copy Markdown
Member Author

el nombre viejo es usado!
no me parece "esta en uso", porque no lo estΓ‘, trata de usarlo.
igual lo cambie a
Β‘usa el nombre viejo!
asi que al pobre lo siguen usando

a.X.b
cuΓ‘l estΓ‘ delante de la X ?
y cual detras ?
uno escribe una palabra detras de otra...
si, es medio tramposo. Asi qe te hice caso y puse "antes" que no deja lugar a dudas.

No deja uar batch aca "onversation" , hice todo en "files changed" y otrass resolvi
vuelvo aca y aparecen alg sin resolver
igual edite y estan los cambios hechos y loss que no, resuesltos.

@joaquinelio

Copy link
Copy Markdown
Member Author

/done

@maksumi
maksumi merged commit 56ea948 into javascript-tutorial:master Aug 16, 2020
@javascript-translate-bot

Copy link
Copy Markdown

Thank you πŸ’– I updated the Progress Issue #17 πŸŽ‰ πŸŽ‰ πŸŽ‰

@joaquinelio
joaquinelio deleted the obmeto branch August 21, 2020 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants