V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
The Go Programming Language
http://golang.org/
Go Playground
Go Projects
Revel Web Framework
RedisMasterNode
V2EX  ›  Go 编程语言

Go 里面的 map 数据初始化应该怎么做,这代码太丑陋了

  •  
  •   RedisMasterNode · 2024-06-05 17:18:06 +08:00 · 3285 次点击
    这是一个创建于 367 天前的主题,其中的信息可能已经有所发展或是发生改变。

    有没有大佬写个简单一点的 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  
       352 天前
    😓要知道是网图➕没有后续就不费那心思写回复了,完全纸上谈兵啊
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2502 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 31ms · UTC 01:52 · PVG 09:52 · LAX 18:52 · JFK 21:52
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.