帮我看看有什么地方可以改进。

kevin999990

1. #include<stdio.h>
   
2. 
   
3. #define PREPAID_BALANCE 10.00
   
4. 
   
5. int main()
   
6.         //Account Balance Program
   
7. {
   
8.         const float band1 = 0.50f, band2 = 0.40f, band3 = 0.20f, sms_rate = 0.10f;//calculating rate
   
9.         float total_call, balance, sms_cost;
   
10.         int min, min1, min2, min3, decide, sms_num, topup_amount;
    
11.         int total_sms=0, total_min=0;//for summary
    
12.         float total_call_charge=0, total_sms_charge=0, total_topup=0; //for summary
    
13. 
    
14.         balance=PREPAID_BALANCE;
    
15.         printf("Thank you for using TARMoblie.\n\n");
    
16. 
    
17. CALL://done
    
18.         {
    
19.                 printf(
    
20.                         "\n==============================================="
    
21.                         "\n\t\tCALL\n"
    
22.                         );
    
23.                 printf("\nYour balance is RM %.2f\n",balance);//show balance in begin
    
24.                 printf("\nESTIMATED Available Talk Time : %.0f minutes",balance/0.51);//estimate call time(inaccurate)
    
25.                 printf("\nCall duration (minute) :");//promt user input
    
26.                 scanf("%d",&min);
    
27.                 fflush(stdin);
    
28. 
    
29.                 if (min>10)
    
30.                         min1=10;
    
31.                 else min1=min;
    
32.                 printf("\nBand1 charge rate : %4d minute * RM 0.50 = RM\t%.2f",min1,min1*band1);
    
33. 
    
34. 
    
35.                 if(min>10)//show band 2 rate if the call duration is more then 10 minutes
    
36.                 {
    
37.                         if(min>20)//if call duration is over 20minute, assign 10 into min2
    
38.                                 min2=10;
    
39.                         else min2=min-10;
    
40.                         printf("\nBand2 charge rate : %4d minute * RM 0.40 = RM\t%.2f",min2,min2*band2);
    
41.                 }
    
42. 
    
43. 
    
44.                 if(min>20)//show band 3 rate if the call duration is more than 20 minutes
    
45.                 {
    
46.                         min3=min-20;//calculate the band 3 duration usage
    
47.                         printf("\nBand3 charge rate : %4d minute * RM 0.20 = RM\t%.2f",min3,min3*band3);
    
48.                 }
    
49. 
    
50.                 if(min<=10)
    
51.                         min2=0;
    
52.                 if(min<=20)
    
53.                         min3=0;
    
54. 
    
55.                 total_call=min1*band1+min2*band2+min3*band3;//calculate this call rate
    
56.                 balance=balance-total_call;//calculate balance
    
57.                 total_min=min+total_min;//for summary, calculate averall call time
    
58.                 total_call_charge=total_call+total_call_charge;//for summary, calculate overall call charge
    
59. 
    
60.                 printf("\n\t\t\t\t\t\t-----\n");
    
61.                 printf("\t\t\tCharge of this call: RM\t%.2f\n",total_call);//show the charge rate
    
62.                 printf("\t\t\t\t\t\t=====\n");
    
63. 
    
64.                 printf("\nNew Balance is RM %.2f\n",balance);
    
65. 
    
66.                 printf("-----------------------------------------------\n");
    
67.                 goto SELECTION;
    
68.         }
    
69. 
    
70. SMS://done
    
71.         {
    
72.                 printf(
    
73.                         "\n==============================================="
    
74.                         "\n\t\tSMS\n"
    
75.                         );
    
76. 
    
77.                 printf("\nYour balance is RM %.2f\n",balance);
    
78.                 printf("\nSMS available : %.0f",(balance-1.90)/0.10);//estimate avalible sms
    
79.                 printf("\nHow many SMS do you want to send :");
    
80.                 scanf("%d",&sms_num);
    
81.                 fflush(stdin);
    
82. 
    
83.                 sms_cost=sms_num*sms_rate;//calcalate sms cost
    
84.                 printf("SMS charge = RM %.2f\n",sms_cost);
    
85. 
    
86.                 balance=balance-sms_cost;//calculate balance
    
87.                 printf("\nNew Balance is RM%.2f\n",balance);
    
88.                 printf("-----------------------------------------------\n");
    
89. 
    
90.                 total_sms_charge=sms_cost+total_sms;//for summary, total sms charge
    
91.                 total_sms=sms_num+total_sms;//for summary, total sms count
    
92. 
    
93.                 goto SELECTION;
    
94.         }
    
95. 
    
96. TOPUP://done
    
97.         {
    
98.                 printf(
    
99.                         "\n==============================================="
    
100.                         "\n\t\tTOP UP\n"
     
101.                         );
     
102.                 printf("\nYour balance is RM %6.2f\n",balance);
     
103. 
     
104.                 printf("\nTop Up Amount : RM ");//promt for topup amount
     
105.                 scanf("%d",&topup_amount);
     
106.                 fflush(stdin);
     
107. 
     
108.                 balance=topup_amount+balance;//calculate balance after topup
     
109.                 total_topup=topup_amount+total_topup;//for summary, calculate overall topup amount
     
110. 
     
111.                 printf("\nNew Balance is RM%6.2f\n",balance);
     
112.                 printf("-----------------------------------------------\n");
     
113. 
     
114.                 goto SELECTION;
     
115.         }
     
116. 
     
117. SELECTION://done
     
118.         {
     
119.                 printf("\n\n");
     
120. 
     
121.                 printf("(1)Call, (2)SMS, (3)TOP UP, (4)EXIT >>");
     
122.                 scanf("%d",&decide);
     
123.                 fflush(stdin);
     
124. 
     
125.                 if(decide==1 || decide==2)
     
126.                 {
     
127.                         if(balance>=2.00)//need at least above RM2.00 to make call or sms
     
128.                         {
     
129.                                 if(decide==1)
     
130.                                         goto CALL;
     
131.                                 if(decide==2)
     
132.                                         goto SMS;
     
133.                         }
     
134.                         else goto CHECK_BALANCE;
     
135.                 }
     
136.                 if(decide==3)
     
137.                         goto TOPUP;
     
138.                 if(decide==4)
     
139.                         goto SUMMARY;
     
140.                 if(decide>4)
     
141.                         goto SELECTION;
     
142.         }
     
143. 
     
144. CHECK_BALANCE:
     
145.         {
     
146.                 while(balance<2.00)//loop topup, if amount is less than RM2.00
     
147.                 {
     
148.                         printf(
     
149.                                 "\n==============================================="
     
150.                                 "\n\t\tNOTICE\n"
     
151.                                 );
     
152.                         printf("\nYour balance is RM %.2f\n",balance);
     
153.                         printf(
     
154.                                 "You need at least RM 2.00 to make call or sms\n"
     
155.                                 "How much to top up? >>RM "
     
156.                                 );
     
157.                         scanf("%d",&topup_amount);//promt for topup amount
     
158.                         fflush(stdin);
     
159. 
     
160.                         balance=topup_amount+balance;//calculate balance atfer topup
     
161.                         total_topup=topup_amount+total_topup;//for summary, calculate overall topup amount
     
162. 
     
163.                         printf("\nNew Balance is RM%6.2f\n",balance);
     
164.                         printf("-----------------------------------------------\n");
     
165.                 }
     
166.                 goto SELECTION;
     
167.         }
     
168. 
     
169. SUMMARY:
     
170.         {
     
171.                 printf(
     
172.                         "\n\n==============================================="
     
173.                         "\n\t\tTRANSACTION SUMMARY\n"
     
174.                         );
     
175. 
     
176.                 printf("\nIn total, you \ttalk \t%d minutes\n",total_min);
     
177.                 printf("\t\tsent \t%d SMS's\n", total_sms);
     
178.                 printf("\nStarting Balance\t\t : RM %6.2f",PREPAID_BALANCE);
     
179.                 printf("\nTotal Amount Topped UP \t\t : RM %6.2f",total_topup);
     
180.                 printf("\nTotal Call Charges \t\t : RM %6.2f",total_call_charge);
     
181.                 printf("\nTotal SMS Charges \t\t : RM %6.2f", total_sms_charge);
     
182.                 printf("\nEnding Balance \t\t\t : RM %6.2f",balance);
     
183. 
     
184.                 printf("\n-----------------------------------------------\n");
     
185.                 printf("\n\n");
     
186.         }
     
187.         system("pause");
     
188.         return 0;
     
189. 
     
190. }
     

