C#, C♯, C＃相談室 Part94

2019/03/20(水) 18:57:36.47

!extend:checked:vvvvv:1000:512
■Visual Studio 2017 Community(無償の統合開発環境)等はこちら
http://www.visualstudio.com/downloads/

■コードを貼る場合はこちら
http://ideone.com/

■前スレ
C#, C♯, C＃相談室 Part93
http://mevius.5ch.net/test/read.cgi/tech/1492818720/

■次スレは>>970が建てる事。
建てられない場合は他を指定する事。

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

2021/11/17(水) 20:50:08.18

ビスマルク「愚者は経験に学び、賢者は歴史に学ぶ」

2021/11/17(水) 22:01:06.93

そもそもコンパイル環境なら、VS Codeすらいらないんじゃ

2021/11/17(水) 22:23:38.77

VSはダメでVSCodeはインストールできるんだ
よくわからない環境だな

2021/11/17(水) 22:48:30.86

普通の会社で使えるのは2017までなんだよな

2021/11/17(水) 23:11:54.33

うち２０２２買ったで

2021/11/17(水) 23:14:07.67

まだ2021年なのに？

2021/11/18(木) 09:00:28.34

ジャップには分からんだろうが世界的には9月からはもう2022年だぞ

2021/11/18(木) 21:37:43.70

じゃあ日本は4月からもう2022年になってるのか

2021/11/19(金) 01:11:50.72

コマンドが格納されたListを順番にサーバへ送信し、
1が返ってきたら次のコマンド送信、1以外かTimeoutしたら即中断みたいな処理を
ractiveでサクッと書けますか？

var cmdList=new List<string>{ cmd1, cmd2, ... };

cmdList
　.ToObservable()
　.TimeOut(　・・・　)
　.SendCommand()
　.WaitResponse()
　.PostRecv(1だったら次のコマンド送信、1以外やタイムアウトなら処理中止);

みたいな感じで。。

2021/11/20(土) 22:43:08.40

cmdList
.ForEach(x => { if(x.SendAsync(cts.Token).Result == 1 ) throw new Exception() });

2021/11/20(土) 22:58:08.09

var results = await cmdList
.ToAsyncEnumerble()
.SlectAwait(async (c) => await c.SendAsync(cts.Token))
.TakeWhileAwait(async (r) => r == 1)
ToArrayAsync();

2021/11/21(日) 00:02:36.10

タイムアウトで値や例外が返ってくるタイプだったらいいけど
時間経過で勝手にタイムアウト判定しろよってタイプだと詰みそう

**925** (ﾜｯﾁｮｲ 8eda-xqyN) · 2021/11/21(日) 00:41:13.81

ReactiveExtentions（System.Reactive名前空間 ver5.0）を使ってます。
自分なりに書いてみましたがこのライブラリの使い方がよくわかってない状態です。
（疑問点をコメントに書きました。分かる方教えてください）

IDisposable d = cmdList
　.ToObservable()
　.Timeout( TimeSpan.FromSeconds( 3000 ) )
　.Select( cmd =>
　{
　　var res = this.SendCommand( cmd ) // 自作のコマンド送信関数;
　　return res;
　} )
　.Where( res => res == 1 )
　.Subscribe(
　　res => /* ここで次のコマンド送信を行う書き方が分からない*/ res.ToString(),
　　ex => Debug.WriteLine( ex.Message /* Timeout発生時 */ ),
　　()=> Debug.WriteLine( "Complete" /* 全ての送信完了？それとも1つだけ送信完了？ */ )
);

>>927
やはりToAsyncEnumerbleからアプローチするべきなんですかね。。

2021/11/21(日) 00:46:46.14

Timeout関数の中身は
TimeSpan.FromMilliseconds( 3000 ) // 3秒
の間違いでした

2021/11/21(日) 10:42:16.16

Rxは監視を行うもの
時系列に沿って要素が流れてくる時に使うもの
なのである要素が流れてきた時に次の要素なんてのはまだ無い
要素が最初から揃ってるListから始めるからわからなくなる
最初から揃ってるなら普通にLinqかforeachを使う
Listでなくストリームならこれでおk

var d = CommandStream
.Select(c => Send(c))
.Timeout(t)
.TakeWhile(r => r == 1)
.Subscribe();

2021/12/10(金) 11:57:57.31

次元の違う配列ではArray.Copy出来ないのでしょうか。

例）
var foo = new int[4];
var bar = new int[3, 4];
ArrayCopy(foo,0,bar,0,4); // 例外発生 "The specified arrays must have the same number of dimensions."

