ふらっとC#,C♯,C＃（初心者用） Part47

886 名前：843 mailto:sage [2009/11/18(水) 04:45:56 ] >>883 いろいろある突っ込みどころ　ありがとうございます。 ・wBlocksの件 アルゴリズムの問題ですね。 たしかに整数演算で簡潔するのでその方がスマートです。 勉強になりました。 ・usingについて 例外が発生するとdisposeされないのでusingを書くべきですが 動作検証用サンプルでしたので急ピッチですのであしからず。 ・切り出しファイルの名前を外から指定について 同じく、動作検証用サンプルですので大目にみてください。 StringFormatもargs指定ミスはコンパイル時に発見されないので +でつないでます。 ・必要なところでvar指定　 2重forで300回近くループするのでfor内部で変数宣言は好ましくないのでは？ ループ内部で使いまわせる変数は外出しすべきだと僕は思います。 また、null代入は無駄なのは分かってますが 「初期化してますよ」ってメッセージをこめたソースを書くのは悪くないと僕は思います。 ちなみにBitmapを毎回new（必然的にGraphicsもnew） してるのは画像サイズが256の倍数でないときに 右端と下端の矩形を描画すると、前回の描画が残ってしまうための回避策です。 Graphics.Clearで思った動作するかも。テストしてないのでなんともいえませんが。 varは メソッドの戻り値の型をそのまま受けるとき linq+foreach使うときなど、ジェネリックの型名なんだっけってなるとき 使用するのはいいと思うのですが、 C#使ってるんだからその場で初期化する変数は型はしっかり書いたほうが いいと自分は思ってます。賛否はあると思いますが。 (型付DataSetのテーブルアダプタみたいに型名が長いと初期化が冗長になるので そういうときは見易さを考慮して左辺はvarで受けたりします。） コーディング規則が決められているときはもちろんそれに従います。

---

---

[ 続きを読む ] / [ 携帯版 ]

read.cgi ver5.27 [feat.BBS2 +1.6] / e.0.2 (02/09/03) / eucaly.net products.
担当:undef