同じような処理の流れを異なる関数として提供すべきかどうか

Question

コード

例えば以下のようなlistの最大値と最小値を更新する処理があったとき

void update_min(){
    auto min = list[0];

    for (int i = 0; i < list.size(); i++)
    {
        if (min > list[i]){
            min = list[i];
        }
    }
}

void update_max(){
    上記と同じような処理
}

と

void update_min_max(){
    auto min = list[0];
    auto max = list[0];

    for (int i = 0; i < list.size(); i++)
    {
        if (min > list[i]){
            min = list[i];
        }

        if (max < list[i]){
            max = list[i];
        }
    }
}

質問

上記の2つのコードではどちらがベストな記述でしょうか。

今回は極端な例でfor文１つであまり変わらなさそうだと思いますが、データ量やfor文のネストが深くなった場合、前者はアクセス回数が増えたり記述量が増えたり無駄が多いなと思いました。しかし見直した時のわかりやすいとも思いました。

どのような場合でどちらの記述が、もしくはこれ以外の最適な記述等、ご教授ください。

Comments

[hidezzz]

質問に例として挙げられている関数に引数が指定されていませんが、実際に引数はなくてlistはグローバル変数ということでしょうか？それとも引数を省略して書かれていて実際には引数でlistを渡しているということでしょうか？
どちらであるかによって判断基準は変わってくると思います。

Answer 1

前者ではじめてパフォーマンスに問題があるなら後者にすれば良いと思います。分かれているものを結合するのは簡単ですが、融合したものを切り離すのは容易ではありません。また、開発初期における過度な最適化はアンチパターンとしても知られています。パフォーマンスチューニングは問題になることが明らかである場合のみ施せばよいのであってそれ以外は可読性、保守性が優先されるべきです。

Comments

[Romira]

迅速な回答ありがとうございます。
可読性を重視して記述したいと思います

Answer 2

(質問へのコメントの重複になりますが…)
質問に例として挙げられている関数に引数が指定されていませんが、実際に引数はなくてlistはグローバル変数ということでしょうか？それとも引数を省略して書かれていて実際には引数でlistを渡しているということでしょうか？
どちらであるかによって判断基準が変わってくると思います。

グローバル変数ということであれば、限られた形で関数が呼ばれることになると推測されるので後者の最大と最小をまとめて書くやり方のほうが良い気がします。

listを引数で渡すのなら、関数の使われ方によってケースバイケースになるのではないでしょうか？
最大値取得処理、最小値取得処理を必要に応じて個別に呼ぶことができるように前者のような関数を独立させた形のほうが関数としての使い勝手が良いケースも考えられると思います。最大値最小値はかならずまとめて取得するようなことが想定されるのなら後者のほうが良いケースもあると思います。