私のC++コードで何が問題になっていますか？私はプログラミングの初心者だ間違った結果

Question

を取得しておくと、私はキロメートルとマイルコンバータアプリケーションを作成しようとしています。コードにコードブロックを使用しています。選択肢2は結果として0を与え続けます。ここで私が入力しています正確なコードです：

#include <iostream> 
using namespace std; 
double choice, value1, result; 
// I'm sure I messed up somewhere after this: 
double value2 = .621371; 
double value3 = 1.609344; 
// Are these two lines^supposed to be here? 
int main() 
{ 
    while (true) { 
     cout << "1. Kilometers to Miles" << endl; 
     cout << "2. Miles to Kilometers" << endl; 
     cout << "3. Exit the Application" << endl; 
     cout << "Please enter a choice: "; 
     cin >> choice; 
     if (choice == 3) 
      break; 
     cout << "Please enter the first value: "; 
     cin >> value1; 
     // This if statement keeps giving me 0: 
     if (choice == 2) 
      result = value1 * value2; 
     // I believe this part here is okay: 
     else if (choice == 3) 
      result = value1/value3; 
     cout << "The result is: " << result << endl; 
    } 
} 

Answer 1

私は今、問題を修正しました。もし誰かが私に下にコメントすることができます、これは私の元のコードよりもはるかに良いですか？基本的に、この新しいコードは皆さんにもっと読みやすく、組織化されていますか？私はこの更新されたコードについてy'allの意見を得るのが大好きです。

#include <iostream> 
float choice, value1, result; 

int main(){ 
    while (true){ 
    std::cout << "Please enter a choice: " << std::endl << 
       "1. Kilometers to Miles" << std::endl << 
       "2. Miles to Kilometers" << std::endl << 
       "3. Exit the Application" << std::endl; 
    std::cin >> choice; 
    if(choice == 3){ 
     break; 
    } 
    std::cout << "Enter the Value you would Like to Convert: "; 
    std::cin >> value1; 
    if (choice == 1){ 
     result = value1 * 0.62137; 
    } 
    else if (choice == 2){ 
     result = value1 * 1.609344; 
    } 
    std::cout << "The result is: " << result << std::endl; 
    } 
} 

Answer 2

はここにあなたのコードを修正する方法は次のとおりです。

1.Indentは適切に（あなたのコードをより読みやすくなります）
2. これは、あなたが動作しないコード作っていますあなたのコードも非常に多くの不必要である：（選択== 3）{ ブレーク場合
。 }文はそれらの後、これらの中括弧を持っている必要がありますあなたのif/else/else ifの
3. ひとつひとつの1：

if(whatever=whatever) { 
    //whatever 
} 

4.この単純なプログラムのためにこれらのbreakステートメントを使用して忘れます。
5.「キロメートルからマイル」を逃した。
6. 終了オプションがめちゃくちゃにされ、

else if(choice == 3) { 
    return 0; // To end the program 
} 

7. ないdoublechoice変数を行い、から

else if(choice == 3) 
result = value1/value3 

を変更することは非常に、非常に、多くを使用しています、はるかに多くのデータ単純なint値（及びどのユーザが3.1（または4.838389、ECT）入力とを入力した場合？
8.あなたは（あなたが3.3マイル/キロメートルをしたい、3.37374662232365マイル/キロメートルを望んでいない。）、あなたの結果以下の精度を確認する必要がありますが、宣言するもの「結果の前にも#include <iomanip>を追加し、setprecision()に追加していることを行うことができます「ある（すなわち：result = setprecision(1) value1 * value2;）あなたは、各STD関数の前に:: STDを書き、あるいは単にusing std::whatever

を書く必要がありますので全体std名前空間を含む
9は、（それは多くをあなたのプログラムが遅くなります）、この場合には不適切です