microsoft / microsoft/TypeScript
Type Guard Issue with Array.prototype.fill, Array Constructor
まだ誰も着手していません。
- 主要言語
- Go
- スター
- 111k
- フォーク
- 14.4k
- 平均マージ
- 1日 19時間
- マージ済み PR(30日)
- 117
説明
Typegurd Issue with Array.prototype.fill, Array Constructor
TypeScript Version: 3.5.1
Search Terms:
- Array.prototype.fill
- fill
- ArrayConstructor
- array implicit any
Code
const foo: number[] = new Array(3).fill("foo"); // accepted
Actual behavior:
This code above is accepted because new Array(3) returns any[], and so fill("foo") returns any[].
I know giving type explicitly to Array like
const foo: number[] = new Array<number>(3).fill("foo"); // error
would work, but I believe the compiler should reject the first one. (This is TypeScript.)
Expected behavior:
Option A. Array.prototype.fill returns narrow type
replace declaration of Array.fill in lib.es2015.core.d.ts like fill<S extends T>(value: S, start?: number, end?: number): S[];, then
const foo: number[] = new Array(3).fill("foo");
// error: "Type 'string[]' is not assignable to type 'number[]'."
// because `fill` is resolved as `fill<string>(value: string, .....): string[]`
const bar = new Array(3).fill(0); // `bar` is resolved as `number[]`
bar.fill("bar"); // error: "Type '"bar"' is not assignable to type 'number'."
const baz: (number|string)[] = new Array(3).fill(0); // accepted.
baz.fill("baz"); // accepted.
Option B. Array Constructor never return implicit any
The problem is that Array constructor returns any[] implicitly. The first code is accepted even with --strict or any other options. It means we always have to care about "Array constructor returns any[]".
Something like #26188 could solve this issue.
Playground Link:
https://www.typescriptlang.org/play/#src=const%20foo%3A%20number%5B%5D%20%3D%20new%20Array(3).fill(%22foo%22)%3B
Related Issues:
#29604
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、リンクされている TypeScript Playground の例を再現し、次に lib.es2015.core.d.ts の Array と ArrayConstructor の宣言を読みます。関連する issue #29604 と #26188 を確認し、提案されているどの動作が意図されたものかを判断します。受け入れられた動作が仕様化され、関連する TypeScript テストでカバーされていれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- typescript
- 領域
- compilers
- issue の種類
- バグ
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100