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

No description provided.

@CLAassistant
Copy link

CLAassistant commented May 17, 2020

CLA assistant check
All committers have signed the CLA.

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.

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

Choose a reason for hiding this comment

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

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

Copy link
Contributor

Choose a reason for hiding this comment

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

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á

Copy link
Contributor Author

@rainvare rainvare Jun 28, 2020

Choose a reason for hiding this comment

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

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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

Copy link
Contributor

@cortizg cortizg Jun 28, 2020

Choose a reason for hiding this comment

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

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

Choose a reason for hiding this comment

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

de acceso

Copy link
Contributor

@cortizg cortizg Jun 27, 2020

Choose a reason for hiding this comment

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

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.

1-js/07-object-properties/02-property-accessors/article.md Outdated Show resolved Hide resolved
},

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

Choose a reason for hiding this comment

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

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

Choose a reason for hiding this comment

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

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

Copy link
Contributor

@cortizg cortizg Jun 27, 2020

Choose a reason for hiding this comment

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

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

Copy link
Contributor Author

@rainvare rainvare Jun 28, 2020

Choose a reason for hiding this comment

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

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

Copy link
Contributor

@cortizg cortizg Jun 28, 2020

Choose a reason for hiding this comment

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

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

Choose a reason for hiding this comment

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

"de acceso" 2 veces
bah, sugiero.

Copy link
Contributor

@cortizg cortizg Jun 27, 2020

Choose a reason for hiding this comment

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

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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

Copy link
Contributor

@cortizg cortizg Jun 28, 2020

Choose a reason for hiding this comment

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

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

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

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

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

Correcciones de terminología.
@cortizg
Copy link
Contributor

cortizg commented Jun 27, 2020

puse algunos comentarios, saludos

Copy link
Contributor

@cortizg cortizg left a comment

Choose a reason for hiding this comment

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

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

1-js/07-object-properties/02-property-accessors/article.md Outdated Show resolved Hide resolved

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

@cortizg cortizg Jun 28, 2020

Choose a reason for hiding this comment

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

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

1-js/07-object-properties/02-property-accessors/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 👻

@cortizg
Copy link
Contributor

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 commented Jun 28, 2020

/done

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.

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.

1-js/07-object-properties/02-property-accessors/article.md Outdated Show resolved Hide resolved
1-js/07-object-properties/02-property-accessors/article.md Outdated Show resolved Hide resolved

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

1-js/07-object-properties/02-property-accessors/article.md Outdated Show resolved Hide resolved
@@ -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:
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
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:

1-js/07-object-properties/02-property-accessors/article.md Outdated Show resolved Hide resolved
1-js/07-object-properties/02-property-accessors/article.md Outdated Show resolved Hide resolved
1-js/07-object-properties/02-property-accessors/article.md Outdated Show resolved Hide resolved
1-js/07-object-properties/02-property-accessors/article.md Outdated Show resolved Hide resolved
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.

Copy link
Member

Choose a reason for hiding this comment

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

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
Member

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 June 29, 2020 12:36
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

/done

@cortizg
Copy link
Contributor

cortizg commented Jun 29, 2020

/done

@joaquinelio joaquinelio self-assigned this Jun 29, 2020
@joaquinelio joaquinelio merged commit 25c66f6 into javascript-tutorial:master Jun 30, 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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants