PHPで「SELECT」してから「DROP TABLE」すると動作しない

Question

$dsn = "sqlite:../sql/base.sqlite";
$pdo = new PDO($dsn);

$stmt = $pdo -> prepare("DROP TABLE '削除テーブル'");
$stmt->execute();

上記ソースですと「削除テーブル」の削除に成功します

$dsn = "sqlite:../sql/base.sqlite";
$pdo = new PDO($dsn);

$st = $pdo->query("SELECT * FROM task WHERE num='1'");
$data = $st->fetch();
$del_table = $data["del_table"];

$stmt = $pdo -> prepare("DROP TABLE '{$del_table}'");
$stmt->execute();

SELECTをしてからテーブル削除をすると何故か削除が行われません
何か書き方に問題があるのでしょうか？

Comments

[m.ts10806]

そもそも正しく想定の情報をSELECTできてますか？

[m.ts10806]

また、try-catchでPDOException拾うように組んでご確認ください。

[jinba]

SELECT後にSELECTした文を使わず
$stmt = $pdo -> prepare("DROP TABLE '削除テーブル'");
としても動作しないので正しく想定の情報はSELECTできた環境で試せています

try-catchいまから試します

[jinba]

try-catch試してみたところ
catchを通らずエラーなく終了するのですがテーブルが削除されません

try内でもSELECTしないと削除されました

[yoorwm]

シングルクォーテーションで文字列を囲むとリテラルとして扱われるのですが、意味が分かっていてシングルクォーテーションにしているのでしょうか？
それとも'削除テーブル'などと本当のテーブル名のように見えないので、その部分は実際に使うテーブル名に置き換えている、という事でしょうか？

[m.ts10806]

try-catch導入したコードを提示してください

[jinba]

string(58) "SQLSTATE[HY000]: General error: 6 database table is locked"

上記エラーが出ておりました

saziさんの回答「closeCursor」をすることで解決できました、ありがとうございます

ちなみにテーブル名がランダムでテーブル名先頭に数字が来てしまう場合があったためシングルコーテーションで囲っておりました
（先頭に数字来ないように修正すればいいのですが…）

Answer 1

selectされた状態が解放されていないのかもしれません。
試しに、select後にcloseCursorしてみてはどうでしょう。

PDOStatement::closeCursor

Comments

[jinba]

$data = $st->fetch();
$del_table = $data["del_table"];
$st->closeCursor();　//追加

上記のよう追加したら思ったように動作しました。

SELECT状態の開放初めて知りました
おかげさまで解決することができましたありがとうございました！

Answer 2

直接の回答ではありませんが、以下のようにしてエラーを表示すれば、解決の糸口がつかめますし、エラーメッセージを読んで、jinba さん自身で問題を把握できるかもしれません。

try {
    $dsn = "sqlite:../sql/base.sqlite";
    $pdo = new PDO($dsn);
    $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); // 追加

    $st = $pdo->query("SELECT * FROM task WHERE num='1'");
    $data = $st->fetch();
    $del_table = $data["del_table"];

    $stmt = $pdo->prepare("DROP TABLE '{$del_table}'");
    $stmt->execute();
} catch (PDOException $e) {
    var_dump($e->getMessage());
}

Comments

[jinba]

try-catch試してみたところ
catchを通らずエラーなく終了するのですがテーブルが削除されません

try内でもSELECTしないと削除されました

[jinba]

追加見落としていました、追加含めもう一度試してみます

[jinba]

string(58) "SQLSTATE[HY000]: General error: 6 database table is locked"

上記エラーが出ておりました
こやってエラーチェックするのですね、次回から自己解決するためにも
とても参考になりましたありがとうございました。

[退会済みユーザー]

> こやってエラーチェックするのですね

そうです。

さらに、「General error: 6 database table is locked」でググれば、解決方法が簡単に見つかります。

Answer 3

反対に質問ですが、なぜ SELECT のタイミングで DROP TABLE したいのですか？そんな使い方をしなければならない場面が思い浮かびません。
DELETE でテーブルのレコードを削除するのではダメなのですか？

Comments

[jinba]

サブテーブルをたくさん使うプログラムを作っており
そのテーブル一覧表のテーブルがあります。

なのでテーブル一覧表からはレコードを削除し
サブテーブル自体は丸ごと削除したいのです

[m.ts10806]

横から失礼します
テーブルの正規化おこなったほうが良さそうです。テーブルの削除が必要なシステムってなかなかないですよ。

[tacsheaven]

やってせいぜい truncate ですかね。デーブルそのものを drop ってのは……それやるくらいなら temporary table 作りますよねえ……

[m.ts10806]

私もtemporary table案件だと思いました。定時処理とかではよく用いますね。

[jinba]

初心者過ぎて気づけてない部分が大きいのですが、かなり強引なシステムなんでしょう…
テーブルdropするようなシステム自体がおかしいことに気付けるよう精進します
ご指摘ありがとうございました。

[Orlofsky]

まともに正規化されていないとか、かなりテーブル設計に無理があるものと想像します。
SQLをシンプルにパフォーマンス良く実行できるように、データベースのテーブルはデータベースの正規化 https://oss-db.jp/measures/dojo_info_04.shtml に沿って設計します。通常、第3正規化まで行います。

[m.ts10806]

業務改善のポイントと同じなのですが「ムリ」「ムダ」「ムラ」の三点で検討すると良いと思います。
初めは多少強引で良いので仮説を立てるところから。その仮説を検証していく形です。
幾ら設計だけにらめっこしても話は進まないですし、動かしたり簡単な検証プログラムを作ったり。大量のサンプルデータを投入してみたり、ですね。
熟練者が見たからって一瞬で見抜けるわけではなく地道な要件確認と検証で詰めていくものなので時間はかかりますが、事故があったときの損害に比べれば小さいものと思います

[jinba]

デザインが本業でシステム的なことに関しては教えていただける方が居らず完全独学状態です。

みなさんのご指摘で新たに勉強しなければならない分野が絞れます

とても参考になりました、ありがとうございます。