[解決済み】ReSharperが「暗黙のうちに捕捉されたクロージャ」と言うのはなぜ?
2022-03-29 20:51:10
質問
次のようなコードがあります。
public double CalculateDailyProjectPullForceMax(DateTime date, string start = null, string end = null)
{
Log("Calculating Daily Pull Force Max...");
var pullForceList = start == null
? _pullForce.Where((t, i) => _date[i] == date).ToList() // implicitly captured closure: end, start
: _pullForce.Where(
(t, i) => _date[i] == date && DateTime.Compare(_time[i], DateTime.Parse(start)) > 0 &&
DateTime.Compare(_time[i], DateTime.Parse(end)) < 0).ToList();
_pullForceDailyMax = Math.Round(pullForceList.Max(), 2, MidpointRounding.AwayFromZero);
return _pullForceDailyMax;
}
今度は、その行にコメントをつけて
リシャーパー
が変更を示唆しています。その意味や、なぜ変更する必要があるのでしょうか?
implicitly captured closure: end, start
どのように解決するのですか?
警告では、変数
end
と
start
は、このメソッド内のラムダが生きているのと同じように生きています。
短い例を見てみましょう。
protected override void OnLoad(EventArgs e)
{
base.OnLoad(e);
int i = 0;
Random g = new Random();
this.button1.Click += (sender, args) => this.label1.Text = i++.ToString();
this.button2.Click += (sender, args) => this.label1.Text = (g.Next() + i).ToString();
}
最初のラムダで、"Implicitly captured closure: g"という警告が表示されます。これは、次のように言っています。
g
はありえない
ガベージコレクション
は、最初のラムダが使用中である限りは
コンパイラは、両方のラムダ式に対応するクラスを生成し、ラムダ式で使用されるすべての変数をそのクラスに配置します。
つまり、私の例では
g
と
i
は、私のデリゲートの実行のために同じクラスで保持されています。もし
g
が多くのリソースを残した重いオブジェクトである場合、ガベージコレクタはそれを回収することができませんでした。なぜなら、このクラスの参照は、ラムダ式のいずれかが使用中である限り、まだ生きているのです。つまり、これはメモリリークの可能性があり、それがR#の警告の理由です。
スプリンター C#では、匿名メソッドは常にメソッドごとに1つのクラスに格納されるため、これを回避する方法が2つあります。
-
無名メソッドではなく、インスタンスメソッドを使用する。
-
ラムダ式の作成を2つのメソッドに分割する。
関連
-
[解決済み】C#はJavaのcharAt()と同等?)
-
[解決済み] DBNullから他の型にオブジェクトをキャストすることができない
-
[解決済み】Sequence contains no matching element(シーケンスにマッチする要素がない
-
[解決済み】バックスラッシュを含むパス文字列のエスケープシーケンスが認識されない件
-
[解決済み】Visual Studio: 操作を完了できませんでした。パラメータが正しくありません
-
[解決済み】2年前のMSDateを把握する【クローズド
-
[解決済み] [Solved] .NETでスレッドの終了を待つには?
-
[解決済み】インデックスが範囲外でした。コレクションパラメータname:indexのサイズより小さく、非負でなければなりません。
-
[解決済み] なぜC#は汎用属性型を禁止しているのですか?
-
[解決済み] ReSharperのGetHashCodeのオーバーライドに'397'が使用されているのはなぜですか?
最新
-
nginxです。[emerg] 0.0.0.0:80 への bind() に失敗しました (98: アドレスは既に使用中です)
-
htmlページでギリシャ文字を使うには
-
ピュアhtml+cssでの要素読み込み効果
-
純粋なhtml + cssで五輪を実現するサンプルコード
-
ナビゲーションバー・ドロップダウンメニューのHTML+CSSサンプルコード
-
タイピング効果を実現するピュアhtml+css
-
htmlの選択ボックスのプレースホルダー作成に関する質問
-
html css3 伸縮しない 画像表示効果
-
トップナビゲーションバーメニュー作成用HTML+CSS
-
html+css 実装 サイバーパンク風ボタン
おすすめ
-
[解決済み】コンパイルエラー「未割り当てのローカル変数を使用しています」が発生したのはなぜですか?
-
[解決済み】ここで「要求URIに一致するHTTPリソースが見つかりませんでした」となるのはなぜですか?
-
[解決済み】C#におけるtypedefの等価性
-
[解決済み】統合マネージドパイプラインモードで適用されないASP.NETの設定が検出された
-
[解決済み】C# - パスに不正な文字がある場合
-
[解決済み】なぜこのコードはInvalidOperationExceptionを投げるのですか?
-
[解決済み】Linq 構文 - 複数列の選択
-
[解決済み] 2つのリストを結合する
-
[解決済み】2年前のMSDateを把握する【クローズド
-
[解決済み】 C# 条件演算子エラー 代入、call、increment、decrement、await、new object 式のみ文として使用可能です。