当前位置: 首页 > 编程日记 > 正文

劣质代码评析——《写给大家看的C语言书(第2版)》附录B之21点程序(一)

  《写给大家看的C语言书(第2版)》是邮电社图灵公司引进翻译的一本C语言入门书,这是一本垃圾书。搞不清图灵为什么引进了这样一本垃圾书。该书作者基本不懂得C编程技术,书中误导、错谬比比皆是。
  该书的附录B给出了一个21点游戏的代码,这是一个很糟糕的C程序,毛病很多,实在不足以为初学者以示范,当反面教材还差不多。考虑到其中的很多毛病初学者也有,所以拿来在此评析一番。
  首先,这段程序可读性极差,甚至可以说是乱作一团。连我这种阅读代码能力很强的人看起来都很费劲,初学者根本就没有可能看懂。
  乱作一团和可读性还不是一回事。可读性指的是代码晦涩难懂,乱作一团则是指不顾起码的编程规范。譬如代码没有基本的缩进

do {
ans = getAns("Hit or stand (H/S)?");
if ( ans = 'H' )
{ playerGetsCard(&numCards,cards,
playerPoints);
}  
}while(ans != 'S' );

任意地截断语句或声明

do { initCardsScreen(cards,playerPoints,dealerPoints,
total, &numCards);
dealerGetsCard(&numCards,cards, dealerPoints);

  代码本来就够乱的了,又塞进了一大堆更乱更不规范的注释。这些注释其实是应该翻译过来的,但是很可惜译者偷懒并没有翻译,对于国内初学者来说,这就更加混乱。混乱的代码和混乱的注释纠结在一起,整个源程序惨不忍睹。

  为了不折磨大家眼球,下面以删除了注释的代码为基础评析。大家也可以顺便比较一下(http://ishare.iask.sina.com.cn/f/20886221.html),看看这些注释是不是起了很坏的作用。如果感觉这里删除了注释的代码更清楚一些,那么无疑原书上的注释是垃圾。

#include <stdio.h>
#include <time.h>
#include <ctype.h>
#include <stdlib.h>#define BELL '\a'
#define DEALER 0
#define PLAYER 1#define ACELOW 0
#define ACEHIGH 1int askedForName = 0;void dispTitle(void);
void initCardsScreen(int cards[52],int playerPoints[2],
int dealerPoints[2], int total[2], 
int *numCards);
int dealCard(int * numCards,int cards[52]);
void dispCard(int cardDrawn,int points[2]);
void totalIt(int points[2],int tatal[2],int who);
void dealerGetsCard(int *numCards,int cards[52],
int dealerPoints[2]);
void playerGetsCard(int *numCards,int cards[52],
int playerPoints[2]);
char getAns(char mesg[]);
void findWinner(int total[2]);main()
{
int numCards;
int cards[52],playerPoints[2],dealerPoints[2],total[2];
char ans;do { initCardsScreen(cards,playerPoints,dealerPoints,
total, &numCards);dealerGetsCard(&numCards,cards, dealerPoints);
printf("\n");
playerGetsCard(&numCards,cards,playerPoints); 
playerGetsCard(&numCards,cards,playerPoints);
do {
ans = getAns("Hit or stand (H/S)?");
if ( ans = 'H' )
{ playerGetsCard(&numCards,cards,
playerPoints);
}  
}while(ans != 'S' );
totalIt(playerPoints,total,PLAYER);
do{
dealerGetsCard(&numCards,cards,dealerPoints);
}while (dealerPoints[ACEHIGH] < 17 );
totalIt(dealerPoints,total,DEALER);
findWinner(total); 
ans=getAns("\nPlay again(Y/N)?");  }while(ans=='Y');return ;}
void initCardsScreen(int cards[52],int playerPoints[2],
int dealerPoints[2], int total[2], 
int *numCards)
{
int sub,val = 1 ;
char firstName[15];
*numCards=52;for(sub=0;sub<=51;sub++){
val = (val == 14) ? 1 : val;
cards[sub] = val;
val++;  }
for(sub=0;sub<=1;sub++)
{ playerPoints[sub]=dealerPoints[sub]=total[sub]=0;}
dispTitle();
if (askedForName==0)
{ printf("What is your first name?");
scanf(" %s",firstName);
askedForName=1;
printf("Ok, %s,get ready for casino action!\n\n",firstName);
getchar();
}
return;        
}void playerGetsCard(int *numCards,int cards[52],
int playerPoints[2])
{
int newCard;
newCard = dealCard(numCards, cards);
printf("You draw:");
dispCard(newCard,playerPoints);
}
void dealerGetsCard(int *numCards,int cards[52],
int dealerPoints[2])
{
int newCard;
newCard = dealCard(numCards,cards);
printf("The dealer draws:");
dispCard(newCard,dealerPoints);
}
int dealCard(int * numCards,int cards[52])
{
int cardDrawn,subDraw;
time_t t;
srand(time(&t));
subDraw = (rand()%(*numCards));
cardDrawn = cards[subDraw];
cards[subDraw] = cards[*numCards -1];
(*numCards)-;
return cardDrawn;
}
void dispCard(int cardDrawn, int points[2])
{
switch(cardDrawn){
case(11): printf("%s\n","Jack");
points[ACELOW] += 10;
points[ACEHIGH] += 10;
break;
case(12): printf("%s\n","Queen");
points[ACELOW] += 10;
points[ACEHIGH] += 10;
break;
case(13): printf("%s\n","King");
points[ACELOW] += 10;
points[ACEHIGH] += 10;
break;
default : points[ACELOW] += cardDrawn;
if(cardDrawn==1)
{ printf("%s\n","Ace");points[ACEHIGH]+= 11;
}
else
{  points[ACEHIGH]+=cardDrawn;printf("%d\n",cardDrawn); }
}
return ;
}
void totalIt(int points[2],int total[2],int who)
{
if ((points[ACELOW] == points[ACEHIGH])||
(points[ACEHIGH] > 21 ))
{ total[who] = points[ACELOW];}else
{ total[who] = points[ACEHIGH];}if (who == PLAYER )
{printf("You have a total of %d\n\n", total[PLAYER]);}
else
{printf("The house stands with a total of %d\n\n", total[DEALER]);}
return;
}
void findWinner(int total[2])
{
if ( total[DEALER] ==  21 )
{ printf("The house wins.\n");return ;}
if ( (total[DEALER] > 21) && (total[PLAYER] > 21))
{ printf("%s", "Nobody wins.\n");return ; }
if ((total[DEALER] >= total[PLAYER])&& (total[DEALER] < 21))
{ printf("The house wins.\n");return ; }
printf("%s%c","You win!\n",BELL);
return;
}
char getAns(char mesg[])
{
char ans;
printf("%s", mesg);
ans = getchar();
getchar();
return toupper(ans);
}
void dispTitle(void)
{
int i = 0 ;
while(i<25)
{ printf("\n");
i++; }
printf("\n\n*Step right up to the Blackjack tables*\n\n");
return ;
}
View Code

  然而尽管去除了脏、乱、差的注释,代码依然很糟糕,例如

{ printf("\n");
i++; }

  更拙劣的是这个

{  points[ACEHIGH]+=cardDrawn;printf("%d\n",cardDrawn); }
}

  注意到中间有个小三(“}”)吗?这风格简直丑疯了,只好再整理一下。

