参照型引数はワンライナーで処理できない？

Question

using (FileStream sr= new FileStream(
        srcName, FileMode.Open, FileAccess.Read))
  {
       //バイト配列"buffer"にdatファイルを読込んでセットします
     sr.Read(buffer,0,buffer.length);
     //値がセットされたバイト配列"buffer"にLINQのメソッドチェーンを付加して色々と処理していきます。
     buffer.skip(10).～メソッドチェーンが続く

    }

ソースの最後2行をワンライナーで書く手段てあったりするのでしょうか。
リーダブルコード本に怒られそうな質問内容ですが、なんとなく気になりまして。

Comments

[iwamoto_takaaki]

それ、C#じゃない。

[Zuishin]

C# でも LINQ でもありませんね。タグの見直しをお願いします。おそらく VB.NET だと思います。

[Zuishin]

いえ、すみません。LINQ は合っていました。最終行ですね。でも VB って {} でしたっけ？

Answer 1

とりあえず C# で書いてみました。

using (var fs = new FileStream(@"hoge.dat", FileMode.Open, FileAccess.Read))
{
	var data = new BinaryReader(fs)
		.ReadBytes((int)fs.Length)
		.Skip(10)
		.ToArray();
}

fs.Length が long なのに対し、ReadBytes() は int なのでキャストしています。ファイルのサイズが int.MaxValue を超えるとうまく動きません。

次のように書いた方が簡単です。

File.ReadAllBytes("hoge.dat").Skip(10).ToArray();

Comments

[nyahonyaho]

ネットからコピペが故のVBフェイント、失礼しました

Answer 2

LINQのメソッドチェーンで書けないか？ということだと思います。LINQを利用するためにはメソッドチェーン初段がLINQをサポートしているなんらかの型（列的なもの）が前提ですので

FileStream.Read(buffer,0,buffer.length)

戻り値が読み込めたバイト数で副作用が主目的であるReadのようなメソッドでは後段につなげることができません。

他の言語やライブラリーではI/Oもパイプラインとして処理しやすいように、副作用によるインターフェースではなく読み込んだデータそのものをbyteの列、あるいは行単位のstringの列として返すようなインターフェースがよく用いられます。FileStreamはそういうものを備えていませんが、そういう使い方をしたい場合はextensionメソッドを自分で定義しLINQのメソッドチェーンの初段となり得るようにすれば利用できるようになります。

public static class MyCSharpExtension {
  // extension method の定義例
  public static IEnumerable<byte> ReadBytes(this FileStream fs) {
    byte[] buffer = new byte[4096];
    for (;;) {
      int len = fs.Read(buffer, 0, buffer.Length);
      if (len == 0)
        break;
      for (int i = 0; i < len; i++)
        yield return buffer[i];
    }
  }
}
...
// 利用例
using (fs = new FileStream(...)) {
  fs.ReadBytes().Where(b => b != ' ').Selet( ... );
}

Comments

[KSwordOfHaste]

あはは。Zuishinさん回答にあるFile.ReadAllBytesがあるから本件は自分で定義するまでもないですね。
経験少ないとすでにあるメソッドを知らずに自分で書いてしまったりと・・・悲しい空回りをしてしまいがちですｗ；

[Zuishin]

ReadByte() は遅いのが難点です。多分内部でバッファリングしているはずなんですが。
でも File.ReadAllBytes() は一度全部読み込んでしまうので、メモリ効率の観点からはこの回答の方が上ですね。使い分けられると思います。

[KSwordOfHaste]

なるほどFileStreamはバッファリングしてくれてないのですね。せめて自分の例はReadを使ってバッファーへ読み込むようなものにすべきということでしょうね。コメントありがとうございました。
---
Zuishinさんコメントを参考に例をバッファリングするようなものに変えておきました。File.ReadAllBytesがあるのでどうでもよいことではありますが。