Q&A
何がわからないのか、challenge_log.txtの内容を追記していただけませんか？
PHPでファイル操作の勉強をしていた時、if文内のfopen,fwrite,fgets,関数を使った条件の書き方について分からない点があったのでお聞きしたいです。以下にコードを記載いたします。
<?php $filename = './challenge_log.txt'; $log = ''; $errors = array(); if($_SERVER['REQUEST_METHOD'] === 'POST'){ $log = date("m月d日 H:i:s") . "\t" . $_POST['comment']; if(($fp = fopen($filename,'a')) !== FALSE){ if(fwrite($fp,$log) === FALSE){ echo 'ファイル書き込み失敗' . $filname; } fclose($fp); } } $data = array(); if(is_readable($filename) === TRUE){ if(($fp = fopen($filename,'r')) !== FALSE){ while(($tmp = fgets($fp)) !== FALSE){ $data[] = htmlspecialchars($tmp, ENT_QUOTES, 'UTF-8'); } fclose($fp); } }else{ $data[] = 'ファイルがありません'; } ?> <!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <title></title> </head> <body> <form method="post"> <p>発言:<input type="text" name="comment"> <input type="submit" value="送信"> </p> </form> <?php foreach($data as $value){?> <p> <?php echo $value; ?></p> <?php } ?> </body> </html>
if(($fp = fopen($filename,'a')) !== FALSE)やwhile(($tmp = fgets($fp)) !== FALSE)なんですが、while($tmp = fgets($fp) !== FALSE)のようにカッコをひとつ削ってもいいように思えます。while文でしたら何故$tmpの前にカッコが必要なのでしょうか？
説明がうまくできず申し訳ないですが、教えて頂けたら幸いです。
コード部分は```で囲ってください（分からなければ質問編集画面でコード部分を選択して<code>ボタンを押してください）
回答3件
0
ベストアンサー
代入演算子=の方が、比較演算子!==より優先度が低いのですが、代入演算子を先に実行させるために括弧が必要です。
PHP: 演算子の優先順位 - Manual
あなたの言っているのは、「(1+2)*3を、1+2*3と書いては駄目なのか？」という質問と同じです。
投稿2017/07/06 21:33
総合スコア84555
0
すみません。訂正です。
ifの中の()だと、booleanとして比較しないですね。。。勘違いしていました。
empty(),is_null(),isset()はちゃんと評価されますが、()だけだとダメです。
ので、以下は間違いです＾＾；
===
$fp や $tmp を外に出すと分かりやすいかもしれません。
php
1$fp = fopen($filename,'a'); 2if(($fp) !== FALSE)
つまり、比較したいのは、$fp の boolean が FALSE であるかどうかです。
PHP 型の比較表
個人的には、if や while の中で代入するとわかりにくいので、分けるのがオススメです。
投稿2017/07/06 11:07
編集2017/07/06 13:28
退会済みユーザー
総合スコア0
0
$tmp = fgets($fp)の実行結果を明示的に判定したいからだと思います。
足し算と掛け算を並べて足し算を先にしたいときは(a+b)*cのように書きますよね。
（ちょっと例は良くないかもしれませんが）
それと同じで、囲うことで明示的にすることになります。
可読性にもかかってきそうです。
投稿2017/07/06 11:01
総合スコア80850
あなたの回答
tips
プレビュー
