鱼C论坛

 找回密码
 立即注册
查看: 1548|回复: 9

[已解决]猴子排序算法中随机数问题

[复制链接]
发表于 2020-5-31 12:39:35 | 显示全部楼层 |阅读模式

马上注册,结交更多好友,享用更多功能^_^

您需要 登录 才可以下载或查看,没有账号?立即注册

x
本帖最后由 Crazyeye 于 2020-5-31 12:48 编辑
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <time.h>

  4. int in_order(int array[], int length);
  5. void shuffle(int array[], int length);
  6. void bogo_sort(int array[], int length);

  7. int in_order(int array[], int length)
  8. {
  9.         int i = 0;

  10.         while (array <= array[i+1] && ++i < length - 1)
  11.                 ;

  12.         if (i == length - 1)
  13.         {
  14.                 return 1;
  15.         }
  16.         else
  17.         {
  18.                 return 0;
  19.         }
  20. }

  21. void shuffle(int array[], int length)
  22. {
  23.         int index, temp, i;
  24.         static int t1, t2;

  25.         srand(t1);//这里t1作为种子
  26.         t1 = rand();//再把随机数赋值给t1,下次再循环种子(t1)就变了,这样每一次调用 srand(t1) 都会产生一个不一样的随机数种子

  27.         t2 = time(NULL);//NULL这里取当前时间应该是一秒一变(+1),那这个t2有啥意义么?前面的t1已经很随机了。
  28.         srand(t1+t2);
  29. 那这个种子就是一个一秒一变的数+随机数产生的随机种子的随机数,有人说为了避免伪随机,因为种子出来的随机数是均匀分布的,
  30. 那整体的看先看t2,相对时间一秒一变,cpu运算那么快,一秒之内这就是个定值!再看t1,虽然每一次调用 srand(t1) 都会产生一个不一样的随机数种子,
  31. 但是,这个还是均匀分布的啊,这个种子产生的随机数是一定的,他作为种子产生的随机数也是一定的啊,一直循环下去还是均匀分布的,
  32. 然后再加一个t2整体都加了一个定值的数,这怎么避免伪随机了?如果只为了随机产生数,t2的意义是啥?我的问题重点就是t2存在的意义,没有行不行,
  33. 或者说没有了会影响执行速度么?


  34.         for (i = 0; i < length; i++)
  35.         {
  36.                 index = rand() % (length - i) + i;//(第二个问题)为啥不直接用 rand() % length,也会影响速度么?怎么影响的
  37.                 if (index != i)
  38.                 {
  39.                         temp = array;
  40.                         array = array[index];
  41.                         array[index] = temp;
  42.                 }
  43.         }
  44. }

  45. void bogo_sort(int array[], int length)
  46. {
  47.         while (!in_order(array, length))
  48.         {
  49.                 shuffle(array, length);
  50.         }
  51. }

  52. int main(void)
  53. {
  54.         int array[] = {73, 108, 111, 118, 101, 70, 105, 104, 67};
  55.         int i, length;
  56.         time_t begin, end;

  57.         begin = time(NULL);

  58.         length = sizeof(array) / sizeof(array[0]);
  59.         bogo_sort(array, length);

  60.         printf("排序后的结果是:");
  61.         for (i = 0; i < length; i++)
  62.         {
  63.                 printf("%d ", array);
  64.         }
  65.         putchar('\n');

  66.         end = time(NULL);
  67.         printf("总共耗时:%ld秒\n", end - begin);

  68.         return 0;
  69. }
复制代码
最佳答案
2020-8-25 18:57:58
qq1256740918 发表于 2020-8-25 15:38
不对,t1 是 static int 类型,是以 0 值初始化的。

没注意,疏忽了……

这样一来,t2 的作用就十分重要了。如果没有 t2,当 t1 初始值为 0 时,后面几次产生的随机数都是可以通过计算的出来的。
小甲鱼最新课程 -> https://ilovefishc.com
回复

使用道具 举报

