Property getters and setters - #194

rainvare · 2020-05-17T21:12:36Z

No description provided.

CLAassistant · 2020-05-17T21:12:41Z

All committers have signed the CLA.

joaquinelio

Para la discusión de glosario, son nombres de elementos JS. ¿los traducimos?
accessor = FUNCION de ACCESO, es una función que accede a propiedades de dato y casi simula ser una. no es "accesorio" aunque una vez programado sea cómodo verlo como tal (la habrían llamado "accessory")
get/set = leer/obtener y escribir/asignar (segun contexto)
getter/setter

joaquinelio · 2020-05-28T02:16:00Z

+El primer tipo son las *propiedades de los datos*. Ya sabemos cómo trabajar con ellas. En realidad, todas las propiedades que hemos estado usando hasta ahora eran propiedades de datos.

-The second type of properties is something new. It's *accessor properties*. They are essentially functions that work on getting and setting a value, but look like regular properties to an external code.
+El segundo tipo de propiedades es algo nuevo. Son las *propiedades de los accesorios*. Estas son esencialmente funciones que trabajan en la obtención y configuración de un valor, pero que parecen propiedades normales de un código externo.


mejor: "propiedades de acceso"
porque el accessor es una función que lee y escribe (get/set)

"configuración" puede ser, pero no es claro (parece obtener y modificar),
prefiero obtención y asignación. Después de la definición las sugiero en inglés pues son palabras muy usadas en programación. get/set getter/setter.

"El segundo tipo de propiedades es algo nuevo. Son las propiedades de acceso (accessor properties). Estas son esencialmente funciones que trabajan en la obtención y establecimiento de un valor, pero que para un código externo parecen propiedades normales ."

El segundo tipo de propiedad es algo nuevo. Son las propiedades de acceso. Esencialmente son funciones que trabajan en la obtención y asignación de valores, pero que parecen propiedades normales fuera del código.

"El segundo tipo de propiedad es..." se deja en singular, verbo "be" lo está

Disculpa, no entiendo. Podrías explicarme de nuevo cuál sería la modificación? @cortizg

Disculpa la demora, soy nueva en esto. Gracias por tus comentarios @joaquinelio

si mi comentario es reciente, creo que ya habian hecho review pero no se hicieron sugerencias de cambio, mi comentario es para que se mejore la traducción un poco, sugerencia: "El segundo tipo de propiedad es algo nuevo. Son las propiedades de acceso. Esencialmente son funciones que trabajan en la obtención y asignación de valores, pero que parecen propiedades normales fuera del código."

joaquinelio · 2020-05-28T02:27:35Z

 ## Getters and setters

-Accessor properties are represented by "getter" and "setter" methods. In an object literal they are denoted by `get` and `set`:
+Las propiedades del accesorio están representadas por los métodos "getter" y "setter". En un objeto con la notación literal se denotan por "get" y "set":


Las propiedades de acceso se representan por los métodos "getter" y "setter". Propiamente en un objeto se denotan por gety set:

creo que se lee mejor.

