JavaでSQL文を条件に合わせて結合したい

Question

実現したいこと

ここに実現したいことを箇条書きで書いてください。
4つの変数(loginId, userName, startBirthDate, endBirthDate)にのいずれかに値が入っていた場合にそれぞれに対応したif文内の処理を実行して変数sqlに結合する。

前提

eclipseで検索画面に入力された値に従ってSQL文を結合していくという処理のコードを書いています。本来であればtoStringメソッドを使用するべきだと思うのですが、今回はif文で必要なSQL文を結合していくやり方でやりたいと思っています。

発生している問題・エラーメッセージ

 public static List<User> search(String loginId, String userName, String startBirthDate,
      String endBirthDate) {

    Connection conn = null;
    List<User> userList = new ArrayList<User>();
    try {

      conn = DBManager.getConnection();



      String sql = "SELECT * FROM user WHERE is_admin = false";
      if ("".equals(loginId)) {
        sql += " AND login_id = ?";
      }
      if ("".equals(userName)) {
        sql += " AND name LIKE ?";
      }
      if ("".equals(startBirthDate)) {
        sql += " AND ? <= birth_date";
      }
      if ("".equals(endBirthDate)) {
        sql += " AND birth_date <= ?";
      }


　　　(以降割愛)

試したこと

プログラミングスクールのメンターには上のコードにあるように条件式の書き方を「"".equals(loginId)」のように書くように指示されたのですが、この場合、変数が空文字の時の処理を記述することになり、目的の
である「変数に値が入った場合の処理」がif文の中に記述できなくなってしまうのではないかと思っております。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

メンターが居るなら直接聞いたほうが早いと思いますけど。(その人に本当にそれしか言われなかったんでしょうか。)

>本来であればtoStringメソッドを使用するべき
どこでどう使用するというのでしょう。

「変数が空文字の時」と「変数に値が入った場合」の違いは理解されているのでしょうか。
そして、 if の条件は真偽値(boolean)であり equals がそれを返すメソッドであること、真偽値は比較/操作できることは理解されていますでしょうか。

[javahack]

メンターの思惑を勝手に推測すると「"".equals(loginId)」を使い、それを応用して答えを導き出すことを期待しているのではないかと。
ヒントは jimbe さんが出してるので自分の頭で考えてみては。

[skychord]

ご返答いただきありがとうございます。
スクールに通っているのでメンターはいるのですが、対応は平日のみで、休日も勉強を進めていきたいと思い質問させていただきました。
変数が空文字の時と値が入った場合の違いや真偽値の比較/操作のことについて改めて調べてみたのですがどのように今回の課題に生かしていくべきか分からなかったので後日メンターに再度相談したいと思います。
ご対応いただきありがとうございました。

[dodox86]

より直接的にヒントを書くと、if文には対応するelse節があるし、真偽値を否定(評価した値を反転、とでも言いましょうか)する ! 演算子がありますね。

Answer 1

if ("".equals(loginId)) { ～ } という if の部分は二つの仕様によって処理が行われています。

1つは、 if(式){} は「"式" がtrue なら後続の({} 内の)処理を行う」ということ、もう1つは String.equals() メソッドが関数的に boolean 型を返すということです。
"".equals(loginId) を 「loginId が空文字なら true」 と訳すなら、この if 文は「"loginId が空文字なら true" が true なら後続の({} 内の)処理を行う」ということになります。

一方、boolean は int 等と同じように == 等によって値の比較が出来、例えば true==true という式は true で false==false も true、 true==false は false 等となります。

これらを踏まえて、 if 文を「loginId が空文字で無いなら処理を行う」に変えてみましょう。
"loginId が空文字で無い" ということは 「"loginId が空文字なら true" が false」 ということが出来ます。これをコード化すると "".equals(loginId)==false となり、 if 文にすると if ("".equals(loginId)==false) { ～ } となります。

また、 boolean は "!" という単項演算子によって値を反転することが出来ます。 !true は false で !false は true です。

"".equals(loginId)==false は !"".equals(loginId)と書くことも出来、 if 文としてif (!"".equals(loginId)) { ～ }とも書けるということです。

Comments

[skychord]

ご回答いただきありがとうございます。
「"".equals(loginId)==false」や「!"".equals(loginId」といった書き方ができるというのはすごく参考になりました。ただ、メンターに空文字ではなく"null"の値が入っていた場合も考えるようにとも言われたことが引っかかっており、その結果「"".equals(loginId)」のように書き方を変えるように指摘されました。
そのため、今回教えていただいた2つの書き方でも何らかの形でloginIdに"null"の値が入っていた場合に思うような処理ができないのではないかと考えています。この点、どのように解消したらよろしいでしょうか？

[jimbe]

"".equals(loginId) で loginId が null だった場合は false になります。
この書き方が loginId.equals("") より良い点は、 loginId が null だった場合に loginId.equals("") では例外( NullPointerException)が発生するのに対し、 "".equals(loginId) は例外は発生しない点です。
しかし loginId が "" でも null でも無い場合に処理をしたいのであれば "".equals(loginId) == false では null が通ってしまう為にそれだけでは使えません。
質問へのコメントとして『変数が空文字の時」と「変数に値が入った場合」の違い』と書きましたのは null を考慮するべきかが質問には書いてなかったので遠回しな言い方となりました。

言ってしまうなら、メンターの言われた事は必要なことの一部でしかありません。
「loginId が null で無く、かつ、 "" でも無い」なら処理を行えば良いのですから、
if(loginId != null && !loginId.equals("")) { ～ }
としてしまえば良いだけです。 && は左が成立しないと右は実行(評価)されませんので、この順で書けば loginId が null でも例外は発生しません。(!loginId.equals("") はメンター推薦(?)の !"".equals(loginId) でも良いですし !loginId.isEmpty() とか !(loginId.length()==0) とか、いっそ "!" を付けないで loginId.length()>0 とか色々あります。)

もし if() の式を && で並べたりしない方法で書けという縛りがあるのであれば、
static boolean isValid(String s) {
return s!=null && !s.equals("");
}
と言ったメソッドを作って
if(isValid(loginId)) { ～ }
とする感じと思います。

[skychord]

詳しく丁寧にご回答いただきありがとうございます。
"".equals(loginId)とloginId.equals("")の違いも分かっていなかったのでそこから整理していただいたことでようやく理解できました。
また、書き方のバリエーションまで考えてくださりありがとうございます。
お陰様で休日にもう少し勉強を進められそうです。