c# 特定の文字を含む行の次の行に特定の文字を含む場合の取得方法

Question

Webからデータを取得し、特定の文字を含む行の次の行に特定の文字を含む場合に、その行を取得します。
StringReaderとReadLineで実装しましたが、もう少しスマートな実装はないでしょうか。

#EXT-X-STREAM･･･640x360･･･
https:･･･abc.m3u8･･･　←これではなく
･
#EXT-X-STREAM･･･640x360･･･
https:･･･xyz.m3u8･･･  ←これを取得したい

var m3u = wc.DownloadString(url);
var sr = new StringReader(m3u);
using(sr)
{
	while (sr.Peek() > -1)
	{
		var line = sr.ReadLine();//1行ずつ取得
		if (line.Contains("#EXT-X-STREAM") && line.Contains("640x360"))
		{
			var nextline = sr.ReadLine();//次の行を読み込む
			if (nextline.Contains("xyz"))
			{
				(nextlineを取り出す);
			}
		}
	}
}

Comments

[m.ts10806]

つまり「１行ずつ読み込みながら特定の文字列がきたときだけその次の行を取得する」というコードを書いてるわけですよね。
いいんじゃないでしょうか。シンプルで分かりやすいです。
複雑な正規表現を書くよりは不具合もないと思います。

[kt_csharp]

そのとおりです。
ネストが深くなるので、簡潔なコードがあればと思いました。

[m.ts10806]

というか「次の１行を取得」という操作は必ず発生するので、むしろこういうやり方がベターなんじゃないでしょうか。
以前私も出力されたテキストの解析やったことありますが、結局全て回して１文字ずつ解析しました。
情報が１行ずつDBに入ってればネストを１つ浅くできるかもしれないくらいの感じですね。
まあ私個人の感覚なので、もしかしたらもっと良い手があるかもしれませんが。

[kt_csharp]

当初、ReadLinesを使ってLINQで書いていたのですが、おっしゃるとおり「次の１行を取得」する必要があるので、この方法に落ち着きました。

[hihijiji]

メモリ効率より速度を優先するなら、
一度string配列にしてZipで自己結合して全ての組み合わせを作り、
マルチスレッドで処理するのはどうでしょう？

[Zuishin]

LINQ を使うなら System.Interactive を Nuget でインストールして Buffer を使うと便利です。

var result = EnumerableEx
.Repeat(0)
.Select(_ => reader.ReadLine())
.TakeWhile(a => a != null)
.Buffer(2, 1)
.Where(a =>
{
return a.Count == 2
&& a[0].Contains("#EXT-X-STREAM")
&& a[0].Contains("640x360")
&& a[1].Contains("xyz.m3u8");
})
.FirstOrDefault();

if (result != null)
{
Console.WriteLine(result[1]);
}

[pepperleaf]

Zuishinさん、回答に書かれた方が、見易くないですか?

[Zuishin]

LINQ を使うならっていう縛りを入れただけで、私は質問のもので十分だと思うんですよね。いわゆる余談です。

[kt_csharp]

Zuishinさんいつもありがとうございます。
さっそく勉強してみます。

Answer 1

今回のWebデータは必ず２行一組の構造になっているので、２行ずつ処理することが可能でした。

using(var sr = new StringReader(m3u))
{
    var line = sr.ReadLine();//1行目を取得
    var nextline = sr.ReadLine();//2行目を取得
    while (nextline != null)
    {
        if (line.Contains("#EXT-X-STREAM") && line.Contains("640x360") && nextline.Contains("xyz"))
        {
           (nextlineを取り出す);
        }
        //以下２行ずつ取得する
        line = sr.ReadLine();//3行目を取得（以下5、7、9･･･行目を取得）
        nextline = sr.ReadLine();//4行目を取得（以下6、8、10･･･行目を取得）
    }
}

Answer 2

元のでも問題無いとは思いますが、ちょっとだけ、手を入れてみると、

using(var sr = new StringReader(m3u))
{
    var line = sr.ReadLine();//1行ずつ取得
	var nextline = sr.ReadLine();//次の行を読み込む
	while (nextline != null)
    {
        if (line.Contains("#EXT-X-STREAM") && line.Contains("640x360") && (nextline.Contains("xyz")))
        {
            (nextlineを取り出す);
		} else {
			line = nextline;
			nextline = sr.ReadLine();//次の行を読み込む
		}
	}
}

書式はちょっと怪しく、動かしてもいませんが、参考として。

Comments

[kt_csharp]

なるほど、最初に２行分読んでおくのですね。
if文が成立した後もlineとnextlineの処理が必要なので、以下のように修正しました。
using(var sr = new StringReader(m3u))
{
var line = sr.ReadLine();//1行目を取得
var nextline = sr.ReadLine();//２行目を取得
while (nextline != null)
{
if (line.Contains("#EXT-X-STREAM") && line.Contains("640x360") && nextline.Contains("xyz"))
{
(nextlineを取り出す);
}
line = nextline;
nextline = sr.ReadLine();
}
}

[pepperleaf]

if()文が成立した次の while()での if()は必ず、失敗すると思えるのですが、成功する場合もあるでしょうか?

[kt_csharp]

失礼しました。質問では該当するものが一つしかないように書きましたが、実際は３つの条件（１行目が「#EXT-X-STREAM」と「640x360」、２行目が「xyz」）を満たすものが複数あります。そのため、if文は複数成功することになります。