Zmień wartości podczas iteracji

Załóżmy, że mam te typy:

type Attribute struct {
    Key, Val string
}
type Node struct {
    Attr []Attribute
}

i że chcę iterować na atrybutach mojego węzła, aby je zmienić.

Bardzo chciałbym móc:

for _, attr := range n.Attr {
    if attr.Key == "href" {
        attr.Val = "something"
    }
}

ale ponieważ attrnie jest wskaźnikiem, to nie zadziała i muszę zrobić:

for i, attr := range n.Attr {
    if attr.Key == "href" {
        n.Attr[i].Val = "something"
    }
}

Czy jest prostszy lub szybszy sposób? Czy można bezpośrednio uzyskać wskazówki range?

Oczywiście nie chcę zmieniać struktur tylko dla iteracji, a bardziej szczegółowe rozwiązania nie są rozwiązaniami.

Nie, żądany skrót nie jest możliwy.

Powodem tego jest to, że rangekopiuje wartości z wycinka, po którym iterujesz. Specyfikacja o zasięgu mówi:

Range expression                          1st value             2nd value (if 2nd variable is present)
array or slice  a   [n]E, *[n]E, or []E   index    i  int       a[i]       E

Tak więc zakres jest używany a[i]jako druga wartość dla tablic / plasterków, co skutecznie oznacza, że ​​wartość jest kopiowana, co sprawia, że ​​oryginalna wartość jest nietykalna.

To zachowanie jest przedstawione w następującym kodzie :

x := make([]int, 3)

x[0], x[1], x[2] = 1, 2, 3

for i, val := range x {
    println(&x[i], "vs.", &val)
}

Kod drukuje całkowicie różne lokalizacje pamięci dla wartości z zakresu i rzeczywistej wartości w wycinku:

0xf84000f010 vs. 0x7f095ed0bf68
0xf84000f014 vs. 0x7f095ed0bf68
0xf84000f018 vs. 0x7f095ed0bf68

Więc jedyne, co możesz zrobić, to użyć wskaźników lub indeksu, co zostało już zaproponowane przez jnml i peterSO.

Wydaje się, że prosisz o coś równoważnego z tym:

package main

import "fmt"

type Attribute struct {
    Key, Val string
}
type Node struct {
    Attr []Attribute
}

func main() {

    n := Node{
        []Attribute{
            {"key", "value"},
            {"href", "http://www.google.com"},
        },
    }
    fmt.Println(n)

    for i := 0; i < len(n.Attr); i++ {
        attr := &n.Attr[i]
        if attr.Key == "href" {
            attr.Val = "something"
        }
    }

    fmt.Println(n)
}

Wynik:

{[{key value} {href http://www.google.com}]}
{[{key value} {href something}]}

Pozwala to uniknąć tworzenia - być może dużej - kopii Attributewartości typu kosztem sprawdzania granic wycinków. W twoim przykładzie typ Attributejest stosunkowo mały, stringodwołania do dwóch wycinków: 2 * 3 * 8 = 48 bajtów na komputerze o architekturze 64-bitowej.

Możesz też po prostu napisać:

for i := 0; i < len(n.Attr); i++ {
    if n.Attr[i].Key == "href" {
        n.Attr[i].Val = "something"
    }
}

Ale sposobem na uzyskanie równoważnego wyniku z rangeklauzulą, która tworzy kopię, ale minimalizuje sprawdzanie granic wycinków, jest:

for i, attr := range n.Attr {
    if attr.Key == "href" {
        n.Attr[i].Val = "something"
    }
}

Dostosowałbym twoją ostatnią sugestię i użył tylko indeksowanej wersji zakresu.

for i := range n.Attr {
    if n.Attr[i].Key == "href" {
        n.Attr[i].Val = "something"
    }
}

Wydaje mi się, że prostsze jest odwoływanie się n.Attr[i]bezpośrednio zarówno do wiersza, który testuje, jak Keyi do wiersza, który ustawia Val, zamiast używać attrdla jednego i n.Attr[i]drugiego.

Na przykład:

package main

import "fmt"

type Attribute struct {
        Key, Val string
}

type Node struct {
        Attr []*Attribute
}

func main() {
        n := Node{[]*Attribute{
                &Attribute{"foo", ""},
                &Attribute{"href", ""},
                &Attribute{"bar", ""},
        }}

        for _, attr := range n.Attr {
                if attr.Key == "href" {
                        attr.Val = "something"
                }
        }

        for _, v := range n.Attr {
                fmt.Printf("%#v\n", *v)
        }
}

Plac zabaw

Wynik

main.Attribute{Key:"foo", Val:""}
main.Attribute{Key:"href", Val:"something"}
main.Attribute{Key:"bar", Val:""}

Alternatywne podejście:

package main

import "fmt"

type Attribute struct {
        Key, Val string
}

type Node struct {
        Attr []Attribute
}

func main() {
        n := Node{[]Attribute{
            {"foo", ""},
            {"href", ""},
            {"bar", ""},
        }}

        for i := range n.Attr {
                attr := &n.Attr[i]
                if attr.Key == "href" {
                        attr.Val = "something"
                }
        }

        for _, v := range n.Attr {
                fmt.Printf("%#v\n", v)
        }
}

Plac zabaw

Wynik:

main.Attribute{Key:"foo", Val:""}
main.Attribute{Key:"href", Val:"something"}
main.Attribute{Key:"bar", Val:""}