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

Objects #321

Merged
merged 34 commits into from Jul 27, 2020
Merged

Objects #321

merged 34 commits into from Jul 27, 2020

Conversation

@maksumi
Copy link
Collaborator

@maksumi maksumi commented Jul 25, 2020

No description provided.

maksumi added 10 commits Jul 17, 2020
Actualizado
Update
@maksumi maksumi changed the title Objects Object Jul 25, 2020
@maksumi maksumi changed the title Object Objects Jul 25, 2020

The short answer is: "ordered in a special fashion": integer properties are sorted, others appear in creation order. The details follow.
La respuesta corta es: "ordenados de una forma especial": las propiedades de enteros se ordenan, los demás aparecen en el orden de la creación. Entremos en detalle.

This comment has been minimized.

@danielgrj

danielgrj Jul 25, 2020
Contributor

Me parece que enteros necesita números para ser más claro. O conservar integer

Suggested change
La respuesta corta es: "ordenados de una forma especial": las propiedades de enteros se ordenan, los demás aparecen en el orden de la creación. Entremos en detalle.
La respuesta corta es: "ordenados de una forma especial": las propiedades de números enteros se ordenan, los demás aparecen en el orden de la creación. Entremos en detalle.
````smart header="¿Propiedades de enteros? ¿Qué es eso?"
El término "propiedad de enteros" aquí significa que una cadena se puede convertir a y desde desde un entero sin nigún cambio.
Comment on lines 448 to 449

This comment has been minimized.

@danielgrj

danielgrj Jul 25, 2020
Contributor

Suggested change
````smart header="¿Propiedades de enteros? ¿Qué es eso?"
El término "propiedad de enteros" aquí significa que una cadena se puede convertir a y desde desde un entero sin nigún cambio.
````smart header="¿Propiedades de números enteros? ¿Qué es eso?"
El término "propiedad de enteros" aquí significa que una cadena se puede convertir a y desde desde un número entero sin nigún cambio.
alert( String(Math.trunc(Number("49"))) ); // "49", same, integer property
alert( String(Math.trunc(Number("+49"))) ); // "49", not same "+49" ⇒ not integer property
alert( String(Math.trunc(Number("1.2"))) ); // "1", not same "1.2" ⇒ not integer property
// Math.trunc es una función incorporada que elimina la parte decimal

This comment has been minimized.

@danielgrj

danielgrj Jul 25, 2020
Contributor

built-in se refiere a las funciones que vienen por defecto dentro de JavaScript

Suggested change
// Math.trunc es una función incorporada que elimina la parte decimal
// Math.trunc es una función nativa que elimina la parte decimal
- To iterate over an object: `for (let key in obj)` loop.
Operadores adicionales:
- Para eliminar una propiedad: `delete obj.prop`.
-Para comprobar si existe una propiedad con la clave proporcionada: `"key" in obj`.

This comment has been minimized.

@danielgrj

danielgrj Jul 25, 2020
Contributor

Suggested change
-Para comprobar si existe una propiedad con la clave proporcionada: `"key" in obj`.
- Para comprobar si existe una propiedad con la clave proporcionada: `"key" in obj`.
1-js/04-object-basics/01-object/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 👻

Copy link
Contributor

@joaquinelio joaquinelio left a comment

hola, coso!
jjjajajaj

1-js/04-object-basics/01-object/article.md Outdated Show resolved Hide resolved

