Получение срезов и карт

Помните, что при передаче карты или среза в качестве параметра функции, если вы сохраняете ссылку на них, пользователи могут их изменять.

Нерекомендуемый подход:

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

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

// Вы пытаетесь изменить d1.trips?
trips[0] = ...

Рекомендуемый подход:

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

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

// Здесь мы изменяем trips[0], но это не повлияет на d1.trips
trips[0] = ...

Возвращение срезов или карт

Точно так же будьте осторожны с тем, чтобы пользователи не изменяли карту или срез, который раскрывает внутреннее состояние.

Нерекомендуемый подход:

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

// Snapshot возвращает текущее состояние
func (s *Stats) Snapshot() map[string]int {
  s.mu.Lock()
  defer s.mu.Unlock()

  return s.counters
}

// snapshot больше не защищен мьютексом
// поэтому любой доступ к снимку будет подвержен состязанию данных и повлияет на stats.counters
snapshot := stats.Snapshot()

Рекомендуемый подход:

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 := stats.Snapshot()