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

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

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

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

コーディング規約

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

解決済

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

Romira
Romira

総合スコア47

コードレビュー

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

コーディング規約

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

2回答

0評価

0クリップ

387閲覧

投稿2021/01/07 17:19

コード

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

C

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(){ 上記と同じような処理 }

C

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

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

良い質問の評価を上げる

以下のような質問は評価を上げましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

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

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

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

teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

  • プログラミングに関係のない質問
  • やってほしいことだけを記載した丸投げの質問
  • 問題・課題が含まれていない質問
  • 意図的に内容が抹消された質問
  • 過去に投稿した質問と同じ内容の質問
  • 広告と受け取られるような投稿

評価を下げると、トップページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

hidezzz

2021/01/07 17:53

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

まだ回答がついていません

会員登録して回答してみよう

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

ただいまの回答率
87.20%

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

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

質問する

関連した質問

同じタグがついた質問を見る

コードレビュー

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

コーディング規約

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