質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

コーディング規約

コーディング規約とは、コードの書き方についての決め事のことです。 文法のことではなく、そのチームなどの中の約束事としてどのような書き方で行うかを定めるもの。 項目の例として、関数や変数の命名規則、コーディングのスタイル、括弧やインデントの書き方などが挙げられます。

Q&A

解決済

2回答

1020閲覧

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

Romira

総合スコア47

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

コーディング規約

コーディング規約とは、コードの書き方についての決め事のことです。 文法のことではなく、そのチームなどの中の約束事としてどのような書き方で行うかを定めるもの。 項目の例として、関数や変数の命名規則、コーディングのスタイル、括弧やインデントの書き方などが挙げられます。

0グッド

0クリップ

投稿2021/01/07 17:19

コード

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

C

1void update_min(){ 2 auto min = list[0]; 3 4 for (int i = 0; i < list.size(); i++) 5 { 6 if (min > list[i]){ 7 min = list[i]; 8 } 9 } 10} 11 12void update_max(){ 13 上記と同じような処理 14}

C

1void update_min_max(){ 2 auto min = list[0]; 3 auto max = list[0]; 4 5 for (int i = 0; i < list.size(); i++) 6 { 7 if (min > list[i]){ 8 min = list[i]; 9 } 10 11 if (max < list[i]){ 12 max = list[i]; 13 } 14 } 15}

質問

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

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

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

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

hidezzz

2021/01/07 17:53

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

回答2

0

ベストアンサー

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

投稿2021/01/07 17:45

A_kirisaki

総合スコア2853

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Romira

2021/01/07 17:53

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

0

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

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

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

投稿2021/01/07 18:01

hidezzz

総合スコア1248

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問