有没有大佬写个简单一点的 helper
帮忙处理一下这代码
1
kneo 2024-06-05 17:24:08 +08:00 via Android
看不见你发的啥玩意……
|
![]() |
2
povsister 2024-06-05 17:26:03 +08:00
自己搓个递归 reflect 初始化。
另外,go 读 nil map 并不会 panic ,你代码的 pattern 是先读,判不存在然后初始化。 读中混写,隐藏两个问题 1. 并发读 map 带来写 race 2. 不存在的 key 造成重新初始化 总结,代码设计太烂 |
![]() |
3
monkeyWie 2024-06-05 17:30:34 +08:00
看起来像是偷懒不想声明 struct ,然后用 map 导致写起来更麻烦了
|
4
billbur 2024-06-05 17:31:34 +08:00
这什么鬼这么长,索引路径可以用一个字符串来做吧,key 就用"a.b.c.e.f"
|
![]() |
5
RedisMasterNode OP PS: 是朋友给的网图,但是还是觉得很有意思,并且也是 Go 里面偶尔会碰到的情况(当有人偷懒不定义结构体时就会这样)
|
6
dyllen 2024-06-05 17:32:51 +08:00
确实挺丑的,看花眼。嵌套的 map 可以重新定义个类型,取个名字,好分辨。
|
![]() |
7
Vegetable 2024-06-05 17:51:21 +08:00 ![]() 这种写法,无论什么语言的 map 写出来都是依托狗屎啊
|
8
fruitmonster 2024-06-05 17:55:36 +08:00
之前写代码发现一个好用的包: https://github.com/tidwall/gjson
|
![]() |
9
ns09005264 2024-06-05 19:00:28 +08:00
就算是都用 map ,至少搞个别名 type 吧。
|
![]() |
10
RedisMasterNode OP @Vegetable python 会略好一点哈哈哈,但是当然本质上是人的问题不是
|
![]() |
11
gitrebase 2024-06-05 19:07:15 +08:00
不是,什么语言的 map 初始化能契合这个鬼需求?
|
12
jfcai 2024-06-05 19:44:29 +08:00
怎么看着像是段子。
|
![]() |
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 ) |
![]() |
15
8355 2024-06-05 20:15:55 +08:00
代码提交量挺大啊。。
|
16
leonshaw 2024-06-05 20:34:29 +08:00 ![]() 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 } |
![]() |
17
Trim21 2024-06-05 21:14:48 +08:00
有泛型了写个 setDefault 就行了
|
18
james122333 2024-06-05 22:44:33 +08:00 via Android
还是写 php 爽
|
![]() |
19
whimsySun 2024-06-06 08:44:21 +08:00
看到这样的代码不删了,还想着怎么初始化呢
|
![]() |
20
lasuar 2024-06-06 16:36:50 +08:00
谁写的,裁了
|
![]() |
21
CEBBCAT 352 天前
😓要知道是网图➕没有后续就不费那心思写回复了,完全纸上谈兵啊
|