投稿番号を「最後の行+1」にしたい

他の似た質問を参考にコードを書いたのですが、
Parse error: syntax error, unexpected '[' in /ファイルのパス(公開できないため伏せています) on line 17てエラーが出てしまいます。
17行目のようです。
$lastline= $lines[count($lines) - 1];が17行目です。
コード中の投稿番号の定義以下にあります。

そして、この部分の書いてる意味がわからないために、どの様に修正して良いかもわかりません。
解説をおねがいしたいです。

以下がコード全体です。

php
1<?php
2/*もしポストで送信されたら以下の処理を行いGET送信でリダイレクトする（更新時の二重書き込み防止）*/
3if ( $_SERVER[ 'REQUEST_METHOD' ] === 'POST' ) { /*もしPOSTで送信されたら*/
4
5  /*フォームで送信された値を受け取り、テキストファイルに保存する。
6  その際1行ごとに「{番号}<>{名前}<>{コメント}<>{投稿された時間}」と言う形で保存する*/
7  /*変数の定義*/
8  $filename = "keijiban.txt"; /*保存するファイル*/
9  $name = $_POST[ 'name' ]; /*投稿者の名前*/
10  $comment = $_POST[ 'comment' ]; /*投稿するコメント*/
11  $date = date( "Y-m-d H:i" ); /*投稿日時*/
12
13  /*投稿番号の定義*/
14  if ( file_exists( $filename ) ) { /*ファイルの存在確認。*/
15  //最後の行にプラス1
16	  $lines=file($filename);
17	  $lastline= $lines[count($lines) - 1];
18    $num = explode('<>',$lastline)[0];
19  } else { /*ファイルが無かった場合変数の定義を１とする*/
20    $num = 1;
21  }
22  //書き込み内容
23  $hozon = $num . "<>" . $name . "<>" . $comment . "<>" . $date . "\n";
24
25  //投稿部分
26  if ( !empty( $_POST[ 'name' ] ) && !empty( $_POST[ 'comment' ] ) ) { /*もし名前とコメントが送信されたら*/
27    $fp = fopen( $filename, "a" ); /*追記モードでファイルを開く*/
28    if ( flock( $fp, LOCK_EX ) ) { /*ロックする*/
29      fwrite( $fp, $hozon ); /*指定したファイルに追記モードで書き込み*/
30      flock( $fp, LOCK_UN );
31    }
32    fclose( $fp );
33  }
34  //削除部分
35  if ( !empty( $_POST[ 'deleteNO' ] ) ) { /*もし、削除番号がポスト送信されたら*/
36    $deleteNO = $_POST[ 'deleteNO' ]; /*削除番号の変数定義*/
37    $ret_array = file( $filename ); /*ファイルを配列として読み込む*/
38
39    $fp = fopen( $filename, "w" );
40    if ( flock( $fp, LOCK_EX ) ) {
41      foreach ( $ret_array as $value ) { /*ループ*/
42        $bangou = explode( "<>", $value ); /*<>で分割し投稿番号取り出す*/
43
44        if ( $deleteNO != $bangou[ 0 ] ) { /*もし読み込んだ番号と送信した番号が違ったら*/
45
46          fwrite( $fp, $value ); /*ファイルに書き込む*/
47          flock( $fp, LOCK_UN ); //ロック開放
48        }
49        fclose( $fp );
50      }
51    }
52
53  }
54
55  header( 'Location: keijiban.php' );
56  exit;
57}
58?>
59<!doctype html>
60<html lang="ja">
61<head>
62<meta charset="utf-8">
63<title>簡易掲示板</title>
64</head>
65
66<body>
67<p>
68<form action="keijiban.php" method="post" >
69  <label for="name-field">お名前<span style="color: red;">【必須】</span><br>
70  </label>
71  <input type="text" name="name" id="name-field" required="required">
72  </p>
73  <p>
74    <label for="comment">コメント<span style="color: red;">【必須】</span></label>
75    <br>
76    <textarea name="comment" cols="30" rows="3" id="comment" required="required"></textarea>
77  </p>
78  <input type="submit" value="投稿">
79</form>
80<?php
81
82
83$filename = "keijiban.txt";
84$ret_array = file( $filename );
85foreach ( $ret_array as $value ) {
86  $bunkatu = explode( "<>", $value );
87  foreach ( $bunkatu as $value2 ) {
88    echo $value2 . "<br>\n";
89  }
90}
91?>
92<form action="keijiban.php" method="post">
93  <P>
94    <label for="deleteNO" >削除対象番号</label>
95    <input type="text" name="deleteNO" id="deleteNO" >
96  </p>
97  <input type="submit" value=”削除”>
98</form>
99</body>
100</html>

m.ts10806

2019/09/09 06:54

１７行目を分かるように指示してください。また、エラーはなるべく全文提示してください。

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

回答2件

ベストアンサー

ざっと見た感じSyntax エラー出てないですね。
＋１してるところがないので＋１されずずっと同じNumになってることは分かります。
なので

