MySQLのOUTER JOINで絞り込んだ結果をPHPで画面に表示させたい

Question

joblistテーブルとskipテーブルがあります。

joblistテーブルは応募者のデータが格納されており、
skipテーブルはuser（企業番号）ごとに不採用者のデータが格納されています。

joblist.jobは求人番号です。
joblist.awuserとskip.awuserは重複している番号があります。
求人番号ごとに、skipテーブルに存在しないawuserの一覧を表示させたいです。

PHPでMySQLには手続き型で接続します。
（オブジェクト指向は使いません）

CREATE TABLE `aw`.`joblist` (
 `id` INT(255) NOT NULL AUTO_INCREMENT ,
 `job` INT(255) NOT NULL ,
 `awuser` INT(255) NOT NULL ,
 `date` DATETIME NOT NULL ,
 PRIMARY KEY (`id`)
) ENGINE = InnoDB;

CREATE TABLE `aw`.`skip` (
 `id` INT(255) NOT NULL AUTO_INCREMENT ,
 `user` VARCHAR(255) NOT NULL ,
 `awuser` INT(255) NOT NULL ,
 PRIMARY KEY (`id`)) ENGINE = InnoDB;

そこで、上記の条件を満たす、以下を作成しました。

$user = 1;
$array = [3533346 ,3562611];

$connection = mysqli_connect('localhost', 'root', '', 'aw');

$query = "SELECT * FROM joblist LEFT OUTER JOIN skip ON (joblist.awuser = skip.awuser) WHERE (user != '$user' OR user IS null) AND job IN ('implode(",", "$array")')";

$select_job_query = mysqli_query($connection, $query);
while($row = mysqli_fetch_array($select_job_query)) {
    echo $myApplicant = $row['awuser'];
}

実行すると
Parse error: syntax error, unexpected ','
が$queryの行に出ます。

惜しいところまできているはずなのですが、
awuserを画面に出力する方法で間違っている箇所をご指摘いただければ幸いです。
よろしくお願いいたします。

Answer 1

SQLインジェクション脆弱性になり得るので，文字列のはめ込みはプレースホルダで。

<?php

$user = 'user';
$array = ['foo', 'bar', 'baz'];

$query = sprintf(
    '
        select * from
        left join skip on joblist.awuser = skip.awuser
        where (user != ? or user is null) and job in (%s)
    ',
    implode(', ', array_fill_keys(array_keys($array), '?'))
);

$stmt = mysqli_prepare($connection, $query);
mysqli_stmt_bind_param($stmt, 's' . str_repeat('s', count($array)), $user, ...$array);
mysqli_stmt_execute($stmt);
$result = mysqli_stmt_get_result($stmt);

while ($row = mysqli_fetch_array($result)) {
    /* ... */
}

どちらかと言うとオブジェクト指向型のほうがおすすめです。手続き型は関数名が長すぎてうんざり…

<?php

$user = 'user';
$array = ['foo', 'bar', 'baz'];

$query = sprintf(
    '
        select * from
        left join skip on joblist.awuser = skip.awuser
        where (user != ? or user is null) and job in (%s)
    ',
    implode(', ', array_fill_keys(array_keys($array), '?'))
);

$stmt = $connection->prepare($query);
$stmt->bind_param('s' . str_repeat('s', count($array)), $user, ...$array);
$stmt->execute();
$result = $stmt->get_result();

while ($row = $result->fetch_array()) {
    /* ... */
}

Comments

[退会済みユーザー]

すんません、横着してしまいました

[terapro]

ご回答ありがとうございます。
本来はこちらの書き方が完成形なのだと思いますが、現状xamppで自分だけが利用している状況ですのでSQLインジェクション対策は考えずに平文で作りました。プレースホルダやオブジェクト指向の考え方は現在の私にはまだ難しく私のレベルでは思ったように再現ができませんでした。今後の学習課題とさせていただきます。
ようやく調べながらコードが書いて動かせるようになったレベルですが、次は他の人にも使ってもらえるようなアプリの制作を目指してこれからも勉強頑張ります。

Answer 2

<?php
$user = 'user';
$array = ['foo', 'bar', 'baz'];

array_walk($array, function (&$val) {
    $val =  "'" . $val . "'";
});
$query = "SELECT * FROM joblist LEFT OUTER JOIN skip ON (joblist.awuser = skip.awuser) WHERE (user != '$user' OR user IS null) AND job IN (" . implode(", ", $array) . ")";
echo $query;

結果

SELECT * FROM joblist LEFT OUTER JOIN skip ON (joblist.awuser = skip.awuser) WHERE (user != 'user' OR user IS null) AND job IN ('foo', 'bar', 'baz')

Comments

[terapro]

無事動きました。私の質問に対して真っ直ぐに回答いただき、誠にありがとうございました。
現状ギリギリ理解できる範疇でした。本当にありがとうございます。