[Go] 这两段代码怎么选择喔？

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

The Go Programming Language

› http://golang.org/

› Go Playground

Go Projects

› Revel Web Framework

这是一个创建于 1644 天前的主题，其中的信息可能已经有所发展或是发生改变。


select {
case l.rConn <- c:
default:
}

if len(l.rConn) < cap(l.rConn) {
	l.rConn <- c
}

15 条回复 • 2021-08-05 11:20:56 +08:00

index90

2021 年 8 月 3 日

当然第一个啊
len(chan) 虽然线程安全的，但很少会这么用，原因是当你前一步执行了 len，下一步就不一定是那个数量了

CEBBCAT

2021 年 8 月 3 日

当然是第一种，第二种我都看不懂你要做什么。倒立吃饭的感觉

yeqown

2021 年 8 月 3 日

nonblocking send channel ？那肯定是第一种更通用，容易理解

SorcererXW

2021 年 8 月 3 日

第二非原子操作，是有问题的

lysS

2021 年 8 月 3 日

@index90 主要是考虑到性能问题
BenchmarkSelect-8 82773339 16.42 ns/op 0 B/op 0 allocs/op
BenchmarkLen-8 738067750 1.571 ns/op 0 B/op 0 allocs/op
BenchmarkLenLock-8 35197221 33.51 ns/op 0 B/op 0 allocs/op

而且 l.rConn 只有一处写入，可能有多处读取

lysS

2021 年 8 月 3 日

@CEBBCAT 就是非阻塞写入，管道没用容量的话会直接丢弃本次数据

keepeye

2021 年 8 月 3 日

第二种不是原子的，两个线程同时判断可能会有一个阻塞

index90

2021 年 8 月 3 日

@lysS 你是不是在 select 或者 if 外包了一层 for 啊？

lysS

2021 年 8 月 3 日

@index90 对的，在一个循环中，只有这一个地方在写入

sunshinev

2021 年 8 月 3 日

第二个一眼看不懂。。。第二眼也没看懂。。

lysS

2021 年 8 月 3 日

@sunshinev 这有啥的，就是避免写入阻塞哒

index90

2021 年 8 月 3 日

直接让它阻塞在 l.rConn <- c 就好啦，为啥还要 select default 或者 if 来跳过啊。
第二种写法很有可能就会阻塞在 l.rConn <- c 的。

lysS

2021 年 8 月 4 日

@index90 逻辑要求就不能阻塞，丢失数据也不能阻塞。

然后 l.rConn 的写入只有这一处，第二种写法也有阻塞的可能？

index90

2021 年 8 月 4 日

@lysS 像上面说的，if len(l.rConn) < cap(l.rConn) 和 l.rConn <- c 不是原子的，所以有阻塞可能。你要非阻塞只能用上面的 select

XTTX

2021 年 8 月 5 日

能用 switch 就不用 if, Bill Kennedy 的淳淳教诲