microsoft / microsoft/TypeScript

multiple cases with the same value are allowed in switch/case

オープン
#53,604 コメント 0 件 リアクション 1 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

Awaiting More Feedback Suggestion
主要言語
Go
スター
111k
フォーク
14.4k
平均マージ
1日 19時間
マージ済み PR(30日)
117

説明

Bug Report

🔎 Search Terms

switch / case
same values on multiple cases

🕗 Version & Regression Information

4.9.4

Playground:
https://www.typescriptlang.org/play?#code/FAMwrgdgxgLglgewgAhgUwM4wBTIG4CGANmGgFzJYBOcEA5sgJTIDewyHlA7nDFABa5CJNE1bAAkBKgEMogOQF5FNlOlIMCImgB0RBHVwAiGXOQBGI4wDckqQCMqaAgGtbUgL7A7phfeXialAaWrr6hsgmsqIATFbuDk6uCV52APRplPwIYEQAJsgQCDDI9qLE+lxoeT7RyIoBqlLBEJraegbGvsgAzPF2Eo7ObnZeEl6p6Fi4DUxAA

💻 Code

// we are allowed to use same value on multiple cases.
// only the first one is executed

function test( value: string ) {
    switch( value ) {
		case 'a': {
			console.log( "case 1");
			break;
		}

		case 'b': {
			console.log( "case 2");
			break;
		}

		// should not be allowed
		case 'a': {
			console.log( "case 3");
			break;
		}
	}
}

test( 'a' )
🙁 Actual behavior

the first case is executed

🙂 Expected behavior

an error should be emit to prevent such error

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

Issue にある TypeScript Playground の例から始め、コンパイラーが重複した switch case の値をどのように検査しているかを追跡します。繰り返された case の値でエラーが発生し、有効な switch cases がこれまでどおりコンパイルおよび実行され続ける状態になれば作業は完了です。関連するコンパイラーテストのカバレッジを追加または更新してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
typescript
領域
compilers
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。