#include <stdio.h>
#include <time.h>
#include <ctype.h>
#include <stdlib.h>#define BELL '\a'
#define DEALER 0
#define PLAYER 1#define ACELOW 0
#define ACEHIGH 1int askedForName = 0;void dispTitle(void);
void initCardsScreen(int cards[52],int playerPoints[2],
int dealerPoints[2], int total[2], 
int *numCards);
int dealCard(int * numCards,int cards[52]);
void dispCard(int cardDrawn,int points[2]);
void totalIt(int points[2],int tatal[2],int who);
void dealerGetsCard(int *numCards,int cards[52],
int dealerPoints[2]);
void playerGetsCard(int *numCards,int cards[52],
int playerPoints[2]);
char getAns(char mesg[]);
void findWinner(int total[2]);main()
{int numCards;int cards[52],playerPoints[2],dealerPoints[2],total[2];char ans;do { initCardsScreen(cards,playerPoints,dealerPoints,total, &numCards);dealerGetsCard(&numCards,cards, dealerPoints);printf("\n");playerGetsCard(&numCards,cards,playerPoints); playerGetsCard(&numCards,cards,playerPoints);do{ans = getAns("Hit or stand (H/S)?");if ( ans == 'H' ){ playerGetsCard(&numCards,cards,playerPoints);}  }while( ans != 'S' );totalIt(playerPoints,total,PLAYER);do{dealerGetsCard(&numCards,cards,dealerPoints);}while (dealerPoints[ACEHIGH] < 17 );totalIt(dealerPoints,total,DEALER);findWinner(total); ans = getAns("\nPlay again(Y/N)?");  }while(ans=='Y');return ;}void initCardsScreen( int cards[52],int playerPoints[2],int dealerPoints[2], int total[2], int *numCards )
{int sub,val = 1 ;char firstName[15];*numCards=52;for(sub=0;sub<=51;sub++){val = (val == 14) ? 1 : val;cards[sub] = val;val++;  }for(sub=0;sub<=1;sub++){ playerPoints[sub]=dealerPoints[sub]=total[sub]=0;}dispTitle();if (askedForName==0){ printf("What is your first name?");scanf(" %s",firstName);askedForName=1;printf("Ok, %s,get ready for casino action!\n\n",firstName);getchar();}return;        
}void playerGetsCard(int *numCards,int cards[52],int playerPoints[2])
{int newCard;newCard = dealCard(numCards, cards);printf("You draw:");dispCard(newCard,playerPoints);
}void dealerGetsCard(int *numCards,int cards[52],int dealerPoints[2])
{int newCard;newCard = dealCard(numCards,cards);printf("The dealer draws:");dispCard(newCard,dealerPoints);
}int dealCard(int * numCards,int cards[52])
{int cardDrawn,subDraw;time_t t;srand(time(&t));subDraw = (rand()%(*numCards));cardDrawn = cards[subDraw];cards[subDraw] = cards[*numCards -1];(*numCards)-;return cardDrawn;
}void dispCard(int cardDrawn, int points[2])
{switch(cardDrawn){case(11): printf("%s\n","Jack");points[ACELOW] += 10;points[ACEHIGH] += 10;break;case(12): printf("%s\n","Queen");points[ACELOW] += 10;points[ACEHIGH] += 10;break;case(13): printf("%s\n","King");points[ACELOW] += 10;points[ACEHIGH] += 10;break;default : points[ACELOW] += cardDrawn;if(cardDrawn==1){ printf("%s\n","Ace");points[ACEHIGH]+= 11;}else{  points[ACEHIGH]+=cardDrawn;printf("%d\n",cardDrawn); }}return ;
}void totalIt(int points[2],int total[2],int who)
{if ( (points[ACELOW] == points[ACEHIGH])||(points[ACEHIGH] > 21 )){ total[who] = points[ACELOW];}else{ total[who] = points[ACEHIGH];}if (who == PLAYER ){printf("You have a total of %d\n\n", total[PLAYER]);}else{printf("The house stands with a total of %d\n\n", total[DEALER]);}return;
}void findWinner(int total[2])
{if ( total[DEALER] ==  21 ){printf("The house wins.\n");return ;}if ( (total[DEALER] > 21) && (total[PLAYER] > 21) ){ printf("%s", "Nobody wins.\n");return ; }if ((total[DEALER] >= total[PLAYER])&& (total[DEALER] < 21)){ printf("The house wins.\n");return ; }printf("%s%c","You win!\n",BELL);return;
}char getAns(char mesg[])
{char ans;printf("%s", mesg);ans = getchar();getchar();return toupper(ans);
}void dispTitle(void)
{int i = 0 ;while(i<25){ printf("\n");i++; }printf("\n\n*Step right up to the Blackjack tables*\n\n");return ;
}
View Code

  这回勉强能看得下去了。现在不难发现main()中

