ファイルを2回読むのをやめようとしています

Question

最大35MBまでのテキストファイルを解析しています。

私は現在、このコードしている：

const string show_BLOCK_DELIMITER = "show "; 
const string Show_BLOCK_DELIMITER = "Show "; 
const string Begin_SHOW_HW_DELIMITER = @"show hardware internal errors"; 

とキャプチャするかを制御このロジック：

if ((line == null) || (trimmed.StartsWith(show_BLOCK_DELIMITER)) 
    || (trimmed.StartsWith(Show_BLOCK_DELIMITER)) 
|| (trimmed.Contains(Begin_SHOW_HW_DELIMITER))) 
    { 
    //stuff happens here 
    } 

私は捕獲方法は、ファイルを読み込まれ、Iで始まる行を打ったとき'ショー'または 'ショー'それは私がキャプチャしたいものかどうかを確認するためにチェックします。もしそうなら、私は 'show'または 'show'で始まる次の行までキャプチャを開始します。新しいコードは、定数 'Begin_SHOW_HW_DELIMTER'の間のものをキャプチャすることになっていますが、ここでは入力のサンプルです。 "Module-N＃"で始まる2つのブロック、つまり文字列があります.Fc-macの行は「show」で始まりますが、コードをキャプチャしたいもののリストにはないのでそれが新しいブロックだと思って、彼らは捕らえられません。この時点で、私はこの新しい部分を扱い、ファイルを2回読むだけの別の方法について疑問に思っています。 if/elseステートメントはすでにかなり複雑です。

モジュール-1＃表示ハードウェアの内部エラー

show hardware internal fc-mac port 1 interrupt-counts

show hardware internal fc-mac port 2 interrupt-counts

モジュール-2＃ショーのハードウェアの内部エラー

show hardware internal fc-mac port 1 interrupt-counts

show hardware internal fc-mac port 2 interrupt-counts

Answer 1

パフォーマンス上の問題がある場合、条件文はディスク読み込みよりずっと安価であるため、ファイルを2度読むと最終的にパフォーマンスが低下します。

複雑な条件文では、保守性が低下し、バグの可能性が生じます。

あなたができることは、ファイルをメモリに読み込むことができます（すべての行を配列にロードするだけです）。ファイルを2回読むのにかかる時間は短くなりますが、メモリを増やすことになります。