Pedoman - Menyalin Irisan dan Peta di Ujung

Menerima Slices dan Map

Ingatlah bahwa ketika sebuah peta atau serpihan dilewatkan sebagai parameter fungsi, jika Anda menyimpan referensi pada mereka, pengguna dapat memodifikasinya.

Pendekatan tidak disarankan:

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

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

// Apakah Anda mencoba memodifikasi d1.trips?
trips[0] = ...

Pendekatan yang disarankan:

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

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

// Di sini kita memodifikasi trips[0], tetapi itu tidak akan memengaruhi d1.trips
trips[0] = ...

Mengembalikan Slices atau Map

Demikian pula, berhati-hatilah terhadap pengguna yang memodifikasi peta atau slice yang mengekspos status internal.

Pendekatan tidak disarankan:

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

// Snapshot mengembalikan status saat ini
func (s *Stats) Snapshot() map[string]int {
  s.mu.Lock()
  defer s.mu.Unlock()

  return s.counters
}

// snapshot tidak lagi dilindungi oleh mutex
// jadi akses apa pun ke snapshot akan menjadi subjek perlombaan data dan memengaruhi stats.counters
snapshot := stats.Snapshot()

Pendekatan yang disarankan:

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 sekarang merupakan salinan
snapshot := stats.Snapshot()