GeezerVison is showing github.com — Object.keys, values, entries by EzequielCaste · Pull Request #275 · javascript-tutorial/es.javascript.info
Conversation
ahora si el verdadero object keys
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
key
@ezzep66 @vplentinax
key no es propiedad:
La llave no es la caja. Ppropiedad es el nombre Y el valor asociado
.keys(obj) no devuelve propiedades, solo sus keys. (devuelve solo sus nombres)
" propiedades que utilizan Symbol como propiedades" definicipn circular.
Propiedad puede tener cualquier valor, KEY debe ser string porque es el nombre. (string o symbol).
En la vida real no hace falta la distinción, pero en este capítulo me parece importante,
Por lo menos, donde se hace evidente la contradicción, sugiero "nombre de propiedad" aunq cuando lo sugerí Valentina me reafirmó "establecimos que se traduce clave"
Y me escudo para no aprobarlo en request por sintaxis Pero que lo apruebe otro ¡ja ja!
pero es tu prerogativa y que esto no lo frene, prefiero avanzar. Merge si quiern , pero arreglen el tilde.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
|
Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
joaquinelio
commented
Jul 6, 2020
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
|
@ezzep66
OTRA COSA
aparece " @vplentinax recentrly edited these files"
no encontré commit a "estos archivos"
supongo se refiere a los originales inglés, su último resync los alteró y en la review no los consideré
edit:
no problem, seguro que sale como conflicto 3way merge
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
key
@ezzep66 @vplentinax
key no es propiedad:
La llave no es la caja. Ppropiedad es el nombre Y el valor asociado
.keys(obj) no devuelve propiedades, solo sus keys. (devuelve solo sus nombres)
" propiedades que utilizan Symbol como propiedades" definicipn circular.
Propiedad puede tener cualquier valor, KEY debe ser string porque es el nombre. (string o symbol).
En la vida real no hace falta la distinción, pero en este capítulo me parece importante,
Por lo menos, donde se hace evidente la contradicción, sugiero "nombre de propiedad" aunq cuando lo sugerí Valentina me reafirmó "establecimos que se traduce clave"
Y me escudo para no aprobarlo en request por sintaxis Pero que lo apruebe otro ¡ja ja!
pero es tu prerogativa y que esto no lo frene, prefiero avanzar. Merge si quiern , pero arreglen el tilde.
❓
me encantaría coincidir contigo pero CADA VEZ que busco en internet alguna de tus afirmaciones, encuentro una refutación:
.keys(obj) no devuelve propiedades
El método Object.keys() devuelve un array de las propiedades names de un objeto, en el mismo orden como se obtienen en un loop normal
Te pido por favor @joaquinelio si vas a afirmar algo por favor que sea basado en una fuente oficial como MDN, NO en tus propias conclusiones.
Fuente
Otra fuente
Google es tu amigo 👍
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
joaquinelio
commented
Jul 7, 2020
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
|
TLDR, sorry. Sabés que podés ignorarme y hacer el merge, este capítulo no va a hacer peores programadores.
key
@ezzep66 @vplentinax
key no es propiedad:
La llave no es la caja. Ppropiedad es el nombre Y el valor asociado
.keys(obj) no devuelve propiedades, solo sus keys. (devuelve solo sus nombres)
" propiedades que utilizan Symbol como propiedades" definicipn circular.
Propiedad puede tener cualquier valor, KEY debe ser string porque es el nombre. (string o symbol).
En la vida real no hace falta la distinción, pero en este capítulo me parece importante,
Por lo menos, donde se hace evidente la contradicción, sugiero "nombre de propiedad" aunq cuando lo sugerí Valentina me reafirmó "establecimos que se traduce clave"
Y me escudo para no aprobarlo en request por sintaxis Pero que lo apruebe otro ¡ja ja!
pero es tu prerogativa y que esto no lo frene, prefiero avanzar. Merge si quiern , pero arreglen el tilde.
❓
me encantaría coincidir contigo pero CADA VEZ que busco en internet alguna de tus afirmaciones, encuentro una refutación:
.keys(obj) no devuelve propiedades
El método Object.keys() devuelve un array de las propiedades names de un objeto, en el mismo orden como se obtienen en un loop normal
Te pido por favor @joaquinelio si vas a afirmar algo por favor que sea basado en una fuente oficial como MDN, NO en tus propias conclusiones.
Fuente
Otra fuente
Google es tu amigo 👍
Bueno, tus fuentes están equivocadas. ¡ja ja!
Googlees hoy mi único amigo... pero ¿no te dijeron que es medio bolacero?
Lo que pasa que esta distinción es innecesaria para el que programa, solo me quejo en este capítulo.
Es perfecto decir car["color"] y tratar el nombre "color" como si fuera la propiedad.
Pero "color" no dice nada si no lo ponés en la cerradura que lo asocia a value. El array es uno de strings.
Debí numerar los argumentos, cada renglon es uno. El que más molesta es la definicion circular, el que key solo puede ser (porque es solo el nombre) string (o symbol, uf) el más contundente.
Voy a usar el ej de tu fuente #otra
prop = "color" // string, es solo el nombre
if (key !== prop) // compara strings. ni key ni prop podrian ser otra cosa. (salvo symbol uf)
object[key] = car[key] // car["brand"]
La propiedad tiene un valor, "blue". pero "color" no.
MDN es a veces peor que nosotros, tiene varios "falsos amigos", pero muchos seguramente se arrastran de antes.
Quizás cuando terminemos me voy para ahí para hacer nuevos adversarios... https://wiki.developer.mozilla.org/es/docs/MDN/Contribute/Editor 🚀
Mentira, estoy ACÁ porque es tutorial, la puerta de entrada al mundo JS. Despues deberían estudiar inglés.
Personal, y no relacionado con key.
Decidí hace un año sumarme a la traducción por un motivo: javascript.info EN es mejor que MDN EN y googledev EN. El mejor golpazo lo tuve con un proyecto con localstorage, antes de que el ruso lo agregara.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Milestone
No milestone
Development
Successfully merging this pull request may close these issues.
Uh oh!
There was an error while loading. Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
|