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

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

ただいまの
回答率

90.38%

  • PHP

    25048questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • MySQL

    7296questions

    MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

  • SQL

    3155questions

    SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

  • phpMyAdmin

    823questions

    phpMyAdminはオープンソースで、PHPで書かれたウェブベースのMySQL管理ツールのことです。

  • MAMP

    293questions

    Mac 上で WordPress などの動的ページのサイトが作れるように環境を構築するフリーソフト

Parse error: syntax error, unexpected ','

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 651

1004

score 62

 前提・実現したいこと

PHPで日記投稿システムを作っています。
php上のSQLで同じユーザーが書き込んだ日記を表示させるコードを実装中に以下のエラーメッセージが発生しました。(ブラウザ上に書かれています。)
実現したいこととしては、メールアドレスごとに過去に該当ユーザーが書いた日記をそのユーザーだけが閲覧できるようにしたいです。

 発生している問題・エラーメッセージ

Parse error: syntax error, unexpected ',' in C:\MAMP\htdocs\mysql\afterLogin.php on line 21


下記の画像はDBの中身になります。例えば、idが10番目のemailがiii@iiiのcontentsを表示させたい場合、

イメージ説明

 afterLogin.php↓

<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="UTF-8">
<title></title>
</head>

<?php

session_start();

$link = mysqli_connect("localhost","root","root","memberapp");

if(mysqli_connect_error()){
    die("DBへの接続失敗!");
}

echo "このユーザーは".$_SESSION['email']."でログインしています。";

//本人の以前の日記を表示させる
$query = "SELECT contents FROM `users` WHERE email = '".($link,$_SESSION['email'])."'";
$result = mysqli_query($link,$query);
echo "前回の投稿".$result;

if(!empty($_POST['text'])){
    $contents = $_POST['text'];
    $stmt = $link->prepare('update users set contents = concat(contents,"\n",?) where email = ?');
    $stmt->bind_param('ss', $contents, $_SESSION['email']);
    $stmt->execute();
    echo "今回の投稿:".$_POST['text'];
    $_SESSION['text'] = $_POST['text'];
}

?>

<form method="post">
    <input type="text" name="text" placeholder="Please fill in the blank">
    <input type="submit" name="submit" value="投稿">
</form>

homework.php↓

<?php

    session_start();

    $link = mysqli_connect("localhost","root","root","memberapp");

    if(mysqli_connect_error()){
        die("DBへの接続失敗!");
    }


    //$query = "SELECT * FROM users WHERE name = '".mysqli_real_escape_string($link,$name)."'";
    if(array_key_exists('email',$_POST) OR array_key_exists('password',$_POST)){

        if($_POST['email'] == ''){
            echo "Eメールアドレスを入力してください";
        } elseif($_POST['password'] == '') {
            echo "パスワードを入力してください";
        } else {

            $query = "SELECT password FROM `users` WHERE email = '".mysqli_real_escape_string($link,$_POST['email'])."'";
            $result = mysqli_query($link,$query);

            $row = $result->fetch_assoc();

            if(mysqli_num_rows($result) > 0){
                if($row["password"] == $_POST['password']){
                    $_SESSION["email"] = $_POST['email'];
                    header("Location:afterLogin.php");
                }else{
                    echo "入力されたパスと値が一致しません。";
                }


            } else {
                $query = "INSERT INTO `users` (`email`,`password`) VALUES ('".mysqli_real_escape_string($link,$_POST['email'])."','".mysqli_real_escape_string($link,$_POST['password'])."')";
                if(mysqli_query($link,$query)){
                    echo "登録されました!";
                } else {
                    echo "登録に失敗しました!";
                }
            }
        }
    }

?>

<form method="post">
    <input type="email" name="email" placeholder="email">
    <input type="password" name="password" placeholder="password">
    <input type="submit" name="submit" value="signin">
</form>

 試したこと

