質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.81%

WHERE句でのCONCATが効かない

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,948

chapp

score 219

お世話になっております。お恥ずかしい内容かと思いますが、表題のWHERE句でのCONCATが効かず、その理由がわからずに困っています。

以下がその実際のSQL

SELECT re_property_data.data_no, re_property_data.data_master_no, re_property_data.data_property_name, re_property_data.data_houi, re_property_data.data_madori1, re_property_data.data_madori2, re_property_data.data_genkyou, re_property_data.data_price, re_property_data.data_kanrihi, re_property_data.data_reikin, re_property_data.data_shikikin, re_property_data.data_kyouekihi, re_property_data.data_chuukai_tesuuryou, re_property_data.data_menseki, re_property_data.data_menseki_tani, re_property_data.data_update_date, re_property_data.data_state, re_master_rent.master_no, re_master_rent.master_member_no, re_master_rent.master_cate_m_no, re_master_rent.master_property_name, re_master_rent.master_pr, re_master_rent.master_addr1, re_master_rent.master_addr2, re_master_rent.master_addr3, re_master_rent.master_kouzou, re_master_rent.master_tatemono_kaisuu_chijou, re_master_rent.master_kenchiku_year, re_master_rent.master_kenchiku_month, re_master_rent.master_train1, re_master_rent.master_train_station1, re_master_rent.master_train_station_toho1, re_master_rent.master_train2, re_master_rent.master_train_station2, re_master_rent.master_train_station_toho2, re_master_rent.master_bas, re_master_rent.master_bas_toho, re_master_rent.master_parking, re_master_rent.master_parking_price, re_master_rent.master_img1, re_master_rent.master_update_date, re_master_rent.master_state, re_master_osusume.mo_no, re_master_osusume.mo_master_no, re_master_osusume.mo_osusume_no, zip.group_code, zip.addr2, zip.addr_no FROM re_property_data LEFT JOIN re_master_rent ON re_property_data.data_master_no = re_master_rent.master_no LEFT JOIN zip ON re_master_rent.master_addr2 = zip.addr2 LEFT JOIN re_master_osusume ON re_master_rent.master_no = re_master_osusume.mo_master_no WHERE re_property_data.data_state = '' AND (CONCAT('re_master_rent.master_train_station1','re_master_rent.master_train_station2') = '五反田' OR CONCAT('re_master_rent.master_train_station1','re_master_rent.master_train_station2') = '目黒') AND re_master_rent.master_cate_m_no = '1'

以下はCONCATを使わない内容で、これだと上手く行く

SELECT re_property_data.data_no, re_property_data.data_master_no, re_property_data.data_property_name, re_property_data.data_houi, re_property_data.data_madori1, re_property_data.data_madori2, re_property_data.data_genkyou, re_property_data.data_price, re_property_data.data_kanrihi, re_property_data.data_reikin, re_property_data.data_shikikin, re_property_data.data_kyouekihi, re_property_data.data_chuukai_tesuuryou, re_property_data.data_menseki, re_property_data.data_menseki_tani, re_property_data.data_update_date, re_property_data.data_state, re_master_rent.master_no, re_master_rent.master_member_no, re_master_rent.master_cate_m_no, re_master_rent.master_property_name, re_master_rent.master_pr, re_master_rent.master_addr1, re_master_rent.master_addr2, re_master_rent.master_addr3, re_master_rent.master_kouzou, re_master_rent.master_tatemono_kaisuu_chijou, re_master_rent.master_kenchiku_year, re_master_rent.master_kenchiku_month, re_master_rent.master_train1, re_master_rent.master_train_station1, re_master_rent.master_train_station_toho1, re_master_rent.master_train2, re_master_rent.master_train_station2, re_master_rent.master_train_station_toho2, re_master_rent.master_bas, re_master_rent.master_bas_toho, re_master_rent.master_parking, re_master_rent.master_parking_price, re_master_rent.master_img1, re_master_rent.master_update_date, re_master_rent.master_state, re_master_osusume.mo_no, re_master_osusume.mo_master_no, re_master_osusume.mo_osusume_no, zip.group_code, zip.addr2, zip.addr_no FROM re_property_data LEFT JOIN re_master_rent ON re_property_data.data_master_no = re_master_rent.master_no LEFT JOIN zip ON re_master_rent.master_addr2 = zip.addr2 LEFT JOIN re_master_osusume ON re_master_rent.master_no = re_master_osusume.mo_master_no WHERE re_property_data.data_state = '' AND (re_master_rent.master_train_station1 = '五反田' OR re_master_rent.master_train_station2 = '五反田' OR re_master_rent.master_train_station1 = '目黒' OR re_master_rent.master_train_station2 = '目黒') AND re_master_rent.master_cate_m_no = '1'