复制代码

我刚学C语言，学到do。。。while 那里罢了。
这是学校给的assignment，已经做完了，但是觉得很长（?）
所以想问下有没有其他方法可以写或者缩短？
另外，因为我有一个朋友说用GOTO很乱水，但是我又不知道有什么其他的方法，请各位指教。

geekman

你的朋友说得没错，使用GOTO并不是个好习惯。C是Structured Language，使用GOTO会打断你的程式的流程，变得Non-Structured，并破坏C语言的Code independence。同时这也会在将来让你变成同事间的怪胎，同时被同事们所厌恶，因为你的Code将会让他们很难Keep track整个程式的flow。

解决方法？细心的理解整个程式的概念，并仔细的设计整个程式的流程，善用Flow Charts，善用Functions。

所以想问下有没有其他方法可以写或者缩短？

我想你陷入所有初学Programming的人的误区了。Program并不是写的越短就越厉害。代码应该清晰，明了，格式整齐才是你目前应该学习的正确方向。超级压缩的代码虽然看起来简短，好像很厉害，但是却是所有人都讨厌的编程方式，因为其他接手你的代码的人必须花费额外的心思才能解读你的代码。

Code Optimization则是另外一回事，那是必须累计了一定的知识和实战经验才能做到的，那不是你现在应该烦恼的，等你Final Year时再说吧。

