- 1 名前：デフォルトの名無しさん mailto:sage [2009/11/01(日) 18:23:31 ]
- このスレッドは
「どんなにくだらないC#プログラミングに関する発言でも誰かが優しくレスをしてくれるスレッド」です。
ほかのスレッドでは恐ろしくて書き込めないような低レベル、もしくは質問者自身なんだか意味がよく分からない質問など、
勇気をもって書き込んでください。
内容に応じて、他スレ・他板へ行くことを勧められる、あるいは誘導される場合がありますがご了承下さい。
>>980を踏んだ人は新スレを建てて下さい。
>>980が無理な場合、話し合って新スレを建てる人を決めて下さい。
前スレ
ふらっとC#,C♯,C＃（初心者用） Part45 (実質46)
pc12.2ch.net/test/read.cgi/tech/1255658236/
- 809 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 17:29:52 ]
- >>802
枚挙に暇が無いっていいうなら
幾つか実例挙げてくれよ
- 810 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 18:18:25 ]
- dynamic使えば見た目は普通のオブジェクト正体はディクショナリとか
変態的なことヤリ放題だな
- 811 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 18:21:44 ]
- 使いすぎるとわけわからなくなるだろ・・・
- 812 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 18:26:59 ]
- 実際、演算やメンバ選択などオブジェクトに対するほとんどの操作を
自由に定義できるんだぞw
- 813 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 18:29:25 ]
- >>812
その代わりに.NET CLRのもっとも強力な点である静的型付けの最適化が無効になるから程々にな。
- 814 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 18:33:38 ]
- dynamicはDLRのキャッシュ使うから意外に遅くないよ
せいぜいボックス化+デリゲート呼び出しのコストが入るだけ
- 815 名前：デフォルトの名無しさん [2009/11/17(火) 18:47:51 ]
- >>809 そのくらい想像してくれよ。簡単な誰でも知ってる経験してる話だよ
ポトペタやパーツの組み合わせ以外にアルゴリズムやロジックや、元々の仕組みを掘り下げた時、
目的だけでなく副次的にや必然から知るようなノウハウってあるだろ
例えば効率のいい分岐、ノード的な表現、ビット演算使った手法など色々
知ってる人には慣れて当たり前に感じる内容でも、知らない人は知らなくて
その点で表現手段が変わってきたり手狭になったり実現できなかったりする。
掘り下げるのも広げるのも自分の技術広げるって意味で同じだから、
興味沸いたら色々試したり調べたりするといいぜって話さ
人はそれを努力と見るかもしれないが、そんな事はどうでもよくて
- 816 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 19:00:02 ]
- ぜんぜん具体的じゃねー
- 817 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 19:02:35 ]
- >>810
だんだん言語仕様が太ってきた気がする
- 818 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 19:02:51 ]
- 関数とメソッドってどう違うんですか
- 819 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 19:06:35 ]
- オブジェクトに紐付いてるかどうかぐらいの違いじゃないの?
- 820 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 19:22:56 ]
- >>818
OSで言えばディレクトリとフォルダの違いのようなもの。さほど気にすることはない。
ただ、はっきり言ってしまえばC#に関数はない。
あるのはdelegateと通常のメソッドと静的メソッドだけだ。
他にイベントハンドラやらラムダやらもあるが、その辺は自分で勉強してくれ。
- 821 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 19:33:43 ]
- え？関数ってreturnでなんか戻ってくるメソッドのことじゃないの？
- 822 名前：デフォルトの名無しさん [2009/11/17(火) 19:42:36 ]
- >>816 なにもかも全部他人に書かせる気か
口移ししないとエサ食えないみたいな事言うなよ
- 823 名前：デフォルトの名無しさん [2009/11/17(火) 19:49:24 ]
- 関数は単純に関数だよ
メソッドはひとかたまりの処理を模したオブジェクトって概念に対して、例えとして
「ふるまい」として名付けられた物
コード部分にだけ逐次的に注目すると変わらないように見えるけど、意味付けが違う、みたいな
例えばC/C++の世界では関数とメソッド（メンバ関数なんて言い方もするが）は、完全に別物
文脈的に話が混線しないならどっちも関数と呼ぶ場合もあるけど、
クラスに紐付いてないただの関数はメソッドとは呼ばない
そんな感じ
- 824 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 19:58:13 ]
- >>815
まあ技術の変わり目にはどんな分野でもこういうアナクロ指向な人っているんだよ。
もちろん、アナクロな彼の言うことにも一理はあるけどね。
そりゃ、洗濯機があるのにあえてたらいで洗濯することで身につく技能だってあるよw
こういう人の欠陥は、視野が狭いから引いた視点から全体を俯瞰して
物事のプライオティを考えることが出来ない点だな。
- 825 名前：デフォルトの名無しさん [2009/11/17(火) 20:04:42 ]
- >>824 ？？誰に向けて言ってんだ？
視野を広げる手段の一つを話してる文脈が理解できないとしたら
ちょっと想像力も読解力も足りなくねｗｗ
なんでアナクロ扱いなんだよ。思考停止はよくないぞ
- 826 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:05:12 ]
- なんだ、しったかかよ
- 827 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:06:50 ]
- >>821
数学でいう関数だとそうだよな。
- 828 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:11:38 ]
- 釣ったほうも釣られたほうもスレ違いだ
- 829 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:12:18 ]
- 単純に
int func(int x,int y){
return x+y;
}
これが関数で
void func(int x,int y){
printf("%d",x+y);
}
これがメソッドだという風に覚えてたわ。
- 830 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:22:22 ]
- イベントのやり方わからん。
.netに最初からついてるフォームとかボタンとかのイベントはわかるんだが・・・
自分でクラスつくるときにどんなイベントを入れればいいのかわからん。
ただのメソッドやデリゲートをクラスに組み込むのと
イベントを組み込むのは機能面でどういう違いがあるんですかい？
メソッドとデリゲートだけじゃ不可能なことでもイベント使ったらできることって例えばどんなの？
- 831 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:35:58 ]
- > ただのメソッドやデリゲートをクラスに組み込むのと
> イベントを組み込むのは機能面でどういう違いがあるんですかい？
メソッドは方向が違う。クラスの利用者がクラスに処理を依頼する形。
デリゲートやイベントはクラスがクラスの利用者に処理を尋ねるようなもの。
で、本質的にはデリゲートは処理の委譲のためのもの、イベントは通知のためのもの。
実際的にはイベントはデリゲートをラップするプロパティみたいなもの。
add/removeの操作限定なので他の利用者がいきなりnull入れたりすることを気にしなくて済む。
> 自分でクラスつくるときにどんなイベントを入れればいいのかわからん。
状態の変化とかをクラスの利用者に通知したいとき。
> メソッドとデリゲートだけじゃ不可能なことでもイベント使ったらできることって例えばどんなの？
デザイナで扱える。
コールバックに優先順位を付けるとかもやろうと思えばできる。
- 832 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:41:04 ]
- >>830
まず、質問はどれかにまとめてくれｗ
C#のイベントはデザインパターンで言えばObserverパターンにあたり、情報の発信元と受け取り先を分離するために使う。
つまり、発信元ではどこに送るのか、受け取り先ではどこから受け取るのかを考える必要がなくなる。
またイベントハンドラの結びつけさえやればどんなオブジェクト同士でも信号のやり取りができる。
だからこそSystem.Formsではすべてのコントロールのイベントを親のFormで受け取ることができるわけだ。
『メソッドとデリゲートだけじゃ不可能』というが、基本的にチューリング完全なら何でもできるので、
できるかどうかではなく、簡単かどうかで考えるといいよ。
ちなみに大抵は初期化時に静的なイベントパイプラインを構築するだけだと思うが、
これを実行中に動的に切り替えるようにすればさらに強力。Stateパターンなんていらねー。
- 833 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:43:47 ]
- デリゲートって何ですか？
- 834 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:45:36 ]
- イベントは一応
他クラスから実行出来無いって保証が付くんだよな
- 835 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:52:59 ]
- >>833
WikipediaやMSDNくらいは読んでから質問してるんだよな？
- 836 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 20:58:31 ]
- サーバへFTP接続する実行ファイルを不特定多数へ公開した場合
ID,パスの流出可能性を出来るだけ抑えるにはどうすればいいと思いますか？
- 837 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 21:03:08 ]
- >>834
リフレクションの前には儚い保証ではあるけどな
- 838 名前：デフォルトの名無しさん [2009/11/17(火) 21:32:17 ]
- >>836 それはまさかIDパスワードを埋め込むつもりって話か
なにをやりたいのか知らないが、俺なら間にCGIでも挟む
直でFTPでなくて80番経由
- 839 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:02:02 ]
- CGI、PHP挟められるならそうするんですが
鯖に環境が入っていないのでそうすることも出来ないです
やっぱり、どう暗号化してもパス埋め込むのは駄目ですかねぇ。。
- 840 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:16:39 ]
- あきらめろよ。
- 841 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:18:33 ]
- 大丈夫って保証は取れないねえ。
それに暗号化してもパケット見たらばれるし。
自PCにならパケットスニファなんて好きなだけ入れられるもんな。
- 842 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:19:04 ]
- そもそもFTPじゃユーザ名パスワードが平文だぜ。
- 843 名前：デフォルトの名無しさん [2009/11/17(火) 22:24:16 ]
- 質問です。.NET3.5 C#3.0
サイズの大きい画像を256四方で分割したいのです。
実際分割してみると、エクスプローラ上では256*256と表示されるのですが、
Fireworksで読み込ませると、256*256, 255*255, 256*256といった具合に、
元の画像のSizeによって256*256にならず、
切り取った画像を256四方に並べると隙間ができてしまいます。
ペイントに貼り付けるとx=0, y=0のラインが真っ黒になります(255*255)。
元画像のImageオブジェクトの
HorizontalResolution,VerticalResolutionは96.01199 なのですが、
コピー先 bmp = new Bitmap(256,256);
のHorizontalResolution,Vert(ry は96.0と異なるのでこの辺で誤差が出てる?
Graphics.FromHwnd にてモニタのdpiを取得すると96.0と表示されます。
でもReadOnlyプロパティなのでbmpのResolutionを96.01199にできない。
矩形画像を正しく切り取り、保存する方法を教えてください。
- 844 名前：843 [2009/11/17(火) 22:25:52 ]
- //ソース
int pixel = 256;
Bitmap bmp = null;
Graphics g = null;
Rectangle rect;
System.IO.FileInfo fInfo = new System.IO.FileInfo(filePath);
Image origin = Image.FromFile(fInfo.FullName);
int wBlocks = (int)Math.Ceiling((double)origin.Width / (double)pixel);
int hBlocks = (int)Math.Ceiling((double)origin.Height / (double)pixel);
Console.WriteLine(filePath);
for (int h = 0; h < hBlocks; h++)
{
for (int w = 0; w < wBlocks; w++)
{
bmp = new Bitmap(pixel, pixel);
g = Graphics.FromImage(bmp);
rect = new Rectangle((w * pixel), (h * pixel), pixel, pixel);
g.DrawImage(origin, 0, 0, rect, GraphicsUnit.Pixel);
string destName = fInfo.Directory + "\\" + fInfo.Name.Replace(fInfo.Extension, string.Empty) + "_" + h.ToString("000") + "_" + w.ToString("000") + ".png";
bmp.Save(destName, System.Drawing.Imaging.ImageFormat.Png);
g.Dispose();
bmp.Dispose();
}
}
origin.Dispose();
- 845 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:26:55 ]
- 抑えるのではなく、逆にIDとパスワードは流出してもOKにすれば問題ない。
- 846 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:30:52 ]
- LockBitsで直接コピーするのが確実じゃね
- 847 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:38:57 ]
- >>845
繋ぐ先のFTPサーバのアカウント操作権があるならそれもありだよね。
ダウンロードのみならanonymous FTPにしちゃうってのもありだ。
- 848 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:43:11 ]
- ユーザーID を本名にして、パスワードをクレジットカード番号とかにしておけばいいな。
- 849 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:45:27 ]
- >>845,847
あー、なるほど。
でもアップロードも必要なので、anonymousは出来ないですね
サーバ側のFTP制御設定弄ってみます。
- 850 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:45:55 ]
- >>844
どうやったかなぁ。とりあえず座標系とかオフセットモードが
関係してた希ガス。PixelOffsetMode.None してみると駄目？
補間モードとかもいじらなあかんかもしれん。
- 851 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:45:55 ]
- そもそも自分でFTPアカウントを発行できない環境でそんな利用の仕方が許されるとは思えない
- 852 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:46:59 ]
- >>843
Fireworks以外のドローソフトだとどうなるの？
- 853 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:56:04 ]
- >>844
ツッコミどころは色々あるが、さしあたりはバグというほどのものはなさそうだな。
FireworksがDPIを真正直に解釈して誤動作起こしてるのかもしれんね。
まずはbmp形式で保存して、バイナリエディタで眺めてみたら？
- 854 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:56:54 ]
- 暗号化ってさ、
そんなに効果あるものなの？
実際使うのなら、起動時に複合化して、
ずっと変数に持たしておいてもいいの？
- 855 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:58:36 ]
- >>854
何を？ パスワードとか大事なモノを扱うときは、SecureString とか使う方がいいね。
- 856 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 22:59:10 ]
- FTPの話は前のスレでも出たな
プロトコルの用途がちげぇって結論になった気がする
- 857 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:06:23 ]
- DrawImageの描画先をint,intじゃなくてRectangleで指定する
- 858 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:07:55 ]
- >>856
だな。まずはその腐った要求仕様を窓から投げ捨てろといいたい。
で、代わりのプロトコルを考えるとするならどうするか、なんだが。
ユーザーのPCから特定のバイナリデータを手元のサーバーに送りつけるというのでいいんだな？
――とはいうものの、考えてみると、案外これというやりやすい方法が見当たらないので困ったもんだ。
さしあたり思いつくのがHTTPのPOSTで送るかメールで送るか、あとRDBにクエリで送りつけるかだが。
IISが運用できるならC#で解決できるからHTTPで決まりだろう。
メールの場合、LinuxやMacならともかく、Windowsは送信できる環境が保証されてないので
メールを正しく送信できることが保証できない。
企業アプリみたいにこちらで動作環境を自由に決められるなら何とでもなるんだけどね。
- 859 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:09:24 ]
- >>854
パスワードを暗号化して埋めこんだんだとしても
IL解読して複合化の手順読みとられたらそれで終りだし
メモリに持ったり通信したりするときに取られちゃ意味ないし
- 860 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:10:09 ]
- >>856
それは、プログラムからFTPは利用しないほうがいいってことですよね？
といっても、CGIやPHPが使えればそちらを使うのですが、
使えないので、ファイルの書き換えを行うには直接FTPするしか無いと思いまして・・・
取りあえずアカウント制御で、ディレクトリ制限を掛けてみました。
これでしばらく挙動見たいと思います
- 861 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:11:50 ]
- どっかのCGI設置可能なレンタルサーバでも借りてゲートウェイでもするしかないな
- 862 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:13:12 ]
- プログラムっていうかクライアントから直には止めた方いいかもね
- 863 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:13:37 ]
- >>860
そのユーザIDとパスワードはFTPアカウント専用なん？
そこが分離されててchrootとかされてるならまだなんぼかマシなんだろうけど。
- 864 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:16:30 ]
- telnetで接続可能なパスワードとかだったら笑えるな
- 865 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:21:01 ]
- >>858
IISも無理です、基本サーバ動作のモノは全てアウトでして。
RDBは思いつきませんでした、が・・・見た感じ無理そうです・・・
>>861 レンタルサーバはちょっと厳しいです
>>863 chrootされてますです
- 866 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:24:51 ]
- ftp内でのコマンドの制限とかquotaとか適切に設定されてるなら
とりあえず大丈夫なのかもしれないね
- 867 名前：デフォルトの名無しさん [2009/11/17(火) 23:26:06 ]
- そもそもなんでこんな危なっかしい事しなきゃいかんのだ？
- 868 名前：デフォルトの名無しさん mailto:sage [2009/11/17(火) 23:31:02 ]
- 自分で試してみましたが、取りあえずchroot周りはうまく制御されてるみたいです
もっと良い方法があれば教えてください。
これでしばらく試してみます。ありがとうございました。
- 869 名前：749 mailto:sage [2009/11/18(水) 00:44:38 ]
- >>750 >>763
遅くなりましたが、ありがとうございます！
こちらにある通りApp.configを修正して実行してみましたが、やはり上手くいきませんでした(固まっちゃって再起動)
ttp://hightechrider.blogspot.com/2009/10/mixed-mode-assembly-errors-after.html
あまり理解できていませんが、最近のランタイムでは使えなくなった可能性が高いということでしょうか。
とりあえず今回は他の方法で実現するという方向性に逃げました。
- 870 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 00:56:52 ]
- つーか、環境ぐらい書けば？ .NET Framework 4.0 なんて入れてんの？
- 871 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 01:01:42 ]
- そりゃ入れてるから「4.0 ランタイムでは読み込めません」なんてメッセージが出るんだろう
- 872 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 01:02:13 ]
- Windows SDKってどこからダウンロードできますか？
ググって飛んだら
申し訳ございません。ご指定のページが見つかりませんでした。
といわれました
- 873 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 01:04:26 ]
- ベータのランタイムなんて入れてんのが悪いんだろ？
- 874 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 01:32:13 ]
- ttp://www.microsoft.com/downloads/details.aspx?FamilyID=c17ba869-9671-4330-a63e-1fd44e0e2505&DisplayLang=en
ttp://www.microsoft.com/downloads/details.aspx?FamilyID=71deb800-c591-4f97-a900-bea146e4fae1&DisplayLang=en
最新はコレ ISOとWebインストール
ttp://www.microsoft.com/downloads/details.aspx?FamilyID=7bbe5eda-5062-4ebb-83c7-d3c5ff92a373&DisplayLang=ja
こっちは日本語版MSDNライブラリの最新
- 875 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 01:34:42 ]
- C#では、C,C++における『CLOCKS_PER_SEC』のようなものってありませんか？
- 876 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 02:03:51 ]
- switch(foo){
case BAR: /*....*/ break;
case BAZ: /*....*/ break;
default:
// バグってなければ絶対こないところ
}
↑のプログラムで default:にいっちゃった時に例外投げたいんだけど、
引数エラーのときの ArgumentException みたいに投げるべき
例外ってある？
- 877 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 02:07:55 ]
- ArgumentExceptionとか
- 878 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 02:10:44 ]
- >>875
のようなものっていわれても
それでなにしたいのさ
- 879 名前：843 [2009/11/18(水) 02:24:33 ]
- >>846
一回のプロセスで一度に大量の画像を分割するので
LockBitsは結構しんどかったりします。
unsafeでポインタ使えば？とか言われそうですが^^;
>>852
ペイントの例をすでに書いています。
ほかのソフトはためしてません。
>>853
色々ある突っ込みどころ 教えてください。
rectだけ宣言時にnull代入してないぞ、とか
そういうのは無しでおねがいします。
bmpで保存すると、
fireworksで表示するとペイントのときのように
x=0,y=0のラインが黒になります。
>>857
描画先をRectangle(0,0,pixel,pixel)にしたら
正常に動作しました。
ありがとうございました。
- 880 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 02:29:46 ]
- >>876
InvalidFooException
- 881 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 02:35:28 ]
- >878
すみません。定数が用意されているならそれを使おうかと思いまして
- 882 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 02:51:33 ]
- >>876
メソッド利用者にパラメータが間違っていることを通知したい場合
→ InvalidEnumArgumentException
コーディングミスによってありえないフローになっているのを検出したい場合
→ Debug.Fail(), Trace.Fail()
- 883 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 03:31:43 ]
- >>879
>857の言うように、GraphicsUnit指定による呼び出し方によって発生する内部の変換処理を避けるわけか。
まあそりゃあそうだわな。そこが今回の不具合の直接の原因なわけだし。
＞色々ある突っ込みどころ 教えてください。
じゃあ手短に。
・wBlocks、整数演算で十分。pixel-1を足してpixelで割ればいい。
・Disposeするオブジェクトはなるべくusing構文で初期化する。明示的な後始末はなるべく書かないのがベター。
・切り出しファイルの名前と形式は外から変えられるように書く。というかこんなところで書くものじゃない。あとstring.Format()
・オブジェクト、=nullの初期化なんてイラネ。何そのC++もどき。むしろ必要なところでvar構文で初期化して使う。
こんなとこかな。
- 884 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 04:07:59 ]
- 大規模なプログラムになるとプロパティ使うんだろうけども
個人で使う小規模なプログラムってメンバ変数をpublicにしおいても問題ないよね？
- 885 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 04:12:18 ]
- 自動実装プロパティにしとけば？
- 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で受けたりします。）
コーディング規則が決められているときはもちろんそれに従います。
- 887 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 05:25:45 ]
- C#ってマイクロソフトが作ったの？
- 888 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 05:37:22 ]
- >>886
俺はおまえさんの主張に大体同意だな
そもそも宗教的なことまで突っ込みを入れるのは回答者の分を超えてると思うよ
- 889 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 06:11:25 ]
- >>886
俺もおまえさん寄りの意見だが、
突っ込んでくださいと言っておきながら、
それは違う、あれは違うと条件を付けて反論するのはどうかと思う。
自信があるなら最初から聞くな。
聞いた以上は真摯に受け止めるか、
変なやつに聞いちゃったとスルーなさい。
- 890 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 07:30:40 ]
- プライドだけは一丁前
- 891 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 07:39:05 ]
- >>889
それはおかしい
教えを請うたからといって間違った回答や、
自分の方針と反する内容を無条件で聞く必要はない
かといってスルーするのは礼を失している
886の対応は真摯な態度だと思うぞ
- 892 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 07:40:20 ]
- >>886
>2重forで300回近くループするのでfor内部で変数宣言は好ましくないのでは？
>ループ内部で使いまわせる変数は外出しすべきだと僕は思います。
少なくとも今回のケースでは、変数宣言を外に書いても速度的にメリット無いよ。
変数宣言を一回通るたびに実行コストがかかるとか杓子定規に考えてない？
- 893 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 07:44:00 ]
- なんでC#はmain()じゃなくてMain()
なんだろうな。
cもc++もjavaもmainで始まったのに、なぜC#はMainなの？それだけ気になる。
- 894 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 07:45:57 ]
- 横だけど・・・
え、そうなの？
スコープ入ったときにスタックに確保、
スコープ出るときにスタックから解放って処理普通にやってんのかと思ってた。
- 895 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 07:47:47 ]
- >>893
命名規約の問題でしょ。
メソッド名はUpperCamelCaseで表すってルールにしてるから"M"ainになってるだけ。
なんでUpperCamelCaseを採用したのかは言語設計者のポリシーなんだろね。
- 896 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 07:56:40 ]
- 黎明期の言語じゃあるまいし
- 897 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 07:57:42 ]
- >>891
無条件で聞けなんてどこにかいてあんの？
ま、どっちもどっちだ
- 898 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 08:01:58 ]
- >>897
>聞いた以上は真摯に受け止め
- 899 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 08:09:20 ]
- >>894
C#で書いたコードをReflectorで逆コンパイルしたときに現れるgotoだらけのコード見たことない？
見てみれば分かるけど、結構オブジェクトの寿命が変わるような書き換えがされている。
変数スコープが終わる前にGCされてファイナライザが呼ばれることがあるのはこのため。
- 900 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 08:12:42 ]
- >>899
なるほど
ちまちましたコードレベルの最適化はほんとに意味なくなってきてるんだなぁ
勉強になった
- 901 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 08:41:55 ]
- >>898
日本語が理解できない人だったんですね。
これは失礼しました。
- 902 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 08:48:42 ]
- >>886
＞StringFormatもargs指定ミスはコンパイル時に発見されないので +でつないでます。
なんでやねん。テストコード書けばテストできる範囲でしょ。
＞2重forで300回近くループするのでfor内部で変数宣言は好ましくないのでは？
実行時最適化が行われる.NET Frameworkの世界でそんな議論しても無意味だよ。
そして、むしろvar構文自体より（便利なものは積極的に使うべきだと思うが）ローカルインスタンスの明示的な初期化(=null;)のほうが問題。
C#ではすべての変数はデフォルト値で初期化されることが保証されているので
object obj=null;
などという初期化は無意味。それどころか変数が使われなくなると初期化なしだとワーニング出してくれるけど
初期化してると出してくれなくなったりするので余計に悪化している。
- 903 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:14:39 ]
- >>893
Javaの皮を被ったDelphiだから
- 904 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:18:19 ]
- varはなんか、なんかちょっと多様したら不味いことがおきるんじゃないかって気がしてならない
具体的に何が起きるのかはわからんけど
dynamicもしかり・・・
- 905 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:19:59 ]
- メソッド名がCamelCaseなのはMSの旧来の伝統
>>904
むしろvar使ったら読みづらくなるようなスコープの長い変数を作るのが悪い
- 906 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:21:56 ]
- >>902
> object obj=null;
> などという初期化は無意味。
可読性的には意味がある
>それどころか変数が使われなくなると初期化なしだとワーニング出してくれるけど
> 初期化してると出してくれなくなったりするので余計に悪化している。
ダウト
- 907 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:23:41 ]
- object obj;
object obj = null;
どう可読性に違いが
- 908 名前：905 mailto:sage [2009/11/18(水) 09:25:02 ]
- すまんPascalCaseだった
フィールドを既定値(default(type)の値)で初期化するとFxCopで警告出るよ
- 909 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:29:30 ]
- FxCop日本語版は有料なのか・・・
- 910 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:33:25 ]
- ttp://msdn.microsoft.com/ja-jp/library/ms182274.aspx
これだな
パフォーマンスは今のコンパイラでは関係ないが冗長だからやめろということらしい
FxCopはコメントのルールだけ外しとけば日本語ソースでも普通に使える
- 911 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:35:13 ]
- ローカル変数の確保は内部的にはメソッドスコープだよ。
そんなのを気にしながらBitmapをループ内で作るって…
- 912 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:39:02 ]
- 世の中富豪的っていうかこまけえ最適化はコンパイラがやってやんぜ!!な方向に
確実に進化してるんだなぁ
- 913 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:39:09 ]
- >>907
皆が皆C#だけやってると思うなよ？
- 914 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:39:59 ]
- nullの方がヌルヌルして入りやすい
- 915 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:42:13 ]
- ローカル変数はデフォルト値で初期化されるって、されないよ。
だから必ずどこかで代入してから参照する必要があり、でないとエラーになる。
もちろん宣言と同時に代入するのでなくとも、どこかで入れればいいが、
宣言で代入するのが普通は分かりやすいけどな（というかできるだけ最初に代入する必要がある場所で宣言する）。
初期化が保証されてるのはフィールドだ。
- 916 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:45:37 ]
- ＞＞ 2重forで300回近くループするのでfor内部で変数宣言は好ましくないのでは？
＞実行時最適化が行われる.NET Frameworkの世界でそんな議論しても無意味だよ。
実行時最適化関係ないよ。
ILの時点でローカル変数はメソッドスコープで確保される。
- 917 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 09:51:52 ]
- 実際ループ内でしか使わない変数ならループ内で宣言してくれた方が
コード読む時も余計なこと考えなくてすむしメソッド化する時とかも便利だしなぁ。
#メソッド化すると呼び出しコスト↑と結局毎回確保になっちゃうけど
- 918 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 10:00:03 ]
- まあちっちゃなメソッドならインライン展開されるけどな。
とはいってもかなり小さい場合だけだったとは思うけど。
- 919 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 10:11:41 ]
- >>915
ttp://msdn.microsoft.com/ja-jp/library/aa691170(VS.71).aspx
これか、なるほど
俺ずっと勘違いしてたわ。
- 920 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 10:16:40 ]
- 最終的にILになれば同じつうならあとは好みの問題だろう・・・
ギリギリの極限まで速度を追い求めない限り
些細な速度UPを得るよりも可読・メンテ性を維持したい
というのが俺の持論
もちろん事務系とかの話でFA系の処理とかになれば必要なところは可読性捨ててでも速度UPとるけど
- 921 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 10:18:50 ]
- みんながどのコードを前提に話してるのか見えないんだが・・・
ループ内で宣言したくないのって言うと、
よっぽどでかいインスタンスを毎回newして捨てるとか
そいういう場合？
- 922 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 10:49:08 ]
- 変数のスコープを最小限にするのは宗教でもなんでもなく常識だろ
宣言だけループ外に置いて、わざわざミスが起こる確率を増加させる必要はない
あと、理由もなくローカル変数を規定値などに初期化するのも良くない
たとえば、未初期化の変数を参照するとコンパイルエラーになるので即座に分かるが、
無駄に初期化していると問題の発覚が遅れ、デバッグに要する時間も増える
- 923 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 12:28:52 ]
- >>922
処理上で意味のある値で初期化するんであって
常に null で初期化すると勘違いしてるから
そういう的外れな反論が飛び出すんだろう
- 924 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 12:45:14 ]
- インスタンス化の削減も微妙だけどな。
もちろんサイズがでかかったりインスタンス化が重いもので
かつ再利用できるものならループ内でのインスタンス化を削減するのもいいが、
何でもかんでも削減すりゃいいってもんじゃない。
よっぽどタイトなループなら別だが。
個人的には、よく言われるStringBuilderの再利用なんかも微妙だと思ってる。
平均的にはある程度効果はあるが、常識のようにやるべきこととは思えない。
- 925 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 12:46:59 ]
- 「理由もなく」初期化すんなよってことであって
意味あって null で初期化すんのはべつにかまわんと思うけどな。
- 926 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 12:48:26 ]
- >>924
そういうのは最適化部分を狭いメソッド内に押し込めればOKってルールでやってるなあ。
- 927 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 12:50:40 ]
- なぜなら、StringBuilderのToStringは内部バッファをそのまま流用してString化するから。
その後StringBuilderに変更を加えた時点でバッファは複製される。
当たり前だけど結局メモリの再確保は起こるわけで、
StringBuilderを新たにインスタンス化するのと大差があるわけではない。
- 928 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 12:56:54 ]
- そんな細かいこと考えるよりもっとロジック工夫して高速化しろよ
- 929 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 12:57:56 ]
- だれに言ってんだよ
- 930 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 13:00:19 ]
- 今日もお前ら絶好調だな
- 931 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 13:04:50 ]
- もっと褒めて
- 932 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 13:22:08 ]
- 結局そういうのもバッドノウハウになってくんだろな
- 933 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 13:27:34 ]
- >>906
スマンカッタ。確かにフィールド、ローカル変数共に初期化してても他で使ってなかったらワーニング出るな。
初期化してないときとワーニングの内容は変わるけど。
＞可読性的には意味がある
意味がないことが多い。922の言うように未初期化参照のコンパイルエラーを出させるほうが素直なコード。
初期化コード自体がif文で分岐してるとかなら初期値を与えることはあるが、それは例外的な措置だよね。
>>923,925
844のBitmap、Graphicsインスタンスのnull初期化は不要だよな？
- 934 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 13:32:32 ]
- まあ初期化なくす前に明らかに宣言位置を変える方向だが。
宣言位置が適切なら、多くの場合その場で初期化が自然になるからね。
もちろん意味のない初期化はするべきではないけど、
上記を守れば基本的にそういうのはなくなるはず。
- 935 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 13:54:00 ]
- >>933
>844のBitmap、Graphicsインスタンスのnull初期化は不要だよな？
不要だがあったからといって悪い訳じゃない
- 936 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 14:05:32 ]
- チームの規約にしたがうレベル
戦ってまで守らなきゃいけない一線ではないわな
どっちにしても
- 937 名前：デフォルトの名無しさん [2009/11/18(水) 14:14:32 ]
- >>935
>Bitmap、Graphicsインスタンスのnull初期化は不要だよな？
インスタンスのnull初期化じゃなくて変数(ry
- 938 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 15:48:55 ]
- formAからｆｏｒｍ.showでformBを開いた後、不要になったformBを×ボタンで閉じます。
この時、formBのリソースは全て開放されますよね？。
- 939 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 15:52:19 ]
- ガベージコレクションの対象にならないと開放されないよ。
- 940 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 15:57:44 ]
- そのｆｏｒｍという変数を保持している間は解放されないかな
- 941 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 16:17:16 ]
- 規定のインスタンスを使用している場合はアプリ終了まで保持されるというような話を聞いた。
（でもこれってたしかVB.NETの話だったような。C#では規定のインスタンスってあるんかな？)
- 942 名前：938 mailto:sage [2009/11/18(水) 16:27:33 ]
- そうなるとFormClosingでちゃんと処理してから終了させた方が良さそうですね
ありがとうございます
- 943 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 16:32:41 ]
- >>942
いあ、そういふ意味ではないんだが。
Form_Closingイベントとメモリ開放は別話。
- 944 名前：デフォルトの名無しさん mailto:sage [2009/11/18(水) 16:43:17 ]
- >>942
モーダルダイアログならFormAで次のようにFormBを表示し
using(Form formB = new FormB())
{
formB.ShowDialog(this);
}
formBフォームの中で使ってるリソースはFormBのDisposeをオーバーライドして行う
|
 |
