パスワードを忘れた? アカウント作成
14057374 journal
日記

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

日記 by delta-keeper

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

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

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

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

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

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

この議論は、delta-keeper (31927)によって「 トモとそのトモ専用」として作成されている。 誰かひとりと仲良くなればコメントさせてもらえるかもね。
typodupeerror

クラックを法規制強化で止められると思ってる奴は頭がおかしい -- あるアレゲ人

読み込み中...