Linee guida - Copia delle slice e delle mappe ai margini

Ricezione di Slices e Maps

Ricorda che quando una mappa o una slice viene passata come parametro di funzione, se ne si memorizza un riferimento, gli utenti possono modificarle.

Approccio non raccomandato:

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

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

// Stai cercando di modificare d1.trips?
trips[0] = ...

Approccio raccomandato:

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

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

// Qui modifichiamo trips[0], ma non influenzerà d1.trips
trips[0] = ...

Restituzione di Slices o Maps

Allo stesso modo, fai attenzione alle modifiche di una mappa o di una slice che espongono lo stato interno.

Approccio non raccomandato:

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

// Snapshot restituisce lo stato attuale
func (s *Stats) Snapshot() map[string]int {
  s.mu.Lock()
  defer s.mu.Unlock()

  return s.counters
}

// snapshot non è più protetto dal mutex
// quindi qualsiasi accesso a snapshot sarà soggetto a una race condition e influenzerà counters di stats
snapshot := stats.Snapshot()

Approccio raccomandato:

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 è ora una copia
snapshot := stats.Snapshot()