return ;

  实际上是

 return 0;

之误。
  在dealCard()函数中,也存在一个明显的错误

 (*numCards)-;

实际上应该是

(*numCards)--;

 --*numCards;

 现在来阅读代码。首先看到的是

int askedForName = 0;

  这个外部变量看起来很别扭。从21点这个问题来看,怎么也不会想到程序会需要这样一个名为askedForName 的变量,对于整个问题来讲,这个变量是无关痛痒的鸡毛蒜皮。把这种鸡毛蒜皮性质的变量提升到外部变量这样重要的代码地位,就如同把家里一件可能几十年还用不到一次的小东西布置在客厅显眼位置一样荒唐。

  那么作者为什么要设置这个外部变量呢?估计有两种可能,一种是缺乏足够的代码写作技能,某段代码实在写不下去了,无可奈何地用了一个外部变量。另一种情况是写代码之前缺乏充分缜密的构思,思路有漏洞,写到中间时才发现,只好用这种办法来补救。后一种情况比前一种更糟糕。这就像盖房子一样,盖到中间才发现继续盖下去,房子的一个墙角就会盖到水塘里一样,之后匆忙修改图纸,房子无法按原计划形状盖成只好不顾了。
  实际上这时应该重新思考,重写代码,而不是修修补补。

(未完待续)