1個づつコピーするしかないのでしょうか？

2021/12/10(金) 12:19:49.36

プリミティブ数値型の配列ならBuffer.BlockCopyできるけど
まあ多次元配列なんて肩身が狭いもんだから仕方ないね

2021/12/10(金) 12:40:02.71

classを配列に保存しているから、BlockCopyもダメなんですよね・・・
あまり数も多くないはずなので、１つずつコピー（代入）していきます。

2021/12/10(金) 13:17:59.80

>>932
若干アンセーフだけど行ごとにならSpan利用すればマネージ型でも出来る
for (int i = 0, len2 = b.GetLength(1); i < b.GetLength(0); i++)
foo.CopyTo(MemoryMarshal.CreateSpan(ref bar[i, 0], len2));

2021/12/10(金) 13:21:12.71

MemoryMarshal.CreateSpan(ref bar[i, 0], 5000)とかやっても確か例外でないから最新の注意を払う必要はある
まぁこうやってもソースの長さがデスト以内であれば問題は生じないけど

2021/12/12(日) 04:59:13.07

入力を検知したときにメソッドを実行する処理において、
要素Aがnullでなく（エラー防止）、Aの子要素Boolがtrueの場合のみ処理をスキップしたい場合に　いい書き方はありますか？
!A.Bool　でメソッドを実行にしてしまうとAがnullだったときに実行されなくなってしまうので……
ひとまずgotoを使って以下のようにしているのですが、もっとスマートな書き方ありますでしょうか

if (Input)
{
if(A != null){
if(A.Bool){
goto INPUTSKIP;
}
}
Method();
}
INPUTSKIP:

2021/12/12(日) 08:06:18.80

null条件演算子 null合体演算子で検索

2021/12/12(日) 08:23:25.29

>>937
こう?
if (Input)
{
if (A != null && A.Bool)
{
// INPUTSKIP
}
else
{
Method();
}
}

2021/12/12(日) 09:36:50.05

>>939
それを反転させて
if (Input)
{
if (A == null || !A.Bool)
{
Method();
}
}

2021/12/12(日) 10:16:02.74

>>937
goto 使いたくないと言うなら A が null か、A.Bool が false の時に処理したいってことだから普通に

if(A == null || !A.Bool){ 処理 }

って書くか>>938が言うnull条件演算子とnull合体演算子を使って

if(!(A?.Bool ?? false)){ 処理 }

でいける
個人的にはこの場合だと上の方がわかりやすいと思う

2021/12/12(日) 10:56:54.04

後藤さんが出てくると素人臭が漂ってくるからな

2021/12/12(日) 12:26:50.16

例外も後藤さんみたいなもんだが

2021/12/12(日) 12:48:33.01

AがnullだったときにORで並べてA.Boolの条件式を使ってしまうとそんなもんないよってエラーを吐くのでは、
と思い込んでいたのですが大丈夫なんですね……！
!A.Boolだけで判定したときに怒られたので頭凝り固まってました！

ありがとうございました！

2021/12/12(日) 13:09:03.73

>>944
いやショートサーキット評価だからエラーにならないだけ
https://docs.microsoft.com/ja-jp/cpp/cpp/logical-or-operator-pipe-pipe?view=msvc-170

2021/12/12(日) 13:17:35.22

>>945
ああーなるほど、ORの場合どれか1つでも満たされてれば実行されるから
左側が条件を満たしてたら右側はもうチェックされないのか！！
良いこと知れました！　ありがとう！

2021/12/12(日) 13:23:43.83

vb.netのorだとエラーになる

2021/12/12(日) 13:31:38.41

>>946>>947
論理演算子2つ並べるのがショートサーキット評価なんだけど
https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q12181009427
次から初心者スレ行ってくれ

2021/12/12(日) 14:02:55.63

>>947
OrElse 使え

>>948
> 論理演算子2つ並べるのがショートサーキット評価なんだけど
とか言う玄人w

2021/12/12(日) 14:06:00.32

vb恐ろしいな。そんなのバグの元じゃん

2021/12/12(日) 15:11:49.02

はあ？

C# ⇔ VB.Net
---------------------
& ⇔ And
&& ⇔ AndAlso
| ⇔ Or
|| ⇔ OrElse

ってなってるだけやぞ

2021/12/13(月) 09:12:51.67

>>937
条件式をネストしたり＆でつないだりと複雑にするのは難読性が高くなるから分けたほうがいいだろ
どういう条件ならMthod()を呼ぶのか明確に

