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

2020/02/19(水) 17:25:41.55

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

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

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

■情報源
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/
・Insider.NET > .NET TIPS - ＠IT
https://www.atmarkit.co.jp/ait/subtop/features/dotnet/dotnettips_index.html
・DOBON.NET　.NET Tips
https://dobon.net/vb/dotnet/index.html

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

2020/02/25(火) 15:01:43.07

おかしい…「モダン言語と現行ライブラリ下においてはシングルトンパターンを適用することは通常開発下ではまずないが」という大前提がまったく共有されていない…
もしかして10年前の感覚で脳内敵を殴りつけてスッキリ喋ってる…？

2020/02/25(火) 15:04:06.03

テメエも皮肉ってればいいと思ってんなよガキ

2020/02/25(火) 15:27:35.00

死んでもstatic使わない、じゃなくて死んでもシングルトン使わない、の話になってるが

2020/02/25(火) 17:25:56.66

singletonがデザパタの中で意義も理解しやすく実装もそこそこ分かりやすく楽で有効に利用できると思わせる範囲が広いから無駄に広まってしまった
他により有効な手段ができても単一インスタンスだからsingletonで！っていう脳死とstaticは結局グローバルだからとにかく駄目！とかいうこれまた脳死が合わさってしぶとく使われているだけじゃない？

2020/02/25(火) 19:02:03.46

どこにstaticはとにくかくダメだなんて話が出てるのか

2020/02/25(火) 20:48:36.91

GoF初版は４半世紀前のJavaすら出る前だぜ
そろそろ新しい版を誰か書いて

2020/02/25(火) 21:46:44.65

初めstaticで足りると思ったんだけど
対象のクラスに
継承やIFいんじゃんクソが
ってなるといつの間にかにシングルトンなってたりするだけだ

2020/02/25(火) 22:40:31.11

>>57
これだよね。
staticクラスやグローバル変数でもいいんだけど呼び出し側に影響させないで実装を拡張できるという。

2020/02/25(火) 23:30:16.22

>>50
シングルトンが初回アクセス時に実行されるのと同様にstaticコンストラクタもそのクラスのいずれかに初回アクセス時に実行されるから同じ事だと思うよ

2020/02/26(水) 01:29:08.48

>>59
初回アクセス時よりも前にロードされることが保証されてるのと
初回アクセスしたそのタイミングで実行されるのとは意味が違う

https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/static-classes-and-static-class-members

仕様としては明記されてないが
static classを含むnamespaceがロードされるタイミングでstatic classもロードされてたはず

2020/02/26(水) 10:56:30.52

型名の一部でしかないnamespaceのロードって何…？

2020/02/26(水) 11:28:50.62

>>60
念の為試してみたら俺が間違ってたわ
すまん

static classでもLazy<T>使えば必要なタイミングまで確実に遅延できた

2020/02/26(水) 12:01:04.52

当たり前なんだけどsingletonでしか実現できない話じゃなくてsingletonなら自然にできる話だからね
singletonじゃなくたって実現方法はいくらでもある
こうすりゃできるなんてわかりきった上で話してるんだよ

2020/02/26(水) 12:04:17.00

>>63
そのsingletonが有用な場面ってのを解説してくれ

2020/02/26(水) 12:08:38.94

>>64
環境的にstaticが使えないとか？C#じゃあんまないんじゃない？
組み込みC環境だとstatic不可とかあるよ

2020/02/26(水) 12:11:40.77

ところでsingletonが有用な場面ってどっから湧いた話？
過去レス確認したけど、俺そんなこと言ってなかったんだけど

2020/02/26(水) 12:16:01.74

>>60
メンバーの話じゃなくてコンストラクタね
static class Human
{
static Human(){} //←この部分は保証されてる
}

**デフォルトの名無しさん** (JP 0H82-SZE8) · 2020/02/26(水) 14:16:45.27