相关文章:

【数学 技巧】2.14计数

有趣的组合数学题&#xff1b;考试时候打满确实挺不容易的…… 题目描述 对于一个 $n$ 阶排列 $p$&#xff0c;我们建立一张无向简单图 $G(p)$&#xff0c;有 $n$ 个节点&#xff0c;标号从 $1$ 到 $n$&#xff0c;每个点向左右两侧最近的比它大的点以及比它小的点连边。 形式化…

冒泡排序 算法

算法思路&#xff1a; 从第一个元素开始遍历&#xff0c;比较当前元素和下一个元素的大小不符合&#xff0c;则交换结束最后一个元素&#xff0c;则重新遍历 实现&#xff1a; void bubble_sort(vector<int> &arr) {for (int i 0;i < arr.size() - 1; i) {//交…

5单个编译总会编译全部_5分钟读懂JavaScript预编译

大家都知道JavaScript是解释型语言&#xff0c;既然是解释型语言&#xff0c;就是编译一行&#xff0c;执行一行&#xff0c;那又何来预编译一说呢?脚本执行js引擎都做了什么呢&#xff1f;今天我们就来看看吧。1-JavaScript运行三部曲语法分析预编译解释执行语法分析很简单&a…

2014百度面试题目---“求比指定整数大且最小的不重复数”解答

题目&#xff1a;给定任意一个正整数&#xff0c;求比这个数大且最小的“不重复数”&#xff0c;“不重复数”的含义是相邻两位不相同&#xff0c;例如1101是重复数&#xff0c;而1201是不重复数。 代码&#xff1a; #include <iostream> using namespace std;bool istha…

3DMAX 批量 场景 对象 导出 .X格式 脚本

一、首先你需要下载一个 Total Commader文件管理软件。利用这个软件你可以收集文件夹下包含子文件夹下的max文件&#xff08;或完整路径&#xff09;打开TotalCMD后使用查找文件&#xff1a;&#xff08;如图红框中的操作&#xff09;1.2.3. 复制文件名和完整路径后粘贴到文本文…

C++复数类面向对象的参考

#include <bits/stdc.h> #include <future> #include <thread>using namespace std;class Complex { public:Complex (double r 0, double i 0): re (r), im (i) {} ///冒号后面是初始化的过程&#xff0c;注意分清初始化和赋值的区别Comp…

快速排序 算法

算法思路 序列终任意选择一个数&#xff0c;把序列分为比这个数大和比这个数小的两个子序列不断重复以上步骤(递归) 代码实现 int partition1(vector<int> &arr, int begin , int end) {int ret arr[begin];int index begin 1;for (int i index;i < end; i…

利用Spring AOP与JAVA注解为系统增加日志功能

Spring AOP一直是Spring的一个比较有特色的功能&#xff0c;利用它可以在现有的代码的任何地方&#xff0c;嵌入我们所想的逻辑功能&#xff0c;并且不需要改变我们现有的代码结构。 鉴于此&#xff0c;现在的系统已经完成了所有的功能的开发&#xff0c;我们需要把系统的操作日…

