type User struct {
  ID    string
  Name  string
  Email string
}

// 読み取り専用処理（値渡し）
func DisplayName(u User) string {
  return u.Name
}

// 状態更新処理（ポインタ渡し）
func UpdateEmail(u *User, email string) {
  u.Email = email
}

func Process(u *User) {
  fmt.Println(u.Name)
}
@Reviewer
処理内容は読み取りのみで更新を伴わないのにポインタを受け取っています。値渡しへ変更し、API設計の副作用明示性を高めてください。

type LargeData struct {
  ID    string
  Name  string
  Email string
  Tags  []string
  Meta  map[string]string
}

func Send(ld LargeData) {
  // 実質毎回大型コピーが発生
}
@Reviewer
LargeDataはポインタ型メンバを含む複合構造です。頻繁に値渡しするとメモリ負荷が高まるため、ポインタ渡しへ変更しコピーコストを削減してください。

func GetConfig() *Config {
  return &Config{
    Retry: 3,
    Debug: true,
  }
}
@Reviewer
返却されたConfigが呼び出し元で更新可能構造か否かが設計上不明確です。読み取り専用用途であれば値返却または不変設計を検討してください。

func (u User) DisplayName() string {
  return u.Name
}

func (u *User) UpdateEmail(email string) {
  u.Email = email
}
@Reviewer
メソッドで値レシーバー・ポインタレシーバーが混在しています。状態変更を含む型なら全体をポインタレシーバーへ統一し一貫性を保ってください。

func Update(u **User) {
  (*u).Name = "Updated"
}
@Reviewer
二重ポインタを使用する設計理由がありません。通常は*Userまでで十分です。API設計の過剰抽象は避けてください。

func Modify(u *User) {
  u.Name = "Changed"
}

func main() {
  u := User{Name: "Alice"}
  Modify(&u)
  fmt.Println(u.Name)
}
@Reviewer
呼び出し側で副作用が発生することが明文化されていません。副作用設計意図をAPIシグネチャでも表現してください。

func GetUser() *User {
  return &User{Name: "Bob"}
}
@Reviewer
呼び出し元がこのUserを更新可能か否か設計意図が不明確です。読み取り専用であれば構造体値返却も検討してください。