-
Notifications
You must be signed in to change notification settings - Fork 226
WeakMap and WeakSet #176
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
WeakMap and WeakSet #176
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
discrepo en dejar Array en las traducciones, este review lo dejo como comentario, pero hay unas traducciones pequeñas que se pasaron por alto como "Like this"
Otra solución diferente podría ser agregar una propiedad como `message.isRead = true` a un mensaje después de leerlo. Como los objetos de mensajes son administrados por otro código, generalmente se desaconseja, pero podemos usar una propiedad simbólica para evitar conflictos. | ||
|
||
Like this: | ||
```js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Como esto:
|
||
--- | ||
|
||
# Almacenar banderas "no leídas" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mencionaba Joaquin que banderas se deja como flags, tal como hace MDN
# Almacenar banderas "no leídas" | ||
|
||
Hay un array de mensajes: | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
array es arreglo, incluso MDN dice arreglo
|
||
# Almacenar fechas de lectura | ||
|
||
Hay un array semejante al de la actividad anterior [previous task](info:task/recipients-read). La sitación es similar: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
array es arreglo https://developer.mozilla.org/es/docs/Glossary/Arreglos
|
||
En la tarea anterior solo necesitábamos almacenar el hecho de "sí / no". Ahora necesitamos almacenar la fecha, y solo debe permanecer en la memoria hasta que el mensaje sea recolectado como basura. | ||
|
||
P.D Las fechas se pueden almacenar como objetos de la clase incorporada `Fecha`, que cubriremos más adelante. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
la clase incorporada (builtin) se llama Date: OJO
|
||
Por lo general, las propiedades de un objeto o elementos de un array u otra estructura de datos se consideran accesibles y se mantienen en la memoria mientras esa estructura de datos permanezca en la memoria. | ||
|
||
Por ejemplo, si colocamos un objeto en un array, mientras el array esté vivo, el objeto también estará vivo, incluso si no hay otras referencias a él. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
array es arreglo https://developer.mozilla.org/es/docs/Glossary/Arreglos
john = null; // sobrescribe la referencia | ||
|
||
*!* | ||
// John se almacena dentro del array, por lo que no será recolectado por el recolector de basura |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
busqueda y reemplazo con editor de texto array por arreglo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
traducciones que pasaron por alto y un typo
1-js/05-data-types/weakmap-weakset/01-recipients-read/solution.md
Outdated
Show resolved
Hide resolved
1-js/05-data-types/weakmap-weakset/02-recipients-when-read/task.md
Outdated
Show resolved
Hide resolved
Please make the requested changes. After it, add a comment "/done". |
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>
…k.md 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>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Un par de cositas tipo "suggest" nomás.
Abandono la pelea de glosario array. Cambialo o no, then merge sin asco.
1-js/05-data-types/weakmap-weakset/01-recipients-read/solution.md
Outdated
Show resolved
Hide resolved
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>
Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉 |
No description provided.