Go: Wie entferne ich ein Element aus einer Scheibe und ändern Sie es in Erinnerung

Question

Ich versuche zu Lernzwecken ein einfaches Server/Client-Chat-Programm zu schreiben, aber ich bin stecken. Ich möchte die Leave Funktion entfernen Sie den Zeiger, der es passiert, und aktualisieren Sie die Scheibe in der Struktur, so dass der Zeiger nicht mehr da ist. Aber es funktioniert nicht.

Beispiel: Input, Output

type Room struct { 
    Name  string 
    Visitors []*net.Conn 
} 


func (r *Room) Leave(pc *net.Conn) { 
    for i, pv := range r.Visitors { 
     //found the connection we want to remove 
     if pc == pv { 
      fmt.Printf("Before %v\n",r.Visitors) 
      r.Visitors = append(r.Visitors[:i], r.Visitors[i+1:]...) 
      fmt.Printf("Before %v\n",r.Visitors)     
      return 
     } 
    } 
} 

Answer 1

Sie verwenden einen Zeiger auf eine Schnittstelle (Visitors []*net.Conn) in Ihrem Room Typ. Sie sollten niemals einen Zeiger auf einen Schnittstellenwert benötigen. Eine Schnittstelle ist value ist wie ein generischer Content-Holder, und ihre Darstellung im Speicher ist anders als eine Struktur, die diese Schnittstelle implementiert.

Sie sollten einfach den Visitors Typ verwenden deklarieren eine Schnittstelle zu sein:

type Room struct { 
    Name  string 
    Visitors []net.Conn // should not be a pointer to interface value 
} 


func (r *Room) Leave(pc net.Conn) { // same thing as above 
    for i, pv := range r.Visitors { 
     // found the connection we want to remove 
     // in the comparison below actual concrete types are being compared. 
     // both these concrete types must be comparable (they can't be slices for example 
     if pc == pv { 
      r.Visitors = append(r.Visitors[:i], r.Visitors[i+1:]...) 
      return 
     } 
    } 
} 

Beachten Sie, dass der Vergleich oben (pc == pv) nicht so trivial ist. Lesen Sie es hier: https://golang.org/ref/spec#Comparison_operators

Beziehen Sie sich auch auf diese Frage: Why can't I assign a *Struct to an *Interface?

Answer 2

Das Entfernen Logik korrekt ist (die Ausgabe auch dies bestätigt). Das Problem ist der Mangel an Synchronisation zwischen mehreren goroutines (Ihre Frage sagt nichts darüber).

Du hat gesagt (in Ihrem Kommentar), dass Sie diese zuerst mit 1 goroutine erhalten arbeiten wollen, und später mit Synchronisation befassen. Aber Ihr Code verwendet bereits mehrere goroutines, so können Sie diesen Luxus nicht haben kann:

//Let a goroutine Handle the connection 
go handleConnection(conn) 

Mehrere goroutines sind Lesen und Schreiben der Room.Visitors Scheibe, so haben Sie keine andere Wahl, als den Zugriff darauf zu synchronisieren.

wäre ein Beispiel sync.RWLock zu verwenden:

utils.go:

type Room struct { 
    Name  string 
    Visitors []net.Conn 
    mux  sync.RWLock 
} 


func (r *Room) Leave(c net.Conn) { 
    r.mux.Lock() 
    defer r.mux.Unlock() 
    for i, v := range r.Visitors { 
     //found the connection we want to remove 
     if c == v { 
      fmt.Printf("Before remove %v\n", r.Visitors) 
      r.Visitors = append(r.Visitors[:i], r.Visitors[i+1:]...) 
      fmt.Printf("After remove %v\n", r.Visitors) 
      return 
     } 
    } 
} 

, wenn auch andere Code Room.Visitors berührt, auch den Code sperren (Sie RWMutex.RLock() verwenden können, wenn Sie es nur gerade lesen) .

Ebenso müssen Sie den Zugriff auf alle Variablen synchronisieren, die von mehreren goroutines gelesen/geändert werden.

berücksichtigen auch das Element Nullung, die nach der Entfernung befreit ist, sonst die darunter liegende Array wird noch diesen Wert halten, den gargabe Kollektor verhinderte freie Speicher, um richtig durch sie verwendet. Weitere Informationen zu diesem Thema finden Does go garbage collect parts of slices?