Laravel クエリビルダでのビット演算について

Question

こんにちは。

MYSQLで作ったWebサービスをherokuに挙げた所、以下コード①の箇所でエラーとなりました。
恐らくherokuでpostgreを使用しているため、SQL文の　`　が対応していないのではないかと思い、
生のSQL文からクエリビルダでの記法に修正しています。

以下②のように修正したのですが、最下部記載のエラーが出てしまいました。

エラー箇所は$query->where("months" & $request->month, "<>", "0");の行で、
"months" & $request->monthが期待通りに動いていないようです。

ここは "months" と $request->month を & でビット演算した結果が <> 0 （0以外）のレコードを抽出したいのですが、
クエリビルダを使用しての正しい書き方をご存知の方、ご教示いただけないでしょうか。

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

//①MYSQLで動いていた処理（生のSQL）
$points = \DB::select("select * from `points` inner join `categorymonths` on `points`.`id` = 
`categorymonths`.`point_id` where `category` = '".$category."' and `months` & ".$month." <> 0");

//②postgreでも動くように上のSQLを修正（クエリビルダ）
$points = \DB::table("points")->join("categorymonths", "points.id", "=", "categorymonths.point_id")
         ->where("category",$request->category)
         ->where(function($query) use ($request){
             $query->where("months" & $request->month, "<>", 0);
         })
         ->get();

エラー

SQLSTATE[42S22]: Column not found: 1054 Unknown column '!' in 'where clause' (SQL: select * from points inner join categorymonths on points.id = categorymonths.point_id where category = dolphin and (! <> 0))

Answer 1

LaravelでSQL文を組み立てるときは、文字列結合でなく、プレースホルダー機能を使ってください。
Laravel 5.8 データベース：利用開始 SELECTクエリの実行

$users = DB::select('select * from users where active = ?', [1]);

LaravelのクエリビルダのSQL文を以下のtoSQLで取得できるので、出力されたSQLをpostgreSQLで直接実行してみて、postgreSQLのヘルプなどを参照しながら、エラーがでないように修正してみてください。
Laravel5でSQLを確認する(お手軽版)

$query ->\DB::table->処理()->処理();
dd($query->toSql(),query->getBindings())

Comments

[cob0822]

プレースホルダーで記載し直した所、解消しました。
原因はやはり、SQLの '(シングルクォーテーション)がMYSQLは対応、Postgresは非対応だったからのようです。
初心者なので、LaravelのSQLでプレースホルダーを使用する点、勉強になりました。
回答ありがとうございました。

Answer 2

そもそも下記の&が問題ではないでしょうか。

$query->where("months" & $request->month, "<>", 0);

$test = 'b';
echo 'a' & $test ;
// `

こうかなと

$query->where("months & ".$request->month, "<>", 0);

これで通るかは未検証です。
元々のSQLが通らない（構文的に）のであればこれも通らなそうに思います。

Comments

[sazi]

>元々のSQLが通らない（構文的に）のであればこれも通らなそうに思います。
ですよね。
なんで、postgresで生SQLを確認しないのかな。
ビット演算とかそのままじゃ駄目っぽい気がします。

[m.ts10806]

「生SQLが通らないからフレームワークの機能で」というのはちょっと安直すぎる気がします。
生SQLがきちんとできないとフレームワークの機能にも正しく反映できないと思います。

[cob0822]

回答ありがとうございました。
原因はやはり、SQLの '(シングルクォーテーション)がMYSQLは対応、Postgresは非対応だったからのようです。

[cob0822]

回答いただいたので補足ですが、
postgresで生SQLを確認した結果、SQLの '(シングルクォーテーション)が原因のようだったので、
シングルクォーテーション使わずに書けるクエリビルダを使用して書こうとしていた次第です。
私の説明不足です。すみません。

[m.ts10806]

クエリビルダでやろうとしていることと今のコメントが合いませんが・・・
ちなみにカラム名に入れられているのはシングルクォートではなくバッククォートです。

[m.ts10806]

もともと「`　が対応していないのではないかと思い」と懸念されているので、外してやってみれば良かったのではないでしょうか。
別にやり方を模索するのは構わないと思いますが、結局関係のないところでエラーになっていては何も進まないと思います。

[sazi]

@cob0822さん
`は「バッククォート」と言います。MySQLでテーブル名やカラム名など予約語などと重なる時に指定するものです。
Postgresで該当するのは"「ダブルクオート」になります。
`を使用する場面が無いという事で非対応とかではありません。

[m.ts10806]

個人的には「予約語でない名称なら何もつけない」でいいように思います。
DBによって解釈やお作法に違いがあるから。そこで戸惑っていては実現したいことが全く進まないように思います。

[cob0822]

何を熱くなっているのかわかりませんが、他のユーザーの方に期待通りの回答をもらい解決済みの
問題ですので、失礼しますね。

[sazi]

@mst10806さん
私も付けない派です。というか付けるとSQL書くのが面倒ですｗ

[sazi]

@cob0822さん
理解のポイントがズレているように感じましたので、もったいないかなと思いコメントしています。
スルーされても特に問題ありません。

[m.ts10806]

cob0822さん
何を勘違いされているのか分かりませんが、
「質問最下部の提示のエラー」と「解決済みにした回答」と「PostgreSQLではバッククォートが使えない」は無関係ですよね。
質問から解決までの流れからすると（質問内容自体も）不自然です。
両方エラーがでたのならそれぞれのエラー内容を関連付けられるように書かないと分かりませんし、
クエリビルダでやろうとしていることもそもそもPHPの書き方を間違っていることからきているので
そこは指摘が必要な場面です（私の回答はそこのみに注力しています）

[m.ts10806]

特に煽るつもりではなくて、
「問題解決」を主軸にしたときに疑問点が多かったための確認と指摘をしています。
スルーされるならスルーされるで構いませんが、後から同じような問題を抱えた人が見たときに「？？？」とならないかどうかは気にしてもらえたらと。