2つの連続した出力が同じ場合にループが壊れます

Question

#include <iostream> 

int main() 
{ 
    float x, y; 
    while (true) 
    { 
    cin >> x; 
    if (x == y) 
     break; 
    y = x; 
    } 
    return 0; 
} 

これは入力された2つの連続する数字が同じである場合に破損するはずのループです。私はいくつかの助けを感謝したい2つの質問があります。

まず、これはタスクを解決する最も効率的な方法のようですか？

第2に、xとyに宣言された値が割り当てられていないのですか（float x=?, y=?）？具体的には、最初にループを入力したときに、x（x == y）と比較するために、yに何らかの値を設定しないでください。

Answer 1

1. 「効率的」とは何ですか？ 「実行時にできるだけ少ないCPUを使用する」という意味では最も効率的ではありませんが、プログラムがユーザーの入力を待っている時間の99.999％を費やしていれば、それは問題になりません。

2. いいえ、そうです。 yが初期化されていないという問題があります（一般に、初期化されていない変数はコードのエラーです）。それを考慮に入れてロジックを再構成する必要があります。

あなたは自分のコードが短い、または単純になりたい場合は、あなたのような何かを行うことができます：何をやって

#include <iostream> 

int main() 
{ 
    float x, y; 
    std::cin >> y; 
    while (std::cin >> x && x != y) { 
     y = x; 
    } 
} 

Answer 2

を（初期値の欠如を除く）正常に見えます。同じ結果を達成する方法はたくさんありますが、スタックオーバーフローはディスカッションサイトではありません。初期値については、そうです、いくつかあるべきです。あなたのコードの簡単な書き換え：

int main() 
{ 
    float x=1; 
    float y=2; 
    bool firstLoop=true; // it is possible that the user first inputs the number 2. To avoid the loop ending before the user inputs the first 2 numbers we use this flag 
    while (true) 
    { 
      cin >> x; 

      if (x == y && !firstLoop) 
       break; 
      y = x; 
      firstLoop=false; 
    } 
    return 0; 
}  

Answer 3

他の方法は次のようになります。

int main() 
{ 
    float x=0; 
    cin >> x; 
    cin >> y; 
    while(x!=y){ 
     x=y; //x will always be the previously entered number 
     cin >> y; //y will always be the newly entered number 


    } 
    return 0; 
} 

EDIT：答えで小さなミスがあった、それはY、