joaquinelio · 2020-05-28T02:29:48Z


  *!*set propName(value)*/!* {
-    // setter, the code executed on setting obj.propName = value
+    // setter, el código ejecutado al obtener obj.propName = value


// setter, el código ejecutado para asignar obj.propName = value

joaquinelio · 2020-05-28T02:47:03Z

 ```

-Now we want to add a "fullName" property, that should be "John Smith". Of course, we don't want to copy-paste existing information, so we can implement it as an accessor:
+Ahora queremos añadir una propiedad de "nombre completo", que debería ser "John Smith". Por supuesto, no queremos copiar-pegar la información existente, así que podemos aplicarla como un accesorio:


accesorio (algo q puede ser inerte) No es, es una función que accede a otras propiedades. pero no sé como ponerlo. Sugeriría dejarlo en inglés
...aplicarla como un "accessor".
Tema para discutir glosario. Si no,
...aplicarla como una "función de acceso".

no deberia de traducir "fullname" se enrreda más... es una propiedad

"Ahora queremos añadir una propiedad "fullName", que podría ser "John Smith". Por supuesto, no queremos copiar-pegar la información existente, asi que podemos implimentarla como propiedad de acceso:"

No creo que se enrede, la idea de la traducción es presentar al lector el equivalente en su lengua. Si te das cuenta en la línea 56 no lo traduje completamente, de esa forma el lector puede identificar el código del ejemplo y comprender el equivalente. @cortizg

Estimada, no siempre se traduce exactamente igual, por experiencia propia.... ademas se siguen los tips de traducción del Autor, y si se puede se mejora!, Gracias por estar pendiente, creo que la ultima palabran la tendrán los reviewers.

joaquinelio · 2020-05-28T02:55:59Z

 ```

-From outside, an accessor property looks like a regular one. That's the idea of accessor properties. We don't *call* `user.fullName` as a function, we *read* it normally: the getter runs behind the scenes.
+Desde fuera, una propiedad accesoria se parece a una normal. Esa es la idea de las propiedades accesorias. No llamamos a " user.fullName" como una función, la leemos normalmente: el "getter" corre detrás de la escena.


"de acceso" 2 veces
bah, sugiero.

Desde fuera, una propiedad de acceso se parece a una normal. Esa es la idea de estas propiedades. No llamamos a "user.fullName" como una función, la leemos con normalidad: corremos el getter destras de escena.

Acá hay dos sujetos, (nosotros) la leemos y (el getter) corre. Tu corrección no tendría concordancia. @cortizg

nosotros usamos el getter, que es un método de acceso que lee la propiedad user.fullName internamente, no veo la inconcordancia. Saludos

vplentinax · 2020-06-18T22:32:05Z

@joaquinelio , veo que hace un mes hiciste las correcciones pero no ha habido respuesta. Usa mejor el botón de "suggestion" para que sea más facil corregirlo por nosotros mismos.

joaquinelio · 2020-06-19T17:05:28Z

No era para suggestion, era para discusión de glosario.
Con glosario podía hacer la suggestion derecho,
No sabía cómo poner "accessor" (que no es accesorio ) . Volverlo al inglés sin preguntar me parecía de mal gusto.

vplentinax · 2020-06-26T05:52:22Z

@joaquinelio @ezzep66 qué hacemos con este... ¿Merge, review?

Correcciones de terminología.

cortizg · 2020-06-27T21:09:48Z

puse algunos comentarios, saludos

cortizg

la leemos normalmente: el "getter" corre detrás de la escena.

Creo que deberia lerse : "...la leemos normalmente: corremos el "getter" detrás de escena"

que leemos normalmente?: "user.fullName", como corremos el "getter" detrás de escena
Se lee mejor

cortizg · 2020-06-28T01:39:18Z

 - **`configurable`** -- lo mismo que para las propiedades de los datos.

-Por ejemplo, para crear un accesorio " Nombre Completo" con "Definir Propiedad", podemos pasar un descriptor con `get` y `set`:
+Por ejemplo, para crear un acceso " Nombre Completo" con "Definir Propiedad", podemos pasar un descriptor con `get` y `set`:


Suggested change

Por ejemplo, para crear un acceso " Nombre Completo" con "Definir Propiedad", podemos pasar un descriptor con `get` y `set`:

Por ejemplo, para crear un acceso `fullName` con `defineProperty`, podemos pasar un descriptor con `get` y `set`:

javascript-translate-bot · 2020-06-28T01:43:39Z

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

cortizg · 2020-06-28T01:50:36Z

@rainvare Hola como estas, Sugerencia, sería buena idea pasar por este enlace: Glosario

Saludos

cortizg · 2020-06-28T02:15:30Z

/done

joaquinelio

Esta sí es review, lo de antes era para never-ending discusión de glosario
Mucho gusto, @rainvare
Presta atención a los marcadores "MD"
asteriscos - pone la palabra en itálica,
acento grave - la resalta, usualmente porque se incluye en el código
y otros.
También puse cambios que se hicieron en el repo inglés, hacen cambios más seguido que nosotros traduccioes. Y por eso es el mejor.
Ah, se superpusieron los cambios tuyos, de carlos y mio
justo se nos ocurrip editarlo al mismo tiempo,
el resultado se ve algo caótico... espero se entienda.

joaquinelio · 2020-06-28T01:33:21Z

 ## Getters and setters

-Accessor properties are represented by "getter" and "setter" methods. In an object literal they are denoted by `get` and `set`:
+Las propiedades de acceso están representadas por los métodos "getter" y "setter". En un objeto con la notación literal se denotan por "get" y "set":


Suggested change

Las propiedades de acceso están representadas por los métodos "getter" y "setter". En un objeto con la notación literal se denotan por "get" y "set":

Las propiedades de acceso se representan por métodos "getter" y "setter" (de obtención y asignación). Propiamente en un objeto se denotan por `get` y `set`:

joaquinelio · 2020-06-28T01:43:59Z

 ```

-Now we want to add a `fullName` property, that should be `"John Smith"`. Of course, we don't want to copy-paste existing information, so we can implement it as an accessor:
+Ahora queremos añadir una propiedad de "nombre completo", que debería ser "John Smith". Por supuesto, no queremos copiar-pegar la información existente, así que podemos aplicarla como un acceso:


Suggested change

Ahora queremos añadir una propiedad de "nombre completo", que debería ser "John Smith". Por supuesto, no queremos copiar-pegar la información existente, así que podemos aplicarla como un acceso:

Ahora queremos añadir una propiedad `fullName`, que debería ser "John Smith". Por supuesto, no queremos copiar-pegar la información existente, así que podemos implementarla como propiedad de acceso:

joaquinelio · 2020-06-28T03:19:35Z

 ```

-As the result, we have a "virtual" property `fullName`. It is readable and writable.
+


El 30 nov, Ilya borró el smat header. Suele hacer esas cosas: por legibilidad, por redundante, por equivocado.
Deberías borrar el bloque desde
98 "Accessor properties are only accessible with get/set" hasta 106
y QUEDA:
97 Como resultado, tenemos una propiedad virtual fullName que puede leerse y escribirse.

javascript-translate-bot · 2020-06-28T03:53:16Z

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

joaquinelio · 2020-06-29T15:36:17Z

Estimada (como todos con su generoso aporte) @rainvare , espero que la multitud no te intimide :-) Quedó un poco desorganizado porque editamos sugerencias al mismo tiempo y se superpusieron.
Las revisiones son aún tema de discusión, a veces la jerga cambia por zonas, a veces tenemos distintos puntos de vista de cómo quien estudie acá se refuerza o desanima.
No hay palabra final, y tu aporte ES valioso. Que aceptes cambios o que defiendas tu original también.
Gracias.

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Co-authored-by: Carlos Ortiz Gutiérrez <56600925+cortizg@users.noreply.github.com>

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

