Skip to content

Object.keys, values, entries #275

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

Merged

Conversation

EzequielCaste
Copy link
Contributor

ahora si el verdadero object keys

Copy link
Member

@joaquinelio joaquinelio left a 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.

@javascript-translate-bot

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

@joaquinelio
Copy link
Member

joaquinelio commented Jul 6, 2020

@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

@EzequielCaste
Copy link
Contributor Author

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 👍

EzequielCaste and others added 3 commits July 6, 2020 11:37
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@joaquinelio
Copy link
Member

joaquinelio commented Jul 7, 2020

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.

@EzequielCaste EzequielCaste removed the request for review from vplentinax July 11, 2020 15:16
Copy link
Contributor Author

@EzequielCaste EzequielCaste left a comment

Choose a reason for hiding this comment

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

ok

@EzequielCaste
Copy link
Contributor Author

/done

Copy link
Contributor Author

@EzequielCaste EzequielCaste left a comment

Choose a reason for hiding this comment

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

ok

Copy link
Contributor Author

@EzequielCaste EzequielCaste left a comment

Choose a reason for hiding this comment

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

ok

@EzequielCaste
Copy link
Contributor Author

/done

Copy link
Contributor Author

@EzequielCaste EzequielCaste left a comment

Choose a reason for hiding this comment

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

/done

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

ok

Copy link
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

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

Ok!

@joaquinelio joaquinelio merged commit b46487d into javascript-tutorial:master Jul 13, 2020
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants