mysqlに接続し、データベースをcsvダウンロードできるphpが動作しません

Question

お世話になります。
phpへリンクを貼った箇所をクリックするとmysqlに接続し、データベースをcsvダウンロードできるものを
作成しておりますが「HTTP ERROR 500」と出て動作しません。ご教示お願いいたします。

<?php
/*
  顧客管理データベース csv出力php
  データベース hikitsugi_db1
  テーブル　   table_hikitsugi
*/
//CSVのファイル名・ヘッダー名を定義
$file_path = "customer.csv";
$export_csv_title = ["ID", "月", "日", "内容", "担当", "カテゴリ", "進行状況"];
$export_sql = "SELECT id, month,day,naiyou,tantou,category,shinkou FROM table_hikitsugi";

//データベースに接続する
require_once 'db_config.php';
$dbh = new PDO('mysql:host=localhost;dbname=hikitsugi_db1;charset=utf8', $user, $pass);
$dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

// 文字コードをSJIS-winにエンコード
foreach ($export_csv_title as $key => $val) {
    $export_header[] = mb_convert_encoding($val, 'SJIS-win', 'UTF-8');
}

//CSV書き込み出力
if (touch($file_path)) {
    $file = new SplFileObject($file_path, "w");

    // write csv header
    $file->fputcsv($export_header);

    // query database
    $sql = "SELECT * FROM table_hikitsugi";
    $stmt = $dbh->query($export_sql);

    // create csv sentences
    while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
        $file->fputcsv($row);
    }
}

// close database connection
$dbh = null;

Comments

[m.ts10806]

回答結果の対応を入れた後のコードで質問内容を更新してください。 少なくとも私の環境では500エラーになるようなエラーは出ていません。

[uep3vjz9wp]

更新完了しました。HTTP ERROR 500　継続中です。

[m.ts10806]

私の回答に書いていますが、コード整形してください。Webサービスもあります。　http://flatsystems.net/php_beautifier.php

[m.ts10806]

画面にエラーが出ないのであればエラーログを確認してください。

[uep3vjz9wp]

ありがとうございます。フォルダ、ファイルのパミッションを777に変更、エラーログのファイル名、場所も変更したのですが、なぜか何も吐き出されておりません。引き続き確認してみます。環境はwin10にvmwareでcentos7です。 }の位置を修正してからは画面が真っ白になるようになりました。

[m.ts10806]

エラー表示はONにしておいてください。　https://qiita.com/shotets/items/3c95aef631b2c5eadae5

[uep3vjz9wp]

ありがとうございます。「　Warning: touch(): Unable to create file customer.csv because Permission denied in /var/www/html/phptry/backup.php on line 35　」と表示。この辺探ってみます。

[m.ts10806]

ファイルの権限の問題ですね。やはりtouch()通ってなかったですね・・・。

[uep3vjz9wp]

お世話になりました。ありがとうございました。

Answer 1

500エラーはサーバー内部エラーです。
ほとんどの場合がプログラムの構文ミスにより起きます。
例えばEclipseのような高機能エディタでも良いですし、Web上で単体実行確認できるサービス（例）でもいいです。
構文チェックを自動で行ってくれるものを使いましょう。それだけで実行前に半分以上の問題が解決できます（ロジック的なところでのミスは全体の3割あるかないかだと思っていいです）

ということで
上記EclipseとOnlinePHPEditor両方で確認すると同じエラーが出ていますね。

構文エラー, unexpected 'EOF'

EOFは
end of file

要は終端です。
何かきちんと閉じられていないときに起きやすいです。

フォーマッターを利用してきちんと整形してみると分かりやすいです。

//CSV書き込み出力

if(touch($file_path)){
    $file = new SplFileObject($file_path, "w");
    
    // write csv header
    $file->fputcsv($export_header);
    
    // query database
    $sql = "SELECT * FROM table_hikitsugi";
    $stmt = $dbh->query($export_sql);
    
    // create csv sentences
    while($row = $stmt->fetch(PDO::FETCH_ASSOC)){
        $file->fputcsv($row);
        
    }
    
    // close database connection
    $dbh = null;


    ?>

if(touch($file_path)){　　の閉じ } がありません。

もちろんこれで「mysqlに接続し、データベースをcsvダウンロード」が正常に動くかどうかは別問題です。
ひとまずコードは綺麗に書きましょう。
閉じ忘れで動かない　良くあることです。

追記を受けて

SplFileObjectを$fileという変数で受けていますが、
質問者さんが閉じられた場所では$fileが宣言されずに使われる可能性があります。
せめて$fileを利用しているところも全て囲ってください。
例：

if (touch($file_path)) {
    $file = new SplFileObject($file_path, 'w');

    // write csv header
    $file->fputcsv($export_header);
    
    // query database
    $sql = 'SELECT * FROM table_hikitsugi';
    $stmt = $dbh->query($export_sql);
    
    // create csv sentences
    while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
        $file->fputcsv($row);
    }
} //←ifここまで

※というかそもそもif(touch($file_path))がtrueじゃなくfalseになっていてSplFileObject()取得できてないということでは？

あと、異常検知したほうがいいかもしれません。
try-catchを利用してみてください。
※未検証　実装イメージのみで捉えてください。

try{
    if (touch($file_path)) {
        $file = new SplFileObject($file_path, 'w');
    
        // write csv header
        $file->fputcsv($export_header);
        
        // query database
        $sql = 'SELECT * FROM table_hikitsugi';
        $stmt = $dbh->query($export_sql);
        
        // create csv sentences
        while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
            $file->fputcsv($row);
        }
    }
}catch(RuntimeException $e){
    echo $e->getMessage();
}

• SplFileObject::__construct

エラー / 例外

filename がオープンできない場合、RuntimeException がスローされます。

蛇足：
画面表示を伴わないPHPプログラムであれば最後の?>は必要ありません。
あくまでその後に画面出力を伴うときにのみ利用します。
※今時ほとんどフレームワークにより構築されるので?>を書くことは稀と思います。

• モダンPHPアンチパターン#ファイルの最後に?>を書く

Answer 2

if(touch($file_path))部分の閉じ括弧が無いような気がします。。。

Comments

[uep3vjz9wp]

早速のご指摘ありがとうございます。
}　　を入れて見ましたが HTTP ERROR 500に変化ありません。他のphpへは問題なく飛んでいます。

[mayoi_maimai]

閉じる場所が違いますね。。
以下でどうですか？
mts10806さんのアドバイスのようにちゃんと成形すると読みやすくなると思いますよ。

if(touch($file_path)){
　　$file = new SplFileObject($file_path, "w");

　　// write csv header
　　$file->fputcsv($export_header);

　　// query database
　　$sql = "SELECT * FROM table_hikitsugi";
　　$stmt = $dbh->query($export_sql);

　　// create csv sentences
　　while($row = $stmt->fetch(PDO::FETCH_ASSOC)){
　　　$file->fputcsv($row);
　　}
}
// close database connection
$dbh = null;

[uep3vjz9wp]

ありがとうございます。閉じる箇所を再度変更したところ、エラーが出ずに画面が真っ白になるようになりました。今度はtry-catchを実施してみます。