maven引入hadoop_如何添加Hadoop依赖通过Maven

匿名用户1级2017-09-09 回答Hadoop开发中需要用到至少不下10个的依赖包&#xff0c;它们相互间的依赖关系比较复杂&#xff0c;不同版本的依赖关系也有所不同&#xff0c;而间接依赖导致的程序错误并不会在运行之前报错&#xff0c;因此确定适合一个版本的依赖包&#xff0c;会…

js 闭包作用

2019独角兽企业重金招聘Python工程师标准>>> 一、变量的作用域 要理解闭包&#xff0c;首先必须理解Javascript特殊的变量作用域。 变量的作用域无非就是两种&#xff1a;全局变量和局部变量。 Javascript语言的特殊之处&#xff0c;就在于函数内部可以直接读取全…

vue实用组件——页面公共头部

可伸缩自适应的页面头部&#xff0c;屏幕适应范围更广泛 效果如下&#xff1a; 代码如下&#xff1a; <template> <div class"site-header"> <div class"logo"><img src"/assets/icons/logo.png" alt"">&…

插入排序 算法

算法思路 维护一段有序数列同时遍历待排序数列&#xff0c;在有序数列中找到合适的位置插入元素 基本代码 实现如下: void insertion(vector<int>& arr){for(int i1;i<arr.size();i){int tempi;for(int ji-1;j>0;j--){//有序序列不断得增加if(arr[temp]<…

线段树入门【转】

文章来自 &#xff1a; http://blog.csdn.net/x314542916/article/details/7837276 学习算法&#xff0c;自己收藏着。 线段树的入门级 总结 线段树是一种二叉搜索树&#xff0c;与区间树相似&#xff0c;它将一个区间划分成一些单元区间&#xff0c;每个单元区间对应线段树中的…

python自动化框架pytest pdf_pytest+python下的UI自动化基础框架

整体设计模式&#xff1a;config目录&#xff1a;存放一些公共的静态文件&#xff0c;如项目名称&#xff0c;配置文件等这些环境变量(可以用其他组件替换&#xff0c;如sql&#xff0c;主要能把配置文件的内容被程序识别)。httptrquest目录&#xff1a;存放接口代码&#xff0…

ny520 最大素因子 筛选法求素数

最大素因子时间限制&#xff1a;1000 ms | 内存限制&#xff1a;65535 KB难度&#xff1a;2 描述 GreyAnts最近正在学习数论中的素数&#xff0c;但是现在他遇到了一个难题&#xff1a;给定一个整数n,要求我们求出n的最大素因子的序数,例如&#xff1a;2的序数是1,3的序数是2…

JAVA_SE之内部类

内部类分类&#xff1a; 1. 成员内部类 静态内部类 非静态内部类 2. 局部内部类 3. 匿名内部类 1. 成员内部类&#xff1a; package com.atguigu.java; /** 类的第5个成员&#xff1a;内部类* 1.相当于说&#xff0c;我们可以在类的内部再定义类。外面的类&#xff1a;外部类。…

希尔排序 算法

算法思路 插入排序的改进版&#xff0c;选择插入距离远的元素选择一个间距&#xff0c;将序列分成很多子序列并行插入排序降低间距&#xff0c;并重复插入元素&#xff0c;直到间距将为1&#xff0c;完成排序。 算法实现 void shell_sort(vector<int> &arr, int b…

解决Apache CXF 不支持传递java.sql.Timestamp和java.util.HashMap类型问题

在项目中使用Apache开源的Services Framework CXF来发布WebService&#xff0c;CXF能够很简洁与Spring Framework 集成在一起&#xff0c;在发布WebService的过程中&#xff0c;发布的接口的入参有些类型支持不是很好&#xff0c;比如Timestamp和Map。这个时候我们就需要编写一…

python教学上机实验报告怎么写_Python基础(下)

不要忘了冒号啊&#xff01;&#xff01;&#xff01;&#xff01;&#xff01;对于基本数据类型的变量&#xff0c;变量传递给函数后&#xff0c;函数会在内存中复制一个新的变量&#xff0c;从而不影响原来的变量。(我们称此为值传递)但是对于表来说&#xff0c;表传递给函数…