rainvare · 2020-06-29T15:48:48Z

/done

cortizg · 2020-06-29T16:25:41Z

/done

javascript-translate-bot · 2020-06-30T01:19:17Z

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

Update article.md

84d9a86

javascript-translate-bot added the review needed label May 17, 2020

javascript-translate-bot requested a review from a team May 17, 2020 21:12

iliakan mentioned this pull request May 17, 2020

Spanish Translation Progress #17

Open

joaquinelio reviewed May 28, 2020

View reviewed changes

Merge branch 'master' into rainvare-patch-1

62996fc

Update article.md

05a92e5

Correcciones de terminología.

Update article.md

f9b2439

javascript-translate-bot removed the review needed label Jun 28, 2020

cortizg suggested changes Jun 28, 2020

View reviewed changes

javascript-translate-bot added the changes requested label Jun 28, 2020

joaquinelio requested changes Jun 28, 2020

View reviewed changes

rainvare and others added 8 commits June 29, 2020 12:36

Update 1-js/07-object-properties/02-property-accessors/article.md

78810a6

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 1-js/07-object-properties/02-property-accessors/article.md

1ddc056

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 1-js/07-object-properties/02-property-accessors/article.md

e9b0519

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 1-js/07-object-properties/02-property-accessors/article.md

0988760

