ふらっと C#,C♯,C＃（初心者用） Part128 [無断転載禁止]©2ch.net

**デフォルトの名無しさん** · 2017/05/02(火) 22:05:40.13

「どんなにくだらないC#プログラミングやVisual C#の使い方に関する質問でも誰かが優しくレスをしてくれるスレッド」です。

他のスレッドでは書き込めないような低レベルな質問、
質問者自身なんだか意味がよく分からない質問、
ググろうにもキーワードが分からないなど、勇気をもって書き込んでください。

内容に応じて他スレ・他板へ行くことを勧められることがあります。ご了承下さい。

なお、テンプレが読めない回答者は邪魔なので後述のC#相談室に移動して下さい。
C#に関係の無い話題や荒らしの相手や罵倒レスはやめてください

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

■前スレ
ふらっと C#,C♯,C＃（初心者用） Part127 [無断転載禁止]©2ch.net
http://echo.2ch.net/test/read.cgi/tech/1489498042/

■関連スレ
C#, C♯, C＃相談室 Part94 [無断転載禁止]©2ch.net
http://echo.2ch.net/test/read.cgi/tech/1492843013/

■コードを貼る場合は↓を使いましょう。
http://ideone.com/
https://dotnetfiddle.net/

■情報源
https://msdn.microsoft.com/en-us/library/gg145045.aspx
http://referencesource.microsoft.com/
VIPQ2_EXTDAT: default:vvvvv:1000:512:----: EXT was configured

**デフォルトの名無しさん** · 2017/05/18(木) 22:40:26.48

>>268
最大数十回程度だったと思う
ただ24時間運用の機器だったから再現テストがめちゃ大変だった
最後は自前で読み書きを実装した

**デフォルトの名無しさん** · 2017/05/18(木) 23:10:55.63

>>270
iniに記録する類の情報なんかアプリ終了時の保存しようよそもそもw
いきなり電源落ちたら困る？
そういうのはハード(UPSとか)で対応しなきゃw

**デフォルトの名無しさん** · 2017/05/19(金) 09:25:16.90

下記のような処理をスマートに書くとどうなるでしょうか
for (int i = 0; i< 100; i++){
if (i< 50){
for (int j = 0; j< 20; j++){
if (j< 10){
hoge[idx] = 1;
}else{
hoge[idx] = 2;
}

idx++;
}
}else{
for (int j = 0; j< 20; j++){
if (j< 10){
hoge[idx] = 3;
}else{
hoge[idx] = 4;
}

idx++;
}
}
}

**デフォルトの名無しさん** · 2017/05/19(金) 09:49:36.16

>>272
その前に質問をスマートに書き込む技術を身に付けるのがいいぞ

**デフォルトの名無しさん** · 2017/05/19(金) 10:06:13.31

痩せろデブ

**デフォルトの名無しさん** · 2017/05/19(金) 10:06:42.03

>>272
const int COUNT = 2000;
int idx = 0;
int[] hoge = new int[COUNT];
var tmp = Enumerable.Range(idx, COUNT).Select(x => x / 10 % 2 + x / 1000 * 2 + 1).ToArray();
Array.Copy(tmp, 0, hoge, idx, COUNT);
idx += COUNT;

**デフォルトの名無しさん** · 2017/05/19(金) 10:14:56.08

>>272
lengthとかcountなりプロパティ使えや

**デフォルトの名無しさん** · 2017/05/19(金) 10:50:07.32

>>272
定数どうにかするべき

**デフォルトの名無しさん** · 2017/05/19(金) 11:06:00.52

ListでAddして行って後からToArrayにするのどうよ

**デフォルトの名無しさん** · 2017/05/19(金) 12:43:58.87

>>272
こんな感じ？
aはない方がいいかも

Action<int> a = (x) =>
{
　　hoge[idx] = x;
　　idx++;
};

Action<int, int> a2 = (first, second) =>
{
　　for (int i = 0; i < 10; i++) a(first);
　　for (int i = 0; i < 10; i++) a(second);
};

for (int i = 0; i < 50; i++) a2(1, 2);
for (int i = 0; i < 50; i++) a2(3, 4);

