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

2019/07/16(火) 15:36:23.74

!extend:checked:vvvvv:1000:512
次スレを立てる時は↑を2行冒頭に書くこと（1行分は消えて表示されない為）

「どんなにくだらないC#プログラミングやVisual C#の使い方に関する質問でも誰かが優しくレスをしてくれるスレッド」です。
他のスレッドでは書き込めないような低レベルな質問、
質問者自身なんだか意味がよく分からない質問、
ググろうにもキーワードが分からないなど、勇気をもって書き込んでください。
内容に応じて他スレ・他板へ行くことを勧められることがあります。ご了承下さい。
なお、テンプレが読めない回答者、議論をしたいだけの人は邪魔なので後述のC#相談室に移動して下さい。
C#に関係の無い話題や荒らしの相手や罵倒レスはやめてください

>>980を踏んだ人は新スレを建てて下さい。
>>980が無理な場合、話し合って新スレを建てる人を決めて下さい。

■関連スレ
C#, C♯, C＃相談室 Part95
https://mevius.5ch.net/test/read.cgi/tech/1508168482/

■コードを貼る場合は↓を使いましょう。
http://ideone.com/
https://dotnetfiddle.net/
■前スレ
ふらっと C#,C♯,C＃（初心者用） Part143
https://mevius.5ch.net/test/read.cgi/tech/1558002486/

■情報源
https://docs.microsoft.com/ja-jp/dotnet/standard/class-libraries
https://docs.microsoft.com/ja-jp/dotnet/csharp/language-reference/index
https://docs.microsoft.com/en-us/dotnet/standard/class-libraries
http://referencesource.microsoft.com/

VIPQ2_EXTDAT: checked:vvvvv:1000:512:: EXT was configured

2019/09/21(土) 15:21:35.32

>>721
Producer-Consumerパターンを使って特定スレッドでだけ処理するのはどう？
BlockingCollection<T>使うと楽に実装できるよ

2019/09/21(土) 15:59:40.26

Dllimortで呼び出す関数の引数にはジェネリックは使えないのでしょうか？

2019/09/21(土) 17:32:33.11

使えない

2019/09/21(土) 17:41:38.79

>>724
どんな使い方を想定してるの？
っていうかそれ、呼び出されたアンマネージド関数側はどうしたらいいの？www

2019/09/21(土) 17:50:18.96

>>726
.NETのジェネリックは実行時に特殊化されるから、技術的にはそんなに無茶な話ではないよ

2019/09/21(土) 17:57:47.91

まあどのみち今のところできないから、いんぽ関数自体はIntPtrで受け取って
Marshal.PtrToStructureとかでごにょごにょするジェネリック関数でラップする感じでやるしかないね
SystemParametersInfoとかでそういう欲求があること自体はわかる

2019/09/21(土) 17:59:11.73

>>727
それじゃ質問者に回答してやってくれ

2019/09/21(土) 18:12:58.46

>>729
質問者を馬鹿にするほどには突飛な発想ではないということだよ
.NETのジェネリックがどのように実装されているかを知っていれば、できてもいいと考えてもおかしくはない

2019/09/21(土) 18:32:15.65

十分突飛だと思いますけどね。

これ、アンマネージドに限らない。
マネージドだろうが、他のアセンブリで既に定義されてる
非ジェネリックなstaticメソッドをジェネリック化したいと言ってるのと同じ。

だから、呼ばれた側は何すりゃいいんだよw

2019/09/21(土) 18:43:55.34

>>731
CのAPIだと、サイズが同一の異なる型を一緒に扱ったり、条件に応じてポインタの指す型の解釈を変えたりとか珍しくないでしょ

2019/09/21(土) 18:44:55.69

>>732
そういうのは共用体とかオーバーロードで解決できるでしょ。
ジェネリックを使うって発想は意味不明だよ

2019/09/21(土) 18:46:58.39

>>726
具体的にはLPVOID引数で自己責任ながら参照型からref、outやintなどの値渡しまで色々な型のオーバーロードを作って実際にちゃんと機能しています
なので一々オーバーロードとそのラップだけで30行ほど使ってしまってる現状が無駄に思えてどうにか方法ないかなと

2019/09/21(土) 19:46:47.84

一例としてはこんな感じ

APIの想定される構造体がBlittableのみならマーシャリング不要なので
unmanaged制約にしてアドレス渡しの方が効率は良い

public static T GetSystemParametersInfo<T>(SPI uiAction, uint uiParam, SPIF fWinIni) where T : struct {
　　T value;
　　var buffer = Marshal.AllocCoTaskMem(Marshal.SizeOf<T>());
　　SystemParametersInfo(uiAction, uiParam, buffer, fWinIni);
　　value = Marshal.PtrToStructure<T>(buffer);
　　Marshal.FreeCoTaskMem(buffer);
　　return value;
}

