ActiveRecord joinメソッドで、SQL文を追加するとシンタックスエラーになる

Question

以下のようなSQLの内容を、railsのモデルで行いたいと考えています。

reservationsテーブルで、cartsテーブルをそれぞれのcart_idとidとでleft joinで結合し、
cartsのレコードが作成されてから5分間経過したものは、結合先の内容をnullにする、という条件を加えます。

SELECT * 
FROM reservations AS r 
LEFT OUTER JOIN carts AS c 
ON r.cart_id = c.id AND c.create_date >= SUBTIME(NOW(), '00:05:00') 
WHERE r.id = 20

rails の　Reservation モデルでは以下のように記述しました。

scope :getReservationStates, lambda {
  select('*')
   .joins('LEFT OUTER JOIN carts AS c ON lock_id = c.id AND c.create_date >= SUBTIME(NOW(), '00:05:00')')
   .where('id = 20')
  }

すると以下のエラーが出ます。

syntax error, unexpected tINTEGER, expecting ')'
...eate_date >= SUBTIME(NOW(), '00:05:00')')
... ^~ ^~

以下のように、SUBTIME(NOW(), '00:05:00')の部分をNOW()に変えるとうまくいきます。

scope :getReservationStates, lambda {
  select('*')
   .joins('LEFT OUTER JOIN carts AS c ON lock_id = c.id AND c.create_date >= NOW()')
   .where('id = 20')
  }

そもそもjoinメソッド内にこのような記述ができないのか、括弧が多すぎるのか、
それとも全くこの方法付では間違いなのか、お教えいただけますと幸いです。

よろしくお願いいたします。

Answer 1

SQL中のシングルクォートはシングルクォートを2つ書く、ってのがあるので

   .joins('LEFT OUTER JOIN carts AS c ON lock_id = c.id AND c.create_date >= SUBTIME(NOW(), '00:05:00')')
                                 ↓
   .joins('LEFT OUTER JOIN carts AS c ON lock_id = c.id AND c.create_date >= SUBTIME(NOW(), ''00:05:00'')')

かも？

Comments

[orori]

すごいです。昨日から検索しまくっていたんですが解決しました、ご指摘のとおりです。
どうしたらこういうことに引っかからなくなるのでしょう。経験つんで基礎力をあげていくしかないのでしょうね・・・
本当に助かりました。

[Orlofsky]

お使いのデータベースについてSQL入門レベルの勉強をしっかりやっておくこと。
同じSQLでもデータベースやそのバージョンによって方言が大きいですから、どのデータベースを使うのかを質問のタグで明示したり、バージョンも明記した方が適切なコメントが付き易いです。

[orori]

ありがとうございます、おっしゃるとおりですね。今後気をつけます。MySQL5.7だったのですが勉強精進します！

Answer 2

表記法の問題です。一重引用符の中に一重引用符を書いたので、そこで文字列が終わると認識されてしまっています。

エスケープするか、外側を二重引用符にしましょう。

.joins("LEFT OUTER JOIN carts AS c ON lock_id = c.id AND c.create_date >= SUBTIME(NOW(), '00:05:00')")

Answer 3

()の始まりと終わりの数があっていないように思います。

select('*')
   .joins('LEFT OUTER JOIN carts AS c ON lock_id = c.id AND c.create_date >= SUBTIME(NOW(), '00:05:00'))')
   .where('id = 20')

↓

select('*')
   .joins('LEFT OUTER JOIN carts AS c ON lock_id = c.id AND c.create_date >= SUBTIME(NOW(), '00:05:00')')
   .where('id = 20')

Comments

[orori]

すみません！本文のほうはコピーする際に間違ったようですが、ご指摘のように直してもエラーは発生するのです。本文のほうも訂正しておきます。大変申し訳ありません、エラーの内容も変更しました。

syntax error, unexpected tINTEGER, expecting ')'
...eate_date >= SUBTIME(NOW(), '00:05:00')')
... ^~

[退会済みユーザー]

解決できず、すみません。自分のローカルで少し試してみたのですが、subtimeの部分がうまく動かないですね。。。

少し煩雑になりますが、
時間計算部分を外でやるというのはどうでしょうか？
time = Time.now - (5 * 60)
select('*')
.joins('LEFT OUTER JOIN carts AS c ON lock_id = c.id AND c.create_date >= ?', time)
.where('id = 20')

また、最適解ではありませんが、ActiveRecordで生SQLを使う方法
参考：https://qiita.com/yut_h1979/items/4cb3d9a3b3fc87ca0435

[orori]

ありがとうございます！今回Zendframework1のアプリをRailsに移植しているのですが、上のような記述が数多く存在し、ご指摘の方法は、もとの記述の形式から異なってしまい、それをすべて適用するのは大変になってしまうのです。しかし、他の方法も模索しているところでした。大変助かりました。