**デフォルトの名無しさん** · 2017/05/19(金) 12:48:21.02

Action<int> a = (x) => hoge[idx++] = x;
で良かったねw

**デフォルトの名無しさん** · 2017/05/19(金) 13:01:35.38

>>279
何やってるのかわかりにくい

**デフォルトの名無しさん** · 2017/05/19(金) 13:09:48.87

そんなテトリスを十行で書いたから偉いみたいな厨房コードより
一番はじめのコードで定数を意味のわかる形にするだけでええと思うで
デバッガでも追いやすいと思うし
１行にありったけ詰め込んじゃうとバグったときに追いにくいし仕様書のどこがどのコードと一致してるかもわかりにくい

**デフォルトの名無しさん** · 2017/05/19(金) 13:14:54.18

言っちゃ悪いけど、この程度のコードが分かりづらいと思うなら
プログラマなんかやめた方がいいよw

>>279みたいな書き方をする目的は行数を減らすことじゃない。

>>272のコードにある重複部分を一つにまとめるとともに、2重ループで構成される
スパゲッティーな構造を平坦化して見通しを良くすること

**デフォルトの名無しさん** · 2017/05/19(金) 13:19:11.93

>>283
俺は少しでも読みにくいと思うなら誰でも読めるコードに倒す派
程度問題になるなら簡単な表現にするよ
何より仕様書に書けない
お前そのラムダ式仕様書でどうやって表現するの?

**デフォルトの名無しさん** · 2017/05/19(金) 13:22:52.04

言っても無駄なタイプだと思うけど、まともなプログラマなら
>>279の方が>>272より何倍も読みやすいと感じるはずだと思うよ。

>>279が読みづらいと思うなら、それは端的に君の能力不足ですw
悪いけどね

まあ、「マジックナンバーガー」なんて本質と無関係のことを吠えてるレベルの人には
分からないかもしれん。

**デフォルトの名無しさん** · 2017/05/19(金) 13:26:48.87

>>285
それは何人のプログラマにアンケートをとったの?
設計を語る割にはあんまりレビュー慣れしてないね
お前が不勉強だから俺のコードがわからないんだって解説が会社で通っているならまあそれがお前の不幸なんだろうな

**デフォルトの名無しさん** · 2017/05/19(金) 13:29:09.30

>まともなプログラマ
ｹﾞﾗｹﾞﾗ

**デフォルトの名無しさん** · 2017/05/19(金) 13:37:02.35

俺はLinq使ってるコードの方がはるかに見やすいと思う

**デフォルトの名無しさん** · 2017/05/19(金) 13:40:46.21

>>288
設計書の内容と一致できる?

**デフォルトの名無しさん** · 2017/05/19(金) 13:42:16.23

読める人が3人以上いれば使って良し
その場合、読めない奴は今後雑魚コーダーとして扱効果査定で技術力マイナス査定
俺らも遊びじゃないからね
何でもかんでも下に合わせるのはジャップランド運動会みたいで悪しき風習だ

**デフォルトの名無しさん** · 2017/05/19(金) 13:42:58.11

初心者の質問スレってわからない大先生がまた出てきたよ

**デフォルトの名無しさん** · 2017/05/19(金) 13:43:46.41

>>289
設計書どれ？

**デフォルトの名無しさん** · 2017/05/19(金) 13:46:06.68

Linq使うなとかvar使うなとかウザイ先生が定期的に湧くね

**デフォルトの名無しさん** · 2017/05/19(金) 13:46:42.22

>>292
じゃあ、設計書無いのにソースだけ見ていいって言ってるわけだ

俺らの仕事は設計書を書いた段階で95%は終ってなきゃいけないよね
いいソースってのは設計書の内容をソースに反映できていることであって
ソースの見てくれがいいことじゃないよ

**デフォルトの名無しさん** · 2017/05/19(金) 13:47:34.14

何言ってんだこいつ

**デフォルトの名無しさん** · 2017/05/19(金) 13:55:45.76