まあぶっちゃけお行儀的にはオーバーロードだらだら並べる方が正解ではあるんだけどねｗ

2019/09/21(土) 19:58:38.68

c# で必殺技みたいなメソッドがあったのですが思い出せません。どなたかご存知ないでしょうか？

2019/09/21(土) 20:03:15.85

>>736
リフレクション？

2019/09/21(土) 22:34:09.60

>>736
privateメソッドを外部から無理矢理呼び出すやつ？

2019/09/21(土) 22:42:59.80

リフレクションならDynamicInvoke()がなんともかっこいいメソッド名
>>736のはおそらくはデストラクタあたりを想定してるんじゃないかと思うけど

2019/09/21(土) 23:13:56.71

DangerousAddRefもなかなか

2019/09/22(日) 01:11:49.71

>>736
オーバーライド！！

2019/09/22(日) 01:26:33.02

オーバーロード！！

2019/09/22(日) 01:39:55.72

>>736
ラムダ式！

2019/09/22(日) 07:51:48.39

>>718
>>719
さんきう
とある製品のRDBMSのSQLに独自の書き方やら関数やら記法やらが混じってるからなんとかしたかったんだけどライブラリとやらを使えばいけるのかな。

2019/09/22(日) 08:33:40.66

1) githubで"sql parser"と検索
2) C#で絞り込む
3) 上から順に試す
4) 好みのが見つかったら改造を試みる

少なくとも1から作るよりは圧倒的に楽だし勉強にもなるよ

2019/09/22(日) 09:28:54.26

>>726
dllってことは既にasm化されてるんだから型なんて関係なく受け取ったもんを問答無用でバイナリとして処理するだけだろ
C#はそういうのをコーディング段階で防いでるだけであって

2019/09/22(日) 12:11:14.62

>>746
～だろ
馬鹿ほどぞんざいで偉そうな口調なのは法則なのかねwww

2019/09/22(日) 12:16:38.76

少なくともC#のジェネリックはstringとbyte[]を透過的に扱ったり、
longをint2つと透過的に扱ったりするための仕組みじゃありませんwww

だからそういうのは共用体やオーバーロードを使うんだってw

2019/09/22(日) 12:28:14.42

端的に答えとけばいいんだよ
余計な講釈を垂れるな

2019/09/22(日) 22:36:43.63

BitmapをBitmapDataにしたいのですが、使用されたパラメータが有効ではありませんと言われてしまいます。
ファイルから読み込んだbitmapは変換できるのですが、このようにコード上で作ったbitmapはダメです。
何がいけないのでしょうか？

Bitmap trimBmp = baseBmp.Clone(rect, baseBmp.PixelFormat);
BitmapData bitData = trimBmp.LockBits(rect, ImageLockMode.ReadOnly, PixelFormat.Format32bppArgb);

2019/09/22(日) 22:44:27.94

rectで切り出した後、その切り出した後のビットマップから
さらにrectで一部分のロックしようとしてるけど大丈夫？　全面じゃない？

2019/09/22(日) 22:53:39.23

>>751
baseBMPからrectで切り抜いてますが、trimBMPにとってはrectは全面だと思うのですがダメなのですかね？

でも
Bitmap bmp = new Bitmap(100,100)
みたいな素画像でもやっぱりLockBitsできないです

2019/09/22(日) 23:53:45.92

>>752
>trimBMPにとってはrectは全面
ではなくbaseBmpのrectの範囲って自分で指定しているよね
BitmapData bitData = trimBmp.LockBits(new Rectangle(0, 0, trimBmp.Width, trimBmp.Height), ImageLockMode.ReadOnly, trimBmp.PixelFormat);
>Bitmap bmp = new Bitmap(100,100)
>みたいな素画像でもやっぱりLockBitsできないです
できる。そっちもちゃんと引数の指定してみて

2019/09/23(月) 01:34:07.15

private Point CalcPopupLoction(Form launcher, Form popupForm) {}

ボタン押したときに上部に表示するモーダルな popup フォームの位置を計算するメソッドを書きました。

ボタンが左の方にあると、親フォームの左枠をはみ出さないように、popupForm は左揃えで表示。
真ん中へんにあると中央揃え、右端ならボタンとpopupForm を右揃えで表示する感じです。

1つ目の引数の良い名前が思い付きません。
主にボタンを想定してる引数です。
launcher （発射台）はどうもしっくりきません。
parent や baseForm はなんかおかしいのでやめました。

