Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Custom errors, extending Error#184

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
cortizg wants to merge2 commits intojavascript-tutorial:masterfromcortizg:es.javascript.info.1-10-2-eh
Closed

Custom errors, extending Error#184

cortizg wants to merge2 commits intojavascript-tutorial:masterfromcortizg:es.javascript.info.1-10-2-eh

Conversation

cortizg
Copy link
Contributor

No description provided.

1-10-2-eh Corregido 7
Copy link
Member

@joaquineliojoaquinelio left a comment
edited
Loading

Choose a reason for hiding this comment

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

Casi,@cortizg. No me deja sugerir, pero podria hacer merge.
Sin embargo, hay tema de glosario, es decir discutible:
WRAPPING. Lo tradujiste como ajuste. Va mas o menos bien, muy entendible, pero en un momento habla de "excepciones de ajuste" como una tecnica generalizada, y eso me molesta, porque está dando un refuerzo a lo que es un WRAPPER: una función que envuelve otra, en unos lados tradujeron como ENVOLTORIO y otros como CONTENEDOR (que acá sonarían forzados). Yo prefería dejar el nombre "wrapper" o aclararlo, pero no sugiero nada y si acuerdan dejarlo así podés ignorar mi request change.
También faltaba 1 (un) acento pero perdí dónde. (No los veo si no hago zoom) No reharía el pr entero por eso.

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

Casi,@cortizg. No me deja sugerir, pero podria hacer merge.
Sin embargo, hay tema de glosario, es decir discutible:
WRAPPING. Lo tradujiste como ajuste. Va mas o menos bien, muy entendible, pero en un momento habla de "excepciones de ajuste" como una tecnica generalizada, y eso me molesta, porque está dando un refuerzo a lo que es un WRAPPER: una función que envuelve otra, en unos lados tradujeron como ENVOLTORIO y otros como CONTENEDOR (que acá sonarían forzados). Yo prefería dejar el nombre "wrapper" o aclararlo, pero no sugiero nada y si acuerdan dejarlo así podés ignorar mi request change.
También faltaba 1 (un) acento pero perdí dónde. (No los veo si no hago zoom) No reharía el pr entero por eso.

Tienes razón, en su momento no vi el error: podria seragrupamiento de excepciones, oEmpacado de excepciones

Copy link
ContributorAuthor

@cortizgcortizg left a comment
edited
Loading

Choose a reason for hiding this comment

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

CorregidoExcepciones de ajuste aEmpacado de excepciones


##Wrapping exceptions
##Excepciones de ajuste
Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Empacado de excepciones


The technique that we describe here is called "wrapping exceptions".
La técnica que describimos aquí se llama "excepciones de ajuste".
Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

La técnica que describimos aquí se llama "empacado de excepciones".


The approach is called "wrapping exceptions",because we take "low level" exceptions and "wrap" them into`ReadError`that is more abstract. It is widely used in object-oriented programming.
El enfoque se llama "excepciones de ajuste",porque tomamos excepciones de "bajo nivel" y las "ajustamos" en`ReadError`que es más abstracto. Es ampliamente utilizado en la programación orientada a objetos.
Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

El enfoque se llama "empacado de excepciones", porque tomamos excepciones de "bajo nivel" y las "empacamos" dentro deReadError que es más abstracto. Es ampliamente utilizado en la programación orientada a objetos.

-Wrapping exceptions is a widespread technique: a function handles low-level exceptions and creates higher-level errors instead of various low-level ones. Low-level exceptions sometimes become properties of that object like`err.cause`in the examples above, but that's not strictly required.
-Podemos heredar de`Error`y otras clases deerrorincorporadas normalmente. Solo necesitamos cuidar la propiedad`name`y no olvidemos llamar `super`.
-Podemos usar`instanceof`para verificar errores particulares. También funciona con herencia. Pero a veces tenemos un objetoerrorque proviene de una biblioteca de terceros y nohay una manera fácil de obtener su clase. Entonces la propiedad`name`puede usarse para tales controles.
-Excepciones de Ajustes es una técnica generalizada: una función maneja excepciones de bajo nivel y crea errores de alto nivel en lugar de varios errores de bajo nivel. Las excepciones de bajo nivel a veces se convierten en propiedades de ese objeto como`err.cause`en los ejemplos anteriores, pero eso no es estrictamente necesario.
Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Empacado de excepciones es una técnica generalizada: una función maneja excepciones de bajo nivel y crea errores de alto nivel en lugar de varios errores de bajo nivel. Las excepciones de bajo nivel a veces se convierten en propiedades de ese objeto comoerr.cause en los ejemplos anteriores, pero eso no es estrictamente necesario.

@cortizg
Copy link
ContributorAuthor

Lo cierro por rama eliminada...

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@joaquineliojoaquineliojoaquinelio requested changes

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@cortizg@javascript-translate-bot@joaquinelio

[8]ページ先頭

©2009-2025 Movatter.jp