if文の問題を解いたので答え合わせしてほしい。

Question

独学でC#学習しています。

#質問内容
if文についてある問題集があり回答例と自分の回答が少し違うのですが、
自分の回答について修正点あれば教えてください。
回答例を見て、自分の回答の方が正しいと思ってしまいました。
bool式はif文にまとめ、最初の&&で第一オペランドしか評価しないので、
問題中の"ありえない値"も評価対象に入ってしまうため&で良いと考えました。
ちなみに実行結果は同じで、おめでとうございます。合格です。でした。
よろしくお願いします。

#問題内容

テストがあります。足きり点が65点です。満点が100点です。下限点が0点です。

足きり点以上、満点以下の場合、合格です。
コンソール画面に『おめでとうございます。合格です。』と表示します。
下限点以上、足きり点未満なら、不合格です。
コンソール画面に『残念です。不合格です。』と表示します。
それ以外の場合、ありえない値なので、『試験の得点が正しく入力されていません。』と表示します。
今回の受験者の実際の得点は75点でした。

#回答内容
自分の回答

using System;

namespace Seigyo
{
    class Control
    {
        static void Main(string[] args)
        {
            int realScore= 75;
            int maxScore = 100;
            int minScore = 0;
            int passingScore = 65;


            if (maxScore >= realScore & passingScore <= realScore)
                Console.WriteLine("おめでとうございます。合格です。");

            else if (minScore >= realScore & passingScore >= realScore)
                Console.WriteLine("残念です。不合格です。");

            else
                Console.WriteLine("試験の得点が正しく入力されていません。");
        }
    }
}

回答例

using System;
 
namespace ConditionalFlowExercise
{
    class MainClass
    {
        static void Main(string[] args)
        {
            var realScore = 75;
            bool examineeHasPassed = (realScore >= 65 && realScore <= 100);
            bool examineeHasFailed = (realScore >= 0 && realScore < 65);
 
            if (examineeHasPassed)
            {
                Console.WriteLine("おめでとうございます。合格です。");
            }
            else if (examineeHasFailed)
            {
                Console.WriteLine("残念です。不合格です。");
            }
            else
            {
                Console.WriteLine("試験の得点が正しく入力されていません。");
            }
        }
    }
}

Answer 1

テストすべき値は、

• 境界の値: 0, 65, 100
• 境界間の値: 30, 75 （任意の値）
• 範囲外の値: -10, 120 （任意の値）

この程度なので、自分で答え合わせして下さい。

Comments

[hillclub]

境界間の値を注視すべきなんですね、演算子使えばいいみたいなので今思うとたしかにこの程度でした。有難うございます！

Answer 2

オレするなら、

if (maxScore < realScore || minScore > realScore){
// ありえない
}else if(passingScore <= realScore){
// 合格
}else{
// 不合格
}

とゆーふーに

Comments

[hillclub]

なるほどこれでも良いわけですね。シンプルで好きなコードです！有難うございます。

[Zuishin]

「これでも良い」ではなく「この方が良い」です。早期リターンとガード節を調べてください。

[hillclub]

ガード節、早期リターン調べ納得しました。方法として確立されていたのですね。
今後見直しを必ずするようにして慣れていこうと思います。有難うございます。

Answer 3

(minScore >= realScore & passingScore >= realScore)

不合格にならなくなってます。

最初の&&で第一オペランドしか評価しないので、

問題中の"ありえない値"も評価対象に入ってしまうため&で良いと考えました。

「"ありえない値"も評価対象に入ってしまう」というのがよくわかりませんが
今回の場合は短絡した方がどちらかというとよいでしょう。
もちろん前提条件として足きり点が満点以下、下限点以上である事は事前に確認が必要ですが

---

&と&&について

状態変化を含む式をオペランドとして渡さない限りboolの&と&&は同値になります。

Comments

[hillclub]

最後のelseに"試験の得点が正しく入力されていません。"
とあるので125点等、入力ミス？する事が想定されるのかと思いました。

それより不合格にならなくなっていた事を教えて頂き有難うございます。
もう一度書き直そうと思います。

[asm]

足きり点が満点以下、下限点以上である場合
足きり点以上ならば必然的に下限点以上です
足きり点以下ならば必然的に満点以下です

Answer 4

先にありえない値を弾く事で、else ifの条件式を省く事ができました。
また、minScore=0;等の変動しない値は定義しなくても良さそうなので直入力しました。
65，55，125それぞれの境界の値で正常動作しました。

    class AndAnd1
    {
        static void Main(string[] args)
        {
            int realScore = 75;


            if (realScore > 100 || realScore < 0)
                Console.WriteLine("試験の得点が正しく入力されていません。");

            else if (realScore >= 65)
                Console.WriteLine("おめでとうございます。合格です。");

            else
                Console.WriteLine("残念です。不合格です。");
        }
    }