5.7. Error Handling (part 2) #93
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
| <!-- or can we continue using combinators? --> | ||
| もちろん現実のコードは、いつもこんなにクリーンではありません。 | ||
| 時には `Option` 型と `Result` 型が混在していることもあるでしょう。 | ||
| そんな時は、明確なケース分析に頼るしかないのでしょうか？ |
There was a problem hiding this comment.
このexplicitは他の部分と同様「明示的な」が良いとおもいます。
あと、"case analysis"の訳ですが、「場合分け」はいかがでしょうか？
数学などでの場合分けによる証明を "proof by case analysis" といったりします。
また、SICPでも"case analysis"は、「場合分け」と訳されているみたいです。(http://sicp.iijlab.net/fulltext/x116.html)
|
数点コメントさせていただきました。 |
| ここでの変更点は、（エラーを文字列に変換する） `map_err(|e| e.to_string())` を、`map_err(CliError::Io)` や `map_err(CliError::Parse)` へ切り替えたことです。 | ||
| こうして *呼び出し元* が、ユーザーに対してどの程度の詳細を報告するか決められるようになりました。 | ||
| `String` をエラー型として用いることは、事実上、呼び出し元からこうした選択肢を奪ってしまいます。 | ||
| `CliError` のような独自の `enum` エラー型を用いることは、 *構造化データ* によるエラーの説明だけでなく、これまでと同様の使いやすさをもたらします。 |
There was a problem hiding this comment.
ここの「構造化データ」も「構造化されたデータ」に統一したほうが良いように思います。
（前回まとめてコメントすべきでしたが、見落としていました。すみません）
|
修正確認しました！ |
- Apply review comment - Fix a broken link
| <!-- 3. A problem parsing the data as a number. --> | ||
| 1. ファイルを開くときの問題 | ||
| 2. ファイルからデータを読み込む時の問題 | ||
| 3. データを数値としてパースするときの問題 |
There was a problem hiding this comment.
細かいですが、1と３と同様に、2も「ファイルからデータを読み込む「とき」の問題」だと思います
Apply review comment
|
@ykomatsu さん、レビューありがとうございました。 レビュー結果を反映したので、マージしようと思います。 |
tatsuya6502
added a commit
that referenced
this pull request
Feb 20, 2016
5.7. Error Handling (part 2)
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 さん
「5.7. エラーハンドリング」のパート２です。レビューをお願いします。（TranslationTable.mdの変更はありません）