5.7. Error Handling (part 4) #101
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| # // handling, like parsing strings as integers or floats. | ||
| // この構造体はCSVファイルの各行のデータを表現します。 | ||
| // 型に基づくデコードは、文字列を整数や浮動小数点数にパースしてしまう | ||
| // といった、核心部分のエラーハンドリングの大半を免除してくれます。 |
There was a problem hiding this comment.
エラーハンドリングの大半を免除してくれます。
「型に基づくデコードは〜を免除してくれます」は直訳気味なので「型に基づくデコードにより〜から解放されます」はどうでしょう。
|
@tatsuya6502 多忙のため長らく音信普通にして申し訳ありませんでした。不在の間レビューやコードの管理ありがとうございました。 |
| @@ -1569,7 +1569,7 @@ enum CliError { | |||
| <!-- number. The error could represent as many error types as you want by adding new --> | |||
| <!-- variants to the `enum` definition. --> | |||
| このエラー型は2種類のエラー、つまり、IOを扱っているときのエラー、または、文字列を通知に変換するときのエラーが起こる可能性を示しています。 | |||
There was a problem hiding this comment.
文字列を「通知」 → 文字列を「数値」
だと思います。
|
お二人とも、お忙しそうですね。そんな中でレビューしていただき、ありがとうございました！ 誤訳、翻訳漏れ、誤字など、いろいろありましたね。やはりレビューは重要だと実感しました。 いただいたコメントには全て合意します。修正を３回に分けてコミットしました。
ご確認をお願いします。 |
|
マージしますね。これで、全体の進捗が 80% を超えました！ |
tatsuya6502
added a commit
that referenced
this pull request
Feb 25, 2016
5.7. Error Handling (part 4)
|
👏 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@KeenS @ykomatsu @kw-udon
「5.7. エラーハンドリング」のパート４「ケーススタディ」です。レビューをお願いします。
これでこの章は終わりですが、パート３はレビューのフィードバックなしでマージしたので、できれば、パート３「標準ライブラリのトレイトによるエラー処理」まで戻ってレビューしていただけると非常に助かります。
よろしくお願いします。