設計書をコードに齟齬なく反映可能
だとすればその設計書はもはや設計書ではなくコードである
コードに置き換えるという無駄な行程を無くしてコンパイラに仕事をさせればよい

設計書をコードに齟齬なく反映不可能
だとすれば設計書至上論者の言ってることはハナから間違いとなる
反映不可能なのに反映したものが良いとはおかしな話だ
反映不可能なら書いたコードが設計書を正しく反映していると何をもって言えるのだろうか

結論
設計書至上主義者は現実が見えていないバカ

**デフォルトの名無しさん** · 2017/05/19(金) 14:01:48.82

設計書は「誰の為に」「何の為に」あるのか理解してる？
まさかプログラマの為に作られてるとでも思ってるの？

**デフォルトの名無しさん** · 2017/05/19(金) 14:02:58.29

>>296
だから設計書書きたく無いって?ｗ
情シスはいつになっても馬鹿だなｗ

**デフォルトの名無しさん** · 2017/05/19(金) 14:05:06.67

>結論
ｹﾞﾗｹﾞﾗ

**デフォルトの名無しさん** · 2017/05/19(金) 14:35:25.16

今回の命題はコードを簡単にするにはどんなやり方がありますか？
ってことなのにありもしない設計書ベースにコードを簡単にすべきではないと主張するとか完全に馬鹿だろw

**デフォルトの名無しさん** · 2017/05/19(金) 14:37:01.65

>>300
簡単?
スマート（？）だろ?

**デフォルトの名無しさん** · 2017/05/19(金) 14:42:01.05

よくわからんから具体例で議論してほしい
どういう設計書なら>>279がだめなの？
ここ初心者スレだから初心者にもわかるようにやってくれ

**デフォルトの名無しさん** · 2017/05/19(金) 14:42:02.36

>>301
>>300の「簡単」を「スマート」に置換しても主旨は全然変わらないからそれでいいよ

**デフォルトの名無しさん** · 2017/05/19(金) 14:55:55.08

>>300
これに尽きる

**デフォルトの名無しさん** · 2017/05/19(金) 15:23:33.91

設計書文化の頭の悪さが露呈した形になるな

**デフォルトの名無しさん** · 2017/05/19(金) 15:32:26.04

>>302
ううんそれは本質じゃないよ
設計書もないのにソースの良し悪しを判断しようとしてるのが不味い

**デフォルトの名無しさん** · 2017/05/19(金) 15:41:42.49

また底辺プログラマくんが来てスレが荒れてるねw
一生拝承システムでも作ってろ

**デフォルトの名無しさん** · 2017/05/19(金) 16:15:41.26

底辺かどうか知らんけど、ラムダ式をトリッキーでも何でもない
ごく普通の使い方しただけで上から目線で「わかりにくい」とか勘弁してほしいねw

>>279はラムダ式を使ってるだけで、考え方自体は普通の構造化プログラミングの方法論通り
共通部分を括り出して分割統治しただけ。

でも構造化プログラミングの段階で躓いてる人って実際割といるよね。
前の職場にも一つのメソッドに全部のロジックを詰め込んでダラダラ書く方が
分かりやすいと思い込んでる人がいて困った

**デフォルトの名無しさん** · 2017/05/19(金) 16:26:40.59

バカが書いた凝ったコードこそが悪夢だからね
コボラーをPGとして使わなきゃいけないなら全部フラットに書くのは実際無難な方法ではある
ウォーターフローが守られてる限りは問題なくスケールするし

**デフォルトの名無しさん** · 2017/05/19(金) 16:34:13.39

>>308
狙ってやってるのか不具合なのかわかりにくいな
普通にif文で条件判定入れてくれると読みやすいんだけど
まあ、それは設計書になんて書いてあるかだけどね

**デフォルトの名無しさん** · 2017/05/19(金) 16:44:34.52

>>310
単純に1ずつ増えるループ変数iがXを超えたかどうか（i>X)みたいなのは
条件判断なんて言わない。

I == Xになるまではiは絶対にX未満、一度Xを超えたらその後は絶対にi>Xに決まってるんだから、
普通に考えてただの無駄、それが言い過ぎなら仕方のない妥協の産物だ。

