【问题标题】:How does this implementation of randomization work in C?这种随机化的实现如何在 C 中工作?
【发布时间】:2019-04-17 15:47:27
【问题描述】:

自从我在 https://www.tutorialspoint.com/c_standard_library/c_function_rand.htm 上找到这个特定的文档后,我一直在考虑这行特定的代码 srand((unsigned)time(&t));。每当我必须生成一些东西时,我都会使用 srand(time(NULL)) 来排序不要每次运行程序时都生成相同的东西,但是当我遇到这个时,我一直在想:srand((unsigned)time(&t))srand(time(NULL)) 之间有什么区别吗?因为在我看来它们似乎在做同样的事情。为什么是否使用了 time_t 变量?为什么srand() 中使用了地址运算符?

#include <stdio.h>
#include<stdlib.h>
int main(){
    int i,n;
    time_t t;

    n = 5;

    srand((unsigned)time(&t)); 

    for (i = 0; i < n; i++) {
        printf("%d\n", rand() % 50);
    }
return(0);
}

【问题讨论】:

  • 仅仅因为它在互联网上,并不意味着它很好。要记住的东西。
  • 这不是位运算符,而是 address-of 运算符。
  • 什么意思?
  • @Taegyung 对不起,你是对的。我没有注意这个运算符的使用上下文。
  • 您可以通过研究time library function 来了解time 是如何被调用的。它解释了正在发生的事情。演员阵容是你应该熟悉的(除非你知道自己在做什么,否则通常会避免)。

标签: c srand


【解决方案1】:

是的,它会产生相同的结果。但是这个例子写得不好。

我会仔细阅读 Tutorialspoint。这是一个以糟糕的 C 代码而闻名的站点,您在 SO 的问题中看到的许多不良习惯都可以追溯到该站点。好的,这是轶事证据,但我确实在这里问过用户为什么他们投出malloc 的结果,他们回答说他们在 Tutorialspoint 上学到了这一点。在这个简短的 sn-p 中,您实际上可以看到(至少)四个示例。

  1. 他们将调用结果转换为 time(),这完全没有必要,只会使代码混乱。
  2. 出于某种原因,他们使用变量t,这在本例中完全没用。如果您阅读time() 的文档,您会发现在此示例中仅传递 NULL 就足够了。
  3. 为什么要使用变量n?对于这个简短的示例,使用硬编码值完全可以。当你使用变量来避免硬编码值时,你应该声明它们const,并给它们一个比n更具描述性的名称。 (好吧,我意识到在写这篇文章时我有点紧张。省略const 并不是什么大不了的事,即使它更可取。而且“n”是一个通用名称,意思是“迭代次数”。而且使用变量而不是硬编码值通常是一件好事。)
  4. 省略了#include&lt;time.h&gt;,如果他们也省略了其余的包含,那就没问题了。
  5. 使用int main() 代替int main(void)

对于 5,我想说,在大多数情况下,这对于 main 函数无关紧要,但是用空括号而不是 int foo(void) 声明其他函数(例如 int foo())可能会导致问题,因为它们意味着不同的东西。来自C standard

使用带空括号的函数声明符(不是原型格式的参数类型声明符)已过时。

这是一个与此相关的问题:What are the semantics of function pointers with empty parentheses in each C standard?

人们也可以争论其他一些事情,但有些人会不同意这些。

  1. 为什么要在 for 循环之外声明 i?自 20 岁的 C99 以来,在内部声明它是合法的。
  2. 为什么用return 0结束函数?从 C99 开始,省略它也是可以的。如果你想返回 0 以外的值,你只需要在 main 中有一个 return。就个人而言,一般来说,我认为“这是一个很好的做法”是一个完整的废话陈述,除非有一些很好的论据来说明为什么它应该是一个很好的做法。

如果您的目标是在没有支持 C99 的编译器的环境中维护非常旧的 C 代码,请记住这些内容。但这有多普遍?

所以如果我要在教程点重写示例,我会这样写:

#include<stdio.h>
#include<stdlib.h>
#include<time.h>
int main(void){
    srand(time(NULL)); 

    for (int i = 0; i < 5; i++) {
        printf("%d\n", rand() % 50);
    }
}

另一个可怕的例子可以在这里找到:https://www.tutorialspoint.com/c_standard_library/c_function_gets.htm

函数gets 已从标准C 中删除,因为它非常危险。然而,该网站甚至没有提到这一点。

另外,他们教你转换malloc https://www.tutorialspoint.com/c_standard_library/c_function_malloc.htm 的结果,这是完全没有必要的。在此处阅读原因:Do I cast the result of malloc?

虽然他们提到malloc 在失败时返回NULL,但他们没有在示例中展示如何正确地对其进行错误检查。 scanf 等函数也是如此。

【讨论】:

  • 具有讽刺意味的是,有时我的算法设计实验室教授会给我们一个链接来研究有关数据结构的一些理论方面,例如双链表:tutorialspoint.com/data_structures_algorithms/…,每次我看到这样的代码让我发疯了。
  • 您应该使用#include &lt;time.h&gt; 并且可能使用int main(void)。我也更喜欢srand(time(0)) :)
  • @pmg time.h 已修复。但是,我很少关心 void 参数,尤其是 main。另外,我更喜欢 time(NULL),因为 time 的参数是一个指针。
  • @pmg 另外,为此,您可以编写一个包装器。 void srand(void) { srand(time(NULL)); },但由于您通常调用srand,每次程序执行只调用一个,我不明白这一点。
  • @Broman:对于毫无意义的事情,我会选择#define INIT_RNG srand(time(NULL /* or 0 */)) :)
猜你喜欢
  • 2014-12-18
  • 2010-12-10
  • 2016-12-30
  • 1970-01-01
  • 2016-05-14
  • 1970-01-01
  • 1970-01-01
  • 2012-08-30
  • 2021-05-26
相关资源
最近更新 更多