条件が3つある条件分岐がうまくいきません

Question

ど素人が質問させていただきます。
実行しても最後のプレミアム席がなぜか出てきません。
どのような書き方をすれば出るのかを教えていただきたく思います。

let age = 18;
let ticket = "あり";
let fanclubticket = "あり";

if (ticket === "あり") {
if (age >= 18) {
console.log("通常席");
} else {
console.log("帰れ");
}
} else if (age >= 18) {
if (ticket === "あり") {
console.log("通常席");
} else {
console.log("帰れ");
}
} else if (age === 18) {
if (ticket === "あり") {
if (fanclubticket === "あり") {
console.log("プレミアム席");
}
}
}

Comments

[m.ts10806]

コードはマークダウンのcode機能を利用してご提示ください。

[satooo]

マークダウンのcode機能すらわからずごめんなさい。調べてきます。

Answer 1

下記のようにしてみればどこを通っているか分かるわけです。

let age = 18;
let ticket = "あり";
let fanclubticket = "あり";

console.log(1);
if (ticket === "あり") {
	if (age >= 18) {
console.log(2);
		console.log("通常席");
	} else {
console.log(3);
		console.log("帰れ");
	}
} else if (age >= 18) {
	if (ticket === "あり") {
console.log(4);
		console.log("通常席");
	} else {
console.log(5);
		console.log("帰れ");
	}
} else if (age === 18) {
	if (ticket === "あり") {
console.log(6);
		if (fanclubticket === "あり") {
console.log(7);
			console.log("プレミアム席");
		}
	}
}


/**
出力結果：
1
2
通常席

**/

ifって「条件に一致した」らもうそこしか入りません。
幾らelseがあっても既に条件一致してればあとの条件は見られません。

なので一発目の「if (ticket === "あり") {」が成り立った時点で
2か3しか選択肢がなく、if (age >= 18) {が成り立つので、そこで終わります。

プレミアム席を出力したいのでしたらこのロジックは出来ません。
fanclubticketの分岐が生命線なのであれば、そこから最初のifを始めれば良いのでは？

ネストせずに&&でつなげれば次のifにいけると思います。
あと条件はもうちょっとまとめても良いのかなと。

下記なんて「チケットあって１８歳以上なら通常席それ以外は”帰れ”」を入れ替えてるだけで同じじゃないですか？

if (ticket === "あり") {
	if (age >= 18) {
		console.log("通常席");
	} else {
		console.log("帰れ");
	}
} else if (age >= 18) {
	if (ticket === "あり") {
		console.log("通常席");
	} else {
		console.log("帰れ");
	}
}

チケット持ってて１８歳未満の場合はどうするんでしょう。条件に入ってない気がします。
「帰れ」でいいなら下記で済むかと。

例：

if (age === 18 && ticket === "あり" && fanclubticket === "あり"){
    console.log("プレミアム席");
}else if(ticket === "あり" && age >= 18){
    console.log("通常席");
}else{
    console.log("帰れ");
}

Comments

[satooo]

これです。これが知りたかったんです。
マークダウンのcodeすらよくわかってなかったに色々と汲んでいただきありがとうございます。

[m.ts10806]

質問は編集できますのでせめてコード部分だけでも今からでも対応しておいてください。
https://teratail.com/questions/238564

Answer 2

内容としてはvnsa7221さん、m.ts10806さんの内容で満たされていますが、「ガード節」という書き方をご紹介します。
分岐する条件が複数あるときに、ifの中にifを書いて……というのを繰り返すと、書くのも読むのも大変なので、特殊な条件（エラーなど）の場合はさっさと return (あるいは throw) しようというものです。

ガード節を意識して、かつあなたのコードの不足（と思われる部分）を修正すると、こんな感じになるかと思います。
「（と思われる部分）」とわざわざ書いたのは、あなたのコードの意図が100%私に伝わらなかったためです。人に見せるコードにはなるべくコメントを入れましょう。
自分だけが見るコードだとしても、数週間方っておいた後から見ると当時の自分が何をやりたかったのか分からんというのはプログラマあるあるです。

function checkYourSheet(age, ticket, fanclubticket) {
  // チケットを持っていない者には鉄槌を下して処理を中断する
  if (ticket !== "あり") return "帰れ";

  // 18歳に満たない者には鉄槌を下して処理を中断する
  if (age < 18) return "帰れ";

  // ちょうど18歳で、ファンクラブチケットを持っている者にはプレミアム席を案内する
  if (age === 18 && fanclubticket === "あり") return "プレミアム席";

  // いずれの条件も満たさない者には通常席を案内する
  return "通常席";
}

console.log(
  'チケットとファンクラブチケットを持つ18歳',
  checkYourSheet(18, "あり", "あり")
);
// -> チケットとファンクラブチケットを持つ18歳 プレミアム席

console.log(
  'チケットとファンクラブチケットを持つ17歳',
  checkYourSheet(17, "あり", "あり")
);
// -> チケットとファンクラブチケットを持つ17歳 帰れ

console.log(
  'チケットはないがファンクラブチケットを持つ19歳',
  checkYourSheet(19, "なし", "あり")
);
// -> チケットはないがファンクラブチケットを持つ19歳 帰れ

console.log(
  '何も持たぬ42歳',
  checkYourSheet(42, "なし", "なし")
);
// -> 何も持たぬ42歳 帰れ

Answer 3

else if (age >= 18)

この部分はageが18以上（18, 19, 20...）の場合は条件を満たします。

仮にこの部分を条件を満たさなかった場合

(age === 18)

となっていますが、ここはageが18の時のみ条件を満たします。

すでに上の条件式でageが18の場合は条件を満たすのでこのままではどう頑張ってもプレミアム席まではたどり着かないです。

else if (age > 18)

このようにすればageが18より大きい（19, 20...）場合条件を満たすようになるのでageが18では条件を満たさず、次の条件式まで到達することができます。

Comments

[m.ts10806]

そもそも ticket === "あり" が満たされてそれで終わってます。
ageの条件幾ら変えたところでプレミアムにはたどり着かないのでは？

[vnsa7221]

第一条件はまずそこですね。
自分の勝手な早とちりで変数部分の値をどのように変えてもたどり着かないのはなぜかと質問いただいているのかなと思ってしまいました。

なのでlet ticket = "なし"に変更したとしてもそのあとの処理でageをどのように変えても2番目の条件式を満たさず、3番目の条件式を満たすようにすることができないので、そちらを変えないといけないですよと回答させていただきました。
ちょっと不親切だったかもしれないですね。

[m.ts10806]

もともとの要件が不明瞭なのもありますね。