Parse error: syntax error, unexpected ','

前提・実現したいこと

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

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

行動規範の内容に同意します

回答3件

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

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

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

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

投稿2018/09/21 05:34

退会済みユーザー

総合スコア0

1004

2018/09/21 05:39

ご回答、ありがとうございます。エディタはBracketsを使用いています。あと、自習用にVisualStudioCodeを使っています。 Netbeansも使ってみます。リアルタイムで指摘してくれるのはありがたいですね。

退会済みユーザー

2018/09/21 05:47

Brackets でも構文チェックをできるようにすることはできるらしいですが、面倒なようです。 https://qiita.com/motof178/items/ec945512c635c4934096

1004

2018/09/21 06:11

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

退会済みユーザー

2018/09/21 06:12

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

1004

2018/09/21 06:13

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

行動規範の内容に同意します

ベストアンサー

単純に

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

↓↓↓

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

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

投稿2018/09/21 05:46

yambejp

総合スコア114779

1004

2018/09/21 06:09 編集

ご回答、ありがとうございます。 $query = SELECT contents FROM `users` WHERE email =mysqli_real_escape_string($link,$_POST['email']); で合っていますか？変更したところ、Parse error: syntax error, unexpected 'contents' (T_STRING) in C:\MAMP\htdocs\mysql\afterLogin.php on line 21 となり、エラーが出ます。

yambejp

2018/09/21 06:15

$query = "SELECT `contents` FROM `users` WHERE `email` = '".mysqli_real_escape_string($link,$_SESSION['email'])."'"; じゃないですか？

1004

2018/09/21 06:46

変更したところ、Recoverable fatal error: Object of class mysqli_result could not be converted to string in C:\MAMP\htdocs\mysql\afterLogin.php on line 23 とエラー内容が変わりましたので、原因を調べています。もし、何か分かれば、教えて下さい。最終的には$resultでDBのcontentsデータを表示できれば目標が達成できます。

yambejp

2018/09/21 06:52

$row = mysqli_fetch_row($result); print $row["contents"]; みたいな感じです

madoka9393

2018/09/21 06:53

横から失礼いたします。以下が参考になるのではないかと。 https://teratail.com/questions/208 Googleで調べればすぐに見つかるんですけどね。(could not be converted to stringで検索した最上段)

1004

2018/09/21 07:04

madoka9393様＞非常に参考になりました。残る課題としては$resultにcontents列の内容を表示させることです。調べてみます。

1004

2018/09/21 07:36 編集

yambejp様＞ご回答ありがとうございます。ご提示いただいた内容で試してみたところ、下記のエラーが出てしまいます。 Notice: Undefined index: contents in C:\MAMP\htdocs\mysql\afterLogin.php on line 24 下記のようにコードを変えました。 //本人の以前の日記を表示させる $query = "SELECT `contents` FROM `users` WHERE `email` = '".mysqli_real_escape_string($link,$_SESSION['email'])."'"; $result = mysqli_query($link,$query); $row = mysqli_fetch_row($result); print($row["contents"]); 改善策はありますでしょうか？(投稿を修正いたしました。)

madoka9393

2018/09/21 07:27

再度横から失礼をば。エラーメッセージとコードをよーく見てみましょう…。

1004

2018/09/21 07:37

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

madoka9393

2018/09/21 07:44 編集

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

1004

2018/09/21 09:25

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

行動規範の内容に同意します

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

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

を

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

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

投稿2018/09/21 05:38

Takumiboo

総合スコア2534

1004

2018/09/21 06: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;　　←２３行目この部分でどこが間違っているのかわかりません。もし、お分かりであれば教えてください。

madoka9393

2018/09/21 06:12 編集

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

Takumiboo

2018/09/21 06:30

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

1004

2018/09/21 06:47

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

行動規範の内容に同意します

あなたの回答