質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.93%

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

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,104

nyahonyaho

score 81

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

    }


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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • iwamoto_takaaki

    2017/04/11 09:24

    それ、C#じゃない。

    キャンセル

  • Zuishin

    2017/04/11 09:30

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

    キャンセル

  • Zuishin

    2017/04/11 09:31 編集

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

    キャンセル

回答 2

checkベストアンサー

+3

とりあえず 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();

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/04/11 10:02

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

    キャンセル

+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( ... );
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/04/11 10:23

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

    キャンセル

  • 2017/04/11 10:34

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

    キャンセル

  • 2017/04/11 10:56 編集

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.93%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る