-PHP- 外部のテキストファイルから文字列を検索し該当箇所の行を指定の色で表示する。

今回、外部からテキストファイルを読み込み、該当箇所red , blue , greenが含まれている行を赤、青、緑で表示するプログラムを作ろうとしてます。
しかし、PHPが実行できないどころか同じくif(strpos($tmp,"red")で文字列の有無を判断させるプログラムも"no"の判定の一点張りでした。

1.このプログラムはstrposで文字判定を行い、該当箇所の文字列の色変更は"で各文字列の色の変更を行うものとします。

なお、回答者の方にも指摘をいただきましたhttps://teratail.com/questions/53555の質問内容にも重複する部分がありますが、文字列を検索する対象は$tmpであっているでしょうか。

お手数をかけますが、間違っている箇所の指摘をお願いします。

PHP
1<!DOCTYPE html>
2<html lang="ja">
3
4<head>
5	<meta charset="UTF-8">
6	<title>外部ファイル読み込み</title>
7</head>
8
9<body>
10	<h1>ファイルの読み込み</h1>
11	<p>PHPでファイルを読み込むには、fopenを使う</p>
12<?php
13$filename = "ex151.txt";
14$i = 1;
15if(file_exists($filename)){
16	$f = fopen($filename, "r");
17	while(!feof($f)){
18		$tmp = fgets($f);
19		echo("{$i}行目：{$tmp}<br>");
20		$i++;
21if(strpos($tmp,"red"){
22echo ("<span style='color:red;'>"{$i}行目：{$tmp}"</span>");
23	}else{
24;
25}
26	fclose($f);
27}else{
28	echo("ファイル{$filename}はありません");
29}
30
31?>
32</body>
33
34</html>
35

IDE Visual Studio 2015 PHP tools for Visual Studio

kei344

2016/11/02 15:20

回答が付いた質問の編集は慎重に行ってください。質問文のコードについて回答にて指摘があった場合は「追記」し、元のコードを編集する場合も「直したこと」がわかるようにしてください。

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

回答2件

PHP
1// 2016/11/03 00:50編集版
2$filename = "ex151.txt";
3$i = 1;
4if(file_exists($filename)){
5    $f = fopen($filename, "r");
6    while(!feof($f)){
7        $tmp = fgets($f);
8        echo("{$i}行目：{$tmp}<br>");
9        $i++;
10if(strpos($tmp,"red"){
11echo ("<span style='color:red;'>"{$i}行目：{$tmp}"</span>");
12    }else{
13;
14}
15    fclose($f);
16}else{
17    echo("ファイル{$filename}はありません");
18}

↓インデントは適切に付ける癖を付けましょう。

PHP
1$filename = 'ex151.txt';
2$i = 0;
3if ( file_exists( $filename ) ) {
4    $f = fopen( $filename, "r" );
5    while( !feof( $f ) ) {
6        $i++;
7        $tmp = fgets( $f );
8        echo "{$i}行目：{$tmp}<br>";
9        if ( strpos( $tmp, "red" ) ) {
10            echo "<span style='color:red;'>{$i}行目：{$tmp}</span>";
11        } else {
12            ;
13        }
14    }
15    fclose( $f );
16} else {
17    echo 'ファイル'.$filename.'はありません';
18}

投稿2016/11/02 16:15

kei344

総合スコア69407

pinsong

2016/11/02 16:23

ありがとうございます。まだ、terapadの時の癖が残りtabを押さずにそのまま続けてしまうことがよくあるので気を付けるようにします。

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

ベストアンサー

最近ほぼ同じような質問を回答したような気がしますが、これでいけますか？

https://teratail.com/questions/53555

###追記
質問のコードの指摘との事ですので

<?php
$filename = "textfile.txt";

if (strpos($tmp,'red') != false){　//　$tmpはこの時点ででてきていないのにいきなり使われている

//　ここはphpを書く<?php ?>　内なので<spanタグなど書くときはechoなどしないといけない
// $iもまだ未登場の変数
<span style="color:red;">"{$i}行目：{$tmp}"</span
}

// 上と同じでまだ$tmpはなにも入っていない状態、
// またstrposの評価、上の(!= false)と( == true)がほぼ等価なので2回でてくるのはおかしい
if(strpos($tmp,'red') == true){
(何もしない)
} 
$i = 1;
// ここから下でファイルを読み込んでいるがここより上の行にあるredが含まれているかなどの判定のためにかかれたであろうコードを利用していないので、正常に動いてもファイルの内容をそのまま出力するだけになっている
if(file_exists($filename)){
    $f = fopen($filename, "r");
    while(!feof($f)){
        $tmp = fgets($f);
        echo("{$i}行目：{$tmp}<br>");
        $i++;
    }
    fclose($f);
}else{
    echo("ファイル{$filename}はありません");
}
?>

投稿2016/11/02 14:32

編集2016/11/02 15:17

hiim

総合スコア1689

pinsong

2016/11/02 14:38

回答ありがとうございます。上記URLは事前に参照しましたが、色を変えたい行に対して同じくstrposしたと思うんですがどうしてもエラーが残ってしまい実行できませんでした。付け加えると、もエラーが表示されてしまいビルドができませんでした。

hiim

2016/11/02 14:48

上記リンクは普通に動作すると思うのですが、、、それでもエラーがでるのであれば前後に何を書いたかが問題な気がします。質問欄に記載されているコードでみてみるとファイルの名前を変数に入れた直後にファイルをオープンも読んでもいないのにいきなりstrposで行を評価しているが、いったいその行はどこからもってきたのか？また行数の$iは何なのかもわからない段階で使っています。その下までいってようやくファイルを開いて、１行づつ読み込んでいる。とりあえず、記載されているコードは書く順序でエラーになると思いますので、上記URLのコードも抜粋しか記載されていないため、使いどころに間違いがあるのではないかな？と思います

pinsong

2016/11/02 15:17

質問文のほうをいくらか編集しました。肝心の$iのほうは、ページに表示する際文字列が何行目にあるのかを表示するものです。説明不足ですみません。 if(strpos)は、とりあえずfopenでテキストファイルを開いてからのほうが処理の順番として適切かと思いfgetsの下、fcloseの上に配置しました。

hiim

2016/11/02 15:19

追記で回答を書き終わったら、質問自体が大幅に編集されてるし。。。。。

hiim

2016/11/02 15:26

編集されたコードに関してコメントするとまず括弧の数おかしいと思います。次に $tmp = fgets($f); echo("{$i}行目：{$tmp} "); としてしまうと色の判定もなにもなくまず行を表示しているその後で if(strpos($tmp,"red"){ echo ("{$i}行目：{$tmp}");//redを赤色にしたいとredが含まれているか判定している、またこの$iをこれより上でインクリメントしているので実際ファイル内の3行目にredがあったとしてもこれでは4行目と表示されてしまう。などでしょうか。。。。

kei344

2016/11/02 15:29

横から失礼します。 style="color:red;"も style='color:red;' か style=\"color:red;\" にする必要がありますね。

hiim

2016/11/02 15:32

本当ですね、（汗、 to 質問者様：ダブルクオーツの中のダブルクオーツはエスケープしましょう

pinsong

2016/11/02 15:39

あ、なるほど。外に"があるから中はシングルクオーテーションってことですね。これは、失礼しました。style='color:red;'ですね。ソースの方、編集しておきます。

kei344

2016/11/02 15:47

To: pinsongさん質問文のコードの「redを赤色にしたい」の行が新たにエラーになっていますよ。