Guía - Copiar Secciones y Mapas en los Límites

Recepción de slices y maps

Recuerde que cuando se pasa un mapa o un slice como parámetro de una función, si se almacena una referencia a ellos, los usuarios pueden modificarlos.

Enfoque no recomendado:

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

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

// ¿Está intentando modificar d1.trips?
trips[0] = ...

Enfoque recomendado:

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

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

// Aquí modificamos trips[0], pero no afectará a d1.trips
trips[0] = ...

Devolución de slices o maps

Del mismo modo, tenga en cuenta que los usuarios pueden modificar un mapa o un slice que expone un estado interno.

Enfoque no recomendado:

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

// Snapshot devuelve el estado actual
func (s *Stats) Snapshot() map[string]int {
  s.mu.Lock()
  defer s.mu.Unlock()

  return s.counters
}

// snapshot ya no está protegido por el mutex
// por lo que cualquier acceso a snapshot estará sujeto a una condición de carrera de datos y afectará a stats.counters
snapshot := stats.Snapshot()

Enfoque recomendado:

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