C++代碼不工作...爲什麼？

Question

set<int> numbers; 
char userLetter; 
int userNumber; 

cout << "Please enter a letter: "; 
cin >> userLetter; 

while(userLetter == 'X') 
{ 
    cout << "Please enter a number: "; 
    cin >> userNumber; 

    if (numbers.find(userNumber) != numbers.begin()) 
    { 
      cout << "Invalid!"; 
      continue; 
    } 
    else 
    { 
      numbers.insert(userNumber); 
    } 

    if(userNumber == 2) 
      cout << "Yes :)"; 
    else 
      cout << "No :("; 

    cout << "Please enter a letter: "; 
    cin >> userLetter; 
} 

這是當我運行該程序會發生什麼：

Please enter a letter: X 
Please enter a number: 2 
Yes :) 
Please enter a letter: X 
Please enter a number: 2 
Yes :) 
Please enter a letter: X 
Please enter a number: 2 
Yes :) 
... 

當然，「X」和「2」的東西，我進入。當我輸入第二個2時，不應該輸出程序「無效」？

任何幫助？

Answer 1

改變這種

(numbers.find(userNumber) != numbers.begin()) 

這個

(numbers.find(userNumber) != numbers.end()) 

爲find迭代器將返回到結束的時候它沒有找到你在找什麼

Answer 2

（假設C++）

numbers.find(userNumber) != numbers.begin()檢查是否userNumber是該集合中的第一個元素，而不是它是否不存在。你想要numbers.find(userNumber) != numbers.end()。

或者，更好的，你可能想

if (!numbers.insert(userNumber).second) 
{ 
    cout << "Invalid!"; 
    continue; 
} 

因爲集::插入返回一個pair第二個元素是插入是否成功 - 見here