Conversation
| if remain in num_to_idx: | ||
| return [num_to_idx[remain], i] | ||
| num_to_idx[num] = i | ||
| raise ValueError("No solution found") No newline at end of file |
There was a problem hiding this comment.
エラーメッセージは小文字始まりのほうがよく見るなと思いました．
というのも，pythonでエラーを起こさせてみると，
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
IndexError: list index out of range
みたく， (エラーの名前) : (エラーメッセージ)
のフォーマットで出力されるので， 大文字始まりで別の文章として独立させなくて良いと思っています．
あとは，今回の"no solution found"のようにわかりやすければエラーメッセージは文章じゃなくてもよいので，その点からも大文字始まりにしなくて良いと思います．
There was a problem hiding this comment.
一方で，エラーメッセージが独立文なら，
-
NASAのテクニカルライティングガイド(の和訳, Amazon)を見てみたんですが，コロンの後の独立文は大文字と小文字のどちらで始めるかは書き手に委ねられているそうです．
-
numpyのメソッドに，エラーメッセージを先頭大文字＆ピリオド付きの独立文にしているものがありました．https://github.com/numpy/numpy/blob/c5ab79c14c98bfda1e60770ffa23a6130f8267b7/numpy/lib/_ufunclike_impl.py#L126
という感じで，大文字始まりももちろん受け入れられるという認識です．
There was a problem hiding this comment.
色々と調べていただきありがとうございます！！知らなかったです！
| if remain in num_to_idx: | ||
| return [num_to_idx[remain], i] | ||
| num_to_idx[num] = i | ||
| raise ValueError("No solution found") No newline at end of file |
There was a problem hiding this comment.
戻り値の型ヒントがlist[int]なので、raiseでエラーを出させるのは良いと思いました。optionalだったらreturn Noneとかで返したい気持ちになりました。
3 participants
https://leetcode.com/problems/two-sum/description/