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

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を渡しているということでしょうか？
どちらであるかによって判断基準は変わってくると思います。