php
1$num = explode('<>',$lastline)[0];

↓

php
1$num = explode('<>',$lastline)[0]+1;

とすればとりあえずはNumは加算されていきます。

ただ、
同じ名称で同じ情報を格納した変数を何回も定義したり（例：$filename）
ファイルが存在しない場合に１としておきながら、画面表示時にはファイルの存在チェックを行っていなかったり
入力がない場合もいったん変数に受けて保存する文字列を作っていたり
画面表示時にHTMLエスケープしてなかったり
fopen～fcloseという結構古い手続きを踏んでたり

実装はあまりよろしくないと思います。

投稿2019/09/09 07:05

編集2019/09/09 07:07

m.ts10806

総合スコア80850

MakotoIshizawa

2019/09/09 07:11

色々ありがとうございます。ご指摘の細かいところも見直していきます。「fopen～fcloseという結構古い手続きを踏んでたり」という部分、もう少し解説お願いできますか。参考になるページだけでも教えていただけると助かります。

m.ts10806

2019/09/09 07:14

基本的にfile() file_put_contents()　で良いと思います。 https://www.php.net/manual/ja/function.file-put-contents.php > この関数は、fopen()、fwrite()、 fclose() を続けてコールしてデータをファイルに書き込むのと等価です。引数flagsにLOCK_EXを入れておけば排他ロックもできます。

MakotoIshizawa

2019/09/09 07:14

+1と書き換えると、今度は18行目に同じエラーが出てしまいました。 DW上では何もエラーにならないのに、アップロードするとブラウザでエラーが出ます。

MakotoIshizawa

2019/09/09 07:18

最初は file_put_contents()で書いてうまくいったんですが、ロックするのにうまくないと指摘をうけ書き直しました。今はどちらでも良いのでしょうか。

m.ts10806

2019/09/09 07:30

んー手元でPHP実行して出なかったので何とも。コードをコピペしなおしてもらえますか？ > ロックするのにうまくないと指摘をうけ書き直しました。今はどちらでも良いのでしょうか。こちらの質問でしょうか。 https://teratail.com/questions/210035 でもご自身で探された記事は「php-5.2.5以前のfile_put_contentsでは」とあります。そんなに古いバージョンでコーディングされてるんですか？念のため排他ロック実際にかかるかどうかなど試してみてください。

MakotoIshizawa

2019/09/09 08:10 編集

エラーについてWindowsだとだめとかあったりしますかね

m.ts10806

2019/09/09 08:15 編集

Syntaxエラーについてはあくまで構文の問題なのでOSは関係ありません。強いてならPHPのバージョンは関係あるかもしれませんが。

MakotoIshizawa

2019/09/09 08:19 編集

そうなんですねべつの書き方で最後の番号に+1できますか?

m.ts10806

2019/09/09 08:21

Syntax出てるの、読み込みの場所じゃないですよね・・・。結局「今現在何行あるか数えてそれに＋１する」のは変わりません。もしこれ以上「他に」なら、データをテキストではなくデータベースに切り替えてください。処理が煩雑になって問題切り分けも難しくなります。コードも冗長になります。

MakotoIshizawa

2019/09/09 08:22

ただ単に行数を投稿番号にするのは出来たんですが、それだと投稿を削除したりすると問題になっとしまいまして。

MakotoIshizawa

2019/09/09 08:26

読み込みの場所ですか。見てみます

m.ts10806

2019/09/09 08:27

単純に＋１するとそうなります。3番まであって3番を削除したら次は3番と。これも仕様次第になります。存在しなくなる番号だから許容するのか、4番から開始にするのか。物理削除でなく論理削除にすれば可能と思いますが、やはり確実にしたいならデータベースですね。

MakotoIshizawa

2019/09/09 08:48

https://jp.piliapp.com/php-syntax-check/ でphp-5.3.29でチェックした結果、エラーが表示されたのでバージョンの問題かもしれません

MakotoIshizawa

2019/09/09 09:14

$num = explode('<>',$lastline)[0]+1; の行を隠すと普通に表示されます。 $num = explode('<>',$lastline)[0]+1; の行を表示し+1を削除すると何故かエラーに。

m.ts10806

2019/09/09 11:51

たぶん。処理の優先順です。PHP7以降では出ないですね。

MakotoIshizawa

2019/09/11 04:42

ありがとうございます。調べてみます。

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

謎のSyntaxエラーはよくわかりませんでしたが、
+1が曲者のようなので、変数を一つ増やして別の行に書いてみたところうまくいきました。
以下が修正内容です。

php
1//元のコード
2 $lines=file($filename);
3 $lastline= $lines[count($lines) - 1];
4 $num = explode('<>',$lastline)[0]+1;
5
6//修正コード
7$lines=file($filename);
8$lastline=$lines[count($lines)-1];
9$num=explode("<>",$lastline);
10$lastnum=$num[0]+1;