mysqli_real_escape_stringとmysqli_queryの引数を減らして、
.mysqli_real_escape_string($_POST['email'])や、
mysqli_query($query)として実行しても「引数が足りません」というエラーになり、
うまくいきませんでした。

 補足情報(FW/ツールのバージョンなど)

windows10
MAMP
Brackets

分かる方、ご教示いただければ幸いです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

+5

これまでの質問の履歴を見ると、単純な文法エラーの解決ができなくて質問を繰り返しているようですが、一体どんな開発環境をお使いですか?

まともな開発環境があれば、このような単純なミスは事前に防ぐことができます。

こちらは提示されたコードを Netbeans に貼り付けたところです。

イメージ説明

コードを書いているときにリアルタイムに指摘してくれるので、こんな質問をしなくて済むようになりますよ、

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/09/21 15:11

    この際、エディタを変えようと思います。Netbeans等、おススメのエディタが他にもあれば教えてください。

    キャンセル

  • 2018/09/21 15:12

    自分はNetbeansしか使っていません。道具は自分で選びましょう。

    キャンセル

  • 2018/09/21 15:13

    なるほど。ご回答いただき、ありがとうございます。

    キャンセル

checkベストアンサー

+1

単純に

($link,$_SESSION['email'])

↓↓↓

mysqli_real_escape_string($link,$_SESSION['email'])

なんでしょうけど、インジェクション対策としては不十分です
引数を渡すときはprepareで処理してください

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/09/21 16:37

    madoka9393様>
    contentsがontentsになっておりましたので、修正させていただきました。

    キャンセル

  • 2018/09/21 16:40 編集

    訂正した上でもUndefined index(=$rowの中にcontentsなんてないよ)なのであれば、
    $rowの中身を確認してみるのはいかがでしょう。
    var_dump()とかprint_rでログや画面に吐かせてみるとか。

    キャンセル

  • 2018/09/21 18:25

    print_rで画面に$rowの中身を表示させ、いろいろ試してみたところ、ゴールまでたどり着けました。
    ありがとうございました。

    キャンセル

+1

エラーメッセージ、読まれました?
afterLogin.phpの21行目にエラーがあると書かれています。

$query = "SELECT contents FROM `users` WHERE email = '".($link,$_SESSION['email'])."'";


$query = "SELECT contents FROM `users` WHERE email = '".$_SESSION['email']."'";


に変更するといかがでしょうか。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/09/21 15:02

    ご回答、ありがとうございます。該当箇所を変更したところ、
    Recoverable fatal error: Object of class mysqli_result could not be converted to string in C:\MAMP\htdocs\mysql\afterLogin.php on line 23とエラーが変わりました。

    //本人の以前の日記を表示させる
    $query = "SELECT contents FROM `users` WHERE email = '".$_SESSION['email']."'";
    $result = mysqli_query($link,$query);
    echo "前回の投稿".$result;  ←23行目
    この部分でどこが間違っているのかわかりません。もし、お分かりであれば教えてください。

    キャンセル

  • 2018/09/21 15:09 編集

    横から失礼いたします。
    $resultをstring型に変換できない…ってエラーメッセージに書いてありますね(could not be converted to string)。
    調べればすぐに見つかるのでエラー内容で検索する癖をつけるとよいかもです。
    #誤記があったので訂正

    キャンセル

  • 2018/09/21 15:30

    http://php.net/manual/ja/mysqli-result.fetch-assoc.php
    手続き型の例を参考にされると良いと思います

    キャンセル

  • 2018/09/21 15:47

    文献のご提示、ありがとうございます。
    プログラミングに不慣れなので勉強してみます。

    キャンセル

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

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

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

  • PHP

    25048questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • MySQL

    7296questions

    MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

  • SQL

    3155questions

    SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

  • phpMyAdmin

    823questions

    phpMyAdminはオープンソースで、PHPで書かれたウェブベースのMySQL管理ツールのことです。

  • MAMP

    293questions

    Mac 上で WordPress などの動的ページのサイトが作れるように環境を構築するフリーソフト