Go 里面的 map 数据初始化应该怎么做,这代码太丑陋了 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
The Go Programming Language
http://golang.org/
Go Playground
Go Projects
Revel Web Framework
RedisMasterNode
V2EX    Go 编程语言
<1>Go 里面的 map 数据初始化应该怎么做,这代码太丑陋了
  •  
  •   RedisMasterNode 2024-06-05 17:18:06 +08:00 3538 次点击
    这是一个创建于 559 天前的主题,其中的信息可能已经有所发展或是发生改变。

    有没有大佬写个简单一点的 helper 帮忙处理一下这代码

    21 条回复    2024-06-20 13:21:06 +08:00
    kneo
        1
    kneo  
       2024-06-05 17:24:08 +08:00 via Android
    看不见你发的啥玩意……
    povsister
        2
    povsister  
       2024-06-05 17:26:03 +08:00
    自己搓个递归 reflect 初始化。
    另外,go 读 nil map 并不会 panic ,你代码的 pattern 是先读,判不存在然后初始化。
    读中混写,隐藏两个问题
    1. 并发读 map 带来写 race
    2. 不存在的 key 造成重新初始化

    总结,代码设计太烂
    monkeyWie
        3
    monkeyWie  
       2024-06-05 17:30:34 +08:00
    看起来像是偷懒不想声明 struct ,然后用 map 导致写起来更麻烦了
    billbur
        4
    billbur  
       2024-06-05 17:31:34 +08:00
    这什么鬼这么长,索引路径可以用一个字符串来做吧,key 就用"a.b.c.e.f"
    RedisMasterNode
        5
    RedisMasterNode  
    OP
       2024-06-05 17:32:42 +08:00
    PS: 是朋友给的网图,但是还是觉得很有意思,并且也是 Go 里面偶尔会碰到的情况(当有人偷懒不定义结构体时就会这样)
    dyllen
        6
    dyllen  
       2024-06-05 17:32:51 +08:00
    确实挺丑的,看花眼。嵌套的 map 可以重新定义个类型,取个名字,好分辨。
    Vegetable
        7
    Vegetable  
       2024-06-05 17:51:21 +08:00   3
    这种写法,无论什么语言的 map 写出来都是依托狗屎啊
    fruitmonster
        8
    fruitmonster  
       2024-06-05 17:55:36 +08:00
    之前写代码发现一个好用的包: https://github.com/tidwall/gjson
    ns09005264
        9
    ns09005264  
       2024-06-05 19:00:28 +08:00
    就算是都用 map ,至少搞个别名 type 吧。
    RedisMasterNode
        10
    RedisMasterNode  
    OP
       2024-06-05 19:03:42 +08:00
    @Vegetable python 会略好一点哈哈哈,但是当然本质上是人的问题不是
    gitrebase
        11
    gitrebase  
       2024-06-05 19:07:15 +08:00
    不是,什么语言的 map 初始化能契合这个鬼需求?
    jfcai
        12
    jfcai  
       2024-06-05 19:44:29 +08:00
    怎么看着像是段子。
    CEBBCAT
        13
    CEBBCAT  
       2024-06-05 19:55:59 +08:00
    分析 1:楼主少贴了一部份代码,整体代码猜测是遍历 []Struct ,构造不同 Struct 分类下的 slice ,内容为 Struct 某个属性。

    分析 2:因为 map[key] = append(map[key], value) 之前要保证 map 的存在,而截图中 map 其实是其他 map 的 value ,zero value 是 nil ,直接写操作会 panic

    解:使用 struct 作为 key 。不要使用 string ,维护起来简直就是噩梦,凭多层 map 还能知道每一层都用的是什么做 key ,写入的时候少一层会因为类型不对而报错,要是用 string 做 key ,那类型系统一点儿都帮不了你。

    注:strcut 字段类型灵活运用嵌套、指针、空指针等特性,value 如果哪天要用结构体,最好统一 new 方法(以防哪天想往 value 里面加个 map )
    CEBBCAT
        14
    CEBBCAT  
       2024-06-05 19:57:35 +08:00
    @billbur #4 衷心劝告楼上老哥三思用 fmt.Sprintf() 做 map 的 key
    8355
        15
    8355  
       2024-06-05 20:15:55 +08:00
    代码提交量挺大啊。。
    leonshaw
        16
    leonshaw  
       2024-06-05 20:34:29 +08:00   1
    func f[K1, K2 comparable, V any, M ~map[K2]V](m map[K1]M, k K1) M {
    t, ok := m[k]
    if !ok {
    t = make(M)
    m[k] = t
    }
    return t
    }
    Trim21
        17
    Trim21  
       2024-06-05 21:14:48 +08:00
    有泛型了写个 setDefault 就行了
    james122333
        18
    james122333  
       2024-06-05 22:44:33 +08:00 via Android
    还是写 php 爽
    whimsySun
        19
    whimsySun  
       2024-06-06 08:44:21 +08:00
    看到这样的代码不删了,还想着怎么初始化呢
    lasuar
        20
    lasuar  
       2024-06-06 16:36:50 +08:00
    谁写的,裁了
    CEBBCAT
        21
    CEBBCAT  
       2024-06-20 13:21:06 +08:00
    要知道是网图没有后续就不费那心思写回复了,完全纸上谈兵啊
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     1015 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 27ms UTC 18:36 PVG 02:36 LAX 10:36 JFK 13:36
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86