2019/09/23(月) 03:14:26.31

>>754
そもそもそんなメソッドいらんでしょ。

ポップアップがどこに表示されるべきかはボタンを持つFormが知っているんだから
そいつに聞く（そいつのプロパティーとして実装）のが一番

2019/09/23(月) 03:28:32.57

あー、右に揃える場合もあるのね。
じゃあボタン持ってるFormに

void ShowModalPopup(Form popup)
みたいなメソッドを持たせればいい

2019/09/23(月) 03:42:05.85

こういう使い回しが効きそうなのはいちいちフォームに実装しないで拡張メソッドにでもした方が

2019/09/23(月) 03:56:41.99

本当はボタン持ってる方のFormに
interface IDialogOwner:IWin32Window
{
　　HorizontalAlignment DialigHAlignment { get; }
　　Rectangle Rectangle { get; }
}
みたいなのをかぶせて、Popupする方のFormでShowDialogをオーバーロードして
void ShowDialog(IDialogOwner owner)
みたいにするのがいいのかも。

酔っぱらってるんでよく分からなくなってきたw

2019/09/23(月) 08:49:12.84

C#でawaitに対応していないメソッドをawaitで非同期実行できるようにする方法を教えてください．

以下のようにしてもActionが実行されませんでした．
private async static void SomeHandler()
{
Console.Write("Burn");

await new Task(new Action(() =>{
Console.Write("[Start]");
Thread.Sleep(2000);
Console.Write("[Completed]");
}));

Console.Write("End");
}

2019/09/23(月) 08:58:35.82

https://docs.microsoft.com/ja-jp/dotnet/api/system.threading.tasks.task?view=netframework-4.8#creating-and-executing-a-task

2019/09/23(月) 12:17:14.67

bitmapinfo とか
dibsection とか
bitmapinfoheader とか
勉強してみたら

**デフォルトの名無しさん** (ｽﾌｯ Sd22-J2vi) · 2019/09/23(月) 14:47:11.75

>>759
TaskCompletionSourceだっけ？うろ覚えですまんが、jsのPromiseオブジェクト的に使える。

2019/09/23(月) 15:20:09.58

taskをstartしてないだけだから...

2019/09/23(月) 15:46:49.67

>>763
ありがとうございます！

2019/09/23(月) 18:09:27.70

どういたしまして！

2019/09/23(月) 19:11:24.16

ID:QdBzCxoka
何の話してんのこの人
>>754
変数名が思いつかないとかはこのスレじゃなく
クラス名・変数名に迷ったら書き込むスレ。Part28 [無断転載禁止]©2ch.net
https://mevius.5ch.net/test/read.cgi/tech/1494147712/

2019/09/23(月) 22:40:56.88

>>766
交通整理バカw
設計が間違ってるのに名前付けて何の意味がある

2019/09/23(月) 22:48:38.96

>>766
レスありがとうございます。
そんなスレあるの知らなかった。

2019/09/23(月) 23:35:14.83

>>754
private Point CalcPopupLoction(Form launcher, Form popupForm) {}

上記は下部のボタンの位置に応じて、大きさのことなるpopupForm の表示位置を計算するメソッド

exportButton_Click 内で、
ShowDialog(); で表示する popupForm に戻り値を設定

https://i.imgur.com/ggjo4td.jpg

2019/09/24(火) 00:01:07.45

>>754
・Loctionて何
・ボタンの置いてあるオーナーウィンドウと対象のボタンを引数に取った方がいいと思う
・計算だけなら全部Rectangleでいいんじゃないの
Rectangle CalcPopupLocation(Rectangle Owner, Rectangle TargetButton, Rectangle Popup)

2019/09/24(火) 00:03:53.21

>>769
だからさ、そのメソッドをどこに書くおつもり？

ポップアップを左右どちらに寄せるべきかを知ってるのは特定のFormだけのはずなのに
その例のllauncherの型がFormってのは変じゃないか？

そもそも何で「左右どちらに寄せるべきかを知ってる」奴のメソッドとして実装しないの？
それなら引数で渡す必要ないと思うんだけど。

2019/09/24(火) 00:09:30.85

>>770
必要な情報は

(1) 左右どちらに揃えるか？(HorizontalAlignment)
(2) 揃える位置のx座標

だけだね

2019/09/24(火) 11:39:03.69

書き込みは、人を憎んでコードを憎まずのこころで行いましょう。

たとえクソコードであっても悪いのは書いた人だけです。
コードやプログラミング言語に罪は一切ありません。
コードの悪口はやめましょう。
クソコードの修正と言わず、リファクタリングといえばいいのです。

