初中级C语言水平自测：srand((int)time(0)); 有没有错？

bruceteen

插科打诨的说，C语言中应该是NULL不是0吧

zylthinking

解毒专家莫非又在规定不允许隐性类型提升了？

OwnWaterloo

这能有什么（技术性的）错？

1. p = 0 与 p = NULL （假设p是指针类型）难道还会有区别不成？
只有在编译器不知道类型的情况下 —— 也就是省略原型与可变长参数 —— 它们在C中才会有区别。

并且这种情况下写NULL还不一定就是好习惯：
i) 写0在C或C++里都错。
ii) 而写NULL在C++中肯定错，在C里依然不一定对 —— 我觉得这还不如前面的都错的结果。
iii) 只有(T*)0才是在C与C++中都对的写法。


2. 关于(int)，写为(unsigned)或者干脆不写直接srand(time(0))也行。
time_t规定必须是算术类型 —— 也就是说不会像pthread_t那样存在被定义为结构体的可能性 —— 肯定能转换。
不进行错误处理，直接将(time_t)-1当作种子也是可以接受的结果。

剩余的可能的错误就是time_t是浮点并且time(0)的返回值超出了unsigned的可表示范围。。。

zylthinking

OwnWaterloo 发表于 2012-09-25 00:19
这能有什么（技术性的）错？

剩余的可能的错误就是time_t是浮点并且time(0)的返回值超出了unsigned的可表示范围

这也不是错， srand 本来就不期待一个确定的输入， 只要不发生内存访问违规， 就算是 'fuck' 也没人敢说是个错误， 你管他截取与否了

OwnWaterloo

回复 6# pmerofc

确实应该改为(unsigned)或者干脆不写转型。
每次看到pthread_create( ... , (void*)f, (void*)x , ... ) 这样的代码就感到无奈，这究竟是谁教的。。。
—— 也就是说应该这么做。


至于这么做的理由，要UB的话都是，怎么会一个是一个不是？

还有种子的问题，不一定会减少一半：

1. time_t -> int -> unsigned(srand的参数) 不一定就是 time_t -> unsigned 的一半
得看time_t的类型，还有time的返回值范围。


2. 即使只有一半
srand(i), i = [0, UINT_MAX/2) 对比 srand(i), i = [0, UINT_MAX]，后者也不一定就会有更好的分布。
得看srand,rand是怎么实现的。


3. 哪怕将代码改为srand(0)（不仅仅是一半，而是UINT_MAX分之一），也不能肯定程序就一定是错的。

mirnshi

回复 8# OwnWaterloo

补充一些弹药：

在一些OS上time_t与机器位数有关，而且是signed的。32位上的就是int，64位的就是long.
在32位上对time()返回值做(int)只能说是没有任何意义。

srand()的参数是unsigned的，到了内部转换为u_long，作为随机种子。