Stream​Reader.​Read​Lineで読み取ったlogファイルの内容を逆順で読み込みたい

Question

表題の通りなのですがlogファイルの中身を逆の順番で読み取ろうとしています。
いくつか試してみたのですがうまく再現できなかった為、お知恵をお借り出来たらと思います。
・試してみたこと
sr.ReadLineを⇒sr.ReadLine().reverseに
↓
System.Linq.Enumerable+ReverseIterator`1[System.Char]と表示されるだけ

sr.Peek()をsr.Peek().reverse()
↓
エラー

リストに一度すべて格納してreverse？そのほかに逆順でループ等々ありそうなのですが
どれが最適解かいまいちわかりません。
お手数おかけしますがご教示いただけますと幸いです。

        private void Line_read()
        {

            try
            {
                string Path = @"C:\Users\user\Documents\test.log";
                using (FileStream fs = new FileStream(Path,
                    FileMode.Open, FileAccess.Read, FileShare.ReadWrite))
                {   
           
                    using (TextReader sr = new StreamReader(fs,
                        Encoding.GetEncoding("UTF-8")))
                    {
                        String line;
                        // Read and display lines from the file until the end of
                        // the file is reached.
                        while (sr.Peek() > -1)
                        {
                            //※逆順で一行読み込んで表示する
                            Debug.WriteLine(sr.ReadLine());
                        }

                        sr.Close();

   
                    }
                }
            }
            catch (Exception e)
            {
                // Let the user know what went wrong.
                Debug.WriteLine("The file could not be read:");
                Console.WriteLine(e.Message);
            }
        }

追記：
記載漏れがありました。すみません。
逆順で１行ずつ回しながらその行の検索を行う、という処理を挟むまでが全容なのですが
リストにすべて格納するというと、全体を通常の順序で配列に取り込み⇒reverseして
配列を逆に、という形になると思います。
当然そこまでは思いつくのですが、そうなると
一度配列に入れる正順ループを行った後、リバースして
さらにループと、２回ループするようで不格好かと思います。
１度のループで配列に取り込まず最初から
逆順に読み込もうとして方法がうまく思いつかない、という旨の質問となります。
コメント頂いた方々には申し訳ありません。
もしよろしければ再度ご教示いただけますと幸いです。
（２度ループするしか方法がないというのであればそれはそれで構いません）
不勉強かつ言葉足らずでで申し訳ない限りです。

Comments

[退会済みユーザー]

> リストに一度すべて格納してreverse？そのほかに逆順でループ等々ありそうなのですが どれが最適解かいまいちわかりません。 お手数おかけしますがご教示いただけますと幸いです。

こちらの方法で機能としては実現できませんか？

[neconekocat]

普通に実装するなら全てを読み込んで行データの配列にするのが正しいと思います。
ただ、増えていくログを監視して動的に追いたい(tail -fやless +f的なもの)と考えているのであれば、
ファイルの改行コードを検索してそこへSeek→そこからReadLineという形での実装が一番かなと思います

[dodox86]

先に皆さんにコメントいただいているように、
> リストに一度すべて格納してreverse？そのほかに逆順でループ等々ありそうなのですが どれが最適解かいまいちわかりません。

「どれが最適解か」などと一番良い方法を最初からやろうとしたり教えてもらおうとするのではなく、最初は愚直でも自分でできる簡単な方法でやればよいです。

[退会済みユーザー]

> 普通に実装するなら全てを読み込んで行データの配列にするのが正しいと思います。

それなら質問者さんは問題なく実装できるのですか？　もし、今の質問者さんのスキルでそれができないということなら、それができてから話をした方が良いのでは？

[neconekocat]

不格好の意味がよくわかりませんが、何が何でも後ろから1行ずつ読みたい、というのであればFileStreamを使ってそういう処理を自分で書くしか無いです。ただし1行読むのにかかる時間はReadLineより長くなるため、全行を処理するつもりなら確実にパフォーマンスが落ちます。(巨大なファイルの最後の数行のみを処理したい等の用途であれば早くなります)
最適解を知りたいのであれば、どういう用途なのかを明確にしなければ誰も答えられません。

[ookura]

neconekocat様
ご回答ありがとうございます。
ご教示いただいたようにFileStreamで処理を作成する用途が一番希望に近しいかと思いました。
一度、その方針で調べてみます。
ベストアンサーに挙げたいのですが、追記・修正依頼でのご回答をいただいていたので、よろしければ回答として再投稿いただくことは可能でしょうか？

[dodox86]

どうもややこしい方向に行っている気がするのですが、行というのはファイルを読んで初めて意識できることで、ファイルの最後の行から読むにはファイルを全部バッファに読むか、あるいは最後の方と思わしき部分（ファイルの後ろの方だけ）をバッファに読んで、直前の行('\n'）の位置まで出力位置を逆に移動するしかありません。File.ReadLines()などを使えば行の意識を.NETのクラスライブラリがやってくれるのでそれを使うのが簡単、と言う話です。「FileStreamやTextReaderクラスを使う」縛りでもあるのでしょうか。

[neconekocat]

質問者さんの別の質問を見た感じ、ログファイルを監視して都度最終行を取得する必要があるので、本当の目的はパフォーマンス向上をしたい、というXY問題ではないかと疑っています。
残念ながら回答が得られていない為、推測止まりですが。

Answer 1

あくまでも例ですが、後ろから1行ずつ読みたいのであればこんな感じの処理が必要になります。
多くの人が指摘していますが、実装する上でこんなモノが本当に必要かどうかはよく考えてください。
特殊な状況を除けばFile.ReadLines().Reverse()等で事足りることが大半だと思います。

なお、このサンプルの1行あたりの処理は遅いです。ファイルによってはTake等で適切に処理しないと地獄を見ます。

private IEnumerable<string> ReverseReadLines(string path)
{
    using var fs = new FileStream(path, FileMode.Open, FileAccess.Read, FileShare.ReadWrite);

    long current;
    long start;
    long end;
    byte[] buff = new byte[100];
    byte[] line;

    fs.Seek(-1, SeekOrigin.End);
    fs.Read(buff, 0, 1);
    // ファイルが改行で終わっている場合は飛ばしておく
    end = buff[0] == 0x0A ? fs.Seek(-2, SeekOrigin.End) : fs.Seek(0, SeekOrigin.End);
    current = fs.Position;

    while (current > 0)
    {
        int size = (int)Math.Min(current, buff.Length);
        fs.Seek(-size, SeekOrigin.Current);
        fs.Read(buff, 0, size);

        for (int i = 0; i < size; i++)
        {
            if (buff[size - i - 1] == 0x0A) // LF
            {
                start = fs.Seek(-i, SeekOrigin.Current);
                line = new byte[end - start];
                fs.Read(line);

                yield return System.Text.Encoding.UTF8.GetString(line);

                // 前の行へ
                end = fs.Seek(-line.Length - 2, SeekOrigin.Current);    // CRLF決め打ち
                break;
            }
        }

        if (current == fs.Position)
        {
            //  読み込んだ範囲内に改行がなかった場合は更に前方を検索
            fs.Seek(-size, SeekOrigin.Current);
        }
        current = fs.Position;
    }

    // 1行目
    line = new byte[end];
    fs.Read(line);

    yield return System.Text.Encoding.UTF8.GetString(line);
}