[已解決]visual c++(陣列檢查重複輸入數字)

fk035372150

題目是要我們輸入20個範圍介於10~100數字的陣列，(但重複的或是超出範圍的要顯示警告訊息並排除)
以下是我的程式碼，檢查重複的地方怪怪的(1.檢查會出錯2.無法跳出while迴圈)
是我用錯方法嗎?還是變數範圍有錯?麻煩高人指點謝謝~
#include <iostream>
using namespace std;

int main()
...

瀏覽完整內容，請先 註冊 或 登入會員

a221962

變數很混亂
另外建議先判定是否有效的輸入，再 insert 到 input[]

fk035372150

a221962 發表於 2013-11-2 09:35 PM

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

變數很混亂
另外建議先判定是否有效的輸入，再 insert 到 input[]

if(input<10 || input>100){
                cout << "Invalid number." << endl;
...

瀏覽完整內容，請先 註冊 或 登入會員

ifiwant

抱歉我手邊只有Dev C++，我參照你所給定的程式做編譯會錯誤，因此做了部份修改(將input修改成input)，另外我將版主您所提供的程式，小幅度的修改一下(註解的部份)。

麻煩版主指教，與交流一下。

1. #include <iostream>
   
2. using namespace std;
   
3. 
   
4. int main()
   
5. {
   
6.   int input[20];
   
7.   int i,j = 1,num = 20;
   
8.   cout << "Enter 20 integers between 10 and 100:" << endl;
   
9.   for(i = 0;i < num;i++){
   
10.     cin >> input[i];
    
11.     if (input[i] < 10 || input[i] > 100) {
    
12.       cout << "Invalid number." << endl;
    
13.       --i;
    
14.       continue; // 當輸入不正確的數字範圍，應該終止往下執行程式
    
15.     }
    
16.     j = 0; // j的初始值應該放在迴圈內，每輸入一個數字，應該重頭開始檢查數字是否已再已輸入的數字內(除了第一次輸入)
    
17.     while(j < i){      
    
18.       if (input[j] == input[i]) { // j應該從0開始到i-1,因為i的值為目前輸入數字的個數
    
19.         cout << "Duplicate number." << endl;
    
20.         --i; // 只有當重覆輸入數字，才需要將i做減1的動作
    
21.         break; // 找到重複的數字後，不再繼續比對後續未比對的數字
    
22.       }
    
23.       j++;               
    
24.     }
    
25.   }
    
26.   cout << "The nonduplicate values are: ";
    
27.   for (i = 0; i < num; i++) {
    
28.     cout <<"  " << input[i];
    
29.   }
    
30.   cout << endl;
    
31.   system("pause");
    
32. return 0;
    
33. }

複製代碼

...

瀏覽完整內容，請先 註冊 或 登入會員

fk035372150

ifiwant 發表於 2013-11-2 10:15 PM [url=forum.php?mod=redirect&goto=findpost&pid=250489824&ptid=9165744][/url]
抱歉我手邊只有Dev C++，我參照你所給定的程式做編譯會錯誤，因此做了部份修改(將input修改成input)，另外 ...

不好意思，剛剛編譯您的程式出現如下錯誤
...

瀏覽完整內容，請先 註冊 或 登入會員

ifiwant

fk035372150 發表於 2013-11-2 10:54 PM

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

不好意思，剛剛編譯您的程式出現如下錯誤
檢查重複只會檢查到前一個輸入的數字並不會檢查整個陣列欸
可能 ...

...

瀏覽完整內容，請先 註冊 或 登入會員

EdisonX

另一個解法，一開始就開一個陣列，大小是 10~100，輸入過的數字就填 1，填入前做確認即可。

1. #include <iostream>
   
2. using namespace std;
   
3. 
   
4. int main()
   
5. {
   
6.     const int LOWER = 10;
   
7.     const int UPPER  = 100;
   
8.     const int ARY_SIZE  = UPPER - LOWER+ 1;
   
9.     const int COUNT = 20;
   
10. 
    
11.    int  NumFlag[ARY_SIZE] = {0};
    
12.    int  input , tIdx;
    
13.    for( idx = 0 ; idx < COUNT ;  ) {
    
14.         cin >> input;
    
15.         if(input < LOWER || input > UPPER) continue; // input error
    
16. 
    
17.         tIdx = input - LOWER;
    
18.         if ( NumFlag[tIdx  ] == 0) { // 沒輸入過
    
19.                 NumFlag[tIdx] = 1;
    
20.                 idx++ ;  // 沒輸入過才把計數器 + 1
    
21.                 cout << " Dump number " << endl;
    
22.         } else {
    
23.                  // 有輸入過就不處理了
    
24.         }
    
25.    }
    
26. 
    
27.    // 其他從頭到尾找輸入過哪些數字就略了
    
28. 
    
29.    return 0;
    
30. }

複製代碼

...

瀏覽完整內容，請先 註冊 或 登入會員

chenvey2

我照你的code修改，應該比較好懂

1. #include <iostream>
   
2. using namespace std;
   
3. 
   
4. int main()
   
5. {
   
6.   int input[20];
   
7.   int i,j = 1,num = 20;
   
8.   cout << "Enter 20 integers between 10 and 100:" << endl;
   
9.   for(i = 0;i < num;i++){
   
10.     cin >> input[i]; // cin要存入整數, input本身是陣列, 要給index
    
11.     if(input[i]<10 || input[i]>100){ // 同上
    
12.       cout << "Invalid number." << endl;
    
13.       --i;
    
14.     }
    
15.     j = 0; // j每次迴圈必須歸零,重頭開始檢查重複的數字
    
16.     while(j<i){ // 檢查index i之前的數字, i本身不用檢查
    
17.       if(input[j] == input[i]) { // 不用j-1
    
18.         cout << "Duplicate number." << endl;
    
19.         --i; // 如果重複, 才把i減一
    
20.       }
    
21.       j++;
    
22.     }
    
23.   }
    
24.   // input是陣列, 必須用迴圈印出所有的值
    
25.   cout << "The nonduplicate values are: ";
    
26.   for(i = 0; i < num; ++i) cout << input[i] << ",";
    
27.   cout << endl;
    
28.   system("pause");
    
29.   return 0;
    
30. }

複製代碼

...

瀏覽完整內容，請先 註冊 或 登入會員

fk035372150

EdisonX 發表於 2013-11-2 11:42 PM

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

另一個解法，一開始就開一個陣列，大小是 10~100，輸入過的數字就填 1，填入前做確認即可。 ...

...

瀏覽完整內容，請先 註冊 或 登入會員

fk035372150

chenvey2 發表於 2013-11-3 12:26 AM

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

我照你的code修改，應該比較好懂

已經解決問題並了解我錯在哪了，感謝您的指教~~^ ^

a221962

1. #include <vector>
   
2. #include <iostream>
   
3. #include <algorithm>
   
4. 
   
5. int main()
   
6. {
   
7.     const int COUNT = 20;
   
8.     const int LOWER = 10;
   
9.     const int UPPER  = 100;
   
10.     int input,index = 0;
    
11.         std::vector<int> numbers;
    
12. 
    
13.     std::cout << "Enter 20 integers between 10 and 100:" << std::endl;
    
14.         while (index < COUNT)
    
15.         {
    
16.                 std::cin >> input;
    
17.         if(input < LOWER || input > UPPER)
    
18.                 std::cout << "Invalid number." << std::endl;
    
19.                 else
    
20.                 {
    
21.                         auto exist = std::find(numbers.begin(),numbers.end(),input);
    
22.                         if(exist != numbers.end())
    
23.                                 std::cout << "Duplicate number." << std::endl;
    
24.                         else
    
25.                         {
    
26.                                 numbers.push_back(input);
    
27.                                 index ++;
    
28.                         }       
    
29.                 }
    
30.         }
    
31. 
    
32.     std::cout << "The nonduplicate values are: " << std::endl;
    
33.         for (int i : numbers)
    
34.         { // c++11 range-based for loop  
    
35.                 std::cout << i << std::endl;
    
36.     }
    
37. 
    
38.     return 0;
    
39. }

複製代碼

...

瀏覽完整內容，請先 註冊 或 登入會員

fk035372150

a221962 發表於 2013-11-3 04:19 AM

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

您的程式確定無誤，不過vector我們要過幾天才會教到，所以有些地方看不太懂!
...

瀏覽完整內容，請先 註冊 或 登入會員

fk035372150

ifiwant 發表於 2013-11-2 11:29 PM

下載: 訪客無法瀏覽下載點，請先 註冊 或 登入會員

不好意思~我遵照您所輸入的數字，並不會產生您圖片上的錯誤耶@@!!
這之間一定有誤會，目前我想到的就只有 ...

...

瀏覽完整內容，請先 註冊 或 登入會員