if(Input){
//事前処理
judge=(A==null) ? true: //Aがnullのとき
A.Bool==false ? true: //Aの子要素がfalseのとき
false; //それ以外
if(judge) Method();
//共通処理
...
}

2021/12/13(月) 09:51:11.81

>>952が分かりやすく見えない…
>>940が誰が見ても一目瞭然でいいと思うわ
シンプルな条件分岐なんだからこだわる必要ない

2021/12/13(月) 11:44:51.75

skipするほうが例外的処理だろうからskip条件を明確にしてearly returnしたほうがいいと思う
論理式の細かい書き方よりも意図が伝わるかどうかのほうが大事

if (ShouldSkip(a)) {
return;
}
Method();

bool ShouldSkip(A a) {
return a?.Bool ?? false;
}

2021/12/13(月) 14:01:49.98

メソッドにすんのはいいけど、>>937 が入力を検知したときメソッドを実行したいって言ってんだからその通りに書けばいい
設計書通りに条件分岐すりゃいいのに無駄に置き換えて実装する人多いけどなんなんだろうね

if(入力検知(input, A)){
Method();
}

bool 入力検知(bool input, A a){
return input && a != null && a.Bool;
}

2021/12/13(月) 14:22:24.69

昼飯食いながら適当だから何か勘違いしてるかもしれないけど、>>937って要は
ある条件を満たした時だけMethodを実行したい、それだけでしょ？

だったら四の五の言わずに素直にそう書くのが一番じゃないの。
if( input && (!A?.Bool ?? false)) Method();

2021/12/13(月) 14:29:51.31

求めてるのは「もっとスマートな書き方」だからな
主観的な問題だから、俺らが言えるのは「こういう書き方もある」ってのだけ
正解（どれが一番か）を決めれるのは本人だけ

2021/12/13(月) 14:47:42.73

コードでも文章でも読み手に「要するにそれってどういうこと？」って思わせないのが一番。
これもコードでも文章でもそうだけど、読みにくいものは書いた本人がそこをよく理解せずに書いている。

2021/12/13(月) 15:06:50.29

ReSharperが薦めるように書き換えたら、何をしてるのかわからなくなった。

2021/12/13(月) 15:10:10.54

最近のc#はマルチパラダイムで構文が増えてカオスだ

2021/12/13(月) 16:35:33.81

>>955,956
>>937を読む限りでは入力を検知する処理と
検知した後に特定の処理をスキップするかどうかを判断する処理は別じゃないのかな？

「A.Boolがtrueなら入力を検知しない(or 入力を無視する)」ってことなら
入力よりも先にA.Boolを確認するよね

2021/12/13(月) 16:52:41.40

ちなみに回答ざっと見てこんなだな
こういうのはコードが長くなっても動作が分かりやすい方がいいと思う

>>939,940,952
おｋ

>>954,955
バグあり

2021/12/13(月) 22:23:58.46

>>960
それ。言語の表現力が豊かでも、自分やチームで一貫性を担保するのが面倒ξ

2021/12/14(火) 09:16:58.78

>>963
そして、コーディング規約が底辺プログラマーに合わせられて生産性が低下する

2021/12/14(火) 20:46:07.43

どうせ好き勝手に言語機能使って上がる生産性なんて微々たるもん

2021/12/14(火) 23:18:34.83

.net の 1.1 で書くとめちゃくちゃ不便で絶望するぞ

2021/12/14(火) 23:43:50.97

linqがある.NET Framework3.5ですら今これでやれって言われたら不便に感じるしなぁ
使えるものは使わせてもらった方が全然楽だわ

2021/12/15(水) 08:20:32.57

色々使えた方が個人的に楽だと思うのは否定しないが、もっともらしく「生産性が上がる」とか言っちゃうと
意外とそうでもなかったり。

2021/12/15(水) 09:41:14.43

>>966
特にList<T>が無いのはキツイわ

2021/12/15(水) 12:10:36.77

心配しなくても今どき1.1で書いたり書かされたりすることはないでしょ

2021/12/15(水) 16:03:58.38

1.1とか極端すぎるやろ

2021/12/15(水) 16:14:56.34

2.0の知識で止まってるヤツも多いよね

2021/12/15(水) 16:57:55.56

さすがに今どきvar使うなとかLINQ使うなとか言われないよね？

2021/12/15(水) 17:25:02.67

>>973
var使うとわからなくなるというのは名称が明示的になってないだけだと思うわ

2021/12/15(水) 17:31:22.22

>>974
ハンガリアン記法?

2021/12/15(水) 18:19:16.59

