Data race & detektornya

Data race adalah dua goroutine yang menyentuh memori yang sama tanpa penyelarasan. Ia tidak selalu membuat program crash; sering ia cuma menghasilkan angka yang salah, sesekali.

Seperti apa bentuknya

type Counter struct{ n int }

func (c *Counter) Naik() { c.n++ }   // BUKAN operasi tunggal:
                                      // baca n, tambah 1, tulis n

func main() {
	c := &Counter{}
	var wg sync.WaitGroup
	for range 1000 {
		wg.Add(1)
		go func() { defer wg.Done(); c.Naik() }()
	}
	wg.Wait()
	fmt.Println(c.n)   // 987. atau 1000. atau 994. berbeda tiap kali.
}

go run -race main.go

==================
WARNING: DATA RACE
Read at 0x00c0000140a0 by goroutine 8:
  main.(*Counter).Naik()
      /app/main.go:6 +0x2c

Previous write at 0x00c0000140a0 by goroutine 7:
  main.(*Counter).Naik()
      /app/main.go:6 +0x44
==================

Laporannya memberi tiga hal: alamat memori yang diperebutkan, dan dua stack trace — siapa membaca dan siapa menulis. Itu biasanya cukup untuk langsung menemukan penyebabnya.

Cara memakainya

go test -race ./...        # ← ini yang dipasang di CI
go run -race ./cmd/server  # saat mengembangkan
go build -race -o server   # binari untuk staging

Race yang khas di aplikasi web

1. Map bersama

// ❌ dua permintaan bersamaan = crash "concurrent map writes"
var cache = map[string]Produk{}

func handler(w http.ResponseWriter, r *http.Request) {
	cache[r.URL.Path] = ambilProduk()   // race
}

2. Field yang ditulis goroutine latar

// ❌ konfigurasi yang di-refresh berkala sambil dibaca handler
type Config struct{ Batas int }

var cfg Config

func muatUlangBerkala() {
	for range time.Tick(time.Minute) {
		cfg = ambilConfig()   // race dengan setiap pembaca
	}
}

// ✅ pointer atomik: pembaca tanpa lock, penulis mengganti utuh
var cfg atomic.Pointer[Config]

func muatUlangBerkala() {
	for range time.Tick(time.Minute) {
		c := ambilConfig()
		cfg.Store(&c)
	}
}

func batas() int { return cfg.Load().Batas }

3. Menulis ke ResponseWriter dari dua goroutine

// ❌ http.ResponseWriter tidak aman dipakai bersamaan
go func() { fmt.Fprintln(w, "a") }()
fmt.Fprintln(w, "b")

Semua penulisan respons harus terjadi di goroutine handler. Kalau pekerjaan dibagi, kumpulkan hasilnya lewat channel lalu tulis di satu tempat.

4. time.Time dan slice yang "dibagikan"

Method mengembalikan slice milik struct, pemanggil mengubahnya dari goroutine lain. Ini kembali ke aturan Fase 0: slices.Clone saat mengembalikan slice internal dari tipe yang dipakai bersama.

Tes yang benar-benar menemukan race

func TestCacheParalel(t *testing.T) {
	c := NewCache()
	var wg sync.WaitGroup

	for i := range 100 {
		wg.Add(2)
		go func() { defer wg.Done(); c.Simpan(fmt.Sprint(i), Produk{}) }()
		go func() { defer wg.Done(); c.Ambil(fmt.Sprint(i)) }()
	}
	wg.Wait()
}

Setelah menemukan race

1. Jangan tambal dengan time.Sleep. Itu mengubah peluang, bukan kebenaran.
2. Tanyakan apakah keadaannya perlu dibagi. Sering jawabannya tidak — beri tiap goroutine salinannya.
3. Kalau perlu dibagi: mutex untuk struktur data, atomic untuk penghitung, channel untuk penyerahan kepemilikan.
4. Tulis tes paralel yang gagal dengan -race sebelum memperbaiki, supaya kamu tahu perbaikannya berhasil.