関心の分離と計算量のトレードオフ

Question

package main

import (
  "fmt"
)

type person struct {
  weight float32
  height float32
}

func GetAvarageBMI1(persons []*person) float32 {
  aveWeight := GetAvarageWeight(persons)
  aveHeight := GetAvarageHeight(persons)
  return aveWeight / (aveHeight * aveHeight)
}

func GetAvarageBMI2(persons []*person) float32 {
  var total float32
  var count float32
  for _, p := range persons {
    total += p.weight / (p.height * p.height)
    count++
  }
  return total / count
}

func GetAvarageWeight(persons []*person) float32 {
  var total float32
  var count float32
  for _, p := range persons {
    total += p.weight
    count++
  }

  return total / count
}

func GetAvarageHeight(persons []*person) float32 {
  var total float32
  var count float32
  for _, p := range persons {
    total += p.height
    count++
  }

  return total / count
}

func main() {
  persons := []*person{
    &person{60, 1.7},
    &person{60, 1.7},
    &person{70, 1.8},
    &person{70, 1.6},
  }

  fmt.Printf("平均BMI1：%v\n", GetAvarageBMI1(persons))
  fmt.Printf("平均BMI2：%v\n", GetAvarageBMI2(persons))
}

例として、このようなコードがあり、
GetAvarageBMI1が関心の分離、GetAvarageBMI2は計算量に特化していると仮定した場合。
どちらを優先すべきなのでしょうか？

Comments

[episteme]

BMIの平均 とは BMIの総和/サンプル数 ですよね?
であるなら、GetAvarageBMI1() は計算式そのものが間違っています。

[shiketa]

平均の平均は平均ではない。GetAvarageBMI1とGetAvarageBMI2の結果がそもそも異なっているのではありませんか。

[dondakehimnan]

確かに間違ってますね...
一応例なので、訂正はしませんが、どちらも同じ内容を計算しているという前提でお願いします！

[episteme]

> 同じ内容を計算しているという前提でお願いします！

だったら「より単純な方」を採用します(僕なら)。