microsoft / microsoft/TypeScript
Generic inference of Overloaded methods only working with the last defined method
オープン
まだ誰も着手していません。
Docs
- 主要言語
- Go
- スター
- 111k
- フォーク
- 14.3k
- 平均マージ
- 2日 4時間
- マージ済み PR(30日)
- 132
説明
Bug Report
🔎 Search Terms
- Generics
- Overload
- Type inference
🕗 Version & Regression Information
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about overloading
⏯ Playground Link
Playground link with relevant code
💻 Code
class SomeClazz {
public overload(param: 'PARAM1'): number;
public overload(param: 'PARAM2'): string;
public overload(param: 'PARAM1' | 'PARAM2'): number | string {
switch (param) {
case 'PARAM1':
return 1337;
case 'PARAM2':
return 'Hi';
}
}
}
function call<Ctx extends { [F in Name]: (this: Context, ...args: any[]) => any }, Name extends string>(
ctx: Ctx,
method: Name,
...args: Parameters<Ctx[Name]>
): void {
ctx[method](...args);
}
const clazz = new SomeClazz();
clazz.overload('PARAM1'); // Works ok
clazz.overload('PARAM2'); // works ok
call(clazz, 'overload', 'PARAM1'); // ERROR: Argument of type '"PARAM1"' is not assignable to parameter of type '"PARAM2"'
call(clazz, 'overload', 'PARAM2'); // Works ok
🙁 Actual behavior
- Only one function overload is taken into account when inferring function parameter types
🙂 Expected behavior
- All function overloads should be taken into account when inferring function parameter types
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、リンクされた Playground または Workbench の再現コードを実行し、2 つのジェネリックな呼び出し箇所を直接のオーバーロード呼び出しと比較します。オーバーロードされたメソッドに対して Parameters<Ctx[Name]> がどのように推論されるかを追跡します。call(clazz, 'overload', 'PARAM1') と call(clazz, 'overload', 'PARAM2') の両方が受け入れられ、報告されているオーバーロードの動作が維持されれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- typescript
- 領域
- compilers
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 35/100