ある強力なFor関数を紹介いたす(C#)
ttp://x0000.net/topic.aspx?id=3630-0

アルファ・ラボ|学術掲示板群
（理系　文系　工学　語学）
ttp://x0000.net/

2020/02/26(水) 15:11:05.12

ただのAggregateでは？

2020/02/26(水) 17:52:39.14

>>64
昨日>>48に書いた通り俺は唯一性に強制力を持たせる必要性については
懐疑的だけど、シングルトン的な発想が有用なケースは結構あるとは思う

例えばUIでいろんな場面で表示する必要があるサブ画面みたいなのがある時、
こんなことやってる人多いんじゃないかな
public partial class Form2 : Form
{
　　public Form2()
　　{
　　　　InitializeComponent();
　　}

　　private static Form2 Singleton { get; set; }

　　public static void ShowSingleton()
　　{
　　　　if (Singleton == null || Singleton.IsDisposed) Singleton = new Form2();
　　　　if (Singleton.WindowState == FormWindowState.Minimized)
　　　　　　Singleton.WindowState = FormWindowState.Normal;
　　　　Singleton.Show();
　　　　Singleton.BringToFront();
　　}
}

2020/02/26(水) 18:15:26.79

>>70
それはシングルトンじゃねーよ
まずシングルトンはnewしない

2020/02/26(水) 18:19:20.31

>>71
一回目は作るべ？

2020/02/26(水) 18:21:41.53

>>71
揚げ足取りはいいけど、よく読んでからにして欲しいよね

2020/02/26(水) 18:22:19.21

>>72
つか、インスタンスは自分自身の中にスタティックで用意する
そのクラスのスタティックプロパティー経由でインスタンスを参照するが
最初の参照時にシングルトンクラスで自身の中に生成する

だから外部で使うクラスはインスタンスを明示的に生成しません

2020/02/26(水) 18:30:13.02

> インスタンスは自分自身の中にスタティックで用意する
private static Form2 Singleton { get; set; }
用意してるな

> 外部で使うクラスはインスタンスを明示的に生成しません
public static void ShowSingleton()
{
　if (Singleton == null || Singleton.IsDisposed) Singleton = new Form2();
してないな

2020/02/26(水) 18:40:31.49

>>62
それ実体が生成されないだけでLazy<T>自体はロードされるだろ

2020/02/26(水) 18:42:00.84

シングルトンじゃなく
シングルトン的な発想が有用なケースって確かに書いてるね

2020/02/26(水) 18:50:11.01

>>77
シングルトンの最大の特徴は、複数のクラスから一つのインスタンスを参照できることだから
それはシングルトン的なものでもない

>>75
シングルトンではSIngletonクラスの内部にたった一つのインスタンスがプライベートで格納されている
それをSingletonのスタティックメソッドやスタティックプロパティーで参照するのがGOFのデザパタだ

2020/02/26(水) 19:43:16.15

シングルトンって、昔は、複数インスタンスが作られる可能性を潰すためにやってたんだろうけどさ
今は、主にDIコンテナへの指示みたいなもんとして役立ってんじゃねえのか？

2020/02/26(水) 20:21:07.70

DIコンテナ使ってるなら、DIコンテナが自動的にインスタンス化して単一のインスタンスを各所へ注入してくれるから、
GoFのシングルトンパターン(というかグローバルインスタンスアンチパターンとでも呼ぶべきもの)は全く必要ないよ

2020/02/26(水) 21:07:30.64

単一かどうかはコンテナへの突っ込み具合で違ってくるんじゃないの
シングルトンでって指定していなければ複数インスタンス化されちゃうでしょう

2020/02/26(水) 21:12:48.10

属性などで指定するだけだよ

2020/02/27(木) 08:46:08.57

>>77
グローバル変数を単にフォームにおいてアクセス制御しただけで次とほぼ変わらないな
staticでいいじゃん側の話に思える

private static Form2 _form;
public static Form2 Hoge{
get{
if(_form == null || _form.IsDisposed)_form ==new Form2();
return _form;
}
}

2020/02/27(木) 08:57:22.75

もうめんどくせえからstaticも中でシングルトン使ってるから
実質シングルトンパターンって事でいいよ

2020/02/27(木) 12:49:09.42

日本人はすぐそういう党派的（どっちの側にお前はつくんだ？｝で幼稚な話になるよねw

>>83
そういう話じゃないってw
>>70のコードびShowSingletonというメソッドはもちろんForm2以外のクラスに
書くことも出来るけど、それが分かりやすいですか？

こういう話だから。

シングルトンがあんまり有用じゃないとしても、
「唯一のオブジェクトを管理したり窓口になったりする役割をそのクラス自身に持たせる」
というシングルトン的なアイデアは応用できる、と言ってるんだけど、
これがそんなに難しい話かなw

2020/02/27(木) 17:27:49.50

そのアイデアを実現する手法としてstaticはどうだって話してたんじゃないないのか

2020/02/27(木) 19:49:51.32

>>85
日本人はーっていうのも相当アホっぽいけどなw
外人だってエディタとかLinux vs. BSDとか宗教戦争しまくり
てか中東の内戦なんてガチの宗教戦争だし

2020/02/27(木) 20:14:51.21

うっかり2つ作らせない方法だからな

2020/02/27(木) 22:15:47.05

>>87
日本人と欧米人の論争の仕方の違いは明らかで、
それは別に俺のオリジナルの見解でも何でもないよw

今のこの話なら、日本人のダメなところは「シングルトンパターンの利点はここで問題点はここで」
という議論が苦手なこと。
日本人は一度決めた「立場」に固執して全肯定か全否定かになりやすい。
好き嫌いを持ち込むべきでない評価に好き嫌いを持ち込みがち。
エディターとかはいいんよ別に好き嫌いで語れば。技術的な話はおかしいでしょ。

板違いだからもう止めるけどね。

2020/02/27(木) 22:46:12.99

>>89
> 今のこの話なら、日本人のダメなところは「シングルトンパターンの利点はここで問題点はここで」
> という議論が苦手なこと。
それでどちらかに決まるなら宗教戦争なんてすぐに解決してるよw
そもそも技術的な話とか言うわりには
> それが分かりやすいですか？
なんて人によって意見が違うことを言い出すから戦争はいつまで経っても終わらないわけで…

2020/02/27(木) 23:52:28.63

>>89
まあ、外人ならってのは幻想だな
そもそも奴らがそんなに頭が良かったのならオブジェクト指向なんて流行ってない

リーナスが言ったようにオブジェクト指向はカルトだ
誰もメリットなんて説明できない
それでも海を超えて日本へ来たのさ

2020/02/28(金) 00:58:28.47

>>90-91
なんか党派的主張のお手本みたいなレスで何だかなあという気持ちになるけど
言っても無駄だねたぶんw

2020/02/28(金) 06:43:53.39

>>92
技術的なお話とか言う割には
> なんか党派的主張のお手本みたいなレスで何だかなあという気持ちになるけど
みたいなレスしかできない時点で…

2020/02/28(金) 09:46:17.62

いつもいつも同じ流れでよく飽きないですねカスども

2020/02/28(金) 14:13:28.76

シングルトン的なものが必要になったら、殆どはDIのインジェクションでなんとかするけど
コレクションの要素にインジェクションなんて鬱陶しいから普通のシングルトンやStatic Class使ったりするな

**デフォルトの名無しさん** (ｽﾌｯ Sdea-S34E) · 2020/02/28(金) 16:17:34.85

シングルトンに依存するコレクションのエレメントとかちょっと悪夢だな。

2020/03/01(日) 05:50:05.37

これが最新ぽいな

2020/03/01(日) 07:54:47.18

42　デフォルトの名無しさん　　2020/03/01(日) 05:43:23.78 ID:wVHwYgLC
なんだ？この糞スレ
と思ったら2016年だとwww
どこから引っ張り出したんだよ

2020/03/02(月) 10:29:43.95

C#というより.NET Framework全般の問題かもしれませんが・・・
ビルドした実行ファイルの拡張子を7ｚにして開けると
　　　.rsrc（フォルダ）
　　　.text
　　　.reloc
というファイルが出てきます

このうち「.text」をエディタで開けると

「C:\Users\ユーザー名\source\repos\プロジェクト名\obj\Release\実行ファイル名.pdb」

の文字列がそのまま入っておりユーザー名まで見えてしまいます

プロジェクトのプロパティなど見てみましたが該当する文字列は設定されていませんでした

この文字列が入らないようにビルドするにはどうすれば良いでしょうか？

2020/03/02(月) 10:31:50.59

mtawmg

2020/03/02(月) 10:56:03.68

プロジェクトのプロパティでデバッグ情報の出力を「なし」か「埋め込み」にする
なしの場合はデバッグが難しいから頒布する用のリリースビルドだけにした方が良い

2020/03/02(月) 11:01:21.30

ググっただけですぐ出て来たが
https://www.atmarkit.co.jp/fdotnet/dotnettips/831stoppdbfile/stoppdbfile.html
http://var.blog.jp/archives/70196817.html
https://qiita.com/lainzero/items/27681ddc96638e33758b

2020/03/02(月) 11:30:46.56

>>101-102

ありがとうございました。
pdbファイルがなんなのか理解できました

2020/03/04(水) 15:33:24.81

yield returnの挙動について知りたいのですが、
例えばHuman[]からある苗字のHumanインスタンスだけを列挙する場合、
yield returnで貰えば新たな配列は生成されないという理解で良いんでしょうか？（無駄なヒープの生成なども無いと）

2020/03/04(水) 15:41:47.96

配列は生成されないけどIEnumerable<T>/IEnumerator<T>の実体クラスのインスタンスは作られるよ
具体的にどんなクラスになるかはildasm.exeとかで確認してみるといいよ

2020/03/04(水) 20:59:26.02

var a = new int[10];
for(......){
......
}

for(......){
var a = new int[10];
......
}

配列変数の場合はどちらの書き方が正しいでしょうか？
下のほうはループを繰り返す毎に配列を宣言してるので無駄だと思うのですが

2020/03/04(水) 21:02:11.58

>>106
意味が違うからどっちが正しいもクソもない

2020/03/04(水) 21:03:35.42

......の内容が大事だよなｗ

2020/03/04(水) 21:18:09.87

>>106
どっちが正しいかは何をしたいかによる

2020/03/04(水) 22:58:57.68

始めたばっかの頃の自分を思い出すと、おそらく>>106がやりたいのはこれ

int[10] a;
for(......){
a = new int[10];
......
}

2020/03/04(水) 23:03:21.69

全然違うと思う

2020/03/05(木) 02:53:17.71

>>106
後者はループ処理毎にaが初期化されるが
前者はループ処理毎にaは初期化されない
って違いがあるので、バグが発生しにくいのは後者で
バグは発生しやすいけど効率がいいのが前者

前のループのaを使わないなら、どちらも正しいよ

2020/03/05(木) 09:03:52.56

for内部で毎回一時バッファが必要ならfor内に書くべきだけど、ループ枚に配列生成は高コストなのでstackalloc使って良いと思うけどな

unsafe for ()
{
var a = stackalloc int[10]; //配列サイズが[200000]ほど以上になるとスタックオーバーフローの恐れが出てくるので注意
a[0] = 123;
}

2020/03/05(木) 09:55:29.22

>>110
それ106と全く同じ事だよ
予め配列変数を用意しておいたところで参照型である配列の変数なんてあくまでポインタに過ぎず肝心の中身はnew毎に生成されるからコスト的には全く変わらない

2020/03/05(木) 10:13:49.21

言語ではなくて設計のことなのですが、C#でデスクトップアプリを作っていて、canvasに描画する処理を書いてます
そこで、描画部分で状態の更新(例えばlabelで表示する値の計算)を一緒にやった方が負荷が減らせるなと思ったのですがこれは設計としていいのでしょうか？
描画部分は複雑になっていくので、そこにcanvasの描画に関係ない処理を持ってくるのはどうなのかと悩んでます
でも一連の処理で共通部分があるので出来る限り負荷を減らしたいです
canvasの描画とその他の更新でうまく分けた方がいいでしょうか

2020/03/05(木) 10:16:15.59

>>113
悪いけど、それは全く意味ないと思うよ
ループ内で配列を生成する必要があるとしたら、それを他の非同期なメソッドなどに渡していて毎回インスタンスを別にしなければならないケースだろう
stackallocだと領域が結局使い回されてしまうから、それでいいんだったら単純に最初にnewした配列をループ内で毎回クリアして使えばよい

2020/03/05(木) 10:28:10.79

>>115
描画処理は今の状態のスナップショットを画面に反映させるだけなんだろ？
そんなもん状態の更新処理と何をどう共有するんだよ
分けるかどうかは別にして、そんな頭がごっちゃになった状態で書かれたコードなんて俺は読みたくないな

2020/03/05(木) 10:30:44.90

>>116
ループ内で生成しないやり方と迷ってるってことはret時にもループ内でも他に渡す盤面は無いってことじゃないかな
なので最高のやり方はSpan<int> a = stackalloc int[10]として最初に確保をしてループ毎にa.Clear()
>>113のやり方も悪くは無いけど使いまわせるのに毎回stackallocするのは無駄

2020/03/05(木) 11:02:10.43

>>117
すみません誤解を招く書き方でした
状態の更新ではなくviewの更新です
viewで表示する値を計算するのに一連の処理が必要ということです
改めて考えてみたら元のデータは頻繁に変わるものではないのでメモ化で対処できそうです

2020/03/05(木) 15:17:30.80

自分で使う用のWindows用WinFormsくらいしか作らない俺もSpan利用したいんだけど、そろそろ.NetCore3.1インスコしてええの？
CoreのWinFormsは不完全みたいな事どっかで聞いたからもうかれこれ1年以上.Net Framework4.7.2しか使ってないんだけど

2020/03/06(金) 12:20:46.76

System.Memoryパッケージ入れれば、slow spanになるけど.NET FrameworkでもSpan<T>が使える

2020/03/06(金) 12:24:09.85

SpanならnugetからSystem.Memoryを入れればNET47でも使えるんじゃ？

2020/03/06(金) 12:24:46.59

あ、かぶった

2020/03/10(火) 13:54:33.08

Hoge[] h = new Hoge[]{...};
ArraySegment<Hoge> slice_of_h = new ArraySegment<Hoge>(h, a, b);
Hoge[] new_instance = slice_of_h.ToArray();

2020/03/11(水) 20:14:38.79

FormアプリケーションのGUIをそのままWebサイトにできますか？

2020/03/11(水) 20:15:50.74

無理

2020/03/12(木) 13:35:24.24

https://docs.microsoft.com/ja-jp/archive/msdn-magazine/2017/september/net-standard-demystifying-net-core-and-net-standard

2020/03/12(木) 14:08:08.81

>>127
懐かしい話だな
.NET Standard はあろうことか 2.1 で .NET Framework を切り捨てたから、もはや .NET Standard には何の意味もなくなり事実上消滅したよ

2020/03/12(木) 14:19:52.98

まだ消滅してないよ

2020/03/12(木) 14:40:04.55

どれがホント？
https://blog.ecbeing.tech/entry/2019/03/28/181956
https://blog.hiros-dot.net/?p=9086

2020/03/12(木) 15:00:48.26

>>130
下の方は誤解を招く書き方(というか本人が理解してなさそう)だね
.NET5は.NET CoreとXamarinの新バージョンを含むが、.NET Frameworkは含まない
.NET Frameworkは4.xが最後のバージョンとなり、(Xamarinを除けば).NET Coreへ一本化される

2020/03/12(木) 20:38:24.41

＞　Xamarin は、テクノロジー自体は、.NET Framework と C# を基盤としていますが、
＞　Mono というライブラリを使用します

2020/03/12(木) 23:53:50.67

>>132
それは明確に間違いだね
Monoは元々MSとは無関係の会社による.NET Frameworkの非公式かつ独自な互換実装で、.NET Frameworkとはソースを共有していない
based on かなんかを誤訳したんじゃね

2020/03/13(金) 00:04:41.45

テクノロジーを基盤してるって書いてるだけじゃん
ライブラリがおかしい

**デフォルトの名無しさん** (ｽﾌｯ Sda2-Wi6o) · 2020/03/13(金) 07:54:41.51

>>133
今はだいぶ本家のを使ってるよ。Monoも。
本家がMITになったからね。

2020/03/13(金) 12:12:59.24

毎度のことだけど用語の定義のあいまいさからくる混乱だろうね

>>132の引用部分を書いた人は「NET FrameworkをVS2002とかの時代の意味で使ってるんでしょ。
当時は仮想OSみたいに言われてた

その定義だと.NET Coreも.NET Frameworkになっちゃうので（以下略

2020/03/14(土) 03:14:07.89

C#の正規表現についてですが...

new Regex(@"^a").IsMatch("abc",0) → true : 納得。
new Regex(@"^a").IsMatch("abc",1) → false　: 納得。
new Regex(@"^b").IsMatch("abc",0) → false : 納得。
new Regex(@"^b").IsMatch("abc",1) → false : 納得...できん！！

第2引数が1なら、^で示す入力文字列の先頭が2文字目になってて欲しいじゃん！？
MSDNとか、この挙動について何か書いてある？見つけられなかったわ。
substringとかしたくないんだよな...

なんでこういう挙動になるか、ご存じの方いますか。

2020/03/14(土) 03:47:52.84

こういう理由らしい
https://stackoverflow.com/questions/5884922/regex-match-startat-and-start-of-string

2020/03/14(土) 03:55:38.88

第二引数は探索の開始位置を指定するのであって検索範囲をどうこうするためのもんじゃないってことかな？
でもその理屈だと二つ目がfalseである説明にならんし流し読みだから解釈間違ってるかもしれないわ

2020/03/14(土) 04:35:36.35

139は間違いっぽいので忘れて
IsMatch("abc",0) → abcを探索
IsMatch("abc",1) → bcを探索

リンク先を見るに^は行頭を表すもんでbは行頭ではないのでfalse
aは探索外なので二つ目もfalseってことかな

https://docs.microsoft.com/en-us/dotnet/standard/base-types/anchors-in-regular-expressions

2020/03/14(土) 09:39:43.32

new Regex()するときにRegexOptions.Multilineを指定して『^』『$』の意味づけを変更・・・しても駄目そうだね
日本語のほうで
https://docs.microsoft.com/ja-jp/dotnet/standard/base-types/regular-expression-options?view=netframework-4.8#multiline-mode
を読んだけど、
あくまで『^』は入力文字列全体（>>137のなら"bc"ではなく"abc"）の先頭、MultiLine指定時は\nを行区切りとした行頭についてのみマッチするんだろう

**137** (ﾜｯﾁｮｲ 61dd-gN7l) · 2020/03/14(土) 11:29:21.74

>>138-141
ありがとう！
俺だけではそれらの情報を見つけられなかったわ。

いったん \G を使用することで何とかなりました。
というか \G の仕様(影響範囲)もよくわからん。
最終のマッチってRegexのインスタンスに保存されるのかな？
そうだと思うけど、今一つ確証が持てない。

あと、 \A が正しいんじゃないか、って思いましたが、\Aは「文字列の先頭」でしかないんだね。
Singlelineの場合は^と同じみたい。

2020/03/17(火) 10:51:26.76

プログラムではなくライセンスについての質問なのですが、NAudioというライブラリを使いたくてプロジェクトに導入したのですがMs-PLライセンスとなっていて、この場合著作権表示等は具体的にどう表示すればいいのでしょうか。
著作権表示自体がいまいち分かりません。ライセンスの文書(LICENSE.txt等)とは別なのでしょうか。
調べているのですが具体的な表記方法が書かれている記事が無く分かりません。
ご教示お願いします。

2020/03/21(土) 13:43:45.29

相談させてください。
以下のような MyClass1 クラスを作ると、
「IDE0054 複合代入を使用」というメッセージが表示されます。

class MyClass1 {
　　int MyProperty { get; set; } = 0;
　　public void IncreaseMyProperty(int x)
　　　　=> MyProperty = MyProperty + x; // ここでメッセージ
}

まあそりゃそうでしょうということで以下の MyClass2 のように直すと、今度は
「IDE0052 C# プライベートプロパティは、
　その get アクセサーが呼び出されることがないため、メソッドに変換できます。」
というメッセージが表示されてしまいます。

class MyClass2 {
　　int MyProperty { get; set; } = 0; // ここでメッセージ
　　public void IncreaseMyProperty(int x)
　　　　=> MyProperty += x;
}

なんだそりゃと思いつつ言われたとおり以下の MyClass3 のように直せば
メッセージは表示されなくなるのですが、こんな直し方でいいのでしょうか。

class MyClass3 {
　　int _MyProperty = 0;
　　void SetMyProperty(int value) => _MyProperty = value;
　　public void IncreaseMyProperty(int x)
　　　　=> SetMyProperty(_MyProperty + x);
}

ご意見をお聞かせいただけると幸いです。よろしくお願いいたします。

2020/03/21(土) 18:45:34.77

MyPropertyが本当に自動実装プロパティなら
そもそもフィールドでいいんでは？

2020/03/21(土) 19:30:58.03

最近のIDEは賢いから定義されてるけど使用されないことが明らかなプロパティやメソッドが見つかったら警告が出るようになってる
>>144のは文法等のお勉強でてきとうなソースを書いてるんだと思うけど、MyClass2なら
MyPropertyをpublicなプロパティにするか、MyPropertyを参照するような処理をちゃんと書けば警告解消するはず
ToString()あたりをオーバーライドして override string ToString() => MyProperty; とか書いてもいいかもね

2020/03/21(土) 20:37:06.12

>>145
レスありがとうございます。
説明が不十分で申し訳ありません。
実際には、以下の様な感じにしていまして、
できればプロパティを使いたいと考えております。

class MyClass4 {
　　int _MyProperty = 0;
　　event EventHandler MyPropertyChanged;
　　int MyProperty
　　{
　　　　get => _MyProperty;
　　　　set {
　　　　　　if (value != _MyProperty) {
　　　　　　　　_MyProperty = value;
　　　　　　　　MyPropertyChanged?.Invoke(
　　　　　　　　　　this, EventArgs.Empty);
　　　　　　}
　　　　}
　　}
　　public void IncreaseMyProperty(int x)
　　　　=> MyProperty += x;
}

2020/03/21(土) 20:37:42.93

>>146
レスありがとうございます。
おっしゃるとおりなのですが、MyProperty += x; で
MyProperty の get アクセサーが参照されていることに
ならないため困ってしまっています。
C# の文法的に、自動実装プロパティでなければ

int _MyProperty = 0;
int MyProperty { set => _MyProperty; }

のような書き方も許されているようなのですが、
MyClasss2 の MyProperty の定義をこのように書き換えると、

MyProperty += x;

のところでエラーが出てしまうのです。

2020/03/21(土) 20:45:29.55

警告とエラーを混同しないで
+=オペレータがget使ってないだけでしょ
開発中で今後使う予定がありまだ未実装なら警告はスルーすればいい

2020/03/21(土) 21:05:47.28

いやだからIDEからは「結局MyPropertyってインクリメントで更新はしてるけどどこからも参照されてないんでしょ、無駄なんじゃない？」と怒られてるわけで
他のC#ソースなりXAMLなりでMyPropertyが参照されてればそのメッセージは出ないはずだけど
例示されてるソースであればMyPropertyだけでなくインクリメントする処理も意味のない無駄な処理だから削ればいいし
GUI側etcのソースがまだ書けてないのであれば中途半端な書きかけソースで警告が出るのは当たり前のお話

というかそもそもでいえば、VisualStudioの設定がデフォルトならそのメッセージは
「コンパイルエラー」でも「警告」でもないただの注意喚起なメッセージでしかないから開発にも実行にも一切支障がないので
とりあえずほっとけばいいと思うんだけど