大量殺人犯や c# プログラマーの醜い心はリファクタリング不可能ですが、コードのリファクタリングはカンタンですよ。

https://www.google.co.jp/amp/s/anond.hatelabo.jp/touch/20180319181104%3fmode=amp

2019/09/24(火) 12:11:17.28

性格の悪さもリファクタリングしてくれ

2019/09/24(火) 14:58:01.43

性格はコードに滲み出るよ

2019/09/24(火) 23:29:02.86

性格は分からないけど姿勢みたいなものは見えると思ってる
こんなコピペばかりして少しはおかしいと思わないのかな？とかこんな古い書き方して言語の文法ざっとでもいいから一通り見たのかな？とか
いいコードを書こうとしてる人とそうでない人の差ってすごくよく分かる。本当に能力無くて頑張って頑張ってるのにひどいって場合もきっとあるんだろうけど

2019/09/24(火) 23:44:57.44

ようわからんけど動いてるからヨシ！

2019/09/25(水) 07:36:43.68

式木がイケてるらしいですが使いどころが解りません
プロの皆さんはどこで使ってますか？

2019/09/25(水) 10:16:37.93

>>778
使わない
高度なフレームワークを作るための機能であり、ドカタグラマには全く必要ないから忘れていいよ

2019/09/25(水) 22:57:19.53

>人を憎んでコードを憎まず

逆じゃないんかおい

2019/09/25(水) 23:01:24.46

自分より給料貰ってる先輩（笑）が
クソコードを産み落としていた場合は人もコードも憎む
というか蔑む

2019/09/25(水) 23:16:09.44

何をもってくそコードと言ってるのか

2019/09/25(水) 23:38:23.64

何を以ってクソコードかは難しいところだが具体例なら無数に挙げられるぞ

switchを使わず延々とif-else if分岐
if(flag) return ture; return false;
一行違うだけのメソッド大量生成
マジックナンバーだらけ
コレクションはほとんどList
かと思えば特にライブラリに必要とされているわけでもないobject[]
もちろんパターンマッチングなんて概念は存在しない
out代わりにrefを使う
boolではなくintをフラグとして使う
クラス内のあらゆる場所から明示されず書き換えられるフィールド
どう見ても親子関係にない継承
メソッド1個300行以上
これがさぞ大変な処理かと思えばリファクタリングしたら40行で書けた
1行に300文字以上
リソースでもFormsでもない普通のクラスが4000行

2019/09/25(水) 23:54:44.99

>>778
ドカタだけどlinqのwhereメソッドの引数を動的に生成するときに使った

ここ参考になるよ
https://www.atmarkit.co.jp/fdotnet/dotnettips/986dynamiclinq/dynamiclinq.html

2019/09/26(木) 00:03:29.50

>>783
ぜんぜん許容範囲だな！
その程度でくそ呼ばわりされたらたまらん

2019/09/26(木) 00:08:48.21

>>783
>>switchを使わず延々とif-else if分岐
>>マジックナンバーだらけ
>>クラス内のあらゆる場所から明示されず書き換えられるフィールド
>>メソッド1個300行以上
>>リソースでもFormsでもない普通のクラスが4000行
今直面してるのにまさに当てはまっててもう

>>コレクションはほとんどList
こっちはListどころか配列しか使わないんだぜ。もう書き方が完全にC

2019/09/26(木) 00:09:16.61

メソッド以外はどれも影響は大きくないし
多少大きくてもテストできればなんとかなる
ちょっとした文化の違い程度の話だ

2019/09/26(木) 00:10:34.86

初心者が一見してくそ呼ばわりするコードはたぶんくそじゃない
わかるから細かい粗が目に付く

普遍的恐怖を知るべきだ

ぐちゃぐちゃな循環参照
テストできないメソッド

う　そ　っ　ぱ　ち　の　名　前

2019/09/26(木) 00:15:27.94

突然現れるlinqの長々としたクエリ式
せてめメソッド式にしてくれ

2019/09/26(木) 00:18:57.61

しかしＣ＃いつのまにやらパターンマッチまで入ってたんか…

2019/09/26(木) 00:28:52.23

sqlを全て文字列結合で作ってるうえに500行以上あったときは後輩に丸投げした

2019/09/26(木) 03:16:32.66

>>780
バグやコードを憎んで、人を憎まず。

こんなこと言ってるプログラマーは意識高い系の偽善者のクズ。

2019/09/26(木) 11:30:46.50

あいつに任せた俺の責任だからって最初から頭数に入れていないから憎みもしない。必要なら全面書き直しもいとわない。

2019/09/26(木) 20:59:42.99

