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

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

ただいまの
回答率

88.10%

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

解決済

回答 2

投稿 編集

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

score 32

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

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

以下がコード全体です。

<?php
/*もしポストで送信されたら以下の処理を行いGET送信でリダイレクトする(更新時の二重書き込み防止)*/
if ( $_SERVER[ 'REQUEST_METHOD' ] === 'POST' ) { /*もしPOSTで送信されたら*/

  /*フォームで送信された値を受け取り、テキストファイルに保存する。
  その際1行ごとに「{番号}<>{名前}<>{コメント}<>{投稿された時間}」と言う形で保存する*/
  /*変数の定義*/
  $filename = "keijiban.txt"; /*保存するファイル*/
  $name = $_POST[ 'name' ]; /*投稿者の名前*/
  $comment = $_POST[ 'comment' ]; /*投稿するコメント*/
  $date = date( "Y-m-d H:i" ); /*投稿日時*/

  /*投稿番号の定義*/
  if ( file_exists( $filename ) ) { /*ファイルの存在確認。*/
  //最後の行にプラス1
      $lines=file($filename);
      $lastline= $lines[count($lines) - 1];
    $num = explode('<>',$lastline)[0];
  } else { /*ファイルが無かった場合変数の定義を1とする*/
    $num = 1;
  }
  //書き込み内容
  $hozon = $num . "<>" . $name . "<>" . $comment . "<>" . $date . "\n";

  //投稿部分
  if ( !empty( $_POST[ 'name' ] ) && !empty( $_POST[ 'comment' ] ) ) { /*もし名前とコメントが送信されたら*/
    $fp = fopen( $filename, "a" ); /*追記モードでファイルを開く*/
    if ( flock( $fp, LOCK_EX ) ) { /*ロックする*/
      fwrite( $fp, $hozon ); /*指定したファイルに追記モードで書き込み*/
      flock( $fp, LOCK_UN );
    }
    fclose( $fp );
  }
  //削除部分
  if ( !empty( $_POST[ 'deleteNO' ] ) ) { /*もし、削除番号がポスト送信されたら*/
    $deleteNO = $_POST[ 'deleteNO' ]; /*削除番号の変数定義*/
    $ret_array = file( $filename ); /*ファイルを配列として読み込む*/

    $fp = fopen( $filename, "w" );
    if ( flock( $fp, LOCK_EX ) ) {
      foreach ( $ret_array as $value ) { /*ループ*/
        $bangou = explode( "<>", $value ); /*<>で分割し投稿番号取り出す*/

        if ( $deleteNO != $bangou[ 0 ] ) { /*もし読み込んだ番号と送信した番号が違ったら*/

          fwrite( $fp, $value ); /*ファイルに書き込む*/
          flock( $fp, LOCK_UN ); //ロック開放
        }
        fclose( $fp );
      }
    }

  }

  header( 'Location: keijiban.php' );
  exit;
}
?>
<!doctype html>
<html lang="ja">
<head>
<meta charset="utf-8">
<title>簡易掲示板</title>
</head>

<body>
<p>
<form action="keijiban.php" method="post" >
  <label for="name-field">お名前<span style="color: red;">【必須】</span><br>
  </label>
  <input type="text" name="name" id="name-field" required="required">
  </p>
  <p>
    <label for="comment">コメント<span style="color: red;">【必須】</span></label>
    <br>
    <textarea name="comment" cols="30" rows="3" id="comment" required="required"></textarea>
  </p>
  <input type="submit" value="投稿">
</form>
<?php


$filename = "keijiban.txt";
$ret_array = file( $filename );
foreach ( $ret_array as $value ) {
  $bunkatu = explode( "<>", $value );
  foreach ( $bunkatu as $value2 ) {
    echo $value2 . "<br>\n";
  }
}
?>
<form action="keijiban.php" method="post">
  <P>
    <label for="deleteNO" >削除対象番号</label>
    <input type="text" name="deleteNO" id="deleteNO" >
  </p>
  <input type="submit" value=”削除”>
</form>
</body>
</html>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • m.ts10806

    2019/09/09 15:54

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

    キャンセル

回答 2

checkベストアンサー

+2

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

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


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

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

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

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

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/09/09 18:14

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

    キャンセル

  • 2019/09/09 20:51

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

    キャンセル

  • 2019/09/11 13:42

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

    キャンセル

0

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

//元のコード
 $lines=file($filename);
 $lastline= $lines[count($lines) - 1];
 $num = explode('<>',$lastline)[0]+1;

//修正コード
$lines=file($filename);
$lastline=$lines[count($lines)-1];
$num=explode("<>",$lastline);
$lastnum=$num[0]+1;

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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