テキストフォームに入力された文字列を「（投稿番号）＜＞（名前）＜＞（コメント）＜＞（投稿日時）」のようにテキストファイルの保存したい。なお、どんどん下に追記されていく形にしたい。

php
1<?php
2
3
4date_default_timezone_set('Asia/Tokyo');
5
6$filename = "mission_2-1_Yusuke10.txt";
7
8$line = file($filename);
9
10$number = count($line);
11
12
13?>
14
15
16<html>
17
18 
19 <form action="2-110.php"  method="post">
20
21
22お名前: <input type="text" name="name"><br>
23
24コメント: <input type="text" name="coment"><br>
25
26<input type="submit" value="送信">
27
28
29</form>
30
31
32</html>
33
34
35
36<?php
37
38
39$fp = fopen($filename, "a");
40
41if($fp)
42
43{fwrite($fp, $number."<>".$_POST['name']."<>".$_POST['coment']."<>".date("Y年m月d日 H:i:s"));
44
45fclose($fp);}
46
47
48?>
49
50

出力結果：
「Warning: file(mission_2-1_Yusuke10.txt) [function.file]: failed to open stream: No such file or directory in /home/tt-662.99sv-coco.com/public_html/2-110.php on line 8
お名前:
コメント:
送信」

となりました。自分としてはfileやcount、fopen、fwrite、fcloseの使い方に自信がありません。根本的にこういう書き方はしないよとか、なんか忘れてるよとか、あったらお願いします。

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

回答2件

php.netでドキュメントを読んでほしいですね。

PHP: file - Manual

ファイルが存在しない場合は E_WARNING レベルのエラーが発生します。

ってことなんだと思います。
そのため、ファイルの存在を先にチェックしてから読み込むようにするのが最初かと。
PHP: file_exists - Manual
でチェックできますよ。

php
1$line = array();
2if (file_exists($filename)) {
3    $line = file($filename, FILE_IGNORE_NEW_LINES);
4}

こんなふうにしておくことで、
ファイルが有ればファイルから読み取って$line配列を作ってくれるし、
ない場合は先に初期化したカラのままの$line配列を使えばいい。

file()のドキュメントを読んでほしいのですが、
FILE_IGNORE_NEW_LINESフラグを指定することで、
読み出した一つ一つの行データの末尾に改行コードがつかないように回避できます。
改行コードがついたままでも、
htmlとして文字列を出力するときテキストとしての改行コードは無視されるので、
見た目上の影響はないかもしれませんが、
その点は「要求仕様をどう解釈して、仕様を満足させるコードをどう書くか」という矜持次第かと。

つづけて、
file()での読み込みは
一行一行のデータの区切りとして改行コードをアテにしているので、
ファイルに追記するときは一行データの最後に(テキストとしての)改行コードが必要になる前提です。

試しに
PHP: file_put_contents - Manual
使ってみますか、

php
1file_put_contents($filename, $number."<>".$_POST['name']."<>".$_POST['coment']."<>".date("Y年m月d日 H:i:s")."\n", FILE_APPEND);

ってことでもいいのかな、たぶん。
「<br/>」という文字列はあくまでhtmlコード上で改行を意図するタグであって、
file()でのデータ区切りには意味を成さないです。

そういえば、これ簡易掲示板だったよね。
file()やfile_get_contents()やfile_get_contents()って
一行書くだけで読み込みや書き込みができて便利なんだけど、
簡易掲示板は複数箇所から同時にアクセスされることを想定しないといけないよ。
同じタイミングで投稿データを受け付けた場合、
追記するだけとはいえファイルが無事に書き込み成功する保証はない。
だから、自分の処理中に他者が書き込まないよう、
ファイルを排他ロックしておく必要があるね。

そこで登場するのが
PHP: flock - Manual
って関数なんだけど、ファイル名を受け取るわけじゃなく、
fopen()したときのファイルハンドルを与える必要がある。
そうするとfile()でラクに読み込む方法が使えなくなる。

fopen()でファイルを開いてファイルハンドルを取得し、
fgets()でファイルから一行データを読み出すのを繰り返して、
fclose()で読み込み終了すると。

書き込みは、
fopen()でファイルを開いてファイルハンドルを取得し、
flock()で排他ロックをかけて、
fwrite()でファイルに書き出して、
fclose()で書き込み終了すると。

投稿2018/10/31 04:22

編集2018/10/31 08:21

退会済みユーザー

総合スコア0

ykk_

2018/10/31 04:43

まだ作られてないテキストファイルを使って、変数を定義するという、プログラミングは上から下に実行されていくという基本中の基本をわかってないことをしてしまいました。しかし、第一回目の書き込みで投稿番号を使う、つまりテキストファイルがある状態で定義される変数を使わなければいけません。結局これを解決するためには1回空ののテキストファイルを作って、そこから投稿番号も定義して、書き込みという流れになると思うのですが、こういうやり方であっているでしょうか？また、その場合はどのようなソースコードを書けばいいでしょうか？

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

php
1
2<html>
3
4 
5 <form action="2-114.php"  method="post">
6
7
8お名前: <input type="text" name="name"><br>
9
10コメント: <input type="text" name="coment"><br>
11
12<input type="submit" value="送信">
13
14
15</form>
16
17
18</html>
19
20
21<?php
22
23touch('mission_2-1_Yusuke114.txt');
24
25
26?>
27
28
29
30
31<?php
32
33
34date_default_timezone_set('Asia/Tokyo');
35
36$filename = 'mission_2-1_Yusuke114.txt';
37
38$line = file($filename);
39
40$number = count($line+1);
41
42
43?>
44
45
46<?php
47
48
49$fp = fopen($filename, "a");
50
51if($_POST["name"] and $_POST["coment"])  
52
53{fwrite($fp, $number."<>".$_POST['name']."<>".$_POST['coment']."<>".date("Y年m月d日 H:i:s"),"<br>");
54
55fclose($fp);}
56
57
58?>