顺带一提，我目前在做着的工作的其中一个Project就有一百九十多万行代码，其中我亲自手打的代码就有三万八千多行（24个Unit Files/Forms），长吗？

Program不是看长短，而是看你的Code能有多自律，你能不能在没有Memory Leak，没有Access Violations，没有Errors，没有Warning 的情形下完成你的工作。

kevin999990

想问下，遇到selection的时候要怎么做？除了goto之外有什么可以用的？
或者有什么例子可以让我看一下？

geekman

如果你的Selection Condition是Numeric Enumerable的，你可以用swich()：

1. 
   
2. char key;
   
3. (或者int key)
   
4. switch(key)
   
5. {
   
6.     case 'a' : doAfunction();
   
7.         break;
   
8.     case 0x33: //do whatever
   
9.         break;
   
10. }
    

复制代码

如果不是Numeric enumerable，可以用if() else...

1. 
   
2. if(str == "I CHOOSE OPTION A")
   
3. {
   
4.     dothis();
   
5. }
   
6. else if(str == "I choose PIKACHU")
   
7. {
   
8.     dothat();
   
9. }
   
10. ...
    

复制代码

kevin999990

geekman 发表于 9-7-2013 02:20 PM
如果你的Selection Condition是Numeric Enumerable的，你可以用swich()：如果不是Numeric enumerable，可以 ...

你好，我把code改成以下了

1. #include <stdio.h>
   
2. <span style="line-height: 1.5;">#define PREPAID_BALANCE 10.00</span>
   
3. 
   
4. void Call_Function();
   
5. void SMS_Function();
   
6. void Topup_Function();
   
7. void Check_Balance();
   
8. void Summary();
   
9. 
   
10. const float band1 = 0.50f, band2 = 0.40f, band3 = 0.20f, sms_rate = 0.10f;
    
11. float total_call, balance, sms_cost;
    
12. int min,min1,min2,min3,sms_num,topup_amount;
    
13. int decide;//for choosing
    
14. int total_sms=0, total_min=0;//for summary
    
15. float total_call_charge=0, total_sms_charge=0, total_topup=0; //for summary
    
16. 
    
17. 
    
18. 
    
19. 
    
20. 
    
21. int main()
    
