关于looping的问题

invisible

回复 40# 可爱的医师小小

1. #include <iostream>
   
2. using namespace std;
   
3. /* Task C */
   
4. int main()
   
5. {
   
6.         char x;
   
7.         cout << "This program will determine is that the";
   
8.         cout <<" alphabet input is vowel.\n";
   
9.         do
   
10.         {
    
11.                 cout << "\nPlease enter an alphabet.\nEnter 'z'to exit.\n";
    
12.                 cin >> x;
    
13.                 if(  x == 'a' || x == 'e' || x == 'i' || x == 'o' || x == 'u'||
    
14.                          x == 'A' || x == 'E' || x == 'I' || x == 'O' || x == 'U')
    
15.                 {
    
16.                         cout << "Character is a VOWEL.\n";
    
17.                 }
    
18.                 else if(  x != 'a' || x != 'e' || x != 'i' || x != 'o' || x != 'u' ) // 多餘
    
19.                 {
    
20.                         cout << "Character entered is " << x <<"\n";
    
21.                 }
    
22.         
    
23.         }while ( x != 'z' && x != 'Z');
    
24.         return 0;
    
25. }
    
26.         
    
27.         
    

复制代码

沒什麼大問題 ，只是這個 if( x != 'a' || x != 'e' || x != 'i' || x != 'o' || x != 'u' ) 多餘的，因為你一開始已經判斷了，所以不需要這一行也可以。還有你的 do while, 你的while 可以放在bracket 後面嗎 =.= ？ 我看的時候還以為你多一個 while loop

可爱的医师小小

回复  可爱的医师小小



沒什麼大問題 ，只是這個 if( x != 'a' || x != 'e' || x != 'i' || x != 'o ...
invisible 发表于 31-10-2011 09:32 PM

好的，谢谢你！

geekman

你可以通过toupper（）或者tolower（）来减少你的if（）的evaluation次数：

1. 
   
2. x = toupper(x);
   
3. if(x == 'A' || x == 'E' || x == 'I' || x == 'O' || x == 'U')
   
4. {
   
5.     //do whatever
   
6. }
   

复制代码

或者

1. 
   
2. x = tolower(x);
   
3. if(x == 'a' || x == 'e' || x == 'i' || x == 'o' || x == 'u')
   
4. {
   
5.     //do whatever
   
6. }
   

复制代码

1. while ( x != 'z' && x != 'Z');

复制代码

你确定是 && ？如果你的意图是让使用者按 Z 键来结束while loop，那你得到的结果将是完全相反的。不管使用者按什么键都会break loop的，因为你的条件是x必须同时不是大写和小写的 Z 才会继续 （反过来说，就是你的条件是x必须同时为大写和小写的 Z，或者 x 是 Z，z 以外的任何字母/符号都会停止循环）。

还有，用switch会更简洁，因为你不必evaluate else if（）：

1. x = tolower(x);
   
2. switch(x)
   
3. {
   
4.     case 'a':
   
5.     case 'e':
   
6.     case 'i':
   
7.     case 'o':
   
8.     case 'u': //it's a vowel
   
9.             break;
   
10.     default: //it isn't
    
11. }

复制代码

可爱的医师小小

你可以通过toupper（）或者tolower（）来减少你的if（）的evaluation次数：或者你确定是 && ？如果你的意图 ...
geekman 发表于 31-10-2011 11:18 PM

又学到东西了，谢谢大大

可爱的医师小小

你可以通过toupper（）或者tolower（）来减少你的if（）的evaluation次数：或者你确定是 && ？如果你的意图 ...
geekman 发表于 31-10-2011 11:18 PM

是哦，我测试了，行得通哦

可爱的医师小小

你可以通过toupper（）或者tolower（）来减少你的if（）的evaluation次数：或者你确定是 && ？如果你的意图 ...
geekman 发表于 31-10-2011 11:18 PM

其实那个&&的是我从我老师的例子参考来的，不是很明白。

1. while ( x != 'Z' || x != 'z');

复制代码

1. while ( x == 'z' && x == 'Z');

复制代码

如果是这样的话就行不通……

geekman

while(condition1 && condition2)
=> while condition1 AND condition2 is true, loop goes on，也就是说两个条件都必须是true才会继续

while(condition1 || condition2)
=> while condition1 OR condition2 is true, loop goes on，也就是说任何一个条件是true都会继续。

其实最好的做法就是我给你的方案：用tolower（）或者toupper（），然后只需要evaluate一个condition:

1. 
   
2. x = tolower(x);
   
3. do
   
4. {
   
5.     //whatever
   
6. }(while x != 'z');
   

复制代码

简单明了。

geekman

回复 46# 可爱的医师小小

行不通的原因是x不可能同时是大写和小写的 Z

on9-shop

你要Return出来嘛~，哈哈~