09/11/18 12:45:14
インスタンス化の削減も微妙だけどな。
もちろんサイズがでかかったりインスタンス化が重いもので
かつ再利用できるものならループ内でのインスタンス化を削減するのもいいが、
何でもかんでも削減すりゃいいってもんじゃない。
よっぽどタイトなループなら別だが。
個人的には、よく言われるStringBuilderの再利用なんかも微妙だと思ってる。
平均的にはある程度効果はあるが、常識のようにやるべきこととは思えない。
925:デフォルトの名無しさん
09/11/18 12:46:59
「理由もなく」初期化すんなよってことであって
意味あって null で初期化すんのはべつにかまわんと思うけどな。
926:デフォルトの名無しさん
09/11/18 12:48:26
>>924
そういうのは最適化部分を狭いメソッド内に押し込めればOKってルールでやってるなあ。
927:デフォルトの名無しさん
09/11/18 12:50:40
なぜなら、StringBuilderのToStringは内部バッファをそのまま流用してString化するから。
その後StringBuilderに変更を加えた時点でバッファは複製される。
当たり前だけど結局メモリの再確保は起こるわけで、
StringBuilderを新たにインスタンス化するのと大差があるわけではない。
928:デフォルトの名無しさん
09/11/18 12:56:54
そんな細かいこと考えるよりもっとロジック工夫して高速化しろよ
929:デフォルトの名無しさん
09/11/18 12:57:56
だれに言ってんだよ
930:デフォルトの名無しさん
09/11/18 13:00:19
今日もお前ら絶好調だな
931:デフォルトの名無しさん
09/11/18 13:04:50
もっと褒めて
932:デフォルトの名無しさん
09/11/18 13:22:08
結局そういうのもバッドノウハウになってくんだろな
933:デフォルトの名無しさん
09/11/18 13:27:34
>>906
スマンカッタ。確かにフィールド、ローカル変数共に初期化してても他で使ってなかったらワーニング出るな。
初期化してないときとワーニングの内容は変わるけど。
＞可読性的には意味がある
意味がないことが多い。922の言うように未初期化参照のコンパイルエラーを出させるほうが素直なコード。
初期化コード自体がif文で分岐してるとかなら初期値を与えることはあるが、それは例外的な措置だよね。
>>923,925
844のBitmap、Graphicsインスタンスのnull初期化は不要だよな？
934:デフォルトの名無しさん
09/11/18 13:32:32
まあ初期化なくす前に明らかに宣言位置を変える方向だが。
宣言位置が適切なら、多くの場合その場で初期化が自然になるからね。
もちろん意味のない初期化はするべきではないけど、
上記を守れば基本的にそういうのはなくなるはず。
935:デフォルトの名無しさん
09/11/18 13:54:00
>>933
>844のBitmap、Graphicsインスタンスのnull初期化は不要だよな？
不要だがあったからといって悪い訳じゃない
936:デフォルトの名無しさん
09/11/18 14:05:32
チームの規約にしたがうレベル
戦ってまで守らなきゃいけない一線ではないわな
どっちにしても
937:デフォルトの名無しさん
09/11/18 14:14:32
>>935
>Bitmap、Graphicsインスタンスのnull初期化は不要だよな？
インスタンスのnull初期化じゃなくて変数(ry
938:デフォルトの名無しさん
09/11/18 15:48:55
formAからｆｏｒｍ.showでformBを開いた後、不要になったformBを×ボタンで閉じます。
この時、formBのリソースは全て開放されますよね？。
939:デフォルトの名無しさん
09/11/18 15:52:19
ガベージコレクションの対象にならないと開放されないよ。
940:デフォルトの名無しさん
09/11/18 15:57:44
そのｆｏｒｍという変数を保持している間は解放されないかな
941:デフォルトの名無しさん
09/11/18 16:17:16
規定のインスタンスを使用している場合はアプリ終了まで保持されるというような話を聞いた。
（でもこれってたしかVB.NETの話だったような。C#では規定のインスタンスってあるんかな？)
942:938
09/11/18 16:27:33
そうなるとFormClosingでちゃんと処理してから終了させた方が良さそうですね
ありがとうございます
943:デフォルトの名無しさん
09/11/18 16:32:41
>>942
いあ、そういふ意味ではないんだが。
Form_Closingイベントとメモリ開放は別話。
944:デフォルトの名無しさん
09/11/18 16:43:17
>>942
モーダルダイアログならFormAで次のようにFormBを表示し
using(Form formB = new FormB())
{
formB.ShowDialog(this);
}
formBフォームの中で使ってるリソースはFormBのDisposeをオーバーライドして行う