fwriteでfwrite(): supplied resource is not a valid stream resourceのエラーがでる。

Question

laravelのコマンドでCSV読み込みや書き込みをしています。

上記を読み込み今月まで書き込み新たにファイルを作っています。
上記でいうと2021/03/01のデータがいらない。

        $test = storage_path('app/public/study05.csv');
        $pf = fopen($test, 'r');
        $test1 = fopen('test.csv', 'w');
        if ($pf === false) {
            echo "ファイルオープン時に失敗したので処理終了します。" . PHP_EOL;
            exit;
        }

        $now = Carbon::today();
        $out_year = substr($now, 0, 4);
        $out_month = substr($now, 5, 2);
        $i = 0;

        while ($line1 = fgets($pf)) {
            $i++;


         // カンマ分割
         $test_field = explode(',', $line1);

         // 日付分割
         $test_date = explode('/', $test_field[0]);

        

            $test2 = $test_field[0].','.$test_field[1].','.$test_field[2];
            $test3 = mb_convert_encoding("$test2", "SJIS-win");
            if($test_date[0] < $out_year || $test_date[1] <= $out_month){
                fwrite($test1, $test3);
            }

        }
        fclose($pf);
        fclose($test1);

上記のように組んだのですが、fwrite($test1, $test3);のところで
fwrite(): supplied resource is not a valid stream resource
のエラーが出ます。fopenが上手くいってないなどありましたが、
値は取れているのでfopen出来ていると思うのですが、何が原因でしょうか？

Comments

[退会済みユーザー]

$testと$test1と$test2と$test3で性格の違うものを扱っていてわかりにくくなっています。fopen()とかするときは例えば$fp（ファイルポインター）や$fh（ファイルハンドラーの略か）などとするとわかりやすいです。中身を類推しやすい変数名にしましょう。

[otn]

openの返り値をチェックしましょう。

[ykkk]

書き込めました。ありがとうございます！

ただ、2月以降の消込ができておりません。

$out_yearには2021
$out_monthには02
が入っています。

現状ですと、
2020/1/1 100
2020/1/1 100
2020/2/1 200
2020/3/1 500

で止まってしまいます。

Answer 1

while ($line1) {のループの中で
fclose($test1);を1回目の実行でファイルをクローズして、
ループなので2回めの実行で無効なファイルポインターについてエラーメッセージになっています。

また、whileループに入る前に一度きり$line1 = fgets($fp);を実行していて、
2回目以降の読み込み処理が動いていません。
実はwhileループの終了条件が満たせず無限ループになっています。

構造の見直しを。

Comments

[ykkk]

書き込めました。ありがとうございます！

ただ、2月以降の消込ができておりません。

$out_yearには2021
$out_monthには02
が入っています。

現状ですと、
2020/1/1 100
2020/1/1 100
2020/2/1 200
2020/3/1 500

で止まってしまいます。

[ykkk]

出来ました。
ご教授ありがとうございました！

[退会済みユーザー]

変数名やインデント量を工夫して、testだらけにしないようにわかりやすくするといいでしょう。
VScodeでもAtomでもプログラミングに向いているエディターを使うと良いです。