```js run
let user = { age: 30 };
let key = "age";
alert( *!*key*/!* in user ); // true, property "age" exists
alert( *!*key*/!* in user ); // true, porque su propiedad "age" si existe dentro del objeto

This comment has been minimized.

@joaquinelio

joaquinelio Jul 25, 2020
Contributor

Suggested change
alert( *!*key*/!* in user ); // true, porque su propiedad "age" si existe dentro del objeto
alert( *!*key*/!* in user ); // true, porque su propiedad "age" existe dentro del objeto

Situations like this happen very rarely, because `undefined` should not be explicitly assigned. We mostly use `null` for "unknown" or "empty" values. So the `in` operator is an exotic guest in the code.
Situaciones como esta suceden raramente ya que `undefined` no debe ser explícitamente asignado. Comunmente usamos `null` para valores "desconocidos" o "vacios". Por lo que el operdaor `in` es un invitado exótico en nuestro código.

This comment has been minimized.

@joaquinelio

joaquinelio Jul 25, 2020
Contributor

Suggested change
Situaciones como esta suceden raramente ya que `undefined` no debe ser explícitamente asignado. Comunmente usamos `null` para valores "desconocidos" o "vacios". Por lo que el operdaor `in` es un invitado exótico en nuestro código.
Situaciones como esta suceden raramente ya que `undefined` no debe ser explícitamente asignado. Comúnmente usamos `null` para valores "desconocidos" o "vacíos". Por lo que el operador `in` es un invitado exótico en nuestro código.

Me suena horrible, pero la norma dice que el acento se arrastra y se transforma en esdrújula
me preguntaste por fundeu...
www.fundeu.es/consulta/comun-comunmente-2784/


```js
for (key in object) {
// executes the body for each key among object properties
// se ejecuta e cuerpo para cada clave entre las propiedades del objeto

This comment has been minimized.

@joaquinelio

joaquinelio Jul 25, 2020
Contributor

Suggested change
// se ejecuta e cuerpo para cada clave entre las propiedades del objeto
// se ejecuta el cuerpo para cada clave entre las propiedades del objeto

Also, we could use another variable name here instead of `key`. For instance, `"for (let prop in obj)"` is also widely used.
Además podriamos usar otros nombres de variables en lugar de `key`. Por ejemplo, `"for (let prop in obj)"` también se usa bastante.

This comment has been minimized.

@joaquinelio

joaquinelio Jul 25, 2020
Contributor

Suggested change
Además podriamos usar otros nombres de variables en lugar de `key`. Por ejemplo, `"for (let prop in obj)"` también se usa bastante.
Además podríamos usar otros nombres de variables en lugar de `key`. Por ejemplo, `"for (let prop in obj)"` también se usa bastante.
@javascript-translate-bot

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

maksumi and others added 11 commits Jul 25, 2020
Co-authored-by: Daniel Garcia <jdanielink@gmail.com>
…urce.js

Co-authored-by: Daniel Garcia <jdanielink@gmail.com>
Co-authored-by: Daniel Garcia <jdanielink@gmail.com>
Co-authored-by: Daniel Garcia <jdanielink@gmail.com>
Co-authored-by: Daniel Garcia <jdanielink@gmail.com>
@maksumi
Copy link
Collaborator Author

@maksumi maksumi commented Jul 26, 2020

/done

@maksumi
Copy link
Collaborator Author

@maksumi maksumi commented Jul 26, 2020

Aló... @joaquinelio @danielgrj
Mil gracias por las sugerencias, sin duda quedó mucho mejor. Tome algunas de forma directa y me di la libertad de modificar algunas otras para una compresión todavía más clara al usuario. Sin duda este ha sido uno de los artículos más arduos que he tomado y con una dificultad un poco más elevada en cuanto a traducción pero me alegra el resultado y haber contado con su ayuda. Otras pequeñas sugerencias la verdad ni les hice caso. En cuanto a las traducciones de MDN, no son la mejor referencia para mí @danielgrj ya que hasta los traductores de MDN a veces no son tan claros. En fin. Excelente fin de semana.
Espero su aprobación...

Copy link
Contributor

@joaquinelio joaquinelio left a comment

Aprobar? ni loco,
Pffff una porquería.

apocope "primera", algun tilde que se me habia escapado, etcetcetcetctcetcetc

mmm debería borrar el comentario para no espantar a tu compatriota, ja ja
que hizo buena revisión
explicale que no hay maldad, solo es que soy algo transtornado.

cambios importantes:
varios "llaves" que eran "corchetes"
por las dudas verifiqué el drae
y llaves no es generico como brackets, es especificamente " { } " (cuando crea obj)
y corchetes " [ ] " (cuando accessdes props.)
entonces donde ilya dice "brackets" hay que fijarse cuales

es confuso hacer review.
creo que deberiamos explicar el uso de los label y el /done nunca les hice caso
pero ahora con el repo por fin prolijo, evitaría ssuperposiciones, es decir dessperdicio de trabajo.
Don bot rules!

1-js/04-object-basics/01-object/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/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 👻

maksumi and others added 7 commits Jul 27, 2020
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@maksumi
Copy link
Collaborator Author

@maksumi maksumi commented Jul 27, 2020

/done

@maksumi
Copy link
Collaborator Author

@maksumi maksumi commented Jul 27, 2020

Jajajaja, sale, @joaquinelio en la desesperación de verle merge lo más pronto a este artículo decidí reemplazar todos los "brackets" olvidándome de los "square brackets".
Por favor, no olvides marcar la casillita de Approve and punch the Submmit Review button, @danielgrj, si estás de acuerdo con el commit final o en su defecto manda otra suggest

@joaquinelio
Copy link
Contributor

@joaquinelio joaquinelio commented Jul 27, 2020

Estoy dormido, así que supongo revisaste y cambiaste asi qe esta vez apruebo sin una mirada
lo unico importante era lo del os brackets, esos cosos que se ponen en los dientes

asusta un poco la velocidad en que aparecen pr,
anduve poco pero quiero darles priorridad para que no se desanimen nadie
el año pasado, con los 80 pr de meses muchos habran abandonado por eso...
tenerlo así prolijito es un orgullo...
mientra vpl hace las sync, si no no sé como aguantaríamos el ritmo

saludos y buenas noches

Copy link
Contributor

@joaquinelio joaquinelio left a comment

ah, no, era acá
aprob

@joaquinelio joaquinelio merged commit 94c1c13 into javascript-tutorial:master Jul 27, 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

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