C语言初学者代码中的常见错误与问题

 更新时间:2013年11月10日 10:14:13   作者:  
C语言初学者犯过的很多错误都非常典型,在初学者中非常普遍,于是整理了一下,应该对其他初学者有借鉴意义

问题
开灯问题

  有n盏灯,编号为1~n,第1个人把所有灯打开,第2个人按下所有编号为2 的倍数的开关(这些灯将被关掉),第3 个人按下所有编号为3的倍数的开关(其中关掉的灯将被打开,开着的灯将被关闭),依此类推。一共有k个人,问最后有哪些灯开着?输入:n和k,输出开着的灯编号。k≤n≤1000

复制代码 代码如下:

#include <stdio.h>
#include <math.h>

int main()
{
   int a[1001],n,k,i,j;

   printf("请分别输入灯和人的数量\n");
   scanf("%d%d",&n,&k);

   while(1) //检验是否超出
   {
      if(k>=1 && k<=1000 && n>=k && n<=1000)
         break;
      else
      {
         printf("数值不符,请重新输入:\n");
         scanf("%d%d",&n,&k);
      }
   }

   for(i=2;i<=k;i++) //每操作一次第i*j个开关,a[i*j]加1
   {
      for(j=1;i*j<=n;j++)
      {
         a[i*j]=a[i*j]+1;
      }
   }

   for(i=1;i<=n;i++) //若操作次数为偶数,表示该位置的灯亮着
   {
      if(a[i]%2==0 && a[i]!=0)
      printf("%d\t",i);
   }

   printf("\n");
   return 0;
}

测试:

“自己测了觉得没问题..在线系统结果是WrongAnswer ..未找出原因..”
 评:

  最明显的错误就是第26行

复制代码 代码如下:

a[i*j]=a[i*j]+1;

由于前面定义a数组为局部auto类别,在不进行初始化的情况下,a中的数据是垃圾值。换句话说,a中的数据是无意义的。因此a[i*j]+1这个表达式没有意义。

  此外,第22行

复制代码 代码如下:

  for(i=2;i<=k;i++) //每操作一次第i*j个开关,a[i*j]加1

在逻辑上也是错误的,缺乏“第1个人把所有灯打开”的步骤。

  另外

复制代码 代码如下:

printf("请分别输入灯和人的数量\n");
   scanf("%d%d",&n,&k);

   while(1) //检验是否超出
   {
      if(k>=1 && k<=1000 && n>=k && n<=1000)
         break;
      else
      {
         printf("数值不符,请重新输入:\n");
         scanf("%d%d",&n,&k);
      }
   }

这段写得很丑,属于典型的谭浩强风格,C语言应该这样写:

复制代码 代码如下:

printf("请分别输入灯和人的数量\n");
while(scanf("%d%d",&n,&k) ,( k <1 || n<k || n>1000) )
{
   printf("数值不符,请重新输入:\n");
}

  代码中还有其他一些毛病,由于不是最主要的问题,这里就不再进一步指出了。

重构:

复制代码 代码如下:

#include <stdio.h>

#define MAXNUM 1000
#define ON 0
#define OFF 1

int main( void )
{
   int light[MAXNUM] = { ON } ; //把所有灯打开
   int n , k ;
   int i ;

   printf( "请分别输入灯和人的数量\n" );
   while ( scanf("%d%d", & n ,& k ) ,( k < 1 || n < k || n > MAXNUM ) )
      printf("数值不符,请重新输入:\n");

   for ( i = 2 - 1 ; i < k ; i ++ ) //第2个人按下所有编号为2 的倍数的开关……
   {
      int j ;
      for ( j = i ; j < n ; j += i + 1)
         light[j] = ! light[j];    
   }

   for ( i = 0 ; i < n ; i ++ )
      if( light[i]==ON )
         printf( "%d " , i + 1 );
   putchar('\n');

   return 0;
}

相关文章

  • 详解C++中typedef 和 #define 的区别

    详解C++中typedef 和 #define 的区别

    这篇文章主要介绍了C++中typedef 与 #define 的区别,本文通过实例代码给大家介绍的非常详细,对大家的学习或工作具有一定的参考借鉴价值,需要的朋友可以参考下
    2021-09-09
  • C++实现TCP客户端及服务器Recv数据筛选处理详解

    C++实现TCP客户端及服务器Recv数据筛选处理详解

    这篇文章主要为大家介绍了C++实现TCP客户端及服务器Recv数据筛选处理详解,有需要的朋友可以借鉴参考下,希望能够有所帮助,祝大家多多进步,早日升职加薪
    2022-10-10
  • C语言对冒泡排序进行升级介绍

    C语言对冒泡排序进行升级介绍

    大家好,本篇文章主要讲的是C语言对冒泡排序进行升级介绍,感兴趣的同学赶快来看一看吧,对你有帮助的话记得收藏一下
    2022-01-01
  • 全面了解结构体、联合体和枚举类型

    全面了解结构体、联合体和枚举类型

    下面小编就为大家带来一篇全面了解结构体、联合体和枚举类型。小编觉得挺不错的,现在就分享给大家,也给大家做个参考。一起跟随小编过来看看吧
    2016-07-07
  • C语言职工信息管理系统源码

    C语言职工信息管理系统源码

    这篇文章主要为大家详细介绍了C语言职工信息管理系统源码,文中示例代码介绍的非常详细,具有一定的参考价值,感兴趣的小伙伴们可以参考一下
    2018-01-01
  • Qt图形图像开发之曲线图表模块QChart库读取/设置X轴的显示区间

    Qt图形图像开发之曲线图表模块QChart库读取/设置X轴的显示区间

    这篇文章主要介绍了Qt图形图像开发之曲线图表模块QChart库读取/设置X轴的显示区间,需要的朋友可以参考下
    2020-03-03
  • C++实现显示MP3文件信息的方法

    C++实现显示MP3文件信息的方法

    这篇文章主要介绍了C++实现显示MP3文件信息的方法,可实现显示如作者、专辑等(libZPlay)信息的功能,需要的朋友可以参考下
    2015-06-06
  • 深入了解C++中map用法

    深入了解C++中map用法

    下面小编就为大家带来一篇深入了解C++中map用法。小编觉得挺不错的,现在就分享给大家,也给大家做个参考。一起跟随想过来看看吧
    2016-06-06
  • C语言数据结构不挂科指南之线性表详解

    C语言数据结构不挂科指南之线性表详解

    线性表是由 n(n≥0)个数据元素组成的有穷序列,这篇文章主要来和大家来了C语言数据结构中的线性表,感兴趣的小伙伴可以跟随小编一起了解一下
    2022-09-09
  • C语言递归实现归并排序详解

    C语言递归实现归并排序详解

    这篇文章主要为大家详细介绍了C语言递归实现归并排序,文中示例代码介绍的非常详细,具有一定的参考价值,感兴趣的小伙伴们可以参考一下, 希望能够给你带来帮助
    2022-03-03

最新评论