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

Object.keys, values, entries #275

Merged

Conversation

@EzequielCaste
Copy link
Contributor

@EzequielCaste EzequielCaste commented Jun 22, 2020

ahora si el verdadero object keys

@EzequielCaste EzequielCaste requested a review from vplentinax Jun 22, 2020
@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jun 22, 2020
@iliakan iliakan mentioned this pull request Jun 22, 2020
159 of 174 tasks complete
Copy link
Contributor

@joaquinelio joaquinelio left a comment

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.

1-js/05-data-types/09-keys-values-entries/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/09-keys-values-entries/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/09-keys-values-entries/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/09-keys-values-entries/article.md Outdated Show resolved Hide resolved
@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
Contributor

@joaquinelio 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

@EzequielCaste EzequielCaste commented Jul 6, 2020

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 Jul 6, 2020
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@joaquinelio
Copy link
Contributor

@joaquinelio 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 Jul 11, 2020
Copy link
Contributor Author

@EzequielCaste EzequielCaste left a comment

ok

@EzequielCaste
Copy link
Contributor Author

@EzequielCaste EzequielCaste commented Jul 11, 2020

/done

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jul 11, 2020
Copy link
Contributor Author

@EzequielCaste EzequielCaste left a comment

ok

Copy link
Contributor Author

@EzequielCaste EzequielCaste left a comment

ok

@EzequielCaste
Copy link
Contributor Author

@EzequielCaste EzequielCaste commented Jul 11, 2020

/done

Copy link
Contributor Author

@EzequielCaste EzequielCaste left a comment

/done

Copy link
Contributor

@joaquinelio joaquinelio left a comment

ok

Copy link
Contributor

@vplentinax vplentinax left a comment

Ok!

@joaquinelio joaquinelio merged commit b46487d into javascript-tutorial:master Jul 13, 2020
1 check passed
1 check passed
license/cla Contributor License Agreement is signed.
Details
@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
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.