Prototype methods, objects without __proto__ #195
Conversation
1-08-04-pm Corregido 14 1-08-04-pm Corregido 15
There was a problem hiding this comment.
Bien, sugiero el rearmado de algunas frases.
no se si se debe modificar
index.md (se usa como link, aunque también para mostrar menú)
los svg (y que no sé cómo probar desde acá) creo haber leído que se llaman con parámetros y no se debían modificar pero no pude encontrar dónde.
1-js/08-prototypes/04-prototype-methods/2-dictionary-tostring/task.md
Outdated
Show resolved
Hide resolved
1-js/08-prototypes/04-prototype-methods/2-dictionary-tostring/task.md
Outdated
Show resolved
Hide resolved
|
|
||
| ```js | ||
| // fully identical shallow clone of obj | ||
| // clon superficial totalmente idéntico de obj |
There was a problem hiding this comment.
no entiendo por qué puso shallow, es obj es simplemente "idéntico", nada más. salvo que signifique copia llana. no lo pondría. Tembién me molesta la redundancia del original, pero lo quiere sobresalir de la copia via loop que sí es superficial.
// clon totalmente idéntico de obj
There was a problem hiding this comment.
"shallow" es para identificar que no tiene objetos anidados en su interior
There was a problem hiding this comment.
¡AAAGGH!
DISCULPEN la falta de atención. metí un err donde no lo había
Debería poner un PR al inglés, poque "idéntico" confunde,
hay obj copiados superficialmente (es CORRECTA la palabra) por lo tanto NO idénticos,
y obj copiados recursivamente , IDÉNTICO y no superficial , creí que se refería a esto
There was a problem hiding this comment.
¿seria entonces: clon superficial totalmente idéntico a obj?
There was a problem hiding this comment.
PFFF
si yo estuviera estudiando esto, abandonaría
quizas sea que la capa superficial sí es idéntica
(porque podrías copiar solo prop y metodos y dejar sin copiar el resto)
No me animo a corregir, tengo que repasar el tema. Haciendo pruebas.
@ezzep66 sí parece entenderlo
There was a problem hiding this comment.
fully identical >> totalmente idéntico
shallow clone >> clon superficial
of obj >> de obj
clon superficial de obj totalmente idéntico
me parece...
There was a problem hiding this comment.
ok: clon superficial de obj totalmente idéntico
|
|
||
| The built-in `__proto__` getter/setter is unsafe if we'd want to put user-generated keys in to an object. Just because a user may enter "__proto__" as the key, and there'll be an error with hopefully easy, but generally unpredictable consequences. | ||
| El getter/setter incorporado de `__proto__` no es seguro si queremos poner claves generadas por el usuario en un objeto. Aunque un usuario puede ingresar `"__proto __"` como clave, y habrá un error, con consecuencias levemente esperanzadoras, pero generalmente impredecibles. |
There was a problem hiding this comment.
( ) , y habrá un error, con consecuencias, con esperanza tranquilas pero generalmente impredecibles.
There was a problem hiding this comment.
yo diría
con consecuencias levemente fáciles, pero generalmente impredecibles.
There was a problem hiding this comment.
eehhhmm...
qué significa "levemente fácil". Definitiva y rotundamente y un completo CREO QUE NO :)
Uno pone el parche, potencialmente peligroso. entonces
"easy" acá no es "fácil" sino "suave" ("easy, boy" cuando tranquilizás al perro)
hopefully easy
"consecuencias que, esperamos, sean leves/imperceptibles/no dañinas."
"con suerte, consecuencias leves o nulas" menos literal
take it easy
|
Please make the requested changes. After it, add a comment "/done". |
|
/done |
There was a problem hiding this comment.
@ezzep66
"apruebo"
pero no me animo a "approve"
tengo miedo que haga un merge automático
¿seguimos alimentando el repo? ¿probamos, pongo approve y vemos si el bot hace merge?
porque todavía no sabemos como manejar el repo actual
|
1 change requested me queda colgado eso... |
|
Aprobé sin leer jaja ¿está hecho? Lead this, @ezzep66 te dejo el merge porque en un punto me supera. el bot gerentea más de lo que pensaba, |
|
Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉 |
No description provided.