Wytyczna - Kopiowanie fragmentów i map na krawędziach

Odbieranie cięć i map

Pamiętaj, że gdy mapa lub cięcie jest przekazywane jako parametr funkcji, jeśli przechowujesz do nich odwołanie, użytkownicy mogą je modyfikować.

Nie zalecane podejście:

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

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

// Czy próbujesz modyfikować d1.trips?
trips[0] = ...

Zalecane podejście:

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

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

// Tutaj modyfikujemy trips[0], ale nie wpłynie to na d1.trips
trips[0] = ...

Zwracanie cięć lub map

Podobnie, należy uważać na modyfikowanie mapy lub cięcia, które ujawnia wewnętrzny stan.

Nie zalecane podejście:

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

// Snapshot zwraca bieżący stan
func (s *Stats) Snapshot() map[string]int {
  s.mu.Lock()
  defer s.mu.Unlock()

  return s.counters
}

// snapshot nie jest już chroniony przez muteks
// więc każdy dostęp do snapshot będzie podlegał wyścigowi danych i wpłynie na stats.counters
snapshot := stats.Snapshot()

Zalecane podejście:

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
}

// snapshot teraz jest kopią
snapshot := stats.Snapshot()