22. {
    
23.     balance=PREPAID_BALANCE;
    
24.     printf("Thank you for choosing TARMoblie.\n");
    
25. 
    
26.     Call_Function();
    
27. 
    
28.     do
    
29.     {
    
30.         printf("\n\n");
    
31.         printf("(1)Call, (2)SMS, (3)TOP UP, (4)EXIT >>");
    
32.         scanf("%d",&decide);
    
33.         fflush(stdin);
    
34.         switch(decide)
    
35.         {
    
36.         case 1:
    
37.         case 2:
    
38.             if(balance>=2.00)
    
39.             {
    
40.                 if(decide==1)
    
41.                     Call_Function();
    
42.                 if(decide==2)
    
43.                     SMS_Function();
    
44.             }
    
45.             else Check_Balance();
    
46.             break;
    
47.         case 3:
    
48.             Topup_Function();
    
49.             break;
    
50.         case 4:
    
51.             Summary();
    
52.             break;
    
53.         default:
    
54.             printf("Please Enter number between 1-4");
    
55.         }
    
56.     }
    
57.     while(decide!=4);
    
58.     return 0;
    
59. }
    
60. 
    
61. void Call_Function()
    
62. {
    
63.     printf(
    
64.         "\n==============================================="
    
65.         "\n\t\tCALL\n"
    
66.     );
    
67.     printf("\nYour balance is RM %.2f\n",balance);//show balance in begin
    
68.     printf("\nESTIMATED Available Talk Time : %.0f minutes",balance/0.51);//estimate call time(inaccurate)
    
69.     printf("\nCall duration (minute) :");//promt user input
    
70.     scanf("%d",&min);
    
71.     fflush(stdin);
    
72. 
    
73.     if (min>10)
    
74.         min1=10;
    
75.     else min1=min;
    
76. 
    
77.     printf("\nBand1 charge rate : %4d minute * RM 0.50 = RM\t%.2f",min1,min1*band1);
    
78.     if(min>10)//will show band 2 rate if the call duration is more then 10 minutes
    
79.     {
    
80.         if(min>20)
    
81.             min2=10;
    
82.         else min2=min-10;
    
83.         printf("\nBand2 charge rate : %4d minute * RM 0.40 = RM\t%.2f",min2,min2*band2);
    
84.     }
    
85.     if(min>20)//show band 3 rate if the call duration is more than 20 minutes
    
86.     {
    
87.         min3=min-20;//calculate the band 3 duration usage
    
88.         printf("\nBand3 charge rate : %4d minute * RM 0.20 = RM\t%.2f",min3,min3*band3);
    
89.     }
    
90. 
    
91.     if(min<=10)
    
92.         min2=0;
    
93.     if(min<=20)
    
94.         min3=0;
    
95. 
    
96.     total_call= min1* band1+min2* band2+min3* band3;//calculate this call rate
    
97.     total_min+=min;//calculate this call time
    
98.     total_call_charge+=total_call;//calculate overall call time
    
99.     balance-=total_call;//calculate balance
    
100. 
     
101.     printf("\n\t\t\t\t\t\t-----\n");
     
102.     printf("\t\t\tCharge of this call: RM\t%.2f\n",total_call);//show the charge rate
     
103.     printf("\t\t\t\t\t\t=====\n");
     
104. 
     
105.     printf("\nNew Balance is RM %.2f\n",balance);
     
106. 
     
107.     printf("-----------------------------------------------\n");
     
108. }
     
109. 
     
110. void SMS_Function()
     
111. {
     
112.     printf(
     
113.         "\n==============================================="
     
114.         "\n\t\tSMS\n"
     
115.     );
     
116. 
     
117.     printf("\nYour balance is RM %.2f\n",balance);
     
118.     printf("\nSMS available : %.0f",(balance-1.90)/0.10);
     
119.     printf("\nHow many SMS do you want to send :");
     
120.     scanf("%d",&sms_num);
     
121.     fflush(stdin);
     
122. 
     
123.     sms_cost=sms_num*sms_rate;
     
124.     printf("SMS charge = RM %.2f\n",sms_cost);
     
125. 
     
126.     balance-=sms_cost;
     
127.     printf("\nNew Balance is RM%.2f\n",balance);
     
128.     printf("-----------------------------------------------\n");
     
129. 
     
130.     total_sms_charge=sms_cost+total_sms;
     
131.     total_sms+=sms_num;
     
132. }
     
133. 
     
134. void Topup_Function()
     
