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 maksumi changed the title Objects Object Jul 25, 2020
@maksumi maksumi changed the title Object Objects Jul 25, 2020
1-js/04-object-basics/01-object/2-hello-object/task.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/2-hello-object/task.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/2-hello-object/task.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/3-is-empty/solution.md Outdated Show resolved Hide resolved

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.
Copy link
Contributor

Choose a reason for hiding this comment

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

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.

Comment on lines 448 to 449
````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.
Copy link
Contributor

Choose a reason for hiding this comment

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

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
Copy link
Contributor

Choose a reason for hiding this comment

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

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`.
Copy link
Contributor

Choose a reason for hiding this comment

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

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
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.

hola, coso!
jjjajajaj

1-js/04-object-basics/01-object/2-hello-object/task.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/2-hello-object/task.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/2-hello-object/task.md Outdated Show resolved Hide resolved
1-js/04-object-basics/01-object/3-is-empty/solution.md Outdated Show resolved Hide resolved
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
Copy link
Member

Choose a reason for hiding this comment

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

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.
Copy link
Member

Choose a reason for hiding this comment

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

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
Copy link
Member

Choose a reason for hiding this comment

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

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.
Copy link
Member

Choose a reason for hiding this comment

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

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.

1-js/04-object-basics/01-object/article.md 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 Murakami and others added 11 commits July 25, 2020 01:15
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>
@@ -1 +1 @@
Just loop over the object and `return false` immediately if there's at least one property.
Solo cre un bucle sobre el objeto y un `return false` inmediatamente si hay al menos una propiedad.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
Solo cre un bucle sobre el objeto y un `return false` inmediatamente si hay al menos una propiedad.
Solo crea un bucle sobre el objeto y, si hay al menos una propiedad, devuelve `false` inmediatamente.

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

Property values are accessible using the dot notation:
Se puede acceder a los valores de las propiedades utilizando la notación de punto:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Aquí no lo tomo como absoluto amigo. "Por" y "de" no hacen diferencia conceptual. Al traductor del artículo en MDN le pareció bien usar "por" y solo eso, vaya, en ambos casos se entiende claro


```js
delete user.age;
```

![user object 3](object-user-delete.svg)

We can also use multiword property names, but then they must be quoted:
También podemos nombrar propiedades con más de una plabra (*multiword*). Pero, de ser así, debemos citarlos `"..."`:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

¡Sí! Pensé lo mismo peero mientras hago labor de traducción a veces se me funde el cerebro y se me acaban las ideas. @danielgrj

1-js/04-object-basics/01-object/2-hello-object/task.md Outdated Show resolved Hide resolved

There's another way to make constant object properties, we'll cover it later in the chapter <info:property-descriptors>.
Hay otra manera de crear propiedades de objeto constantes, las cubriremos después en el capítulo <info:property-descriptors>.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Tampoco quise dar a entender "objetos constantes" Es clara la confusión gramatical. Gracias por la correción

@maksumi
Copy link
Collaborator Author

maksumi commented Jul 26, 2020

/done

@maksumi
Copy link
Collaborator Author

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
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.

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 Murakami and others added 7 commits July 26, 2020 22:43
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 commented Jul 27, 2020

/done

@maksumi
Copy link
Collaborator Author

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
Member

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
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.

ah, no, era acá
aprob

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

Successfully merging this pull request may close these issues.

None yet

5 participants