こういうコードを見て無駄でエレガントでないと感じないならプログラマ向いてないよマジで

**デフォルトの名無しさん** · 2017/05/19(金) 16:58:16.77

>>311
ううんそれは元のコードが仕様通りだったらの話
範囲で入る値が異なるコードを書くなら明示的にif文で条件判定を書いて欲しいんだよね
だからラムダのコードの方は俺には偶然によっかかって辛うじて成り立ってるコードに見えちゃう

**デフォルトの名無しさん** · 2017/05/19(金) 17:19:07.19

何のこっちゃ
要求されているのは機能的に等価でよりスマートなコード以上でも以下でもないでしょ

YAGNIじゃないけど、こんなところで無限にありうる仕様変更の可能性に対して
備えてないからダメなんてただの言い掛かりでしかない

**デフォルトの名無しさん** · 2017/05/19(金) 17:57:50.30

ちなみに272氏は単純にこう書こうとは思わんかったん？
for (int i = 0; i< 100; i++){
if (i<**) hoge[i]=1;
if (i>=** && i<**) hoge[i]=2;
：
if (i>**) hoge[i]=4;
}

**デフォルトの名無しさん** · 2017/05/19(金) 18:20:26.57

>>313
違うよ
お前のコードは不具合にしか見えねーって言ってるの

**デフォルトの名無しさん** · 2017/05/19(金) 18:27:07.86

>>315
具体的にどうぞ
まあ無理しなくていいよ

**デフォルトの名無しさん** · 2017/05/19(金) 18:28:57.76

linqクソ遅いやん
for文の方が何倍か速いよね確か

**デフォルトの名無しさん** · 2017/05/19(金) 18:30:03.38

100倍だぞ100倍って主張が昔あったような…

**デフォルトの名無しさん** · 2017/05/19(金) 18:31:48.04

っていうかさあ、ループ変数が単純に増減するループの中で
ループ変数を定数と比較してるコードを見たら、なんだか気持ち悪いとか何か間違ってると
感じるのが普通だけど、>>312みたいに逆にそうでないとしっくりこないって感じる人間もいるんだから
世の中いろいろだね

**デフォルトの名無しさん** · 2017/05/19(金) 18:52:01.40

添字アクセスはバグの温床だからforはなるべく使わない派だわ

**デフォルトの名無しさん** · 2017/05/19(金) 18:58:05.71

そだね

**デフォルトの名無しさん** · 2017/05/19(金) 21:06:53.85

using ( var stream = new MemoryStream( hoge ) )
{
for ( int i = 0; i < 50; i++ )
{
for ( int j = 0; j < 10; j++ )
{
stream.WriteByte( 1 );
}

for ( int j = 0; j < 10; j++ )
{
stream.WriteByte( 2 );
}
}

for ( int i = 0; i < 50; i++ )
{
for ( int j = 0; j < 10; j++ )
{
stream.WriteByte( 3 );
}

for ( int j = 0; j < 10; j++ )
{
stream.WriteByte( 4 );
}
}

idx = ( int )stream.Position;
}

**デフォルトの名無しさん** · 2017/05/19(金) 21:22:07.20

教えてください
ディスクトップの特定の場所の色が変わったらAを実行するというコードを書きたいのですが
どの様に書けばよいのでしょうか？
よろしくお願いします

**デフォルトの名無しさん** · 2017/05/19(金) 21:25:59.99

>>323
それは画面キャプチャが必要でそれだけでも一筋縄じゃできないような
タイマで監視するにしてもコスト大きすぎるし
他の条件は無いのか？

**デフォルトの名無しさん** · 2017/05/19(金) 21:33:43.42

>>324
ありがとうございます

アプリから通知が来た時に右下に通知の吹出が出るので
その時にAというプログラムを実行したいと考えているのですが難しそうですね

**デフォルトの名無しさん** · 2017/05/19(金) 21:39:37.11

通知にフックすればいいじゃん

**デフォルトの名無しさん** · 2017/05/19(金) 21:48:19.14

>>323
ディスクトップｗｗｗ

