私のループがループしない理由を知らない

Question

私はプログラミング時に新しく、サウナの温度を華氏から摂氏に変換して熱を上げるべきかどうかをユーザーに知らせるプログラムに取り組んでいます元の入力番号に応じて下に移動します。私はコードの大部分をやったことがありますが、今は73度以下または77度以上の数字を書くときにループしない理由がわかりません。私の目が見つからないような問題は誰にも見えますか？

using System; 

namespace BastunKP 
{ 
    class Program 
    { 
     public static int FahrToCels(int fahr) 
     { 
      int tempCels = (fahr - 32) * 5/9; 
      return tempCels; 
     } 

     public static void Main(string[] args) 
     { 
      Console.WriteLine("Skriv in Fahrenheit: "); 
      int fahr = int.Parse(Console.ReadLine()); 
      int tempCels = FahrToCels(fahr); 

      do 
      { 
       if (tempCels < 73) 
       { 
        Console.WriteLine("Temperaturen är för kallt, skruva upp lite!"); 
       } 
       else if (tempCels > 77) 
       { 
        Console.WriteLine("Temperaturen är för varmt, skruva ner lite!"); 
       } 
       else 
       { 
        Console.WriteLine("Temperaturen är nu lagom, hoppa in!"); 
        return; 
       } 

       fahr = int.Parse(Console.ReadLine()); 
       tempCels = FahrToCels(fahr); 
      }   
      while (tempCels < 73 && tempCels > 77); 
     } 
    } 
} 

私はまた、教師が高いグレードのために、私は摂氏華氏に変換するための場所を数式に見て、それは、二重にするべきであると述べているが、私はこれを行う方法を知らない私の割り当てについて質問がありますまったく変わりません。事前に

おかげ

Answer 1

ようこそ：あなたは論理的||演算子ではなく、論理的&&演算子を使用する必要があります！さて、あなたの質問にお答えしましょう：

最初に、あなたのdo-whileループを考えてみましょう。

do { 
    if (tempCels < 73) { 
     // Temperature too high 
     Console.WriteLine("Temperaturen är för kallt, skruva upp lite!"); 
    } else if (tempCels > 77) { 
     // Temperature too low 
     Console.WriteLine("Temperaturen är för varmt, skruva ner lite!"); 
    } else { 
     // Temperature just right, hop in! 
     Console.WriteLine("Temperaturen är nu lagom, hoppa in!"); 
     return; 
    } 

    fahr = int.Parse(Console.ReadLine()); 
    tempCels = FahrToCels(fahr); 

} 
while (tempCels < 73 || tempCels > 77); 

ご覧のとおり、不要なelse条件を削除しました。現時点で何が起こるかは、すべての条件がチェックされていることです（temp < 73、temp > 77、および73 < temp < 77）。

1つの間違いは、他の回答でも指摘されていますが、|| (OR)の代わりに&& (AND)だったということです。そしてもちろん、値が今73の下および77 :)

上の両方にすることはできません、私は/また、いくつかのスタイリングを指摘して、私はあなたが「修正」すべきだと思う一般的なものが欲しい：

1）あなたの一時的な変換方法には、不要な変数の作成と割り当てが含まれています。

public static int fahrToCels(int fahr) { 
     // It returns just the same, without needing to create a new, 
     // temporary temperature variable! 
     return (fahr - 32) * 5/9; 
    } 

2）これは議論の余地があるかもしれないが、一般的な命名規則は、その関数名はキャメルケースで書かれていると言う：あなたは、それはこのように、それなしで全く同じように動作させることができます。

3）このシナリオでは特に問題はありませんが、アプリケーションをスケールアップ（または、より大きなものを操作）すると問題になることがあります。 もっと説明的な名前を付けるのが最善です（大きなプロジェクトでは混乱するかもしれません）。繰り返しますが、これは大きな問題ではなく、将来のために考慮すべきものです。

P.S.私は私の例で変数名を変更していませんでしたが、あなたが示したコードにもっと読みやすく/関連性を持たせるためです。

EDIT：要求ごととして

、ここdouble型として値を維持する方法です。

// Notice the return type and the property types are both double. 
    public static double fahrToCels(double fahr) { 
     return (fahr - 32) * 5/9; 
    } 

この方法では、値は整数のみでなければならない、と分裂の奇妙な結果を生成しない - 彼らはあまりにもdouble型のものとすることができます！

ここで、double型の変数を関数に渡す必要があります。そうしないと、型エラーが発生します。

ヒント：

double fahr = int.Parse(Console.ReadLine()); 

は（のような、言う、17.7）ユーザーが非整数値を渡すことができますし、それが適切に保存されます。

ヒント＃2：あなたは本当にフライ変換に行いたい場合は

、あなたはこのようにこれを達成することができます（例えば、値）：

int ourInteger = 4; 
double ourNewDoubleNumber = (double)ourInteger/23; 

あなたは種類や型のキャストについての詳細を読むことができますここに：Types and Type Casting

Answer 2

tempCels < 73 && tempCels > 77は真なることはありません！

は、おそらくあなたは、一時は77より73未満以上であるが、誰が知っているときに実行するように||を望んでいました。

Answer 3

while条件が真であるときにループしますが、tempCelsは同時に<73と>77にはなりません！その状態を修正すれば正常に動作します。

Answer 4

tempCels（またはその任意の数）は、同時に73より小さく77より大きくすることはできません。 StackOverflowのに

do { 
    // code 
} while (tempCels < 73 || tempCels > 77); 
// Here ---------------^