質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.03%

phpのエラーについて

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 2,054

score 12

phpからmysqlへのinsertでエラーが出て困っております。
LOAD DATA LOCAL INFILEをしたいのですが、

Fatal error: Call to a member function query() on null in /var/www/html/php/IN_DATA_test2.php on line 16

のエラーが解決できません。
 $stmt = $dbh->prepare($sql);時点で、dbhの持つ値に問題があるということなのでしょうか。

現在、フォームからの値の入力はできているためドライバ関連のエラーはなく、
sql文単体では問題なく動作していることを確認済みです。

どなたか、お知恵を貸していただけないでしょうか。

<?php

try{
    $dbh = new PDO('mysql:dbname=GPS;host=tk2-218-18679.vs.sakura.ne.jp',
                   'root',
                    array(PDO::MYSQL_ATTR_LOCAL_INFILE => true));

    print('<br>');

    if ($dbh == null){
        print('接続に失敗しました。<br>');
    }else{
        print('接続に成功しました。<br>');
    }

    $dbh->query('SET NAMES utf8');
    $sql = 'LOAD DATA LOCAL INFILE "/GPS_log.csv" INTO TABLE test FIELDS TERMINATED BY "," LINES TERMINATED BY "\n";';
    $stmt = $dbh->prepare($sql);
    $stmt->execute();

}catch (PDOException $e){
    print('Error:'.$e->getMessage());
    die();
}

$dbh = null;

?>

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m6u

    2015/10/08 15:23 編集

    「on line 16」の16行目にあるセンテンスは何でしょうか?

    キャンセル

  • aine

    2015/10/08 15:41

    下記が16行目です。
    $sql = 'LOAD DATA LOCAL INFILE "/GPS_log.csv" INTO TABLE test FIELDS TERMINATED BY "," LINES TERMINATED BY "\n";';

    キャンセル

回答 2

checkベストアンサー

+2

エラーそのものは、16行目のPDO::queryメソッドで発生しています。

$dbh->query('SET NAMES utf8');

しかし、問題の本質はこの部分では無いようです。

まず、PDOの使い方が誤っています。
例示されたコードで、PDOの初期化を行っている箇所を見やすく改行すると、以下のようになります。
(以降のコードでは安全のため、ホスト名をexample.comに置き換えています)

$dbh = new PDO(
    'mysql:dbname=GPS;host=example.com',
    'root',
    array(PDO::MYSQL_ATTR_LOCAL_INFILE => true)
);

第一引数にDSN(mysql:dbname=GPS;host=example.com)、第二引数にusername(root)を指定しているところまでは問題ありませんが、第三引数にはpasswordの文字列を指定します。
しかしこのコードでは、第四引数に指定するはずのoptionsを第三引数に指定してしまっています。

PHP: PDO::__construct - Manual

簡単にいえば、パスワードの指定忘れです。これがこのコードがエラーとなる直接の原因です。

また別の問題として、エラー処理が不適切という点も挙げられるでしょう。
10~14行目で、PDOの初期化に失敗した場合にエラーメッセージが表示されるようになっていますが、「接続に失敗しました。」と表示した後も処理を続けています。

if ($dbh == null){
    print('接続に失敗しました。<br>');
}else{
    print('接続に成功しました。<br>');
}

正しく実装するならば、「接続に失敗しました。」と表示した時点で例外を投げるなりして処理を終了するべきでしょう。

私ならば以下のように書きます。

<?php

try{
    /**
     * PDOのコンストラクタに指定する引数を正しく指定。
     * また文字セットのutf8は、DSNで指定する。
     * (文字セットをDSNで指定するやり方は、PHP5.3.5以前ではできません)
     */
    $dbh = new PDO(
        'mysql:dbname=GPS;host=example.com;charset=utf8',
        'root',
        'password',
        array(
            // 発生したエラーが例外としてスローされるよう指定
            PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
            PDO::MYSQL_ATTR_LOCAL_INFILE => true,
        )
    );

    /**
     * PDOへの接続に失敗した場合、エラーを例外としてスロー(投げる)
     */
    if (!$dbh) {
        throw new RuntimeException('接続に失敗しました。');
    }

    echo '接続に成功しました。';

    $sql = 'LOAD DATA LOCAL INFILE "/GPS_log.csv" INTO TABLE test FIELDS TERMINATED BY "," LINES TERMINATED BY "\n";';
    // ユーザ入力を伴わず、また結果を必要としていないため、PDO::execメソッドを使用
    $dbh->exec($sql);
} catch (PDOException $e) {
    /**
     * PDOで発生した例外の処理
     */
    echo 'PDO Error: ' . $e->getMessage();
} catch (Exception $e) {
    /**
     * PDO以外で発生した例外の処理
     */
    echo 'Error: ' . $e->getMessage();
}

参考:
PHPでデータベースに接続するときのまとめ - Qiita
PHPでCSVファイルをUTF8に変換したい - PHP | 【OKWave】

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/10/08 16:12

    解答そのものからはズレますが、ホスト名を直接記載するのは危険です。
    ホスト名だけでも、質問のコードがどこのサーバのMySQLに接続しようとしているのかおおよその検討がついてしまいます。
    私の解答のように、ホスト名を別のものに置き換えたほうが良いでしょう…
    (example.comは例示用に確保されているものなので、安全です)

    キャンセル

  • 2015/10/08 16:18

    ありがとうございます。無事insertでき、解決しました。

    キャンセル

  • 2015/10/08 16:18

    ホスト名についてもありがとうございます。書き換えます。

    キャンセル

0

pdoが接続に失敗しています。
そのため、$dbhがnullとなり、nullに対してquery()というメンバ関数を呼ぼうとしています。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/10/08 15:47

    確認した所、おっしゃるとおり、接続失敗のエラーがありました。

    下記コードでは接続成功でしたが・・・arrayをつけたことで、書き方を変えたのがいけなかったでしょうか。

    $dsn = 'mysql:dbname=men;host=tk2-218-18679.vs.sakura.ne.jp';
    $user = 'root';
    $password = '';

    try{
    $dbh = new PDO($dsn, $user, $password);

    可能であれば、正しい書き方を教えていただけませんか。

    キャンセル

  • 2015/10/08 16:06

    sounisi5011さんの回答を参考にしてください。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.03%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る