思う通りの条件分岐にならない

Question

初心者です。思うように条件分岐できません。
firstTap で button9num　かつ　secondTap で　button1num になれば正解
または
firstTap で button1num　かつ　secondTap で　button9num になれば正解
としたくて以下のようにしました。
この状態だと正解になるのですが
firstTap = button1num
secondTap = button9numにすると
残念になってしまいます。
ご指導よろしくお願いいたします。

firstTap = button9num
secondTap = button1num

if ((firstTap == button1num && firstTap == button9num) || (firstTap == button9num && secondTap == button1num)){
    print("正解")
}else{
    print("残念")
}

Comments

[fuzzball]

>>残念になってしまいます

firstTap = button1num
secondTap = button9num
であっても、
button1num と button9num が等しければ「正解」になります。

Answer 1

if (firstTap, secondTap) == (button1num, button9num) 
　　|| (firstTap, secondTap) == (button9num, button1num) {
  print("正解")
} else {
  print("残念")
}

こうとか

switch (firstTap, secondTap) {
case (button1num, button9num),
     (button9num, button1num):
  print("正解")
default:
  print("残念")
}

こうとか

すると読みやすい＆＆間違いにくいかなぁ

Comments

[vitoyaokatsu]

たいへん勉強になりました。恥ずかしいミスでしたが、工夫次第でミスを減らせることがわかりました。caseはあまり馴染めていなかったので、勉強していきたいと思いました。ありがとうございました。

Answer 2

ifの()の中、firstTap が3回出てきてsecondTap が1回出てきますね、おかしいですね

Comments

[vitoyaokatsu]

自分自身に残念と言われているようで恥ずかしいです。優しいご指摘ありがとうございました。

Answer 3

                                 ↓
if ((firstTap == button1num && firstTap == button9num) || (firstTap == button9num && secondTap == button1num)){

Comments

[vitoyaokatsu]

ありがとうございました。単純なミスにも気づかない自分が情けなかったです。ありがとうございました。