**デフォルトの名無しさん** · 2017/05/19(金) 22:02:05.77

>>271
設定とかを書いてる訳じゃないよ
詳しくは書けないけど要するに WritePrivateProfileString なんて使うのがバカって言う結論
装置制御用のPCだからUPSはもちろんハングアップしたら外部のハードウェアで電源Off/Onまでやるようになってる

**デフォルトの名無しさん** · 2017/05/19(金) 23:45:52.56

>>320
じゃあforeachで

**デフォルトの名無しさん** · 2017/05/19(金) 23:47:34.46

>>323
指定した座標のカラーを取れば良い

**デフォルトの名無しさん** · 2017/05/19(金) 23:53:05.65

通知ウィンドウ(?)をSpy++とかで調べておいて、そのウィンドウが出てるかポーリングするとか。

**デフォルトの名無しさん** · 2017/05/20(土) 01:29:25.58

>>325
色はダメ

別のアプリの通知がでて同じ色の吹き出しが出たらどう区別すんの？

**デフォルトの名無しさん** · 2017/05/20(土) 10:07:05.79

うんこマーク付けとけうんこマーク

**デフォルトの名無しさん** · 2017/05/20(土) 16:56:47.34

>>272の質問の人何も言わずにとんずらしちゃったけど、
ひょっとしてラムダ式が分かりづらいって文句言ってたのが質問者本人だったのかなw

**デフォルトの名無しさん** · 2017/05/20(土) 17:17:57.91

>>306
まさにお前が言うな

**デフォルトの名無しさん** · 2017/05/20(土) 17:42:51.92

C#スレこんな低レベルだったっけ

**デフォルトの名無しさん** · 2017/05/20(土) 17:51:55.39

初心者だからみんな低レベルだよ

**デフォルトの名無しさん** · 2017/05/20(土) 17:58:30.08

>>334
初心者にもわかりやすいコードがいいね

**デフォルトの名無しさん** · 2017/05/20(土) 18:07:46.65

>>334
コードをスマートに書き換えて、ってそもそも質問じゃないじゃん
ネタ振ってスレの様子見て遊んでいる人が混ざっているのだからいつまでも構わないでくれ

**デフォルトの名無しさん** · 2017/05/20(土) 18:36:04.13

まあ、初心者に的確な判断を要求するのも酷だな

**323** · 2017/05/20(土) 18:52:37.15

>>327
し、初心者なので許してくださいww

>>330
なるほど～その線でしらべてみます
…って思ったけどさっぱりわかりませんorz何処か誘導お願いします

>>332
大丈夫です。
ほかのアプリは黄色い色の通知はしないのでいけると思います

**デフォルトの名無しさん** · 2017/05/20(土) 20:24:06.43

>>339
もっと綺麗に書けそうに思うけど自分ではどうしたらいいか分からない、
っていうのも立派な質問だしスレの趣旨に反していると思えない

**324** · 2017/05/20(土) 20:29:39.57

>>341
その通知を出すアプリが何かわかればアドバイスできる人もいるかもしれない
色は問題ありすぎだから忘れろｗ
今のところ>>326が一番適切な回答だと思う

**デフォルトの名無しさん** · 2017/05/20(土) 20:29:47.52

>>341
デスクトップをキャプチャしてイメージにする。その特定の座標の色を取得する。

まずはそこから

**デフォルトの名無しさん** · 2017/05/20(土) 20:31:11.74

次に上に書いてあるようの、デスクトップの画面更新をフックする。

WM_PAINTとかかな

**デフォルトの名無しさん** · 2017/05/20(土) 20:55:22.87

色はダメだってｗ
OSがアップデートしたりソフトの使用が変わったら何の意味もない

もしそのアプリがc#と絡んでるならそっちをいじれ

**デフォルトの名無しさん** · 2017/05/20(土) 21:05:00.27

通知のフックって具体的にはどうやるの。
アプリの中身知らなくても出来るもん?

**デフォルトの名無しさん** · 2017/05/20(土) 21:06:58.07

デスクトップの色解析はスジが悪いよな。
たまたま似たような画面が重なったら誤動作するし。

