Q&A
（質問は編集できます）コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564
前提・実現したいこと
1<>名前<>コメント<>時間の形で書かれているテキストファイルをfileで読み取り、foreachで配列にしたあと
explodeで分割してそれぞれの値にしたいです
間違いがあれば教えてください
発生している問題・エラーメッセージ
explodeを入れると
array
array
array
となります
explodeをコメントアウトすると正常です
エラーメッセージ
エラーではなさそうです
該当のソースコード
<!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <title>mission_3-2</title> </head> <body></form> <?php $filename="mission_3-2.txt"; $namae= ($_POST["namae"]); $comment = ($_POST["comment"]); $date = date("Y年m月d日 H:i:s"); if (file_exists($filename)) { $num = count(file($filename))+1; } else { $num = 1; } $format = $num."<>".$namae."<>".$comment."<>".$date; if (!empty($namae)||!empty($comment)){ $fp = fopen($filename,'a'); fwrite($fp, $format."\n"); fclose($fp); } $filename="mission_3-2.txt"; $lines = file($filename,FILE_IGNORE_NEW_LINES); foreach($lines as $line){ $lin = explode("<>", $line); print $lin."</br>"; } ?> </body> </html> php<form action="" method="post"> <input type="text" name="namae" value="名前"> <input type="text" name="comment" value="コメント"> <input type="submit" name="sousin">
なんとなく、やりたいのは explode() ではなく str_replace() のような気がするなあ。
一応今回はexplodeを使う課題なんですよね
そちらもきになるのでしらべてみます！
回答2件
0
ベストアンサー
「使い方」ならPHPマニュアルを確認するのが最も確実です。
explode — 文字列を文字列により分割する
explode ( string $delimiter , string $string [, int $limit = PHP_INT_MAX ] ) : array
**文字列の配列を返します。**この配列の各要素は、 string を文字列 delimiter で区切った部分文字列となります。
返却値がarray
つまり「配列」なので、そのままprintではなくループ処理を使って取り出します。
処理結果受け取った変数がどういう内容になっているのかを確認する癖もつけましょう。
var_dump()
蛇足：
テキストファイルにデータを保存し取り扱うやり方は面倒で組みにくいだけでほとんど実務で使われることがありません。もっときちんとロジックで悩んだ方が良いので、早い段階でデータベースに切り替えてください。
投稿2020/10/17 09:18
編集2020/10/17 09:31総合スコア80850
回答ありがとうございます
foreachというのもループを繰り返すためのものですよね？
1度explodeに引き渡したためにまたループ処理が外れたということでしょうか…？
知識不足で申し訳ないです
蛇足について
アドバイスありがとうございます
今はプログラミングの初心者向けインターン中なのでこの形式でやっていますが、インターンが終わって知識をある程度蓄えたらすぐにデータベースに移行したいと思います！！
>1度explodeに引き渡したためにまたループ処理が外れたということでしょうか…？
いいえ。
ループしている$linesの１要素$lineをさらに配列にしたので、配列にした結果、またループする必要があるということです。
ただ、やりたいことは何でしょうか？
「どのような情報をどのような形で表示したいか」で決める必要があります。
データは
num."<>".$namae."<>".$comment."<>".$date
で保存されているわけですし、そこからどのように表示したいのか？
ここは仕様なので、作る人が決める必要があります。
それ次第で、配列にする必要があるのか、配列にした後どう扱うか、変わってきます。場合によりループではなく個別で取り出すようにもなります。
＞インターンが終わって知識をある程度蓄えたら
インターンでこの形を教えているなら微妙ですね。
その会社の業務でテキストへの保存を使ってるわけではないでしょうし。
教材があまり良くなさそう。
遅れてしまい申し訳ないです
arrayの場合はそれに合わせた書き方があるんですね、ありがとうございます！
インターンではあるんですけど少し特殊な形態でして、就職支援が強いので多少教材が微妙でも目をつむって頑張ります
「このやり方覚えても就職しても使う機会がない」です。
一応IT系の会社何社かが合同で開催しているインターンなんですけどまさかそこまで実用性ないとは…
データベースのことを調べると明らかにこちらのほうが便利ですね
セキュリティもしっかりしていますし
基本的な書き方は似たようなところも多そうなので一応しっかりやりたいと思います
書き方は全然違いますよ。
あとあくまでPHPからすると外部の仕組みなので「SQL」という専用のコマンドを利用してデータを操作することになります。
PHPとテキストファイル、PHPとデータベース
PHP自体は一緒なので使われるコマンド？（echoとかJavaでいうところのSystem.out.println）みたいなのは大体同じだろうと思ったのですがまさかここから全然違ったり…？
何を懸念しているのかよくわかりません
SQLを使用することで構成がガラッと変わるくらいなら甘んじて受け入れますが、使う語彙までがまるっきり変わったりしたら大変そうだなあと思っただけです
語彙
んー。
Java知ってるならほぼ同等の書き方にはなりますけど、そういうことではなく？
まあプログラムは日進月歩の世界なので、今あるソースを捨てて作り直しなんて日常茶飯事ですよ。書いたとおりにしか動かないので「惜しい」とかはありません。
0
変数$linは配列になるので添字をつけましょう。
（誤）
PHP
1print $lin."</br>";
（正）
PHP
1print $lin[0]."-".$lin[1]."-".$lin[2]."-".$lin[3]."<br/>";
投稿2020/10/17 12:52
総合スコア55
それだけならimplodeで簡略的に書けます。
あなたの回答
tips
プレビュー