135. {
     
136.     printf(
     
137.         "\n==============================================="
     
138.         "\n\t\tTOP UP\n"
     
139.     );
     
140.     printf("\nYour balance is RM %6.2f\n",balance);
     
141. 
     
142.     printf("\nTop Up Amount : RM ");
     
143.     scanf("%d",&topup_amount);
     
144.     fflush(stdin);
     
145. 
     
146.     balance+=topup_amount;
     
147.     total_topup+=topup_amount;
     
148. 
     
149.     printf("\nNew Balance is RM%6.2f\n",balance);
     
150.     printf("-----------------------------------------------\n");
     
151. }
     
152. 
     
153. void Check_Balance()
     
154. {
     
155.     while(balance<2.00)
     
156.     {
     
157.         printf(
     
158.             "\n==============================================="
     
159.             "\n\t\tNOTICE\n"
     
160.         );
     
161.         printf("\nYour balance is RM %.2f\n",balance);
     
162.         printf(
     
163.             "You need at least RM 2.00 to make call or sms\n"
     
164.             "How much to top up? >>RM "
     
165.         );
     
166.         scanf("%d",&topup_amount);
     
167.         fflush(stdin);
     
168. 
     
169.         balance+=topup_amount;
     
170.         total_topup+=topup_amount;
     
171. 
     
172.         printf("\nNew Balance is RM%6.2f\n",balance);
     
173.         printf("-----------------------------------------------\n");
     
174.     }
     
175. }
     
176. 
     
177. void Summary()
     
178. {
     
179.     printf(
     
180.         "\n\n==============================================="
     
181.         "\n\t\tTRANSACTION SUMMARY\n"
     
182.     );
     
183. 
     
184.     printf("\nIn total, you \ttalk \t%d minutes\n",total_min);
     
185.     printf("\t\tsent \t%d SMS's\n", total_sms);
     
186.     printf("\nStarting Balance\t\t : RM %6.2f",PREPAID_BALANCE);
     
187.     printf("\nTotal Amount Topped UP \t\t : RM %6.2f",total_topup);
     
188.     printf("\nTotal Call Charges \t\t : RM %6.2f",total_call_charge);
     
189.     printf("\nTotal SMS Charges \t\t : RM %6.2f", total_sms_charge);
     
190.     printf("\nEnding Balance \t\t\t : RM %6.2f",balance);
     
191. 
     
192.     printf("\n-----------------------------------------------\n");
     
193.     printf("\n\n");
     
194. }
     

复制代码

我把code改成以上的会不会比用goto来的好？
应为我觉得这样子做跟用goto好象是一样的东西。。。
想请教一下，有哪些地方可以改进的？



附上flowchart

flowchart 我不是很会做，请问这样就可以了吗，是否要加其他东西？

谢谢

geekman

1）Top Up 和 Check Balance的功能重复了。C语言基本教科书都会这么说：“A function should do only what it is meant to do.” 一个Function只需做它应该做的事。Check Balance却跑去做Top Up的事情，过界了喂。

2）

1. case 1:
   
2.         case 2:
   
3.             if(balance>=2.00)
   
4.             {
   
5.                 if(decide==1)
   
6.                     Call_Function();
   
7.                 if(decide==2)
   
8.                     SMS_Function();
   
9.             }
   
10.             else Check_Balance();
    
11.             break;

复制代码

多余了。
如果是我会这样做：

1. case 1：Call_Function();
   
2.         break;
   
3. case 2: SMS_Function();
   
4.         break;
   
5. ...
   
6. void Call_Function()
   
7. {
   
8.    Check_Balance();
   
9.    
   
10.    ...// do whatever call function do
    
11. }
    
12. void SMS_Function()
    
13. {
    
14.     // same as above
    
15. }
    
16. void Check_Balance()
    
17. {
    
18.     if(balance <= 2)
    
19.         Top_Up();
    
20.     //else balance is good and nothing need to be done
    
21.     //or you may choose to print the balance here.
    
22. }
    

复制代码

应为我觉得这样子做跟用goto好象是一样的东西。。。

随便你，喜欢继续用GOTO就继续用吧，当你进入OOP的区域（C++）的时候，你就会知道滋味了。或者你可以考虑将Assembly当成你的终生以及唯一的编程语言，那个语言倒是不忌讳GOTO（其实C里面的GOTO就是用inline assembler来处理的。