上次在用函式来传达你的心意> 0 <中我们把while回圈内在做什麽用函式名称诠释出来,看起来像是在阅读一篇文章一样,这次让我带你看main里面还有什麽可以改进的部分,让整个main更直觉好懂。
第一眼我看见的是seats[10]这个阵列宣告长度的数值,10在很多地方用到而且这个数字是在这个范例内永远是固定值,所以我们可以用一个叫做size来取代他,也方便我们之後长度修改,再来看到宣告变数i, j的地方,由於我们之前把函式切割出去了,这两个变数就不需要存在。
int main()
{
int seats[10] = {99, 0, 10, 31, 0, 42, 70, 67, 0, 0};
int i, j, seat, number;
showSeats(seats, 10);
getUserInput(seat, number);
while(number != 0){
if(seats [seat - 1] == 0){
seats[seat - 1] = number;
showSeats(seats, 10);
getUserInput(seat, number);
}
else{
printf("Sorry, seat is taken.\n");
scanf("%d %d", &seat, &number);
}
}
bubble_sort(seats, 10);
showSeats(seats, 10);
printf("\n");
printf("***************\n");
return 0;
}
变数宣告这个区块我感觉上是差不多了,你有其他想法也欢迎提出来跟我讨论^ ^,再来我想往提示使用者输入这整个区块来做个修改,让我帮你切割出我说的地方。
int main()
{
const int size = 10;
int seats[size] = {99, 0, 10, 31, 0, 42, 70, 67, 0, 0};
int seat, number;
showSeats(seats, size);
getUserInput(seat, number);
while(number != 0){
if(seats [seat - 1] == 0){
seats[seat - 1] = number;
showSeats(seats, size);
getUserInput(seat, number);
}
else{
printf("Sorry, seat is taken.\n");
scanf("%d %d", &seat, &number);
}
}
bubble_sort(seats, size);
showSeats(seats, size);
printf("\n");
printf("***************\n");
return 0;
}
提示使用者输入这整个区块的流程可以分成几点:
首先第1点和第2点我们已经有了分别是showSeats和getUserInput,3和4我想再加两个新函式,分别式isSeatValid跟isContinueInput,我们可以用这几个函式来重新组织看看。
showSeats(seats, size);
getUserInput(seat, number);
while(number != 0){
if(seats [seat - 1] == 0){
seats[seat - 1] = number;
showSeats(seats, size);
getUserInput(seat, number);
}
else{
printf("Sorry, seat is taken.\n");
scanf("%d %d", &seat, &number);
}
}
太棒了!!看看我们修改後的成果,整个区块越来越精简了,而且阅读性增加更多,试着一行一行过去就像看文章一样,他要做的事情就跟你想得差不多,这是让人觉得最舒服的程序码。
原本的while回圈我更换成do...while,因为使用者是先输入才判断之後要做的事,do...while会更符合整个流程概念,让我们把整个修改结果整个合并一下。
do{
showSeats(seats, size);
getUserInput(seat, number);
if(isSeatValid(seats, size, seat)){
seats[seat - 1] = number;
}
}while(isContinueInput(number));
void showSeats(int seats[], int size)
{
printf("*seating*\n");
for(int i = 0; i < size; ++i){
printf("%d ", seats[i]);
}
}
void getUserInput(int& seat, int& number){
printf("\n");
printf("***************\n");
printf("Please input the seat (0~9) and number(-1 to end game)\n");
scanf("%d %d", &seat, &number);
}
bool isSeatValid(int seats[], int size, int seat){
if(seat - 1 < 0 || seat - 1 >= size){
printf("Sorry, seat is invalid.\n");
return false;
}
if(seats[seat - 1] != 0){
printf("Sorry, seat is taken.\n");
return false;
}
return true;
}
bool isContinueInput(int number){
return number != 0;
}
int main()
{
const int size = 10;
int seats[size] = {99, 0, 10, 31, 0, 42, 70, 67, 0, 0};
int seat, number;
do{
showSeats(seats, size);
getUserInput(seat, number);
if(isSeatValid(seats, size, seat)){
seats[seat - 1] = number;
}
}while(isContinueInput(number));
bubble_sort(seats, size);
showSeats(seats, size);
printf("\n");
printf("***************\n");
return 0;
}
到这边你可能会对isContinueInput有个疑问,这个函式里面只有一行有需要为了这一行新增一个函式吗?没错很多人会觉得没有必要,对我自己来说number != 0这行没办法直接说明他到底想做什麽,我必须切换我的思维才能了解这是根据使用者输入数字来决定要不要继续,思维上的转换会导致分心,在看main的时候我只想专注在抽象的想法上,而不是实作细节,这会帮助我减少理解程序码所花的时间,相信也可以帮助其他看程序码的人,要记住你一定会跟别人一起合作写程序码,能够为别人多做一点都是值得的。
整个重新架构的部分就先到这里,你可以回头看这篇变数命名的善意来比较看看阅读性上的差异,相信你会感受很深,很感谢你这麽有耐心看到最後,过程中可能有些讲解上不太明确的部分都欢迎留言告诉我,或是你有任何想法也都欢迎,之後还有其他可以重新修正的程序码范例我也会继续分享给你,让我们继续往更美好程序码迈进,我们下次见 ^ ^
大家好,我是毛毛。ヾ(´∀ ˋ)ノ 废话不多说开始今天的解题Day~ 46. Permutation...
本篇简述如何使用 terraform 中 debug 除错 赛後文章会整理放到个人的部落格上 htt...
前言: Activity 上如果有很多的功能(例如系统的设定...),画面的就会变的很长,或是转跳好...
前序求值 程序说明 相关函式 get_value():计算并回报结果 说明: 将引入的两个运算元与运...
最少改动 x 最大收益 大纲 简单的任务 ? 神奇的软件 耦合性三大原则 ADP , 无环依赖原则 ...