「バグや問題を「MSのせい」にする技術者、いませんか？」記事へのコメント

本当にバグの場合もあるけど

[yoshimo (15798)]

Windows CE 5.0の.NET Framework 2.0(C#)で
はまった内容：

悪い例：
Form form = new HogeForm();
form.ShowDialog();
form.Dispose();

良い例：
Form form = new HogeForm();
form.ShowDialog();
form.Dispose();
form = null;

最後にnullを設定しないと、このフォームがガベージコレクトされず、
メモリリークします。なんだそれ。

マイクロソフトのサポートに問い合わせたところ、
マイクロソフト側でも現象を確認したそうで、
回答が「最後でnullを設定してね」でした。
マイクロソフトはバグとは言いませんでしたが、どう見てもバグですね。

メモリリークするよ～、あんたのコードが悪いんでしょ～、
なんとかしてよ～、とつつかれて大変でした。

Re:

[Anonymous Coward]

面倒なものですなあ。

ところでそれ、最近の流行でいえばRuby流というか、こういう感じには書けないんでしょうか？

HogeForm.ShowDialog do |form|
    xxxxx
end

Rubyのブロックの使い方の1つとして、
それこそ今回のよう(?)な、リソース開放の手順を隠蔽する道具として使う、というのが有るもので。

open(filename) do |f|
  f.xxxx
end

などと書くと「ファイルを開いてブロック内でxxxxさせて、それが終わるとファイルを閉じる」ってのをやってくれる。

こういう書き方が出来るならば、Disposeだろうが=nullだろうが何が来ようが怖くない。みんな隠蔽できるのだから。

この

Re:本当にバグの場合もあるけど

[Tsann (15931)]

ところでそれ、最近の流行でいえばRuby流というか、こういう感じには書けないんでしょうか？

HogeForm.ShowDialog do |form|
        xxxxx
end

using構文で書けます。

using( var form = new HogeFrom() ){
  form.ShowDialog();
}

formのスコープが明確になるし、Dispose()の呼び出しも保証されます。

Re:本当にバグの場合もあるけど

[live-gon (30327)]

formのスコープが明確になるし、Dispose()の呼び出しも保証されます。

そう。そしてusingは使っていないがちゃんとDisposeしているのにメモリリークするというのが元に書かれたバグでしょう。CE の環境がないので試せていませんが。

HogeForm.ShowDialog do |form|
        xxxxx
end

これをこんな風に書く必要があるんでしょう。

HogeForm.ShowDialog do |form|
        xxxxx
        form = null
end

なんだそれと思ったらその感覚が正しい(笑)

--
LIVE-GON(リベゴン)