比较有用的样式

背景图水平垂直居中 background:#ebebeb url(/Images/BlogHTImg/bkht_jia.jpg) center center no-repeat; 背景图居左垂直居中 background:#ebebeb url(/Images/BlogHTImg/bkht_jia.jpg) left center no-repeat; background:#ebebeb url(/Images/BlogHTImg/bkht_jia.jpg) 5px…

Python:线程之定位与销毁

背景 开工前我就觉得有什么不太对劲&#xff0c;感觉要背锅。这可不&#xff0c;上班第三天就捅锅了。 我们有个了不起的后台程序&#xff0c;可以动态加载模块&#xff0c;并以线程方式运行&#xff0c;通过这种形式实现插件的功能。而模块更新时候&#xff0c;后台程序自身不…

选择排序 算法

算法思路 维护一段有序数列&#xff0c;同时遍历待排序数列&#xff0c;找到最小的元素插入有序数列中重复&#xff0c;直到待排序数列没有剩余元素 代码实现 void select_sort(vector<int> &arr) {for (int i 0;i < arr.size(); i) {int temp arr[i];int in…

hdu2236 无题II 最大匹配 + 二分搜索

中文题目&#xff0c;题意大家都明白。 看到“不同的行和列”就觉得要用二分匹配来做。要求最大值与最小值的差值最小&#xff0c;是通过枚举边的下限和上限来完成。 枚举过程是这样的&#xff0c;在输入的过程可以记录下边权的最大值MAX和最小值MIN。那么他们的边权的差值的最…

python十大标准_python对标准类型的分类

python的标准类型可以按照三种方式分类。一、按存储模型分类按存储模型分可以分为原子(标量)类型和容器类型。原子(标量)类型指对象(这里的对象不是对象数据类型&#xff0c;而是任何可能的值)的值只能含有一种数据类型&#xff0c;比如数值和字符串。容器类型指它们的值可以含…

mysql慢查询开启及分析方法

最近服务维护的公司的DB服务器&#xff0c;总是会出现问题&#xff0c;感觉需要优化一下了&#xff0c;登陆上去&#xff0c;发现慢查询日志都没有开&#xff0c;真是惭愧&#xff0c; 故果断加上慢查询日志&#xff0c;经过分析sql记录&#xff0c;发现问题很多&#xff0c;开…

如何在调试页面的时候清除页面的缓存?

1.按F12,弹出下图 2.点击右上角的三个点: 3.点击settings 4.找到Network,下面的Disable cache(while DevTools is open) 转载于:https://www.cnblogs.com/studybrother/p/10396990.html

JAVA图片处理--缩放,切割,类型转换

import java.io.*; import java.awt.*; import java.awt.image.*; import java.awt.Graphics; import java.awt.color.ColorSpace; import javax.imageio.ImageIO;public class ChangeImageSize {/** *//*** 缩放图像* param srcImageFile 源图像文件地址* param result …

文本框自动提示_Excel办公小技巧,使用艺术字与文本框,就是那么的简单

Excel中的艺术字同时拥有文字和图形两种对象的属性&#xff0c;不仅可以修改其中的内容&#xff0c;还可以调整形状的大小、设置边框以及内部填充等效果&#xff0c;常在编辑表格标题或者输入一些比较有提示性的文本时使用&#xff0c;在突出关键内容的同时美化表格效果添加艺术…

Linux之父盟友分道扬镳 直言开源模式软肋

Linux之父盟友分道扬镳 直言开源模式软肋2005-09-06 12:53:00标签&#xff1a;linux职场开源休闲从1993年起&#xff0c;Larry McVoy就一直是Linux之父Linus Torvalds最忠实的盟友之一。 然而经历了这些年后&#xff0c;McVoy开始相信&#xff0c;开源这种风靡一时、纷纷被…

身份证第18位计算

本文计算方式源自 百度百科&#xff0c;根据计算方式&#xff0c;Java计算代码如下文所示。 计算方法 1、将前面的身份证号码17位数分别乘以不同的系数。从第一位到第十七位的系数分别为&#xff1a;7&#xff0d;9&#xff0d;10&#xff0d;5&#xff0d;8&#xff0d;4&…