Diretriz - Copiando Slices e Mapas nas Bordas

Recebendo Slices e Maps

Lembre-se de que, ao passar um mapa ou um slice como parâmetro de função, se você armazenar uma referência a eles, os usuários poderão modificá-los.

Abordagem não recomendada:

func (d *Driver) SetTrips(trips []Trip) {
  d.trips = trips
}

trips := ...
d1.SetTrips(trips)

// Você está tentando modificar d1.trips?
trips[0] = ...

Abordagem recomendada:

func (d *Driver) SetTrips(trips []Trip) {
  d.trips = make([]Trip, len(trips))
  copy(d.trips, trips)
}

trips := ...
d1.SetTrips(trips)

// Aqui modificamos trips[0], mas não afetará d1.trips
trips[0] = ...

Retornando Slices ou Maps

Da mesma forma, esteja atento aos usuários que modificam um mapa ou slice que expõe o estado interno.

Abordagem não recomendada:

type Stats struct {
  mu sync.Mutex
  counters map[string]int
}

// Snapshot retorna o estado atual
func (s *Stats) Snapshot() map[string]int {
  s.mu.Lock()
  defer s.mu.Unlock()

  return s.counters
}

// snapshot já não está protegido pelo mutex
// portanto, qualquer acesso a snapshot estará sujeito a corrida de dados e afetará stats.counters
snapshot := stats.Snapshot()

Abordagem recomendada:

type Stats struct {
  mu sync.Mutex
  counters map[string]int
}

func (s *Stats) Snapshot() map[string]int {
  s.mu.Lock()
  defer s.mu.Unlock()

  result := make(map[string]int, len(s.counters))
  for k, v := range s.counters {
    result[k] = v
  }
  return result
}

// agora snapshot é uma cópia
snapshot := stats.Snapshot()