Co-authored-by: Carlos Ortiz Gutiérrez <56600925+cortizg@users.noreply.github.com>

Update 1-js/07-object-properties/02-property-accessors/article.md

3cee180

Co-authored-by: Carlos Ortiz Gutiérrez <56600925+cortizg@users.noreply.github.com>

Update 1-js/07-object-properties/02-property-accessors/article.md

c19bdf8

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 1-js/07-object-properties/02-property-accessors/article.md

67303a9

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update article.md

0cb05bb

javascript-translate-bot added review needed and removed changes requested labels Jun 29, 2020

javascript-translate-bot requested a review from a team June 29, 2020 15:48

cortizg approved these changes Jun 29, 2020

View reviewed changes

javascript-translate-bot added needs +1 and removed review needed labels Jun 29, 2020

joaquinelio added 2 commits June 29, 2020 19:29

Update 1-js/07-object-properties/02-property-accessors/article.md

fc70112

Update 1-js/07-object-properties/02-property-accessors/article.md

039e27b

cortizg approved these changes Jun 29, 2020

View reviewed changes

joaquinelio approved these changes Jun 29, 2020

View reviewed changes

joaquinelio self-assigned this Jun 29, 2020

joaquinelio merged commit 25c66f6 into javascript-tutorial:master Jun 30, 2020

	Por ejemplo, para crear un acceso " Nombre Completo" con "Definir Propiedad", podemos pasar un descriptor con `get` y `set`:
	Por ejemplo, para crear un acceso `fullName` con `defineProperty`, podemos pasar un descriptor con `get` y `set`:

	Las propiedades de acceso están representadas por los métodos "getter" y "setter". En un objeto con la notación literal se denotan por "get" y "set":
	Las propiedades de acceso se representan por métodos "getter" y "setter" (de obtención y asignación). Propiamente en un objeto se denotan por `get` y `set`:

	Ahora queremos añadir una propiedad de "nombre completo", que debería ser "John Smith". Por supuesto, no queremos copiar-pegar la información existente, así que podemos aplicarla como un acceso:
	Ahora queremos añadir una propiedad `fullName`, que debería ser "John Smith". Por supuesto, no queremos copiar-pegar la información existente, así que podemos implementarla como propiedad de acceso:

		```

		As the result, we have a "virtual" property `fullName`. It is readable and writable.

Conversation

rainvare commented May 17, 2020

Uh oh!

CLAassistant commented May 17, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

joaquinelio left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

rainvare Jun 28, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cortizg Jun 28, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cortizg Jun 27, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cortizg Jun 27, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

rainvare Jun 28, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cortizg Jun 28, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cortizg Jun 27, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cortizg Jun 28, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vplentinax commented Jun 18, 2020

Uh oh!

joaquinelio commented Jun 19, 2020

Uh oh!

vplentinax commented Jun 26, 2020

Uh oh!

cortizg commented Jun 27, 2020

Uh oh!

cortizg left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cortizg Jun 28, 2020 • edited by joaquinelio Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

javascript-translate-bot commented Jun 28, 2020

Uh oh!

cortizg commented Jun 28, 2020

CLAassistant commented May 17, 2020 •

edited

Loading

rainvare Jun 28, 2020 •

edited

Loading

cortizg Jun 28, 2020 •

edited

Loading

cortizg Jun 27, 2020 •

edited

Loading

cortizg Jun 27, 2020 •

edited

Loading

rainvare Jun 28, 2020 •

edited

Loading

cortizg Jun 28, 2020 •

edited

Loading

cortizg Jun 27, 2020 •

edited

Loading

cortizg Jun 28, 2020 •

edited

Loading

cortizg left a comment •

edited

Loading

cortizg Jun 28, 2020 •

edited by joaquinelio

Loading

joaquinelio left a comment •

edited

Loading