Получение срезов и карт
Помните, что при передаче карты или среза в качестве параметра функции, если вы сохраняете ссылку на них, пользователи могут их изменять.
Нерекомендуемый подход:
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()