クラス変数を使用しないメソッドはstaticとすべき？

追記

用語的に「クラス変数」とするとstaticなフィールドを指す様子なので、「インスタンス変数を使用しないメソッドはstaticとすべき？」の方が正しいので、補記。

本文

基本的には、こう言えるんじゃじゃないかという気がするんです。↓みたいなのとか。

    private void rollback(Connection conn) {
        if (conn != null) {
            try {
              conn.close();
            } catch (SQLException e1) {
              throw new DataAccessException(e1.getMessage(), e);
            }
        }
    }

http://www.nulab.co.jp/designPatterns/designPatterns3/designPatterns3-4.html

もしくは、引数を親クラスのメンバにしてしまって、親クラスメソッドとして実装か。あー、そうすると「開いた人が閉じる」ルールでいいのか。

いずれにしろ、リファクタリングの対象としていいと思う。