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

Property getters and setters #194

Merged
merged 14 commits into from Jun 30, 2020

Conversation

@rainvare
Copy link
Contributor

@rainvare rainvare commented May 17, 2020

No description provided.

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es May 17, 2020
@CLAassistant
Copy link

@CLAassistant CLAassistant commented May 17, 2020

CLA assistant check
All committers have signed the CLA.

@iliakan iliakan mentioned this pull request May 17, 2020
159 of 174 tasks complete
Copy link
Contributor

@joaquinelio joaquinelio left a comment

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


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.

This comment has been minimized.

@joaquinelio

joaquinelio May 28, 2020
Contributor

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

This comment has been minimized.

@cortizg

cortizg Jun 27, 2020
Contributor

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á

This comment has been minimized.

@rainvare

rainvare Jun 28, 2020
Author Contributor

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

This comment has been minimized.

@rainvare

rainvare Jun 28, 2020
Author Contributor

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

This comment has been minimized.

@cortizg

cortizg Jun 28, 2020
Contributor

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


## 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":

This comment has been minimized.

@joaquinelio

joaquinelio May 28, 2020
Contributor

de acceso

This comment has been minimized.

@cortizg

cortizg Jun 27, 2020
Contributor

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.

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

This comment has been minimized.

@joaquinelio

joaquinelio May 28, 2020
Contributor

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

@@ -34,7 +34,7 @@ let user = {
};
```

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:

This comment has been minimized.

@joaquinelio

joaquinelio May 28, 2020
Contributor

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

This comment has been minimized.

@cortizg

cortizg Jun 27, 2020
Contributor

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:"

This comment has been minimized.

@rainvare

rainvare Jun 28, 2020
Author Contributor

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

This comment has been minimized.

@cortizg

cortizg Jun 28, 2020
Contributor

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.

@@ -53,11 +53,11 @@ alert(user.fullName); // John Smith
*/!*
```

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.

This comment has been minimized.

@joaquinelio

joaquinelio May 28, 2020
Contributor

"de acceso" 2 veces
bah, sugiero.

This comment has been minimized.

@cortizg

cortizg Jun 27, 2020
Contributor

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.

This comment has been minimized.

@rainvare

rainvare Jun 28, 2020
Author Contributor

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

This comment has been minimized.

@cortizg

cortizg Jun 28, 2020
Contributor

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

@vplentinax
Copy link
Contributor

@vplentinax vplentinax commented Jun 18, 2020

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

@joaquinelio joaquinelio commented Jun 19, 2020

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

@vplentinax vplentinax commented Jun 26, 2020

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

Correcciones de terminología.
@cortizg
Copy link
Contributor

@cortizg cortizg commented Jun 27, 2020

puse algunos comentarios, saludos

Copy link
Contributor

@cortizg cortizg left a comment

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


- **`get`** -- una función sin argumentos, que funciona cuando se lee una propiedad,
- **`set`** -- una función con un argumento, que se llama cuando se establece la propiedad,
- **`enumerable`** -- lo mismo que para las propiedades de los datos,
- **`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`:

This comment has been minimized.

@cortizg

cortizg Jun 28, 2020
Contributor

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

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

@cortizg
Copy link
Contributor

@cortizg cortizg commented Jun 28, 2020

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

Saludos

@cortizg
Copy link
Contributor

@cortizg cortizg commented Jun 28, 2020

/done

Copy link
Contributor

@joaquinelio joaquinelio left a comment

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.


## 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":

This comment has been minimized.

@joaquinelio

joaquinelio Jun 28, 2020
Contributor

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`:
@@ -34,7 +34,7 @@ let user = {
};
```

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:

This comment has been minimized.

@joaquinelio

joaquinelio Jun 28, 2020
Contributor

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:
user.fullName = "Alice Cooper";
alert(user.name); // Alice
alert(user.surname); // Cooper
```

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

This comment has been minimized.

@joaquinelio

joaquinelio Jun 28, 2020
Contributor

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

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 Jun 29, 2020

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.

rainvare and others added 8 commits Jun 29, 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: Carlos Ortiz Gutiérrez <56600925+cortizg@users.noreply.github.com>
Co-authored-by: Carlos Ortiz Gutiérrez <56600925+cortizg@users.noreply.github.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@rainvare
Copy link
Contributor Author

@rainvare rainvare commented Jun 29, 2020

/done

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jun 29, 2020
@cortizg
Copy link
Contributor

@cortizg cortizg commented Jun 29, 2020

/done

joaquinelio added 2 commits Jun 29, 2020
@joaquinelio joaquinelio self-assigned this Jun 29, 2020
@joaquinelio joaquinelio merged commit 25c66f6 into javascript-tutorial:master Jun 30, 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

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