MS公式はvar推奨してるけどさすがにintとかstringとか基本的な型はvarにはしないな

2021/12/15(水) 21:30:02.16

明示的に型を書かないとダメなところ以外全部varで書くよ

2021/12/15(水) 21:44:12.20

最近はnew()で書けるようになったから、極力varはやめてる
Hoge hoge = new();

実装当初の想定通りlinqの戻りくらいかなvar、使うの。

2021/12/15(水) 23:22:27.06

new()使えるのってc#いくつからだっけ

2021/12/15(水) 23:43:54.17

はやくおれのレベルまで上がって来い
上で待ってるぞ

2021/12/16(木) 00:21:19.35

new()で書けるようになったけど、コンストラクタで初期設定値渡すようにしちゃうから全然使わんなぁ。
セッター潰してコンストラクタで設定するのが好き

2021/12/16(木) 07:48:14.53

>>979
C# 9.0

2021/12/16(木) 09:44:30.84

C#がどんどんボクのしらないものになっていってる
いっそC##やDと名乗っていただきたい
（Dは既にある、は受け付けない）

2021/12/16(木) 10:15:25.85

C#に毛が生えた程度だろ
C丼でええがな

2021/12/16(木) 10:54:52.18

c井

2021/12/16(木) 11:01:26.24

C⛩

2021/12/16(木) 11:42:40.19

C升

2021/12/16(木) 11:47:41.44

>>983
ほんと、氷川きよしくらい変わってしまった

2021/12/16(木) 23:46:42.90

if(item is {} a){
...
}
null判定でこんな書き方あるんだな。Resharperさんに提案されて初めて知ったわ。
個人的にはあまり使いたくないけど。

2021/12/17(金) 00:40:01.77

>>989
それやるとaの色がローカル変数色にならず白のままになるのが不満なんだけど俺の環境だけか？

2021/12/17(金) 02:39:12.84

C林

2021/12/17(金) 11:21:38.48

>>989
C#9なら
if (item is not null)
って実に直感的に書けるから早くこっちにこいよ…

ところでその式でaで受ける意味なくね？

2021/12/17(金) 13:42:40.65

C極

2021/12/19(日) 09:44:58.73

WinFormsを独学でやっているのですが教えて下さい
そこそこの規模のソフトは作れるようになってますが
VSで開発していくうえで、フォルダ構成ってどう考えればいいですかね
いままではDB関連をDBフォルダ
フォームはFormにまとめてさらにその中に機能別にフォルダを
細かいクラスファイルはClassというフォルダに
DBとの受け渡し（sｑｌ書いてFormとの受け渡しModelみたいなもの？）は
Modelというフォルダにまとめています

プロの皆様はチームで作っていかれる際に
会社によりけりかもしれませんが一般的なフォルダ構成の考え方ってあるんでしょうか

2021/12/19(日) 09:58:37.55

そんなもの存在しないから適当でいいよ
まぁあえて口出しするならFormフォルダはFormクラスと名前が被るからやめとけって位かな

2021/12/19(日) 10:00:48.95

なんのフォルダなのか理解できない
プロジェクト（visual studio上でのプロジェクト）内じゃフォルダ切ることはないな
プロジェクトがある程度大きくなったらクラスライブラリに分ける

2021/12/19(日) 10:16:13.92

シングルアセンブリ構造なのかな

2021/12/19(日) 10:33:08.48

>>995
そうなんですね、気をつけます

>>996
たとえば画面数でも１００超えて、機能部分のClassファイルも１００越えてくると
いちいち該当ファイル探すのが面倒でフォルダに分けたって感じです
前はPHPのフレームワークとか触ったときは
たとえばcontroller　model viewとフォルダも階層構造みたいになっていて
そういう感覚もあります
チームでやられていると、どこに何のファイルが散らばっているか
わからなくなったりしないのですか？　きちんと仕様書があるから関係ないのですかね

やっぱりクラスライブラリ（DLL化？）もちゃんと理解しないと駄目ですよね
一人で作ってるので、DLL化するほうが手間な気がして避けてました

2021/12/19(日) 10:44:47.17

どこに何がは命名の問題もあるんじゃないかな
classも変数も先頭、末尾にそれが何か分かるように名前つけてるから名前順にソートされてればすぐわかる

2021/12/19(日) 10:53:09.81

>>998
機能ごとにProjectを作るんだよ。

**1001** · Over 1000

このスレッドは１０００を超えました。
新しいスレッドを立ててください。
life time: 1004日 15時間 55分 34秒