Throw when xxxOptions are null

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

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

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
35/100
issue の種類
リファクタリング
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
csharp
領域
backend-api-design

調査の方向性

C# コードベースで、xxxOptions を受け取り、現在 null を新しいオプションインスタンスに置き換えているオーバーロードを検索します。まず参照されている pull request のディスカッションを確認し、その後、影響を受けるすべてのオーバーロードと、それに対応するパラメーターなしのものを特定します。これらのオーバーロードが null のオプションを拒否し、パラメーターなしの呼び出しがデフォルトのオプションオブジェクトを構築すれば完了です。

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

説明

Easy fix Stabilization Up for grabs

From https://github.com/libgit2/libgit2sharp/pull/1068/files#r31650592

We rely on the following pattern throughout the codebase.

public Things DoThings()
{
     return DoThings(null);
}

public Things DoThings(ThingsOptions options)
{
     options = options ?? new ThingsOptions();
     [...]
}

Let's change this to the following one

public Things DoThings()
{
     return DoThings(new ThingsOptions());
}

public Things DoThings(ThingsOptions options)
{
     Ensure.ArgumentNotNull(options);
     [...]
}
主要言語
C#
スター
3.5k
フォーク
925
30日以内にマージされた PR はありません

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

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

はじめの一歩

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

libgit2/libgit2sharp のほかの issue

libgit2/libgit2sharp の issue をすべて見る

似ている issue

C# の issue をもっと見る

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

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