Javaコードアンチパターン 初級からプロまで知っておきたいミスと回避策 第1回議事録

[ 戻る ]

Java読書会 議事録

「Javaコードアンチパターン 初級からプロまで知っておきたいミスと回避策」を読む会 第1回

実施概要

• 参加者: 高橋(智)、高橋(徹)、加藤、井上、遠藤、岩室、根本(記)

第1章：開発ツールと品質向上

1.1 静的解析ツール

• Eclipseのクリーンアップ設定: if文の後のスペースの有無など、フォーマットの統一性が議論となった。
• SpotBugs: FindBugsの後継（フォーク版）。本家は更新が停止しているためこちらを推奨。
• SonarQube: 誤検知の頻度について検討が必要。
• CodeQL: NoSQLのSQLインジェクション検知にも対応。

1.5 ミューテーションカバレッジ

コードの境界条件を書き換えた際、テストコードがそれを検知できるか（テスト自体の品質）を測定する手法。

• 参考URL: PITest (pitest.org)

1.6 動的解析

• Java Pathfinder (NASA): デッドロック検出ツール。大量のケースを網羅的に試行する。
  • GitHub - javapathfinder
• DLCheck: マルチスレッドにおけるデッドロック検知に特化。
  • GitHub - devexperts/dlcheck

第2章：式（Expressions）

2.1 数値演算子の優先順位

2.1.1 ビットシフト

• 優先度: シフト演算子（<<, >>）は四則演算より優先度が低い。
• ベストプラクティス:
  • 可読性とバグ防止のため、四則演算（/ や *）を優先して使用する。
  • hashCode は手動で計算せず、IDEの自動生成機能を利用する。

    【解説の訂正】最適化について: 現代のJITコンパイラが実行時に最適化を行うため、手動でシフト演算に書き換える必要性は低いです。

2.2.1 〜 2.2.3 条件演算子（三項演算子）

• 優先順位: && は || より優先される。
• 三項演算子の罠: 優先度が低いため、括弧がないと予期せぬ評価順序になりやすい。中間変数を利用することで回避を推奨。

  【重要】型変換の注意: 片方が基本型、もう片方がラッパークラスの場合、ラッパークラス側はアンボクシングされます。nullの場合、NullPointerException が発生するため注意が必要です。

2.3 加算と文字列結合の混同

• + 演算子で文字列と数値を混在させない。
• MessageFormat.format や String.format の利用を推奨。

    // 良い例
    String entryName = MessageFormat.format("Entry#{0}", index + 1);
  

2.4 複数行の文字列（テキストブロック）

• Java 15以降で導入された テキストブロック (""" ... """) を活用する。
• 古いバージョンのJava VM（Java 8等）では動作しないため、ターゲットバージョンに注意が必要。

2.5 単項演算子の誤用

• x =+ y; は x = (+y); と解釈され、代入演算子 += とは全く異なる動きになる。
• 文字リテラルの前の + は数値を加算する処理になるため、文字列結合を意図する場合は明示的に " で囲む等の対応が必要。

宿題事項

以上

[ 戻る ]