Skip to content

Instantly share code, notes, and snippets.

@taichi
Last active June 8, 2017 08:24
Show Gist options
  • Star 2 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save taichi/bdb32d515d9ffd7b04c2f630cb63cb55 to your computer and use it in GitHub Desktop.
Save taichi/bdb32d515d9ffd7b04c2f630cb63cb55 to your computer and use it in GitHub Desktop.

https://speakerdeck.com/rtechkouhou/javaru-men

p.68 7って数値リテラルが大分ヤバい

p.69 entrySetメソッドを使うべき 単にListに変換したいだけなら、謎のDTOとかいうオブジェクトの必要性を感じない

List<Map.Entry<String, String>> list = new ArrayList<>(arg.entrySet());

p.72 例外オブジェクトを使ったロールバックのコードが完全におかしい。 フレームワーク内にこういうコードがあるのは避けられないが、業務アプリ内にあるなら間違っている。 というか、素人に例外オブジェクトを触らせるな。

p.74 例外を握りつぶしても良い状況は極めて例外的なものであり、一般的には存在しない。 NOP付ければ握りつぶしても良いように見えるので極めて有害。

p.75 自然順序でソートするだけなら、Comparator作る必要などない。 何か特別な順序でソートするためにComparator作るにしても一々定数にする必要はない。 パフォーマンスは類推するのではなく、計測してから対応すべき。

p.78 こういうチマチマとした最適化に気を使う必要はない。類推するな計測せよ。

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