delta-keeperの日記： モジュール結合度に関する考え方

最近受けるコードレビューの指摘の中で、イマイチ指摘自体に同意できないものがある。

レビューアの癖なのか知らないが、とにかく制御結合をさせたがる。
たとえば以下のような修正。

修正前:
int GetFileData_A(string& strData)
{
      // セクション1のキーAから値を取得
}

int GetFileData_B(string& strData)
{
      // セクション2のキーBから値を取得
}

修正後:
int GetFileData(string& strData, int iDataType)
{
      // セクションとキーをiDataTypeに基づいて設定
      // ファイル読み込み
}

これは簡単な例だけど、積極的にモジュール結合度上げるのってどうなのか?
メソッドで分離していればコードは理解しやすいが、引数で制御する関数はパラメータも知らないといけない。
コード量は減るかもしれないが見通しが悪くならないか?