両者の違いは、WHERE句の中で、その違う箇所は

(CONCAT('re_master_rent.master_train_station1','re_master_rent.master_train_station2') = '五反田' OR CONCAT('re_master_rent.master_train_station1','re_master_rent.master_train_station2') = '目黒')

または

(re_master_rent.master_train_station1 = '五反田' OR re_master_rent.master_train_station2 = '五反田' OR re_master_rent.master_train_station1 = '目黒' OR re_master_rent.master_train_station2 = '目黒')

この部分ですが、何かおかしいところはあるのでしょうか?
各カラムには単体の駅名だけが記録されており(インデックス設定などもしていない)、その駅名を検索する内容となってます。

CONCATの方がスマートに記述できるのでこちらを使いたいと思っているのですが、そもそも誤った使い方なのか?それさえ不明でご教示いただければと質問した次第です。

アドバイスのほど、よろしくお願いいたします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+1

SQLではシングルクォートで囲むと文字列として扱われます。

AND CONCAT('re_master_rent.master_train_station1','re_master_rent.master_train_station2') = '五反田'AND 're_master_rent.master_train_station1re_master_rent.master_train_station2' = '五反田' 


として扱われますから、'五反田'のデータは見つかりません。

SQLを横1行に書くのが主義のようですが、見易さは考えた方が良いです。

FROM re_property_data rpd  -- テーブルの別名を設定
LEFT JOIN re_master_rent rmr
ON rep.data_master_no = rmr.master_no
中略
WHERE rpd.data_state = ''
AND (rmr.master_train_station1 in ('五反田', '目黒')
OR   rmr.master_train_station2 in ('五反田', '目黒'))
AND  rmr.master_cate_m_no = '1'


現象を再現できる最小限のコードで質問することも必要です。
CREATE TABLE文も提示するのもマナーです。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/05/16 09:17

    内容がまるっと被ってました。

    キャンセル

  • 2019/05/16 10:56

    Orlofskyさん

    早々のアドバイスをありがとうございます。
    また、一列表示など見易さに欠けていました。(phpで作っているのですが)エディタ上では改行を行っているものの、ブラウザ上では一行になっていました。現状報告の気持ちが優先してのことでしたが、結果見にくいものとなっていました。以降、気をつけます。

    キャンセル

0

WHERE re_master_rent.master_train_station1 = '五反田' 
   OR re_master_rent.master_train_station2 = '五反田' 
   OR re_master_rent.master_train_station1 = '目黒' 
   OR re_master_rent.master_train_station2 = '目黒'


上記を簡潔にということならInを使用します。

WHERE re_master_rent.master_train_station1 In ('五反田', '目黒') 
   OR re_master_rent.master_train_station2 In ('五反田', '目黒')  


CONCAT()は文字列の結合なので、見当違いです。

Nullじゃないものを優先という事ならCOALESCE()

WHERE coalesce(re_master_rent.master_train_station1, re_master_rent.master_train_station2) In ('五反田', '目黒') 

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/05/16 11:01

    saziさん

    アドバイスをありがとうございます。
    またCOALESCE()を使っての内容にも感謝です。

    CONCAT()の知識が曖昧でしたが、今回のご指摘で理解することが出来ました。ありがとうございます。

    ただ、感謝に気持ちはお二方共にあるのですが、内容が被っていたとのことで、はじめにアドバイスを下さったOrlofskyさんをベストアンサーとさせていただきます。評価するようで恐縮なのですが・・・ご理解ください。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.81%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る