发表于 2020-5-31 12:40:58 | 显示全部楼层
多行注释是 /**/,不是 //
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 0 反对 1

使用道具 举报

 楼主| 发表于 2020-5-31 12:44:26 | 显示全部楼层
永恒的蓝色梦想 发表于 2020-5-31 12:40
多行注释是 /**/,不是 //


我知道怎么了。。。。就是为了问问题啊,我//一行到底不好看就换了几行,好看点啊。懒得打/*
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2020-5-31 12:47:40 | 显示全部楼层
Crazyeye 发表于 2020-5-31 12:44
我知道怎么了。。。。就是为了问问题啊,我//一行到底不好看就换了几行,好看点啊。懒得打/*
那这个种子就是一个一秒一变的数+随机数产生的随机种子的随机数,有人说为了避免伪随机,因为种子出来的随机数是均匀分布的,
那整体的看先看t2,相对时间一秒一变,cpu运算那么快,一秒之内这就是个定值!再看t1,虽然每一次调用 srand(t1) 都会产生一个不一样的随机数种子,
但是,这个还是均匀分布的啊,这个种子产生的随机数是一定的,他作为种子产生的随机数也是一定的啊,一直循环下去还是均匀分布的,
然后再加一个t2整体都加了一个定值的数,这怎么避免伪随机了?如果只为了随机产生数,t2的意义是啥?我的问题重点就是t2存在的意义,没有行不行,
或者说没有了会影响执行速度么?
你愿意直接 srand(t1) 都行,未初始化内存块的值本身就随机
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

 楼主| 发表于 2020-5-31 12:52:19 | 显示全部楼层
永恒的蓝色梦想 发表于 2020-5-31 12:47
你愿意直接 srand(t1) 都行,未初始化内存块的值本身就随机

哦,就是没啥影响。。。怎么折腾都没啥大区别。。。
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2020-8-16 19:07:41 | 显示全部楼层
为什么小甲鱼老师的程序执行不出来?
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2020-8-25 15:38:31 | 显示全部楼层
永恒的蓝色梦想 发表于 2020-5-31 12:47
你愿意直接 srand(t1) 都行,未初始化内存块的值本身就随机

不对,t1 是 static int 类型,是以 0 值初始化的。

评分

参与人数 1荣誉 +5 鱼币 +5 贡献 +5 收起 理由
永恒的蓝色梦想 + 5 + 5 + 5 感谢纠错!

查看全部评分

小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2020-8-25 18:57:58 | 显示全部楼层    本楼为最佳答案   
qq1256740918 发表于 2020-8-25 15:38
不对,t1 是 static int 类型,是以 0 值初始化的。

没注意,疏忽了……

这样一来,t2 的作用就十分重要了。如果没有 t2,当 t1 初始值为 0 时,后面几次产生的随机数都是可以通过计算的出来的。
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2020-8-25 20:53:30 | 显示全部楼层
主函数中调用函数 bogo_sort(array, length); 这里的 array 是以数组名作为实参传递给 bogo_sort 函数的形参,它传递的只是 array[] 这个数组的首地址,而不是 array[0] 的值,所以后面的 array <= array[i + 1] 就报错了,导致了内存不可读
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

发表于 2020-8-25 22:19:46 | 显示全部楼层
  1. int in_order(int array[], int length)
  2. {
  3.         int i = 0;

  4.         while (array <= array[i+1] && ++i < length - 1)
  5.                 ;
  6.         ......

复制代码


将所有函数里的 array 替换成 array[ i ] 之后调试,系统的报错没有了,但新的问题出现了,整个程序进入了死循环,原因是 in_order() 这个函数用 while 循环了半天,什么都没做,仅仅是返回 1,返回 0 而已
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 1 反对 0

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

小黑屋|手机版|Archiver|鱼C工作室 ( 粤ICP备18085999号-1 | 粤公网安备 44051102000585号)

GMT+8, 2025-7-6 16:05

Powered by Discuz! X3.4

© 2001-2023 Discuz! Team.

快速回复 返回顶部 返回列表