なんでListがいかんのか？

2019/09/26(木) 21:12:02.17

ビットで判定とるのがいかんのか？

2019/09/26(木) 21:18:55.30

7f63-aXvPからクソグラマーの臭いがする

2019/09/26(木) 21:21:22.68

>>794
時と場合を考えて最適なコレクションを検討しろ

>>795
特別な理由がないなら褒められないな

2019/09/26(木) 21:37:16.13

ZDDをboolの配列で評価するの？orz

2019/09/26(木) 21:40:05.70

>>798
特別な理由がないならと言ってるだろうが

2019/09/26(木) 22:03:13.49

windows formsの書き方についてなんですけど
メイン画面にタブがいっぱいあってそれを切り替えて……みたいなアプリなんです
んで中身見たらFormクラスが1万行オーバーで。画面一つだからって何もかも突っ込んでる感じなんです
画面というかフォーム一つで1クラスみたいなのが通常の書き方なんでしょうか

2019/09/26(木) 22:10:14.52

普通はブロックに分ける。

2019/09/26(木) 22:13:41.16

>>800
タブ毎にユーザーコントロールを作ればスッキリはすると思うよ

2019/09/26(木) 22:23:35.32

>>800
「通常」はどうか知らないけど例えば
MDIアプリケーションを作成する
https://dobon.net/vb/dotnet/form/mdiapplication.html
こういう作り方もある
あと一つのフォームに普通のコードはともかくコントロール(InitializeComponentのなか)を多く詰め込みすぎると起動に時間がかかるようになる
現状で問題ないのならそこは気にすることは無いが

2019/09/26(木) 22:53:44.29

>>800
parcial class でタブごとにファイルわける。

2019/09/26(木) 23:38:48.19

なるほど参考になりました。
今思えば既にタブごとに部分クラスわけしてファイル別になってたような気がしないでもない……となると実質一クラス10万行か。笑えない

2019/09/27(金) 06:43:42.33

>>800
1000行超える時点でオブジェクト指向できてない証拠

2019/09/27(金) 08:33:35.75

フォーム以外で使わない処理を
わざわざ外部のファイルやクラスに作る必要なんてある？

2019/09/27(金) 08:42:46.73

>>807
それで可読性が高まるならやる意味はある

**デフォルトの名無しさん** (ｽﾌｯ Sd7f-mDN4) · 2019/09/27(金) 09:06:13.84

C#は1ファイルに全部書けるので必要ないです

2019/09/27(金) 10:07:45.04

再利用しないならそれでいいんじゃね？

2019/09/27(金) 22:47:27.06

お前を含めた全人類が未来永劫そのコードに手を加えない保証があるなら好きにすればいい

2019/09/27(金) 23:28:58.48

ググったらタブ大量にあるような場合>>802がクラス分割のためのメジャーな手法なのね

2019/09/28(土) 08:43:54.27

游ゴシックの英数文字をMeasureStringで取得した矩形を使ってDrawStringすると高さ不足で表示されない

矩形を使わなきゃ表示はできるから良いけどもやもやする

2019/09/28(土) 10:37:47.08

フォント周りおかしいのはＭＳ伝統では
ないだろうか？

Ｑ

2019/09/28(土) 14:09:54.66

そんなざっくりした話じゃ、どうせお前自身のコードのバグだろって
反応しか返ってこないと思うw

2019/09/29(日) 01:52:26.51

インターフェイスと抽象クラスってどう使い分ければ良いのでしょう？

ググってもよくわかりません。

2019/09/29(日) 02:47:00.08

インターフェースなら継承関係のない複数のクラスのように
振る舞うことができる

2019/09/29(日) 02:49:22.44

大雑把言えば
プロパティとフィールド（値）を保持するのがクラス
クラスのメソッド（挙動）を公開するのがインターフェイス

2019/09/29(日) 02:54:07.57

それが抽象クラスとなにが違うのだ？と彼は聞いているわけで

2019/09/29(日) 03:01:23.20

インターフェイスは値を持たないし
原則として挙動の実装も持たせない

2019/09/29(日) 03:20:49.12

>>816
使い分けで悩む必要はないと思うよ。
インターフェイスで目的が達成できるならあえて抽象クラスを使う理由はないのだから。

たぶん使い分けが分からないのではなく、インターフェイスの使いどころが分からないだけじゃないのか。
よくわからなくてもとにかく実際コードを書いて使ってみることが重要。
別に難しい話じゃないので使ってるうちに自ずと分かる。たぶん

2019/09/29(日) 03:35:49.46

多重継承的なことがしたくなったらインターフェースを使う
くらいな感じでいいのかもね