**デフォルトの名無しさん** · 2017/05/20(土) 21:27:00.53

Deepラーニングで誤動作回避ですよ

**323** · 2017/05/20(土) 21:38:51.77

>>343
MultiChartsというチャートソフトなんですけど、通知をフックすると言うことは…ネイティブコードの解析が必要？
それが出来れば最高なんですけど私のスキルでそこまで出来ません

他に何も入っていない専用PCなので仕様はほとんど変わらないので
とりあえずスジが悪いかも知れませんが>>344さんの方法で調べてみます

**デフォルトの名無しさん** · 2017/05/20(土) 21:45:16.85

>>331の方針の方がはるかにいいぞ

**デフォルトの名無しさん** · 2017/05/21(日) 00:34:18.82

フックするってどうゆう意味だってばよ？？

**デフォルトの名無しさん** · 2017/05/21(日) 00:40:20.10

>>352
SetWindowsHookEx

**デフォルトの名無しさん** · 2017/05/21(日) 01:25:55.95

オープンソースで公開するってんならOSやソフトの仕様を考慮する必要があるけど、
全くの個人使用なら色で判断するってのはそう悪いアプローチでもないと思う

autohotkeyとかでできそうだけど

**デフォルトの名無しさん** · 2017/05/21(日) 02:12:38.30

複数valueを登録できるdictionaryはありますか？

**デフォルトの名無しさん** · 2017/05/21(日) 02:16:03.09

Dictionary<T0, Tupple<T1,T2 ･･･>>

**デフォルトの名無しさん** · 2017/05/21(日) 02:19:36.59

TuppleじゃなくてTupleだった失礼

**デフォルトの名無しさん** · 2017/05/21(日) 03:21:47.56

Lookup

**デフォルトの名無しさん** · 2017/05/21(日) 12:44:06.39

>>356
tupleを活用する事で可能なんですね。
ありがとう、試してみます。

**デフォルトの名無しさん** · 2017/05/21(日) 13:01:14.66

タプルをコレクションに入れるのはバグの温床
dynamicに匿名型突っ込む方がまだマシ

**デフォルトの名無しさん** · 2017/05/21(日) 13:18:52.22

>>360
どんなバグが出るんですか?

**デフォルトの名無しさん** · 2017/05/21(日) 14:51:50.29

妙な小細工するより素直に複数の値の入れ物の型を定義した方が
分かりやすく使いやすいと思うけど

**デフォルトの名無しさん** · 2017/05/21(日) 15:27:42.93

個人的には普通に格納用クラス作ったほうがいいと思うけどね

**デフォルトの名無しさん** · 2017/05/21(日) 17:07:27.54

>>361
要素を取り違えるバグだね
入れてるところまで遡らないと何番目の要素が何を意味するのか分からない
個人的には、タプルは即座に要素をバラして変数に展開するべきで
タプル型の変数やコレクションは決して使ってはいけない

**デフォルトの名無しさん** · 2017/05/21(日) 17:32:54.07

ごく短期間なスコープでならタプルも便利だけどね

**デフォルトの名無しさん** · 2017/05/21(日) 17:35:37.21

タプルを単一の値としてコレクションや変数に入れたことによる要素の取り違えに起因するバグはPythonだとよくある
C#7.0のタプルはその問題点を考慮して要素にラベルを付けられるようになってるね

**デフォルトの名無しさん** · 2017/05/21(日) 18:46:01.80

絶滅危惧種だとは思うけど、コードレベルの設計書が必要な人達って未だいるんだね (´・ω・`)

**デフォルトの名無しさん** · 2017/05/21(日) 19:01:05.84

普通にいるだろ
ていうかそういう奴はどれだけ詳細なコード設計を渡してもそれだけではコード書けなくて、
必ず類似したプログラムをサンプルとして渡してコピペさせてやる必要がある

**デフォルトの名無しさん** · 2017/05/21(日) 19:09:16.32

横着しないでマシな人を雇った方がいいですよ…

**デフォルトの名無しさん** · 2017/05/21(日) 19:12:49.79

新米ってどこにでも居るけどな